Do not supply default settings, let the component supply defaults

diff --git a/src/pgwui_server/__init__.py b/src/pgwui_server/__init__.py
index 57cbbb6e7bae12e6180f511389748f3b02974fc8..68e43e80d10a7088921b2c16b8691063dd75bc04 100644 (file)
--- a/src/pgwui_server/__init__.py
+++ b/src/pgwui_server/__init__.py
@@ -126,12 +126,6 @@ def validate_setting_values(settings):
     boolean_setting('pgwui.validate_hmac', settings)
 
 
-def supply_default_settings(settings):
-    '''Supply sensible defaults for omitted settings.
-    '''
-    settings['pgwui.default_db'] = ''
-
-
 def do_validate_hmac(settings):
     '''True unless the user has specificly rejected hmac validation
     '''
@@ -157,7 +151,6 @@ def validate_settings(settings):
     for key in settings.keys():
         abort_on_bad_setting(key)
     validate_setting_values(settings)
-    supply_default_settings(settings)
     validate_hmac(settings)
 
 

diff --git a/tests/test___init__.py b/tests/test___init__.py
index 469989d8a63e7634cd8bb97acbbd5970e781a8c2..b7588e8b28688072479cc943a5cafb50a71d8e73 100644 (file)
--- a/tests/test___init__.py
+++ b/tests/test___init__.py
@@ -143,12 +143,6 @@ def test_validate_setting_values(monkeypatch):
     assert boolean_setting_called
 
 
-# supply_default_settings()
-def test_supply_default_settings():
-    '''Something is changed in the supplied settings'''
-    assert {} != pgwui_server_init.supply_default_settings({})
-
-
 # do_validate_hmac()
 
 def test_do_validate_hmac_none():
@@ -218,8 +212,6 @@ def test_validate_settings(monkeypatch):
                         mock_abort_on_bad_setting)
     monkeypatch.setattr(pgwui_server_init, 'validate_setting_values',
                         lambda *args: None)
-    monkeypatch.setattr(pgwui_server_init, 'supply_default_settings',
-                        lambda arg: arg)
     monkeypatch.setattr(pgwui_server_init, 'validate_hmac',
                         lambda *args: None)
     settings = {'key1': 'value1',