diff options
author | Lars Wirzenius <liw@liw.fi> | 2013-10-30 11:40:53 +0000 |
---|---|---|
committer | Lars Wirzenius <liw@liw.fi> | 2013-10-30 11:40:53 +0000 |
commit | 262056afcc2838dc7ac5e430744e99945d535fff (patch) | |
tree | 98ebf567c4e3f9d5716d013d6995a934bda74373 /cliapp/settings_tests.py | |
parent | 7efa8ac96f1debcfe35b4203f7b9d442dea02587 (diff) | |
download | cliapp-262056afcc2838dc7ac5e430744e99945d535fff.tar.gz |
Nicely report unknown variables in config files
Previously we would just raise an ugly exception, now there's a clear
error message.
Diffstat (limited to 'cliapp/settings_tests.py')
-rw-r--r-- | cliapp/settings_tests.py | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/cliapp/settings_tests.py b/cliapp/settings_tests.py index 7a44d81..b617d50 100644 --- a/cliapp/settings_tests.py +++ b/cliapp/settings_tests.py @@ -336,6 +336,19 @@ bar = ping, pong self.assertEqual(self.settings['foo'], 'red') self.assertEqual(self.settings['bar'], ['blue', 'white']) + def test_load_configs_raises_error_for_unknown_variable(self): + + def mock_open(filename, mode=None): + return StringIO.StringIO('''\ +[config] +unknown = variable +''') + + self.assertRaises( + cliapp.UnknownConfigVariable, + self.settings.load_configs, + open=mock_open) + def test_load_configs_ignore_errors_opening_a_file(self): def mock_open(filename, mode=None): |