diff options
| author | Davanum Srinivas <dims@linux.vnet.ibm.com> | 2013-04-23 09:54:22 -0400 |
|---|---|---|
| committer | Davanum Srinivas <dims@linux.vnet.ibm.com> | 2013-05-20 21:05:22 -0400 |
| commit | 4ff33b0390fff7e623be7c2242002e32a47eb855 (patch) | |
| tree | 5b126191bc62466fa1f55768b2dc1f345172a39a /tests/unit/db/sqlalchemy | |
| parent | 97bb81ddbcc47343c78e0a6efe724878fcb35ecb (diff) | |
| download | oslo-4ff33b0390fff7e623be7c2242002e32a47eb855.tar.gz oslo-4ff33b0390fff7e623be7c2242002e32a47eb855.tar.xz oslo-4ff33b0390fff7e623be7c2242002e32a47eb855.zip | |
Specify database group instead of DEFAULT
At the request of Quantum folks, Let us switch from DEFAULT to
database for the database related options. This will help with
migration etc.
DocImpact
Fixes LP# 1171837
Change-Id: If602a6a7cc0f2a202632dd14574fea60dce4b589
Diffstat (limited to 'tests/unit/db/sqlalchemy')
| -rw-r--r-- | tests/unit/db/sqlalchemy/test_sqlalchemy.py | 45 |
1 files changed, 45 insertions, 0 deletions
diff --git a/tests/unit/db/sqlalchemy/test_sqlalchemy.py b/tests/unit/db/sqlalchemy/test_sqlalchemy.py index 7cbe31d..b18825a 100644 --- a/tests/unit/db/sqlalchemy/test_sqlalchemy.py +++ b/tests/unit/db/sqlalchemy/test_sqlalchemy.py @@ -37,6 +37,51 @@ class TmpTable(BASE, models.ModelBase): foo = Column(Integer) +class SessionParametersTestCase(test_utils.BaseTestCase): + + def test_deprecated_session_parameters(self): + paths = self.create_tempfiles([('test', """[DEFAULT] +sql_connection=x://y.z +sql_min_pool_size=10 +sql_max_pool_size=20 +sql_max_retries=30 +sql_retry_interval=40 +sql_max_overflow=50 +sql_connection_debug=60 +sql_connection_trace=True +""")]) + test_utils.CONF(['--config-file', paths[0]]) + self.assertEquals(test_utils.CONF.database.connection, 'x://y.z') + self.assertEquals(test_utils.CONF.database.min_pool_size, 10) + self.assertEquals(test_utils.CONF.database.max_pool_size, 20) + self.assertEquals(test_utils.CONF.database.max_retries, 30) + self.assertEquals(test_utils.CONF.database.retry_interval, 40) + self.assertEquals(test_utils.CONF.database.max_overflow, 50) + self.assertEquals(test_utils.CONF.database.connection_debug, 60) + self.assertEquals(test_utils.CONF.database.connection_trace, True) + + def test_session_parameters(self): + paths = self.create_tempfiles([('test', """[database] +connection=x://y.z +min_pool_size=10 +max_pool_size=20 +max_retries=30 +retry_interval=40 +max_overflow=50 +connection_debug=60 +connection_trace=True +""")]) + test_utils.CONF(['--config-file', paths[0]]) + self.assertEquals(test_utils.CONF.database.connection, 'x://y.z') + self.assertEquals(test_utils.CONF.database.min_pool_size, 10) + self.assertEquals(test_utils.CONF.database.max_pool_size, 20) + self.assertEquals(test_utils.CONF.database.max_retries, 30) + self.assertEquals(test_utils.CONF.database.retry_interval, 40) + self.assertEquals(test_utils.CONF.database.max_overflow, 50) + self.assertEquals(test_utils.CONF.database.connection_debug, 60) + self.assertEquals(test_utils.CONF.database.connection_trace, True) + + class SessionErrorWrapperTestCase(test_utils.BaseTestCase): def setUp(self): super(SessionErrorWrapperTestCase, self).setUp() |
