Permalink
Browse files

renamed option to a standard name

  • Loading branch information...
1 parent b5f3f0b commit fb678dd69115721c0dce587d86b8e59429c7dded @twobraids twobraids committed Mar 25, 2013
@@ -402,10 +402,10 @@
# converter: configman.converters.timedelta_converter
check_in_frequency='00:00:01:00'
- # name: database
+ # name: database_class
# doc: the class of the registrar's database
# converter: configman.converters.class_converter
- database='socorro.external.postgresql.connection_context.ConnectionContext'
+ database_class='socorro.external.postgresql.connection_context.ConnectionContext'
# name: database_host
# doc: the hostname of the database
@@ -29,7 +29,7 @@ class RegistrationError(Exception):
class ProcessorAppRegistrationClient(RequiredConfig):
required_config = Namespace()
required_config.add_option(
- 'database',
+ 'database_class',
doc="the class of the registrar's database",
default=ConnectionContext,
from_string_converter=class_converter
@@ -65,7 +65,7 @@ def __init__(self, config, quit_check_callback=None):
logger - a logger object"""
self.config = config
- self.database = config.database(config)
+ self.database = config.database_class(config)
self.transaction = config.transaction_executor_class(
config,
self.database,
@@ -45,7 +45,7 @@ def test_basic_setup(self):
app_description='app description',
values_source_list=[{
'logger': mock_logging,
- 'database': mock_postgres
+ 'database_class': mock_postgres
}]
)
m_registration = mock.Mock()
@@ -74,7 +74,7 @@ def test_checkin_done(self):
app_description='app description',
values_source_list=[{
'logger': mock_logging,
- 'database': mock_postgres
+ 'database_class': mock_postgres
}]
)
m_registration = mock.Mock()
@@ -125,7 +125,7 @@ def test_checkin_not_necessary(self):
app_description='app description',
values_source_list=[{
'logger': mock_logging,
- 'database': mock_postgres
+ 'database_class': mock_postgres
}]
)
m_registration = mock.Mock()
@@ -173,7 +173,7 @@ def test_requested_processor_id(self):
app_description='app description',
values_source_list=[{
'logger': mock_logging,
- 'database': mock_postgres
+ 'database_class': mock_postgres
}]
)
m_registration = mock.Mock()
@@ -230,7 +230,7 @@ def test_select_host_mode_success(self):
app_description='app description',
values_source_list=[{
'logger': mock_logging,
- 'database': mock_postgres,
+ 'database_class': mock_postgres,
'processor_id': 'host',
}]
)
@@ -299,7 +299,7 @@ def test_select_forcehost_mode_success(self):
app_description='app description',
values_source_list=[{
'logger': mock_logging,
- 'database': mock_postgres,
+ 'database_class': mock_postgres,
'processor_id': 'forcehost',
}]
)
@@ -370,7 +370,7 @@ def test_select_host_mode_not_found_start_new(self):
app_description='app description',
values_source_list=[{
'logger': mock_logging,
- 'database': mock_postgres,
+ 'database_class': mock_postgres,
'processor_id': 'host',
}]
)
@@ -445,7 +445,7 @@ def test_select_forcehost_mode_not_found_start_new(self):
app_description='app description',
values_source_list=[{
'logger': mock_logging,
- 'database': mock_postgres,
+ 'database_class': mock_postgres,
'processor_id': 'forcehost',
}]
)
@@ -517,7 +517,7 @@ def test_select_host_mode_not_dead_fail(self):
app_description='app description',
values_source_list=[{
'logger': mock_logging,
- 'database': mock_postgres,
+ 'database_class': mock_postgres,
'processor_id': 'host',
}]
)

0 comments on commit fb678dd

Please sign in to comment.