WebSearch: fix bare imports in test_filetable for dotted-path loading

[eduralph]

Summary

WebSearch/tests/test_filetable.py (added in WebSearch 1.10.7, commit e3750a0) imports models, constants and db_file_table without a package prefix:

from models import DBFileTableConfig
from constants import DuplicateHandlingMode, DB_FILE_TABLE_DIR
from db_file_table import DBFileTable, DuplicateEntryError

Those resolve only when WebSearch/ itself is on sys.path — i.e. when the test is loaded via unittest discover from inside tests/. Under the dotted-path invocation that fork CI's unit-test-linux job uses (python3 -m unittest WebSearch.tests.test_filetable from the addons-source root, with PYTHONPATH=.), the imports look for a top-level models module and the test module fails to load:

ImportError: Failed to import test module: test_filetable
ModuleNotFoundError: No module named 'models'

This is the same package-shadowing class of bug as 0012691 — the dotted-path loader is what surfaces it; discover from tests/ hides it.

Fix

Add the sys.path.insert(0, …parent dir…) prologue that TMGimporter/tests/test_libtmg.py and Form/tests/test_form_validator.py already use for sibling-module imports — the established in-tree convention for this exact case. No behavioural change beyond the imports.

Verification

• Repro: cd addons-source && PYTHONPATH=. python3 -m unittest -v WebSearch.tests.test_filetable -> ImportError / ModuleNotFoundError.
• After this patch: Ran 28 tests in 0.008s -- OK.

Reproduced & verified in a sibling testbed (gramps-testbed) that runs every addon's tests/test_*.py via the dotted-path form, which is how this trap surfaces in the first place.

Remarks

This is one of the fixes required for the CI PR I've proposed to run clean on the repos.

🤖 Generated with Claude Code

[GaryGriffin]

Added your fix of test_filetable.py to branch 6.1 fixes for unittest - #832

I think this PR should just be a single file change, I suspect you need to rebase to gramps60 to separate out PR 820 and this change.

Please review PR 832 for the changes for 6.1 . I am not too familiar with unittest and would appreciate the review.

[eduralph]

Thanks @GaryGriffin — rebased onto gramps-project:maintenance/gramps60 so the diff now contains only WebSearch/tests/test_filetable.py (+11/−1). Apologies for the noise; the original head branch was based on top of a fork-local merge of the proposed CI/CD work (PR 820's predecessor), which is what dragged everything else into the diff.

Happy to review #832 — I'll leave specific comments there. One thing to flag in advance: in #832's WebSearch/tests/test_filetable.py the existing import unittest is removed from the top of the file, and the new gi.require_version block above it raises unittest.SkipTest in its except branch — but unittest is only re-imported lower down. So if gi (or the GTK 3.0 require) ever fails, the except branch hits NameError: name 'unittest' is not defined instead of cleanly skipping. The other five files in #832 don't have this problem because their original import unittest at the top is preserved.

I'll detail it as a review on #832 with a concrete suggested fix.