Description
There are some things we can potentially clean up in regards to setting the Test Mode in the various pipelines and in the Test Framework. The benefit would be to make our source more self-documenting and easier to understand:
-
We conditionally allow record mode in https://github.com/Azure/azure-sdk-for-net/blob/main/eng/pipelines/templates/jobs/live.tests.yml#L139-L142, in order to support https://github.com/Azure/azure-sdk-for-net/tree/main/sdk/core/Azure.Core.TestFramework#recording-data-plane-tests-on-ci, but this workflow does not seem to be used by many. We should consider deprecating it, or removing it.
-
We set the test mode to None in our live test pipelines which relies on the fact that None is equivalent to Live. We do this ostensibly for back compat reasons with T1, but this backcompat may no longer be needed/used.
-
We can set the AZURE_TEST_MODE to Playback in the archetype-sdk-client.yml, so that we are not relying on the default Test Framework behavior.
-
If we don't do any/all of the above clean ups, we should at least document these oddities in the appropriate yml files/Test Framework README.