Skip to content

Keep reco counts report tests on mock backends - #3588

Merged
adamtheturtle merged 1 commit into
mainfrom
fix/3586-reco-count-report
Sep 9, 2026
Merged

Keep reco counts report tests on mock backends#3588
adamtheturtle merged 1 commit into
mainfrom
fix/3586-reco-count-report

Conversation

@adamtheturtle

@adamtheturtle adamtheturtle commented Sep 9, 2026

Copy link
Copy Markdown
Member

Closes #3586.

Summary

  • Keep the documented reco-count report endpoint in the mock.
  • Run its dedicated contract tests against the mock backends only.
  • Statically exclude its real-Vuforia combinations from the shared cross-cutting tests while retaining those tests on both mock backends.
  • Remove the response-triggered skip logic, so unrelated real-service failures are not hidden.
  • Document why the mock retains an endpoint which is permanently unavailable to this project's real-Vuforia test account.

Vuforia still documents the endpoint, and it previously worked against the real service. It now returns an HTML 404 for every database available to the test account, including a newly created database whose ID and server keys were read directly from Target Manager. The mock therefore keeps the useful documented behavior without pretending that this project can continue verifying it against the real service.

Validation

  • uv run --group=dev prek run --all-files
  • uv run --group=dev prek run --hook-stage pre-push --all-files
  • dedicated reco-count report tests: 50 passed
  • applicable cross-cutting reco-count cases: 48 passed, with 27 real-account combinations statically excluded

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.

Keep reco-count report tests on mock backends while the real endpoint is unavailable

1 participant