Nir Soffer has posted comments on this change.

Change subject: tests: Add time per test if nose-timer is available
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.ovirt.org/#/c/30171/1/tests/testrunner.py
File tests/testrunner.py:

Line 48:     timer_plugin = TimerPlugin()
Line 49:     timer_plugin.enabled = True
Line 50:     timer_plugin.timer_ok = 500  # Okay <= 500ms
Line 51:     timer_plugin.timer_warning = 5000  # Warn > 5s
Line 52:     timer_plugin.timer_no_color = False  # Enable color output
These defaults are not correct for all tests.

Some tests are slow because there is no other way to test without waiting, and 
for these the coloring means nothing.

This should be per test method or per class setting, to allow meaningful output.
Line 53: 
Line 54: from testValidation import SlowTestsPlugin, StressTestsPlugin
Line 55: 
Line 56: import zombiereaper


-- 
To view, visit http://gerrit.ovirt.org/30171
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I4960b30532a84abd259fb268c7e40a1751847a96
Gerrit-PatchSet: 1
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Antoni Segura Puimedon <[email protected]>
Gerrit-Reviewer: Antoni Segura Puimedon <[email protected]>
Gerrit-Reviewer: Dan Kenigsberg <[email protected]>
Gerrit-Reviewer: Francesco Romani <[email protected]>
Gerrit-Reviewer: Nir Soffer <[email protected]>
Gerrit-Reviewer: Yaniv Bronhaim <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
[email protected]
https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches

Reply via email to