Irit Goihman has posted comments on this change.

Change subject: functional tests: fix supervdsmServer test
......................................................................


Patch Set 19: Verified+1

(1 comment)

nose.config: INFO: Ignoring files matching ['^\\.', '^_', '^setup\\.py$']
09:37:08 functional.supervdsmFuncTests.TestSuperVdsmRemotly
09:37:08     testKsmAction                                               OK
09:37:08     testPingCall                                                OK
09:37:08 
09:37:08 ----------------------------------------------------------------------
09:37:08 XML: /tmp/nosetests-el7.xml
09:37:08 ----------------------------------------------------------------------
09:37:08 Ran 2 tests in 0.052s

https://gerrit.ovirt.org/#/c/61902/16/tests/functional/supervdsmFuncTests.py
File tests/functional/supervdsmFuncTests.py:

Line 34: 
Line 35: class TestSuperVdsmRemotly(TestCaseBase):
Line 36: 
Line 37:     @forked
Line 38:     @testValidation.ValidateRunningAsRoot
> You don't need this, we have @forked decorator for running tests in a child
Done
Line 39:     def testPingCall(self):
Line 40:         dropPrivileges()
Line 41:         proxy = supervdsm.getProxy()
Line 42:         self.assertTrue(proxy.ping())


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I8c7e381809261cf330ef4f19c3fec5cd38254b1b
Gerrit-PatchSet: 19
Gerrit-Project: vdsm
Gerrit-Branch: master
Gerrit-Owner: Irit Goihman <igoih...@redhat.com>
Gerrit-Reviewer: Dan Kenigsberg <dan...@redhat.com>
Gerrit-Reviewer: Irit Goihman <igoih...@redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Nir Soffer <nsof...@redhat.com>
Gerrit-Reviewer: Yaniv Bronhaim <ybron...@redhat.com>
Gerrit-Reviewer: gerrit-hooks <automat...@ovirt.org>
Gerrit-HasComments: Yes
_______________________________________________
vdsm-patches mailing list
vdsm-patches@lists.fedorahosted.org
https://lists.fedorahosted.org/admin/lists/vdsm-patches@lists.fedorahosted.org

Reply via email to