diff options
author | Tony Asleson <tasleson@redhat.com> | 2022-08-30 12:47:14 -0500 |
---|---|---|
committer | Tony Asleson <tasleson@redhat.com> | 2022-09-16 10:49:37 -0500 |
commit | f5876a1f3fb729d255ea0bc3974d2891381620b3 (patch) | |
tree | 0325d7ccd599b151dce6bf82f2f74f376eee173e /test/dbus | |
parent | e5c41b94b841abe1ce3c23c814f7f7c31048b6e8 (diff) | |
download | lvm2-f5876a1f3fb729d255ea0bc3974d2891381620b3.tar.gz |
lvmdbustest: Skip test_nesting if scan_lvs is not enabled
Diffstat (limited to 'test/dbus')
-rwxr-xr-x | test/dbus/lvmdbustest.py | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/test/dbus/lvmdbustest.py b/test/dbus/lvmdbustest.py index 772313246..c932d0830 100755 --- a/test/dbus/lvmdbustest.py +++ b/test/dbus/lvmdbustest.py @@ -2040,6 +2040,17 @@ class TestDbusService(unittest.TestCase): return new_pv_object_path + @staticmethod + def _scan_lvs_enabled(): + cmd = ['lvmconfig', '--typeconfig', 'full', 'devices/scan_lvs'] + config = Popen(cmd, stdout=PIPE, stderr=PIPE, close_fds=True, env=os.environ) + out = config.communicate() + if config.returncode != 0: + return False + if "scan_lvs=1" == out[0].decode("utf-8").strip(): + return True + return False + def test_nesting(self): # check to see if we handle an LV becoming a PV which has it's own # LV @@ -2053,6 +2064,8 @@ class TestDbusService(unittest.TestCase): if dm_dev_dir != '/dev': raise unittest.SkipTest('test not running in real /dev') + if not TestDbusService._scan_lvs_enabled(): + raise unittest.SkipTest('scan_lvs=0 in config, unit test requires scan_lvs=1') pv_object_path = self.objs[PV_INT][0].object_path if not self.objs[PV_INT][0].Pv.Name.startswith("/dev"): raise unittest.SkipTest('test not running in /dev') |