summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTony Asleson <tasleson@redhat.com>2023-03-09 11:25:58 -0600
committerTony Asleson <tasleson@redhat.com>2023-03-10 12:51:53 -0600
commit9714f3ec4f3d4526a33781baf706c24930b6f26c (patch)
tree71c3fb9cd8aa1ab2d143356d6c7be77ab92e7076
parent2359023009db40112f4220899f0983684d897765 (diff)
downloadlvm2-9714f3ec4f3d4526a33781baf706c24930b6f26c.tar.gz
lvmdbustest: Only inject 1 missing key error
Previously we were injecting a missing key in the lv, vg, and pv. Given the order of processing in lvmdbusd, this prevented us from exercising all the error paths. Change to returning just 1 instead.
-rwxr-xr-xtest/dbus/lvm_error_inject.py10
1 files changed, 7 insertions, 3 deletions
diff --git a/test/dbus/lvm_error_inject.py b/test/dbus/lvm_error_inject.py
index d87286740..98520845c 100755
--- a/test/dbus/lvm_error_inject.py
+++ b/test/dbus/lvm_error_inject.py
@@ -79,6 +79,7 @@ def write_some(q, stream, remaining=False, binary=False):
def del_random_key(src_dict):
keys = list(src_dict.keys())
pick = random.randint(0, len(keys) - 1)
+ debug("%s will be deleted" % keys[pick])
del src_dict[keys[pick]]
@@ -87,16 +88,19 @@ def inject_key_error(output_json):
for r in output_json['report']:
if 'lv' in r:
for i in r['lv']:
+ debug("deleting a lv key")
del_random_key(i)
- break
+ return
if 'vg' in r:
for i in r["vg"]:
+ debug("deleting a vg key")
del_random_key(i)
- break
+ return
elif 'pv' in r:
for i in r["pv"]:
+ debug("deleting a pv key")
del_random_key(i)
- break
+ return
def inject_exit_error(output_json, val):