openstack
/
keystone
Code Issues Proposed changes

Trivial: Remove app_conf kwarg from testing setup 

The RestfulTestCase classes in both rest.py and test_v3.py both
defined an app_conf kwarg in their setUp() method signatures that
was never used. This commit removes app_conf since it doesn't make
sense to define it if it's not used.

Change-Id: Ide8303363c662747e1527f7de553dba40606986e
This commit is contained in:
Lance Bragstad 2018-08-22 17:34:32 +00:00
parent 6a7220a754
commit 25a5d48d1c
2 changed files with 2 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
keystone/tests/unit/rest.py
View File

@ -52,7 +52,7 @@ class RestfulTestCase(unit.TestCase):
# default content type to test
content_type = 'json'
def setUp(self, app_conf='keystone', enable_sqlite_foreign_key=False):
def setUp(self, enable_sqlite_foreign_key=False):
super(RestfulTestCase, self).setUp()
self.auth_plugin_config_override()

3
keystone/tests/unit/test_v3.py
View File

@ -196,10 +196,9 @@ class RestfulTestCase(unit.SQLDriverOverrides, rest.RestfulTestCase,
config_files.append(unit.dirs.tests_conf('backend_sql.conf'))
return config_files
def setUp(self, app_conf='keystone', enable_sqlite_foreign_key=False):
def setUp(self, enable_sqlite_foreign_key=False):
"""Setup for v3 Restful Test Cases."""
super(RestfulTestCase, self).setUp(
app_conf=app_conf,
enable_sqlite_foreign_key=enable_sqlite_foreign_key)
self.empty_context = {'environment': {}}