Update test suite

textbook · Apr 17, 2016 · f6cc575 · f6cc575
1 parent 1fd8fd6
commit f6cc575
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 11 deletions.
diff --git a/flash/services/__init__.py b/flash/services/__init__.py
@@ -37,7 +37,7 @@ def define_services(config):
     for settings in config:
         name = settings['name']
         if name not in SERVICES:
-            logger.warning('unknown service {!r}'.format(name))
+            logger.warning('unknown service %r', name)
             continue
         services[uuid4().hex] = SERVICES[name].from_config(**settings)
     return services
diff --git a/tests/services/test_define_service.py b/tests/services/test_define_service.py
diff --git a/tests/test_utility_functions.py b/tests/test_utility_functions.py
@@ -12,7 +12,12 @@
 def test_parse_config_from_env(logger, getenv, define_services):
     result = parse_config()
 
-    assert result == {'name': 'foo', 'services': define_services.return_value}
+    assert result == {
+        'name': 'foo',
+        'project_name': 'unnamed',
+        'services': define_services.return_value,
+        'style': 'default',
+    }
     getenv.assert_called_once_with('FLASH_CONFIG')
     logger.info.assert_called_once_with(
         'loading configuration from environment',
@@ -27,7 +32,12 @@ def test_parse_config_from_env(logger, getenv, define_services):
 def test_parse_config_from_file(logger, getenv, define_services):
     result = parse_config()
 
-    assert result == {'name': 'foo', 'services': define_services.return_value}
+    assert result == {
+        'name': 'foo',
+        'project_name': 'unnamed',
+        'services': define_services.return_value,
+        'style': 'default',
+    }
     getenv.assert_called_once_with('FLASH_CONFIG')
     logger.info.assert_called_once_with('loading configuration from file')
     define_services.assert_called_once_with([])