openstack
/
keystone
Code Issues Proposed changes

Fix test_admin to expect admin endpoint 

This fix makes test_admin expect admin endpoint in a response preserving
the correct behaviour for test_public

Change-Id: I2769eddb94918e16d3589d6237841c961eab435a
Closes-Bug: #1478629
This commit is contained in:
Alexey Miroshkin 2015-07-28 03:59:49 -07:00
parent 37b81c95ed
commit 5b6968d0af
1 changed files with 7 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
keystone/tests/unit/test_versions.py
View File

@ -956,6 +956,11 @@ class VersionSingleAppTestCase(tests.TestCase):
link['href'] = port
def _test_version(self, app_name):
def app_port():
if app_name == 'admin':
return CONF.eventlet_server.admin_port
else:
return CONF.eventlet_server.public_port
app = self.loadapp('keystone', app_name)
client = tests.TestClient(app)
resp = client.get('/')
@ -965,18 +970,15 @@ class VersionSingleAppTestCase(tests.TestCase):
for version in expected['versions']['values']:
if version['id'].startswith('v3'):
self._paste_in_port(
version, 'http://localhost:%s/v3/' %
CONF.eventlet_server.public_port)
version, 'http://localhost:%s/v3/' % app_port())
elif version['id'] == 'v2.0':
self._paste_in_port(
version, 'http://localhost:%s/v2.0/' %
CONF.eventlet_server.public_port)
version, 'http://localhost:%s/v2.0/' % app_port())
self.assertThat(data, _VersionsEqual(expected))
def test_public(self):
self._test_version('main')
@utils.wip('waiting on bug #1381961')
def test_admin(self):
self._test_version('admin')