Browse Source

tests: fix test_053_qrexec_vm_service_eof_reverse

and not mark it as expected failure anymore. Note the removal of the
expected failure isn't just about the changes here, but also about the
actual fix on the qrexec side (ffafd01 "Fix not closed file descriptors in
qubes-rpc-multiplexer" commit in core-qrexec repository).
Marek Marczykowski-Górecki 4 years ago
parent
commit
f62a861140
1 changed files with 2 additions and 2 deletions
  1. 2 2
      qubes/tests/integ/vm_qrexec_gui.py

+ 2 - 2
qubes/tests/integ/vm_qrexec_gui.py

@@ -260,7 +260,6 @@ class TC_00_AppVMMixin(object):
         self.assertEqual(stdout, b'test\n',
             'Received data differs from what was expected')
 
-    @unittest.expectedFailure
     def test_053_qrexec_vm_service_eof_reverse(self):
         """Test for EOF transmission VM(src)<-VM(dst)"""
 
@@ -268,6 +267,7 @@ class TC_00_AppVMMixin(object):
             self.testvm1.start(),
             self.testvm2.start()]))
         self.create_remote_file(self.testvm2, '/etc/qubes-rpc/test.EOF',
+                '#!/bin/sh\n'
                 'echo test; exec >&-; cat >/dev/null')
 
         with self.qrexec_policy('test.EOF', self.testvm1, self.testvm2):
@@ -284,7 +284,7 @@ class TC_00_AppVMMixin(object):
             except asyncio.TimeoutError:
                 self.fail("Timeout, probably EOF wasn't transferred")
 
-        self.assertEqual(stdout, b'test',
+        self.assertEqual(stdout, b'test\n',
             'Received data differs from what was expected')
 
     def test_055_qrexec_dom0_service_abort(self):