tests: fix file descriptor leak

QubesOS · May 25, 2017 · 9cdf9a0 · 9cdf9a0
1 parent c435de0
commit 9cdf9a0
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 1 deletion.
diff --git a/qubesadmin/tests/app.py b/qubesadmin/tests/app.py
@@ -323,7 +323,6 @@ def test_004_qubesd_call_payload_stream_proc(self):
         with open(tmpdir + '/payload') as payload:
             self.assertEqual(payload.read(), 'some payload\n')
 
-
     def test_010_run_service(self):
         self.listen_and_send(b'0\0')
         with mock.patch('subprocess.Popen') as mock_proc:

diff --git a/qubesadmin/tests/storage.py b/qubesadmin/tests/storage.py
@@ -158,6 +158,7 @@ def test_040_import_data(self):
         input_proc = subprocess.Popen(['echo', '-n', 'some-data'],
             stdout=subprocess.PIPE)
         self.vol.import_data(input_proc.stdout)
+        input_proc.stdout.close()
         self.assertAllCalled()