Ayal Baron has posted comments on this change. Change subject: Add betterThreadingTests ......................................................................
Patch Set 1: I would prefer that you didn't submit this (3 inline comments) .................................................... File tests/betterThreadingTests.py Line 76: sleep(2) can't we make the times shorter? every sleep we add in tests adds time to wait when running the test suite It would be even better if we could avoid sleep altogether... Line 147: def testWait1L(self): when running the suite it would be easier on the eye to have more descriptive names. No one would know that testWait1L means test wait condition for 1s with lock Line 167: sleep(2) maybe pass the function to run as parameter so that only the "NotPass" test would have a sleep and then too a shorter one if possible -- To view, visit http://gerrit.ovirt.org/3495 To unsubscribe, visit http://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I63e1892937ae20fd2577d49faf69a617b4a17c04 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Saggi Mizrahi <[email protected]> Gerrit-Reviewer: Ayal Baron <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Peter V. Saveliev <[email protected]> _______________________________________________ vdsm-patches mailing list [email protected] https://fedorahosted.org/mailman/listinfo/vdsm-patches
