Skip to content

Commit fe3c728

Browse files
authored
refactor(search): decompose 520-line monolithic search handler into focused per-source functions (closes #99) (#102)
* fix: initial implmentation * fix: typecheck error * fix: add test for search helpers * fix; review findings * fix: nitpack and outside diff comments * fix: Do not expose raw exception text in API warnings.
1 parent 937dc91 commit fe3c728

6 files changed

Lines changed: 1214 additions & 490 deletions

File tree

.github/workflows/tests.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -114,7 +114,7 @@ jobs:
114114
# Pytest fixtures (tests/conftest.py) build a temp workspaceStorage and
115115
# exercise Flask routes via app.test_client(). Only listed files — not
116116
# `pytest tests/` — to avoid re-collecting unittest.TestCase classes above.
117-
run: python -m pytest tests/test_api_endpoints.py tests/test_pdf_export.py -v --tb=short
117+
run: python -m pytest tests/test_api_endpoints.py tests/test_pdf_export.py tests/test_search_helpers.py -v --tb=short
118118

119119
# ── PyInstaller desktop build (Windows only, once per workflow) ────────
120120
# Closes #44. Builds the onedir bundle and smoke-tests --help so the

0 commit comments

Comments
 (0)