diff options
| author | Brant Knudson <bknudson@us.ibm.com> | 2014-01-06 18:27:29 -0600 |
|---|---|---|
| committer | Brant Knudson <bknudson@us.ibm.com> | 2014-01-10 12:43:24 -0600 |
| commit | 99dfe67a3a29f81d31c1b3de7c76f500d17c7496 (patch) | |
| tree | 659260d0d5b26d8b4cba519e1118777cb5988735 | |
| parent | e74220afa370452a696d7f63b85d27c0e0b53974 (diff) | |
| download | python-keystoneclient-99dfe67a3a29f81d31c1b3de7c76f500d17c7496.tar.gz | |
auth_token tests close temp file descriptor
The auth_token middleware tests were failing to close the
file descriptor of the named temporary file.
The fix strategy is to use tempfile.NamedTemporaryFile in a
context manager rather than mkstemp.
Change-Id: Ie20e2b4faa6a0d6bb8c4817ed2c9a04adee96f0f
| -rw-r--r-- | keystoneclient/tests/test_auth_token_middleware.py | 7 |
1 files changed, 5 insertions, 2 deletions
diff --git a/keystoneclient/tests/test_auth_token_middleware.py b/keystoneclient/tests/test_auth_token_middleware.py index 6476db5..31e7754 100644 --- a/keystoneclient/tests/test_auth_token_middleware.py +++ b/keystoneclient/tests/test_auth_token_middleware.py @@ -277,8 +277,11 @@ class BaseAuthTokenMiddlewareTest(testtools.TestCase): self.middleware = auth_token.AuthProtocol(fake_app(self.expected_env), self.conf) self.middleware._iso8601 = iso8601 - self.middleware.revoked_file_name = tempfile.mkstemp( - dir=self.middleware.signing_dirname)[1] + + with tempfile.NamedTemporaryFile(dir=self.middleware.signing_dirname, + delete=False) as f: + pass + self.middleware.revoked_file_name = f.name self.addCleanup(cleanup_revoked_file, self.middleware.revoked_file_name) |