Replace mocha-junit-reporter with Mocha builtin xunit reporter (Azure#27992)

mocha-junit-reporter doesn't support --parallel:
michaelleeallen/mocha-junit-reporter#127

Also update core packages to remove dependency on mocha-junit-reporter

Author: jeremymeng

common/tools/dev-tool/src/commands/run/testNodeJSInput.ts

@@ -21,11 +21,13 @@ export const commandInfo = makeCommandInfo(
 );
 
 export default leafCommand(commandInfo, async (options) => {
+  const reporterArgs =
+    "--reporter ../../../common/tools/mocha-multi-reporter.js --reporter-option output=test-results.xml";
   const defaultMochaArgs = `${
     (await isModuleProject())
       ? "-r source-map-support/register.js"
       : "-r ../../../common/tools/esm-workaround -r esm -r source-map-support/register"
-  } --reporter ../../../common/tools/mocha-multi-reporter.js --full-trace`;
+  } ${reporterArgs} --full-trace`;
   const updatedArgs = options["--"]?.map((opt) =>
     opt.includes("**") && !opt.startsWith("'") && !opt.startsWith('"') ? `"${opt}"` : opt,
   );

common/tools/dev-tool/src/commands/run/testNodeTSInput.ts

@@ -22,9 +22,11 @@ export const commandInfo = makeCommandInfo(
 
 export default leafCommand(commandInfo, async (options) => {
   const isModuleProj = await isModuleProject();
+  const reporterArgs =
+    "--reporter ../../../common/tools/mocha-multi-reporter.js --reporter-option output=test-results.xml";
   const defaultMochaArgs = `${
     isModuleProj ? "" : "-r esm "
-  }-r ts-node/register --reporter ../../../common/tools/mocha-multi-reporter.js --full-trace`;
+  }-r ts-node/register ${reporterArgs} --full-trace`;
   const updatedArgs = options["--"]?.map((opt) =>
     opt.includes("**") && !opt.startsWith("'") && !opt.startsWith('"') ? `"${opt}"` : opt,
   );

common/tools/mocha-multi-reporter.js

@@ -3,26 +3,22 @@
 "use strict";
 
 const Mocha = require("mocha");
-const MochaJUnitReporter = require("mocha-junit-reporter");
 
 /**
  * Usage :
- * - Meant to be used to leverage Mocha's Spec reporter as well as mocha-junit-reporter for the test runs
- * - Provide the relative path to this file from your `sdk/service/package-folder/` as the reporter options to mocha
- * - Example - `--reporter ../../../common/tools/mocha-multi-reporter.js`
+ * - Meant to be used to leverage Mocha's buitin Spec and XUnit reporters
+ * - Provide the relative path to this file from your `sdk/service/package-folder/` to mocha
+ * - Example - `--reporter ../../../common/tools/mocha-multi-reporter.js --report-option ...`
  *
  * Refer the following docs for more customizations on reporters
  * https://mochajs.org/api/tutorial-custom-reporter.html
  *
  * @class MultiReporter
  */
 class MultiReporter {
-  constructor(runner) {
-    // Spec reporter is provided as part of mocha library
-    // Invoking the spec reporter with the runner
-    new Mocha.reporters.Spec(runner);
-    // Invoking mocha-junit-reporter to generate XML reports of test summaries for CI
-    new MochaJUnitReporter(runner);
+  constructor(runner, options) {
+    new Mocha.reporters.Spec(runner, options);
+    new Mocha.reporters.XUnit(runner, options);
   }
 }
 

sdk/core/abort-controller/package.json

@@ -92,7 +92,6 @@
     "karma-mocha-reporter": "^2.2.5",
     "karma-sourcemap-loader": "^0.3.8",
     "mocha": "^10.0.0",
-    "mocha-junit-reporter": "^2.0.0",
     "c8": "^8.0.0",
     "prettier": "^2.5.1",
     "rimraf": "^3.0.0",

sdk/core/core-amqp/package.json

@@ -60,7 +60,7 @@
     "test:node": "npm run clean && tsc -p . && npm run unit-test:node && npm run integration-test:node",
     "test": "npm run clean && tsc -p . && npm run unit-test:node && npm run bundle && npm run unit-test:browser && npm run integration-test",
     "unit-test:browser": "karma start --single-run",
-    "unit-test:node": "cross-env TS_NODE_FILES=true TS_NODE_COMPILER_OPTIONS=\"{\\\"module\\\":\\\"commonjs\\\"}\" mocha -r ts-node/register --reporter ../../../common/tools/mocha-multi-reporter.js  --timeout 50000  --full-trace  --exclude \"test/**/browser/*.spec.ts\" \"test/**/*.spec.ts\"",
+    "unit-test:node": "cross-env TS_NODE_FILES=true dev-tool run test:node-ts-input --no-test-proxy=true",
     "unit-test": "npm run unit-test:node && npm run unit-test:browser"
   },
   "engines": {
@@ -104,7 +104,6 @@
     "karma-chrome-launcher": "^3.0.0",
     "karma-mocha": "^2.0.1",
     "mocha": "^10.0.0",
-    "mocha-junit-reporter": "^2.0.0",
     "c8": "^8.0.0",
     "prettier": "^2.5.1",
     "puppeteer": "^19.2.2",

sdk/core/core-auth/package.json

@@ -79,7 +79,6 @@
     "eslint": "^8.0.0",
     "inherits": "^2.0.3",
     "mocha": "^10.0.0",
-    "mocha-junit-reporter": "^2.0.0",
     "prettier": "^2.5.1",
     "rimraf": "^3.0.0",
     "typescript": "~5.2.0",

sdk/core/core-client-rest/package.json

@@ -84,7 +84,6 @@
     "karma-mocha": "^2.0.1",
     "karma-sourcemap-loader": "^0.3.8",
     "karma": "^6.2.0",
-    "mocha-junit-reporter": "^2.0.0",
     "mocha": "^10.0.0",
     "prettier": "^2.5.1",
     "rimraf": "^3.0.0",

sdk/core/core-client/package.json

@@ -100,7 +100,6 @@
     "karma-mocha-reporter": "^2.2.5",
     "karma-sourcemap-loader": "^0.3.8",
     "mocha": "^10.0.0",
-    "mocha-junit-reporter": "^2.0.0",
     "prettier": "^2.5.1",
     "rimraf": "^3.0.0",
     "sinon": "^17.0.0",

sdk/core/core-http-compat/package.json

@@ -63,7 +63,6 @@
     "@azure/test-utils": "^1.0.0",
     "@azure/dev-tool": "^1.0.0",
     "@microsoft/api-extractor": "^7.31.1",
-    "mocha-junit-reporter": "^2.0.0",
     "mocha": "^10.0.0",
     "eslint": "^8.0.0",
     "rimraf": "^3.0.0",

sdk/core/core-lro/package.json

@@ -116,7 +116,6 @@
     "karma-mocha-reporter": "^2.2.5",
     "karma-sourcemap-loader": "^0.3.8",
     "mocha": "^10.0.0",
-    "mocha-junit-reporter": "^2.0.0",
     "c8": "^8.0.0",
     "prettier": "^2.5.1",
     "rimraf": "^3.0.0",