Skip to content

Supply prepared request URLs through backend fixtures - #3746

Merged
adamtheturtle merged 1 commit into
mainfrom
shared-prepared-request-urls-20261013
Oct 13, 2026
Merged

adamtheturtle merged 1 commit into
mainfrom
shared-prepared-request-urls-20261013

Conversation

@adamtheturtle

Copy link
Copy Markdown
Member

Shared endpoint builders still hardcode the Vuforia service URLs even though shared SDK clients now consume backend URL fixtures. Pass base_vws_url or base_vwq_url into all eleven prepared endpoint fixtures. Update the missing Authorization/Date header assertions to identify the query API by its configured URL, so those tests work when the services share a hostname and use different ports.

This three-file follow-up to #3745 advances #3738, part of #3737. Further URL and backend decoupling remains in #3738.

Validation:

  • Python 3.15: all 44 targeted missing-header cases pass across the in-memory and Flask backends, using synthetic credentials with retries disabled. Twenty-two real-Vuforia cases are skipped locally.
  • Reused the same tests from a separate consumer harness with MockVWS configured at http://127.0.0.1:5006 and http://127.0.0.1:5007: all 22 endpoint/header cases pass. The three changed files have 100% statement and branch coverage across the targeted runs.
  • 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.

Full-suite coverage and real-Vuforia validation remain for hosted CI.

@adamtheturtle
adamtheturtle enabled auto-merge (squash) October 13, 2026 12:13

This branch was successfully deployed

2 active deployments
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