Harden real Vuforia tests against transient failures and exhausted allowance - #3527
Merged
adamtheturtle merged 7 commits intoSep 5, 2026
Merged
Conversation
adamtheturtle
force-pushed
the
adamtheturtle/salvage-model-target-ci-hardening
branch
from
September 5, 2026 00:32
ca54b45 to
02e792e
Compare
adamtheturtle
deleted the
adamtheturtle/salvage-model-target-ci-hardening
branch
September 5, 2026 00:33
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Recovers seven commits of real-Vuforia test hardening that were stranded on the
dependabot/uv/coverage-7.16.0branch (#3521). That branch had been used as a working branch, which meant Dependabot could no longer rebase it, and the work was blocking a routinecoveragebump.The commits are rebased onto current
mainand preserved individually:Conflict resolution
The branch predated #3522, which reworked the same files. Every conflict was prose that #3522 had already rewritten more completely, so
main's wording was kept in each case. No behavioural change was resolved away.Some of the original work has since landed independently via #3522, so the net diff here is smaller than the seven commits suggest.
Worth a look in review
assert_model_target_statusherepytest.skips when the real Vuforia reportsTRAINING_ALLOWANCE_EXCEEDED, whereasmain(via #3522) reports the same condition as an xfail. Both behaviours are now present in the same function. That is a deliberate design call rather than a merge artefact, and it should be settled one way or the other before this merges.Verification
ruff format --check .andruff check .cleanpytest --skip-docker_build_tests --skip-docker_in_memory --skip-mock --skip-real .— 218 passed, 2290 skippedThe real-Vuforia paths these commits touch are only exercised by the credentialed CI job, so they are unverified locally by construction.
🤖 Generated with Claude Code
https://claude.ai/code/session_013uNQs54YVsYTu2tQEUmrho