openstack/python-cinderclient
Code Issues Proposed changes

Fix test_version_discovery test

Browse Source 
Something changed in requests/urllib3 library and now it sends
requests with url in a lower-case. It affects test_version_discovery
test because we use url in camel-case.

This patch changes test url to lower-case to be compatible with old
and new behaviours.

Change-Id: I4ae9715b124adb0a2e0115c9b00ab16576665f72
Closes-Bug: #1643874
This commit is contained in:
Ivan Kolodyazhny
2016-11-22 14:48:41 +02:00
parent c99fb20067
commit d3b6d542dd
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
cinderclient/tests/unit/fixture_data/keystone_client.py
View File

@@ -247,7 +247,7 @@ def keystone_request_callback(request, context):
context.headers["X-Subject-Token"] = token_id context.headers["X-Subject-Token"] = token_id
context.status_code = 201 context.status_code = 201
return token_data return token_data
elif "WrongDiscoveryResponse.discovery.com" in request.url: elif "wrongdiscoveryresponse.discovery.com" in request.url:
return str(WRONG_VERSION_RESPONSE) return str(WRONG_VERSION_RESPONSE)
else: else:
context.status_code = 500 context.status_code = 500

2
cinderclient/tests/unit/test_shell.py
View File

@@ -108,7 +108,7 @@ class ShellTest(utils.TestCase):
_shell = shell.OpenStackCinderShell() _shell = shell.OpenStackCinderShell()
sess = session.Session() sess = session.Session()
os_auth_url = "https://WrongDiscoveryResponse.discovery.com:35357/v2.0" os_auth_url = "https://wrongdiscoveryresponse.discovery.com:35357/v2.0"
self.register_keystone_auth_fixture(mocker, os_auth_url) self.register_keystone_auth_fixture(mocker, os_auth_url)
self.assertRaises(DiscoveryFailure, self.assertRaises(DiscoveryFailure,