summaryrefslogtreecommitdiff
path: root/cliapp/settings_tests.py
diff options
context:
space:
mode:
authorLars Wirzenius <liw@liw.fi>2013-10-30 11:40:53 +0000
committerLars Wirzenius <liw@liw.fi>2013-10-30 11:40:53 +0000
commit262056afcc2838dc7ac5e430744e99945d535fff (patch)
tree98ebf567c4e3f9d5716d013d6995a934bda74373 /cliapp/settings_tests.py
parent7efa8ac96f1debcfe35b4203f7b9d442dea02587 (diff)
downloadcliapp-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.py13
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):