Update DVR references to point to fork rather than venmo #903
Conversation
This file contains 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
ghost
added
the
jairmyree
approved these changes
Jun 11, 2021
tjprescott
added a commit
that referenced
this pull request
Jul 27, 2021
* Initial commit for AzureTest. * AzureTest DVR commit. * Rename "AzureTestClient" to "ResourceUtilityClient". Eliminate MSAL dependency for AzureTest. * Add util function to AzureTest to get environment variables (moved out of Chat). * DVR updates. (#896) * Update DVR references to point to fork rather than venmo (#903) * Update DVR references to point to fork rather than venmo. * Remove static linkage to DVR. * Recordings on a per-test basis (#904) * Feature/recording location (#907) * Environment Variable for sdk root * Revert project.pbxproj * Edit to outputDirectory construction * Env Variable TEST_MODE works as expected (#909) * Environment Variable for sdk root Enforce that DVR record-playback honors environment variables * Environment Variables work as intended with the new DVR updates * Undo Podfile Changes Local Changes that need to be removed * Slight edit to record mode Co-authored-by: Jair Myree <t-jairmyree@microsoft.com> * Automated means of creating test resources (#912) * Add resource group delete operation. * Progress. * Project setting fixes. * Add test-resources.bicep file. * Update bicep file. * Target python script in test resource deploy post-script. * Remove ResourceUtil files and dependencies. * Revert to previous output style. * Remove empty environment variable. (#919) * Injectable test settings (#924) * Commit test-settings.plist and git-ignore. * TestSettings protocol. * Update Podfile for DVR * Don't use local DVR * Chat test updates. * Add recordings to project. * Update prepare_chat_tests.py * Update deployment targets. * Add podspec. * Add readme. * Fix hard-coded links. * Scrub headings and commit chat test recordings. (#935) * Remove headers. * Add scrubbed recordings. * Feature/scrub subscription ids (#932) * Major edits toward scrubbing subscription IDs * Minor edits on scrubbing * Update sdk/test/AzureTest/Scrubbing.swift Co-authored-by: Travis Prescott <tjprescott@users.noreply.github.com> * Update sdk/test/AzureTest/Scrubbing.swift Co-authored-by: Travis Prescott <tjprescott@users.noreply.github.com> * Update sdk/test/AzureTest/Scrubbing.swift Co-authored-by: Travis Prescott <tjprescott@users.noreply.github.com> * Update sdk/test/AzureTest/Scrubbing.swift Co-authored-by: Travis Prescott <tjprescott@users.noreply.github.com> * Recommit: Files changes got left out of last commit * Requested Draft PR changes * Clean up after rebase * Testing response scrubbing in progress * Finished Scrubbing SubscriptionIDs * Adjustments made to PR based on review comments. * Edits to PR based on review comments * Update sdk/test/AzureTest/AzureTestTests/AzureTestTests.swift Co-authored-by: Travis Prescott <tjprescott@users.noreply.github.com> * More edits based of pr review * More edits to pr based off of review comments * More edits to PR based off suggestions in review * More edits to PR from review comments * Removed unused functions and extensions * Edits to PR based off review comments * Edits to PR from review comments * Edits based off PR review comments Co-authored-by: Jair Myree <t-jairmyree@microsoft.com> Co-authored-by: Travis Prescott <tjprescott@users.noreply.github.com> * RPF: RecordableXCTestCase (#963) * Add base class for recordable tests. * Run swiftformat. * CI fixes. * Fix podspec source_files. * Fix project structure. * Fixes * Fix broken links. * CI fixes. * Remove test-settings.plist reference. * Re-record tests. * Fix tests for playback. Co-authored-by: Jair <67484440+jairmyree@users.noreply.github.com> Co-authored-by: Jair Myree <t-jairmyree@microsoft.com>
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.
By pointing to our fork, we can iterate more quickly on issues that arise with DVR.