Add tests for get/setitem and KeyError, and fix getitem to raise it.

1 files changed, 6 insertions, 0 deletions

diff --git a/cliapp/settings_tests.py b/cliapp/settings_tests.py
index 788c2f8..1c035a9 100644
--- a/cliapp/settings_tests.py
+++ b/cliapp/settings_tests.py
@@ -52,6 +52,12 @@ class SettingsTests(unittest.TestCase):
     def test_does_not_have_foo_setting_by_default(self):
         self.assertFalse('foo' in self.settings)
 
+    def test_raises_keyerror_for_getting_unknown_setting(self):
+        self.assertRaises(KeyError, self.settings.__getitem__, 'foo')
+
+    def test_raises_keyerror_for_setting_unknown_setting(self):
+        self.assertRaises(KeyError, self.settings.__setitem__, 'foo', 'bar')
+
     def test_adds_string_setting(self):
         self.settings.add_string_setting(['foo'], 'foo help')
         self.assert_('foo' in self.settings)