Reorganize Babel Validation for more extensibility

[gaurav]

Pure-ish refactor, no service-behavior change. Moves the code under tests/common/ into an importable src/babel_validation package and updates the Google Sheet test modules to match.

What changes

• Split the monolithic tests/common/google_sheet_test_cases.py into:
  • core/testrow.py — TestRow / TestStatus / TestResult
  • services/{nodenorm,nameres}.py — CachedNodeNorm / CachedNameRes
  • sources/google_sheets/{google_sheet_test_cases,blocklist}.py
• Gsheet test modules now import from the new locations and parametrize lazily in pytest_generate_tests via tests/_pytest_helpers.deselected_by_markexpr, so marker-deselected runs never hit the network.
• Migrate tests/pytest.ini into [tool.pytest.ini_options]; add a hatchling build that packages src/; add filelock (gsheet disk cache).
• Move test_env.py → tests/test_environment/.

Two incidental one-liners ride along in the final test files (NameRes biolink_type passed as a list; description identifiers as lists not sets) — kept as-is so #67 rebases cleanly.

Verify

• uv run pytest -m unit --collect-only -q — full suite imports/collects offline (no network).
• uv run pytest tests/test_environment/test_env.py — live Google Sheet download+parse through the moved code.

🤖 Generated with Claude Code

[Copilot]

Pull request overview

Refactors shared Google Sheet–backed test utilities into an importable src/babel_validation library package, updates existing test modules to use the new import paths, and makes Google Sheet/blocklist parametrization lazy so marker-deselected runs avoid network access.

Changes:

• Carves Google Sheet parsing + shared datatypes into src/babel_validation (core/services/sources) and updates tests to import from the new locations.
• Adds lazy parametrization via pytest_generate_tests + tests/_pytest_helpers.deselected_by_markexpr to avoid collection-time network fetches for deselected runs.
• Adds disk caching + file lock for Google Sheet CSV downloads; migrates pytest config into pyproject.toml and adds hatchling wheel build configuration.

Reviewed changes

Copilot reviewed 14 out of 23 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
uv.lock	Adds filelock and updates lock metadata for editable local package.
pyproject.toml	Adds filelock, hatchling build config, and moves pytest settings under [tool.pytest.ini_options].
tests/pytest.ini	Removes legacy pytest.ini in favor of pyproject.toml configuration.
tests/conftest.py	Deletes tempdir Google Sheet CSV cache on run start (controller-only) and keeps existing pytest configuration/fixtures.
tests/_pytest_helpers.py	Adds helper to pre-evaluate -m mark expressions to skip network-backed parametrization when deselected.
tests/init.py	Establishes tests as a package (enabling imports like tests._pytest_helpers).
tests/test_environment/test_env.py	Updates import to new Google Sheet test-case module location.
tests/nodenorm/test_nodenorm_from_gsheet.py	Switches to lazy Google Sheet parametrization and new import path.
tests/nodenorm/test_nodenorm_descriptions.py	Changes description identifier collections from sets to lists.
tests/nameres/test_nameres_from_gsheet.py	Switches to lazy Google Sheet parametrization, new import path, and sends biolink_type as a list.
tests/nameres/test_blocklist.py	Switches blocklist loading to lazy parametrization and new import path.
src/init.py	Establishes src as a package so src.babel_validation.* imports work.
src/babel_validation/init.py	Initializes the new library package namespace.
src/babel_validation/core/init.py	Initializes core subpackage.
src/babel_validation/core/testrow.py	Introduces TestRow, TestStatus, and TestResult in the library.
src/babel_validation/services/init.py	Initializes services subpackage.
src/babel_validation/services/nodenorm.py	Adds CachedNodeNorm wrapper with in-process caching for NodeNorm calls.
src/babel_validation/services/nameres.py	Adds CachedNameRes wrapper with in-process caching for NameRes calls.
src/babel_validation/sources/init.py	Initializes sources subpackage.
src/babel_validation/sources/google_sheets/init.py	Initializes Google Sheets source subpackage.
src/babel_validation/sources/google_sheets/google_sheet_test_cases.py	Implements tempdir CSV cache + file lock and uses extracted TestRow.
src/babel_validation/sources/google_sheets/blocklist.py	Adds blocklist Google Sheet loader + dataclass representation.
CLAUDE.md	Updates architecture/docs to reflect new library layout and import paths.

Comments suppressed due to low confidence (2)

src/babel_validation/sources/google_sheets/google_sheet_test_cases.py:18

• The library module is using an absolute import through the top-level src package. Using a relative import here avoids coupling internal modules to the packaging layout (and makes the planned rename away from src.babel_validation easier), while still working with the current structure.
  src/babel_validation/sources/google_sheets/google_sheet_test_cases.py:42
• The cache filename uses only the first 8 hex chars of an MD5, which makes collisions plausible across different Google Sheet IDs; a collision would cause one sheet’s CSV to be reused for another sheet ID (incorrect test data). Using a longer/stronger hash (or the full digest) removes this risk with negligible downside.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.