Nir Soffer has posted comments on this change. Change subject: tests: Add test_missing_tags for block storage ......................................................................
Patch Set 2: Code-Review-1 (1 comment) Test looks ok, but I want to remove these data files instead of adding new files, and this test is not testing lvm instead of blockSD. https://gerrit.ovirt.org/#/c/56171/2/tests/blocksdTests.py File tests/blocksdTests.py: Line 65 Line 66 Line 67 Line 68 Line 69 Why not return fake lv instead of reading lvm output? This tests lvm module, not blockSD. And now we have to keep more test data files, and mention them in makefiles etc. Why work hard when we have easier way? -- To view, visit https://gerrit.ovirt.org/56171 To unsubscribe, visit https://gerrit.ovirt.org/settings Gerrit-MessageType: comment Gerrit-Change-Id: Idb789cf146a4a9919e4dc864a0ae744e06510cec Gerrit-PatchSet: 2 Gerrit-Project: vdsm Gerrit-Branch: master Gerrit-Owner: Adam Litke <[email protected]> Gerrit-Reviewer: Adam Litke <[email protected]> Gerrit-Reviewer: Jenkins CI Gerrit-Reviewer: Nir Soffer <[email protected]> Gerrit-Reviewer: gerrit-hooks <[email protected]> Gerrit-HasComments: Yes _______________________________________________ vdsm-patches mailing list [email protected] https://lists.fedorahosted.org/mailman/listinfo/vdsm-patches
