openstack/python-keystoneclient
Code Issues Proposed changes

Merge "Don't use private last_request variable"

Browse Source
This commit is contained in:
Jenkins
2014-02-17 04:53:17 +00:00
committed by Gerrit Code Review
parent 8b0c5eea3c d0f129637f
commit 43f8870f66
2 changed files with 13 additions and 13 deletions
Download Patch File Download Diff File Expand all files Collapse all files

24
keystoneclient/tests/test_auth_token_middleware.py
View File

@@ -289,9 +289,9 @@ class BaseAuthTokenMiddlewareTest(testtools.TestCase):
def assertLastPath(self, path):
if path:
self.assertEqual(path, httpretty.httpretty.last_request.path)
self.assertEqual(path, httpretty.last_request().path)
else:
self.assertIsInstance(httpretty.httpretty.last_request,
self.assertIsInstance(httpretty.last_request(),
httpretty.core.HTTPrettyRequestEmpty)
if tuple(sys.version_info)[0:2] < (2, 7):
@@ -369,7 +369,7 @@ class DiabloAuthTokenMiddlewareTest(BaseAuthTokenMiddlewareTest,
super(DiabloAuthTokenMiddlewareTest, self).setUp(
expected_env=expected_env)
httpretty.httpretty.reset()
httpretty.reset()
httpretty.enable()
self.addCleanup(httpretty.disable)
@@ -1106,7 +1106,7 @@ class CertDownloadMiddlewareTest(BaseAuthTokenMiddlewareTest,
self.assertEqual(f.read(), data)
self.assertEqual("/testadmin/v2.0/certificates/signing",
httpretty.httpretty.last_request.path)
httpretty.last_request().path)
def test_fetch_signing_ca(self):
data = 'FAKE CA'
@@ -1119,7 +1119,7 @@ class CertDownloadMiddlewareTest(BaseAuthTokenMiddlewareTest,
self.assertEqual(f.read(), data)
self.assertEqual("/testadmin/v2.0/certificates/ca",
httpretty.httpretty.last_request.path)
httpretty.last_request().path)
def test_prefix_trailing_slash(self):
self.conf['auth_admin_prefix'] = '/newadmin/'
@@ -1136,12 +1136,12 @@ class CertDownloadMiddlewareTest(BaseAuthTokenMiddlewareTest,
self.middleware.fetch_ca_cert()
self.assertEqual('/newadmin/v2.0/certificates/ca',
httpretty.httpretty.last_request.path)
httpretty.last_request().path)
self.middleware.fetch_signing_cert()
self.assertEqual('/newadmin/v2.0/certificates/signing',
httpretty.httpretty.last_request.path)
httpretty.last_request().path)
def test_without_prefix(self):
self.conf['auth_admin_prefix'] = ''
@@ -1158,12 +1158,12 @@ class CertDownloadMiddlewareTest(BaseAuthTokenMiddlewareTest,
self.middleware.fetch_ca_cert()
self.assertEqual('/v2.0/certificates/ca',
httpretty.httpretty.last_request.path)
httpretty.last_request().path)
self.middleware.fetch_signing_cert()
self.assertEqual('/v2.0/certificates/signing',
httpretty.httpretty.last_request.path)
httpretty.last_request().path)
def network_error_response(method, uri, headers):
@@ -1206,7 +1206,7 @@ class v2AuthTokenMiddlewareTest(BaseAuthTokenMiddlewareTest,
'revoked_token_hash': self.examples.REVOKED_TOKEN_HASH
}
httpretty.httpretty.reset()
httpretty.reset()
httpretty.enable()
self.addCleanup(httpretty.disable)
@@ -1338,7 +1338,7 @@ class CrossVersionAuthTokenMiddlewareTest(BaseAuthTokenMiddlewareTest,
self.assertEqual(self.response_status, 200)
self.assertEqual("/testadmin/v2.0/tokens/%s" %
self.examples.UUID_TOKEN_DEFAULT,
httpretty.httpretty.last_request.path)
httpretty.last_request().path)
class v3AuthTokenMiddlewareTest(BaseAuthTokenMiddlewareTest,
@@ -1388,7 +1388,7 @@ class v3AuthTokenMiddlewareTest(BaseAuthTokenMiddlewareTest,
'revoked_token_hash': self.examples.REVOKED_v3_TOKEN_HASH
}
httpretty.httpretty.reset()
httpretty.reset()
httpretty.enable()
self.addCleanup(httpretty.disable)

2
keystoneclient/tests/test_discovery.py
View File

@@ -316,7 +316,7 @@ class ClientDiscoveryTests(utils.TestCase):
ip = '192.168.1.1'
self.assertCreatesV3(auth_url=BASE_URL, original_ip=ip)
self.assertThat(httpretty.httpretty.last_request.headers['forwarded'],
self.assertThat(httpretty.last_request().headers['forwarded'],
matchers.Contains(ip))
def test_discover_bad_args(self):