Skip to content

Supply recognition count seeding through a backend fixture - #3743

Merged
adamtheturtle merged 1 commit into
mainfrom
decouple-recognition-fixture-20261013
Oct 13, 2026
Merged

adamtheturtle merged 1 commit into
mainfrom
decouple-recognition-fixture-20261013

Conversation

@adamtheturtle

Copy link
Copy Markdown
Member

Shared target-summary and recognition-report tests currently pass a concrete backend identity to a seeding helper that imports backend setup. Supply seeding through a typed seed_recognition_counts pytest fixture instead, so the call sites only request the operation they need. Backend setup supplies either the running in-memory mock's operation or the target-manager HTTP request.

This four-file change follows #3742 and advances #3738, part of #3737. Further backend decoupling remains in #3738.

Validation:

  • Python 3.15: all 60 recognition-report and seeded-summary tests pass across the in-memory and Flask backends, using synthetic credentials with retries disabled.
  • The new fixture's executable lines and both backend branches are covered. The operation protocol and both changed test modules have 100% coverage in the targeted run.
  • Pre-commit and pre-push hooks pass, including CI partition validation, Mypy, Pyright, public type completeness, Ty, and Pyrefly. Pylint passes for all changed files, and git diff --check passes.

The local run is targeted; full-suite coverage and real-Vuforia validation remain for hosted CI.

@adamtheturtle
adamtheturtle merged commit dd45e86 into main Oct 13, 2026
11 checks passed
@adamtheturtle
adamtheturtle deleted the decouple-recognition-fixture-20261013 branch October 13, 2026 11:40

This branch was successfully deployed

1 active deployment
development — b3ce988e Deployed Oct 13, 2026 by adamtheturtle via pages #2369
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.

1 participant