Change in vdsm[master]: testlib: Show fully qualified class name

danken at redhat.com danken at redhat.com
Mon Jul 13 14:10:37 UTC 2015


Dan Kenigsberg has submitted this change and it was merged.

Change subject: testlib: Show fully qualified class name
......................................................................


testlib: Show fully qualified class name

We typically have multiple test case classes in the same module, and
looking in the tests results it is not clear where the tests are
located.

    RoundTests
        test_round(n=0, size=1024, result=0)                        OK
        test_round(n=1, size=1024, result=1024)                     OK
        test_round(n=1024, size=1024, result=1024)                  OK
        test_round(n=1025, size=1024, result=2048)                  OK
        test_round(n=3.14, size=1024, result=1024)                  OK
    StopwatchTests
        test_debug                                                  OK
        test_info                                                   OK
        test_notset                                                 OK

Now we show also the module name:

    utilsTests.RoundTests
        test_round(n=0, size=1024, result=0)                        OK
        test_round(n=1, size=1024, result=1024)                     OK
        test_round(n=1024, size=1024, result=1024)                  OK
        test_round(n=1025, size=1024, result=2048)                  OK
        test_round(n=3.14, size=1024, result=1024)                  OK
    utilsTests.StopwatchTests
        test_debug                                                  OK
        test_info                                                   OK
        test_notset                                                 OK

Change-Id: I560ba4a0b4efcc7c53d85a830b8a8d4f5365ce5e
Signed-off-by: Nir Soffer <nsoffer at redhat.com>
Reviewed-on: https://gerrit.ovirt.org/43432
Continuous-Integration: Jenkins CI
Reviewed-by: Francesco Romani <fromani at redhat.com>
Reviewed-by: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Reviewed-by: Dan Kenigsberg <danken at redhat.com>
---
M tests/testlib.py
1 file changed, 2 insertions(+), 1 deletion(-)

Approvals:
  Piotr Kliczewski: Looks good to me, but someone else must approve
  Nir Soffer: Verified
  Jenkins CI: Passed CI tests
  Dan Kenigsberg: Looks good to me, approved
  Francesco Romani: Looks good to me, but someone else must approve



-- 
To view, visit https://gerrit.ovirt.org/43432
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I560ba4a0b4efcc7c53d85a830b8a8d4f5365ce5e
Gerrit-PatchSet: 2
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <danken at redhat.com>
Gerrit-Reviewer: Francesco Romani <fromani at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsoffer at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: automation at ovirt.org


More information about the vdsm-patches mailing list