[Freeipa-devel] [PATCH] Fix backend.Executioner unit test.

Pavel Zuna pzuna at redhat.com
Wed Jan 13 16:46:41 UTC 2010


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.

Pavel
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Fix-backend.Executioner-unit-test.patch
Type: application/mbox
Size: 4070 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20100113/e8cff1c7/attachment.mbox>


More information about the Freeipa-devel mailing list