Skip to content

Commit 0064b12

Browse files
committed
address comments
Signed-off-by: Jenia Peimer <jpeimer@redhat.com>
1 parent b811a25 commit 0064b12

File tree

2 files changed

+5
-3
lines changed

2 files changed

+5
-3
lines changed

tests/storage/vm_export/conftest.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -216,7 +216,7 @@ def rhel_vm_for_snapshot_with_content(
216216
),
217217
) as vm:
218218
running_vm(vm=vm)
219-
write_file_via_ssh(vm=vm, filename=TEST_FILE_CONTENT, content=TEST_FILE_NAME)
219+
write_file_via_ssh(vm=vm, filename=TEST_FILE_NAME, content=TEST_FILE_CONTENT)
220220
yield vm
221221

222222

utilities/storage.py

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -646,7 +646,9 @@ def write_file_via_ssh(vm: virt_util.VirtualMachineForTests, filename: str, cont
646646
run_ssh_commands(host=vm.ssh_exec, commands=cmd, wait_timeout=TIMEOUT_2MIN, sleep=TIMEOUT_5SEC)
647647

648648

649-
def run_command_on_vm_and_check_output(vm, command, expected_result):
649+
def run_command_on_vm_and_check_output(
650+
vm: virt_util.VirtualMachineForTests, command: str, expected_result: str
651+
) -> None:
650652
"""Run command on RHEL VM via SSH and verify expected result is in output.
651653
652654
Args:
@@ -664,7 +666,7 @@ def run_command_on_vm_and_check_output(vm, command, expected_result):
664666
sleep=TIMEOUT_5SEC,
665667
)[0].strip()
666668
expected_result = expected_result.strip()
667-
assert expected_result in cmd_output, f"Expected '{expected_result}' in output '{cmd_output}'"
669+
assert expected_result == cmd_output, f"Expected '{expected_result}' in output '{cmd_output}'"
668670

669671

670672
def run_command_on_cirros_vm_and_check_output(vm, command, expected_result):

0 commit comments

Comments
 (0)