steno_dictionary: don't allow clear on a read-only dictionary

openstenoproject · Apr 29, 2021 · 03c4915 · 03c4915
1 parent 3e7fae6
commit 03c4915
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 0 deletions.
diff --git a/plover/steno_dictionary.py b/plover/steno_dictionary.py
@@ -96,6 +96,7 @@ def __getitem__(self, key):
         return self._dict.__getitem__(key)
 
     def clear(self):
+        assert not self.readonly
         self._dict.clear()
         self.reverse.clear()
         self.casereverse.clear()

diff --git a/plover_build_utils/testing/steno_dictionary.py b/plover_build_utils/testing/steno_dictionary.py
@@ -77,6 +77,7 @@ def fake_asset_only(p, r, v=None):
     @pytest.mark.parametrize('method_name, args', (
         ('__delitem__', (VALID_KEY,)),
         ('__setitem__', (VALID_KEY, 'pouet!')),
+        ('clear'      , ()),
         ('save'       , ()),
         ('update'     , ()),
     ))