From a0f616f14e7190718dd0489e33bdffcde95c1855 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Marek=20Marczykowski-G=C3=B3recki?= Date: Wed, 21 Jun 2017 00:13:23 +0200 Subject: [PATCH] tests: fix checking exit code Since run_for_stdio raise an exception for non-zero exit code, it isn't ignored anymore. So, check if qrexec-client-vm return expected value, instead of keep ignoring it. QubesOS/qubes-issues#2861 --- qubes/tests/integ/vm_qrexec_gui.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/qubes/tests/integ/vm_qrexec_gui.py b/qubes/tests/integ/vm_qrexec_gui.py index 00443ba6..f9c86069 100644 --- a/qubes/tests/integ/vm_qrexec_gui.py +++ b/qubes/tests/integ/vm_qrexec_gui.py @@ -315,7 +315,7 @@ class TC_00_AppVMMixin(qubes.tests.SystemTestsMixin): stdout, _ = self.loop.run_until_complete(asyncio.wait_for( self.testvm1.run_for_stdio('''\ /usr/lib/qubes/qrexec-client-vm dom0 test.Abort \ - /bin/cat /dev/zero'''), + /bin/cat /dev/zero; test $? -eq 141'''), timeout=10)) except asyncio.TimeoutError: self.fail("Timeout, probably stdout wasn't closed")