fix module error and test optional dependencies (#142)

* fix error and test optional dependencies

* fix tests

* fix coverage

.coveragerc

@@ -10,3 +10,4 @@ omit =
 exclude_lines =
     pragma: no cover
     if __name__ == .__main__.:
+    except .*ImportError.*:

.github/workflows/tests.yml

@@ -18,6 +18,7 @@ jobs:
       matrix:
         os: [ubuntu-latest]
         python-version: ['3.8', '3.9', '3.10', '3.11', '3.12', '3.13-dev']
+        env: [{ MINIMAL: "true" }, { MINIMAL: "false" }]
         include:
         # common versions on MacOS
         - os: macos-latest
@@ -55,8 +56,12 @@ jobs:
     # package setup
     - uses: actions/checkout@v4
 
-    - name: Install package and dependencies
-      run: pip install .[dev]
+    - name: Install dependencies
+      run: python -m pip install ".[dev]"
+
+    - name: Install full dependencies
+      if: ${{ matrix.env.MINIMAL == 'false'}}
+      run: python -m pip install --ignore-installed ".[marisa-trie]"
 
     # tests
     - name: Lint with flake8
@@ -77,7 +82,7 @@ jobs:
 
     # coverage with default version
     - name: Upload coverage to Codecov
-      if: matrix.python-version == '3.11'
+      if: ${{ matrix.env.MINIMAL == 'false' && matrix.python-version == '3.11' }}
       uses: codecov/codecov-action@v4
       env:
         CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}

pyproject.toml

@@ -104,7 +104,6 @@ marisa-trie = [
     "platformdirs == 4.2.2",
 ]
 test = [
-    "simplemma[marisa-trie]",
     "pytest == 8.3.2",
     "pytest-cov == 5.0.0",
 ]

simplemma/strategies/dictionaries/trie_directory_factory.py

@@ -4,8 +4,15 @@
 from pathlib import Path
 from typing import List, Mapping, Optional
 
-from marisa_trie import BytesTrie, HUGE_CACHE  # type: ignore[import-not-found]
-from platformdirs import user_cache_dir
+try:
+    from marisa_trie import BytesTrie, HUGE_CACHE  # type: ignore[import-not-found]
+    from platformdirs import user_cache_dir
+except ImportError:
+
+    class BytesTrie:  # type: ignore[no-redef]
+        def __init__(self):
+            raise ImportError("marisa_trie and platformdirs packages not installed")
+
 
 from simplemma import __version__ as SIMPLEMMA_VERSION
 from simplemma.strategies.dictionaries.dictionary_factory import (

tests/strategies/dictionaries/test_trie_dictionary_factory.py

@@ -4,12 +4,22 @@
 from unittest.mock import call, patch
 
 import pytest
-from marisa_trie import BytesTrie  # type: ignore[import-not-found]
+
+try:
+    from marisa_trie import BytesTrie  # type: ignore[import-not-found]
+
+    HAS_MARISA = True
+except ImportError:
+    HAS_MARISA = False
 
 from simplemma.strategies.dictionaries.trie_directory_factory import TrieWrapDict
 from simplemma.strategies import TrieDictionaryFactory
 
 
+if not HAS_MARISA:
+    pytest.skip("skipping marisa-trie tests", allow_module_level=True)
+
+
 def test_exceptions() -> None:
     # missing languages or faulty language codes
     dictionary_factory = TrieDictionaryFactory(use_disk_cache=False)