Nir Soffer has posted comments on this change. Change subject: iscsi: Add ChapCredentials equality and hash tests ......................................................................
Patch Set 1: Code-Review-1 (2 comments) Need to fix typos https://gerrit.ovirt.org/#/c/43989/1/tests/iscsiTests.py File tests/iscsiTests.py: Line 102: c1 = iscsi.ChapCredentials("username", ProtectedPassword("password")) Line 103: c2 = iscsi.ChapCredentials("username", ProtectedPassword("password")) Line 104: self.assertEqual(hash(c1), hash(c2)) Line 105: Line 106: def test_subclass_diffrent_hash(self): > diffrent->different Will fix in next version Line 107: class Subclass(iscsi.ChapCredentials): Line 108: pass Line 109: c1 = iscsi.ChapCredentials("username", ProtectedPassword("password")) Line 110: c2 = Subclass("username", ProtectedPassword("password")) Line 116: ]) Line 117: def test_not_equal_different_hash(self, user1, user2, pass1, pass2): Line 118: c1 = iscsi.ChapCredentials(user1, ProtectedPassword(pass1)) Line 119: c2 = iscsi.ChapCredentials(user2, ProtectedPassword(pass2)) Line 120: self.assertNotEqual(hash(c1), hash(c2)) > Why do we need this tests ? See my reply in https://gerrit.ovirt.org/#/c/43988/2/tests/storageServerTests.py -- To view, visit https://gerrit.ovirt.org/43989 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: I7f3fde727c179936ba480f5b4cf4c24cacb195b8 Gerrit-PatchSet: 1 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Nir Soffer <[email protected]> Gerrit-Reviewer: Ala Hino <[email protected]> Gerrit-Reviewer: Allon Mureinik <[email protected]> Gerrit-Reviewer: Dan Kenigsberg <[email protected]> Gerrit-Reviewer: Freddy Rolland <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: [email protected] Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
