Catch ValueError by Voikko.analyze

annif/analyzer/voikko.py

@@ -5,6 +5,8 @@
 
 import voikko.libvoikko
 
+from annif.exception import OperationFailedException
+
 from . import analyzer
 
 
@@ -27,7 +29,13 @@ def __getstate__(self) -> dict[str, str | None]:
     def _normalize_word(self, word: str) -> str:
         if self.voikko is None:
             self.voikko = voikko.libvoikko.Voikko(self.param)
-        result = self.voikko.analyze(word)
+        try:
+            result = self.voikko.analyze(word)
+        except ValueError as err:
+            raise OperationFailedException(
+                f"Voikko error in analysis of word '{word}'"
+            ) from err
+
         if len(result) > 0 and "BASEFORM" in result[0]:
             return result[0]["BASEFORM"]
         return word

tests/test_analyzer_voikko.py

@@ -1,8 +1,11 @@
 """Unit tests for voikko analyzer in Annif"""
 
+from unittest import mock
+
 import pytest
 
 import annif.analyzer
+from annif.exception import OperationFailedException
 
 voikko = pytest.importorskip("annif.analyzer.voikko")
 
@@ -18,3 +21,15 @@ def test_voikko_finnish_analyzer_normalize_word():
     assert analyzer._normalize_word("xyzzy") == "xyzzy"
     assert analyzer._normalize_word("vanhat") == "vanha"
     assert analyzer._normalize_word("koirien") == "koira"
+
+
+def test_voikko_analyze_valueerror():
+    analyzer = annif.analyzer.get_analyzer("voikko(fi)")
+    with mock.patch(
+        "voikko.libvoikko.Voikko.analyze",
+        side_effect=ValueError,
+    ):
+        with pytest.raises(
+            OperationFailedException, match="Voikko error in analysis of word 'kissa'"
+        ):
+            assert analyzer._normalize_word("kissa")