Pavel Zuna wrote:
Before the patch that allows to create unshared instances of Connectible, all Connection objects were deleted at once in destroy_context(). It made sense at the time, because there was always at most one Connection per Connectible subclass and Connectible.disconnect() was called only interanally by the Executioner class. Now that we can make arbitrary connections, it makes more sense to delete the Connection object when Connectible.disconnect() is called.

Long story short: I fixed the Disconnect class in the unit test, so that it behaves like Connectible.disconnect() and deletes the Connection object when called.


ack, pushed to master

Freeipa-devel mailing list

Reply via email to