Make the spell check wrapper more robust, solves #933

Author: vkbo

novelwriter/core/spellcheck.py

@@ -75,17 +75,26 @@ def setLanguage(self, theLang, projectDict=None):
     def checkWord(self, theWord):
         """Wrapper function for pyenchant.
         """
-        return self.theDict.check(theWord)
+        try:
+            return self.theDict.check(theWord)
+        except Exception:
+            return True
 
     def suggestWords(self, theWord):
         """Wrapper function for pyenchant.
         """
-        return self.theDict.suggest(theWord)
+        try:
+            return self.theDict.suggest(theWord)
+        except Exception:
+            return []
 
     def addWord(self, newWord):
         """Add a word to the project dictionary.
         """
-        self.theDict.add_to_session(newWord)
+        try:
+            self.theDict.add_to_session(newWord)
+        except Exception:
+            return False
 
         if self.projectDict is not None and newWord not in self.projDict:
             newWord = newWord.strip()

tests/test_core/test_core_spell.py renamed to tests/test_core/test_core_spellcheck.py

@@ -48,6 +48,13 @@ def testCoreSpell_Enchant(monkeypatch, tmpDir):
         assert spChk.listDictionaries() == []
         assert spChk.describeDict() == ("", "")
 
+    # Break the enchant package, and check error handling
+    spChk = NWSpellEnchant()
+    spChk.theDict = None
+    assert spChk.checkWord("word") is True
+    assert spChk.suggestWords("word") == []
+    assert spChk.addWord("word") is False
+
     # Load the proper enchant package (twice)
     spChk = NWSpellEnchant()
     spChk.setLanguage("en", wList)