summaryrefslogtreecommitdiff
path: root/src/leap/keymanager/tests
diff options
context:
space:
mode:
authorFolker Bernitt <fbernitt@thoughtworks.com>2015-09-21 15:59:53 +0200
committerKali Kaneko <kali@leap.se>2015-09-24 12:09:32 -0400
commit4a090d6e405415607f9c811a7961f8dc0cdd2af0 (patch)
treec27d0ffd265b63600f1712449b9e2c5f6504bd59 /src/leap/keymanager/tests
parent4e82d6ef4dc5e3bb311bf5b80b1e92d67cb0f346 (diff)
[bug] keep combined file longer in scope
In previous commit 9546348c, the combined bundle ca was not long enough in scope and was therefore deleted when it actually was used. Adopted test to check whether file is deleted.
Diffstat (limited to 'src/leap/keymanager/tests')
-rw-r--r--src/leap/keymanager/tests/test_keymanager.py8
1 files changed, 5 insertions, 3 deletions
diff --git a/src/leap/keymanager/tests/test_keymanager.py b/src/leap/keymanager/tests/test_keymanager.py
index 7b90ae1..b2722b2 100644
--- a/src/leap/keymanager/tests/test_keymanager.py
+++ b/src/leap/keymanager/tests/test_keymanager.py
@@ -20,7 +20,7 @@
Tests for the Key Manager.
"""
-
+from os import path
from datetime import datetime
import tempfile
from leap.common import ca_bundle
@@ -369,8 +369,7 @@ class KeyManagerKeyManagementTestCase(KeyManagerWithSoledadTestCase):
@inlineCallbacks
def test_fetch_uses_combined_ca_bundle_otherwise(self):
- with tempfile.NamedTemporaryFile() as tmp_input, \
- tempfile.NamedTemporaryFile() as tmp_output:
+ with tempfile.NamedTemporaryFile() as tmp_input, tempfile.NamedTemporaryFile(delete=False) as tmp_output:
ca_content = 'some\ncontent\n'
ca_cert_path = tmp_input.name
self._dump_to_file(ca_cert_path, ca_content)
@@ -390,6 +389,9 @@ class KeyManagerKeyManagementTestCase(KeyManagerWithSoledadTestCase):
expected = self._slurp_file(ca_bundle.where()) + ca_content
self.assertEqual(expected, self._slurp_file(tmp_output.name))
+ del km # force km out of scope
+ self.assertFalse(path.exists(tmp_output.name))
+
def _dump_to_file(self, filename, content):
with open(filename, 'w') as out:
out.write(content)