Complete Model Target API fidelity and refresh credentials - #3451
Draft
adamtheturtle wants to merge 4 commits into
Draft
Complete Model Target API fidelity and refresh credentials#3451adamtheturtle wants to merge 4 commits into
adamtheturtle wants to merge 4 commits into
Conversation
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.
What changed
TestSuccessfull-dataset.ziparchives containingMTDataset.datandMTDataset.xmlWhy
CI slot 1 returned Vuforia
403 ProjectSuspended, blocking every open PR. The previous Model Target credentials were revoked and lacked advanced and State-Based scopes, leaving much of the mock's behavior unverified.Validation
ci_secretsfiles to the same schemafull-dataset.zipthrough the advanced status/download routestests/mock_vws/test_model_target_web_api.py: 747 passing before the credential-management addition; all added lifecycle tests pass against all three backendstests/mock_vws/test_query.py::TestSuccess: 21 passed, including real VuforiaCloses #2114
Closes #3192
Closes #3193
Closes #3194
Closes #3195
Closes #3196
Closes #3202
Closes #3344
Closes #3345
Closes #3348
Closes #3441
Closes #3444