[Recorder] Rebrand recorder-new 1.0.0 as recorder 2.0.0

rush.json

@@ -716,11 +716,6 @@
       "projectFolder": "sdk/test-utils/recorder",
       "versionPolicyName": "utility"
     },
-    {
-      "packageName": "@azure-tools/test-recorder-new",
-      "projectFolder": "sdk/test-utils/recorder-new",
-      "versionPolicyName": "utility"
-    },
     {
       "packageName": "@azure-tools/test-credential",
       "projectFolder": "sdk/test-utils/test-credential",

sdk/storage/CONTRIBUTING.md

@@ -57,7 +57,7 @@ The environment variable **TEST_MODE** controls how the tests are running.
 - Else If TEST_MODE = "playback" (or if the TEST_MODE is not set or set to an invalid value),
   - Existing recordings are played back as responses to the HTTP requests in the tests
 
-Please refer to the [guidelines on Record and Playback](https://github.com/Azure/azure-sdk-for-js/blob/main/sdk/test-utils/recorder/GUIDELINES.md) for more details.
+Please refer to the [guidelines on Record and Playback](https://github.com/Azure/azure-sdk-for-js/blob/c06170fbaf39e81496f8009a0da93d470d8c9f88/sdk/test-utils/recorder/GUIDELINES.md) for more details.
 
 ### Emulator Tests
 
@@ -156,7 +156,7 @@ npm run test
 
 `npm run test` would run the the tests in both node and the browser.
 
-**Link** - [Guidelines for record and playback - `@azure-tools/test-recorder`](https://github.com/Azure/azure-sdk-for-js/blob/main/sdk/test-utils/recorder/GUIDELINES.md)
+**Link** - [Guidelines for record and playback - `@azure-tools/test-recorder`](https://github.com/Azure/azure-sdk-for-js/blob/c06170fbaf39e81496f8009a0da93d470d8c9f88/sdk/test-utils/recorder/GUIDELINES.md)
 
 ## Pull Requests
 

sdk/tables/perf-tests/data-tables/package.json

@@ -8,7 +8,7 @@
   "author": "",
   "license": "ISC",
   "dependencies": {
-    "@azure/data-tables": "^12.1.2",
+    "@azure/data-tables": "^13.0.2",
     "@azure/test-utils-perf": "^1.0.0",
     "dotenv": "^8.2.0",
     "uuid": "^8.3.0"

sdk/template/template/karma.conf.js

@@ -4,14 +4,8 @@
 // https://github.com/karma-runner/karma-chrome-launcher
 process.env.CHROME_BIN = require("puppeteer").executablePath();
 process.env.RECORDINGS_RELATIVE_PATH =
-  require("@azure-tools/test-recorder-new").relativeRecordingsPath();
+  require("@azure-tools/test-recorder").relativeRecordingsPath();
 require("dotenv").config();
-const {
-  jsonRecordingFilterFunction,
-  isPlaybackMode,
-  isSoftRecordMode,
-  isRecordMode,
-} = require("@azure-tools/test-recorder");
 
 module.exports = function (config) {
   config.set({
@@ -32,15 +26,13 @@ module.exports = function (config) {
       "karma-env-preprocessor",
       "karma-coverage",
       "karma-junit-reporter",
-      "karma-json-to-file-reporter",
-      "karma-json-preprocessor",
     ],
 
     // list of files / patterns to load in the browser
     files: [
       "dist-test/index.browser.js",
       { pattern: "dist-test/index.browser.js.map", type: "html", included: false, served: true },
-    ].concat(isPlaybackMode() || isSoftRecordMode() ? ["recordings/browsers/**/*.json"] : []),
+    ],
 
     // list of files / patterns to exclude
     exclude: [],
@@ -49,7 +41,6 @@ module.exports = function (config) {
     // available preprocessors: https://npmjs.org/browse/keyword/karma-preprocessor
     preprocessors: {
       "**/*.js": ["env"],
-      "recordings/browsers/**/*.json": ["json"],
       // IMPORTANT: COMMENT following line if you want to debug in your browsers!!
       // Preprocess source file to calculate code coverage, however this will make source file unreadable
       //"dist-test/index.browser.js": ["coverage"]
@@ -69,7 +60,7 @@ module.exports = function (config) {
     // test results reporter to use
     // possible values: 'dots', 'progress'
     // available reporters: https://npmjs.org/browse/keyword/karma-reporter
-    reporters: ["mocha", "coverage", "junit", "json-to-file"],
+    reporters: ["mocha", "coverage", "junit"],
 
     coverageReporter: {
       // specify a common output directory
@@ -87,11 +78,6 @@ module.exports = function (config) {
       properties: {}, // key value pair of properties to add to the <properties> section of the report
     },
 
-    jsonToFileReporter: {
-      filter: jsonRecordingFilterFunction,
-      outputPath: ".",
-    },
-
     // web server port
     port: 9876,
 
@@ -126,9 +112,6 @@ module.exports = function (config) {
     browserNoActivityTimeout: 600000,
     browserDisconnectTimeout: 10000,
     browserDisconnectTolerance: 3,
-    browserConsoleLogOptions: {
-      terminal: !isRecordMode(),
-    },
 
     client: {
       mocha: {

sdk/template/template/package.json

@@ -87,7 +87,7 @@
     "@azure/dev-tool": "^1.0.0",
     "@azure/eslint-plugin-azure-sdk": "^3.0.0",
     "@azure/identity": "^2.0.1",
-    "@azure-tools/test-recorder": "^1.0.0",
+    "@azure-tools/test-recorder": "^2.0.0",
     "@microsoft/api-extractor": "^7.18.11",
     "@types/chai": "^4.1.6",
     "@types/chai-as-promised": "^7.1.0",
@@ -122,7 +122,6 @@
     "source-map-support": "^0.5.9",
     "typescript": "~4.2.0",
     "util": "^0.12.1",
-    "@azure-tools/test-recorder-new": "~1.0.0",
     "@azure-tools/test-credential": "~1.0.0"
   },
   "//sampleConfiguration": {

sdk/template/template/test/public/configurationClient.spec.ts

@@ -3,11 +3,8 @@
 
 import { assert } from "chai";
 import { Context } from "mocha";
-
 import { ConfigurationClient } from "../../src";
-
-import { env } from "@azure-tools/test-recorder";
-import { Recorder } from "@azure-tools/test-recorder-new";
+import { Recorder, assertEnvironmentVariable } from "@azure-tools/test-recorder";
 import { createTestCredential } from "@azure-tools/test-credential";
 
 // When the recorder observes the values of these environment variables in any
@@ -25,7 +22,7 @@ const replaceableVariables: Record<string, string> = {
 function createConfigurationClient(recorder: Recorder): ConfigurationClient {
   // Retrieve the endpoint from the environment variable
   // we saved to the .env file earlier
-  const endpoint = env.APPCONFIG_ENDPOINT;
+  const endpoint = assertEnvironmentVariable("APPCONFIG_ENDPOINT");
 
   // We use the createTestCredential helper from the test-credential tools package.
   // This function returns the special NoOpCredential in playback mode, which
@@ -73,8 +70,8 @@ describe("[AAD] ConfigurationClient functional tests", function () {
   });
 
   it("predetermined setting has expected value", async () => {
-    const key = env.APPCONFIG_TEST_SETTING_KEY;
-    const expectedValue = env.APPCONFIG_TEST_SETTING_EXPECTED_VALUE;
+    const key = assertEnvironmentVariable("APPCONFIG_TEST_SETTING_KEY");
+    const expectedValue = assertEnvironmentVariable("APPCONFIG_TEST_SETTING_EXPECTED_VALUE");
 
     const setting = await client.getConfigurationSetting(key);