]> git.apps.os.sepia.ceph.com Git - ceph-ci.git/commitdiff
ceph-volume: delete test_lvs_list_is_created_just_once
authorRishabh Dave <ridave@redhat.com>
Fri, 6 Dec 2019 07:40:35 +0000 (13:10 +0530)
committerRishabh Dave <ridave@redhat.com>
Tue, 28 Jan 2020 15:25:58 +0000 (20:55 +0530)
lisitng.py doesn't call api.Volumes anymore. Therefore, this test is
redundant.

Signed-off-by: Rishabh Dave <ridave@redhat.com>
src/ceph-volume/ceph_volume/tests/devices/lvm/test_listing.py

index efbb460f96033e46acc72885d9e6e2cf0c4afe01..d823d6411649c142f300f4cf092ee30db3ce5dd8 100644 (file)
@@ -79,24 +79,6 @@ class TestList(object):
         with pytest.raises(SystemExit):
             lvm.listing.List([]).list(args)
 
-    def test_lvs_list_is_created_just_once(self, monkeypatch, is_root, volumes, factory):
-        api.volumes_obj_create_count = 0
-
-        def monkey_populate(self):
-            api.volumes_obj_create_count += 1
-            for lv_item in api.get_api_lvs():
-                self.append(api.Volume(**lv_item))
-        monkeypatch.setattr(api.Volumes, '_populate', monkey_populate)
-
-        args = factory(format='pretty', device='/dev/sda1')
-        with pytest.raises(SystemExit):
-            lvm.listing.List([]).list(args)
-
-        # XXX: Ideally, the count should be just 1. Volumes._populate() is
-        # being called thrice out of which only twice is moneky_populate.
-        assert api.volumes_obj_create_count == 2
-
-
 class TestFullReport(object):
 
     def test_no_ceph_lvs(self, volumes, monkeypatch):