Skip to content

Cherry-pick test suite from tests branch into dev - #188

Merged
tkswanson merged 2 commits into
mainfrom
copilot/research-repository-structure
Aug 24, 2026
Merged

Cherry-pick test suite from tests branch into dev#188
tkswanson merged 2 commits into
mainfrom
copilot/research-repository-structure

Conversation

Copilot AI commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

The tests branch contained a complete database-layer test suite (108 tests) and CI workflow that had not been merged into dev. This brings those commits forward.

Changes

  • Test suite (matchypatchy_package/tests/): 108 pytest tests covering database/setup.py, database/mpdb.py, database/media.py, and database/location.py
  • conftest.py: Stubs out PyQt6, cv2, and related modules at collection time so DB-layer tests run headless without a display server; includes _VersionMock to handle PyQt6 version integer comparisons; provides tmp_db and populated_db fixtures
  • pyproject.toml: Added [tool.pytest.ini_options] with testpaths = ["tests"]
  • CI workflow (.github/workflows/tests.yml): Runs pytest with pytest-cov on push/PR to main; uploads coverage to Codecov
  • 3 source bug fixes in location.py, media.py, and mpdb.py that were caught while writing the tests

Note: mpdb.py had a cherry-pick conflict — the tests branch was adding thread-local chroma/collection properties that already exist on dev from a later PR. HEAD was kept.

Copilot AI and others added 2 commits August 24, 2026 18:23
Co-authored-by: tkswanson <4371698+tkswanson@users.noreply.github.com>
Co-authored-by: tkswanson <4371698+tkswanson@users.noreply.github.com>
@tkswanson
tkswanson marked this pull request as ready for review August 24, 2026 18:30
@tkswanson
tkswanson merged commit d0bf50a into main Aug 24, 2026
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants