Upgrade to Node.js 20 and Improve CI Test Output (#529)

* Upgrade to Node.js 20 and remove deprecated mock server build step

- Update all pipelines to use Node 20.x instead of Node 16.x
- Remove obsolete packages/tfs-mock-server build step from common-steps.yml
- Update package.json engines to require Node >=20.0.0
- Fixes EBADENGINE warnings for shelljs, path-scurry, and system-architecture dependencies
- All 181 tests passing locally

* Add mocha-multi-reporters for better CI test output

- Install mocha-multi-reporters package
- Configure test:ci to output both spec (console) and xunit (XML) formats
- Pipeline will now show detailed test progress in logs while still generating test-results.xml for Azure DevOps
- All 181 tests passing

* packge version update for node 20

* updating node in matrix test as well

---------

Co-authored-by: Tarun Ramsinghani <tarunramsinghani@users.noreply.github.com>

Author: tarunramsinghani

.azure-pipelines/common-steps.yml

@@ -6,9 +6,9 @@ steps:
     clean: true
 
   - task: UseNode@1
-    displayName: Use Node 16
+    displayName: Use Node 20
     inputs:
-      version: "16.x"
+      version: "20.x"
 
   - task: NpmAuthenticate@0
     inputs:
@@ -17,12 +17,6 @@ steps:
   - script: npm i -g npm@8.19.4 --force
     displayName: Use npm version 8.19.4
 
-  - bash: |
-      cd packages/tfs-mock-server
-      npm ci
-      npm run build
-    displayName: Install and Build Mock Server
-
   - bash: |
       npm ci
       npm run build
@@ -34,7 +28,6 @@ steps:
       export TFX_TRACE="$TFX_TRACE"
       npm run test:ci
     displayName: Run Tests
-    continueOnError: true
 
   - task: PublishTestResults@2
     displayName: Publish Test Results

.azure-pipelines/multi-os-matrix.yml

@@ -17,21 +17,21 @@ stages:
       displayName: Run tests on all Node/OS combinations
       strategy:
         matrix:
-          windows_node16:
+          windows_node24:
             imageName: 'windows-2022'
-            nodeVersion: '16.x'
+            nodeVersion: '24.x'
           windows_node20:
             imageName: 'windows-2022'
             nodeVersion: '20.x'
-          ubuntu_node16:
+          ubuntu_node24:
             imageName: 'ubuntu-24.04'
-            nodeVersion: '16.x'
+            nodeVersion: '24.x'
           ubuntu_node20:
             imageName: 'ubuntu-24.04'
             nodeVersion: '20.x'
-          macos_node16:
+          macos_node24:
             imageName: 'macos-14'
-            nodeVersion: '16.x'
+            nodeVersion: '24.x'
           macos_node20:
             imageName: 'macos-14'
             nodeVersion: '20.x'
@@ -47,28 +47,8 @@ stages:
         - task: NpmAuthenticate@0
           inputs:
             workingFile: .npmrc
-        - script: |
-            if [[ "$(nodeVersion)" =~ ^(10|12|14|16|18) ]]; then
-              npm i -g npm@8.19.4 --force
-              npm --version
-            else
-              npm i -g npm@10 --force
-              npm --version
-            fi
-          displayName: Use compatible npm version for Node (Linux/macOS)
-          condition: ne( variables['Agent.OS'], 'Windows_NT' )
-        - script: |
-            if "%nodeVersion%" == "16.x" (
-              npm i -g npm@8.19.4 --force
-              npm --version
-            ) else if "%nodeVersion%" == "20.x" (
-              npm i -g npm@10 --force
-              npm --version
-            ) else (
-              npm --version
-            )
-          displayName: Use compatible npm version for Node (Windows)
-          condition: eq( variables['Agent.OS'], 'Windows_NT' )
+        - script: npm i -g npm@latest --force
+          displayName: Use latest npm version
         - script: npm ci
           displayName: npm ci
         - script: npm run build

.azure-pipelines/release-pipeline.yml

@@ -52,11 +52,11 @@ extends:
             displayName: Publish TFX CLI to npm
             condition: eq(variables['Build.SourceBranchName'], 'master')
 
-        # Install node 16 for CodeQL 3000
+        # Install node 20 for CodeQL 3000
         - task: NodeTool@0
-          displayName: Use node 16
+          displayName: Use node 20
           inputs:
-            versionSpec: "16.x"
+            versionSpec: "20.x"
     - stage: Release
       displayName: Release to Latest
       trigger: manual
@@ -68,9 +68,9 @@ extends:
               clean: true
 
             - task: NodeTool@0
-              displayName: Use node 16
+              displayName: Use node 20
               inputs:
-                versionSpec: "16.x"
+                versionSpec: "20.x"
             - script: npm i -g npm@8.19.4 --force
               displayName: Use npm version 8.19.4
 

.mocha-multi-reporters.json

@@ -0,0 +1,6 @@
+{
+  "reporterEnabled": "spec, xunit",
+  "xunitReporterOptions": {
+    "output": "test-results.xml"
+  }
+}

.vscode/settings.json

@@ -8,5 +8,6 @@
         "tmp/": true
     },
 	"editor.insertSpaces": false,
-    "typescript.tsdk": "./node_modules/typescript/lib"
+    "typescript.tsdk": "./node_modules/typescript/lib",
+    "azure-pipelines.1ESPipelineTemplatesSchemaFile": true
 }

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "tfx-cli",
-  "version": "0.22.7",
+  "version": "0.23.0",
   "description": "CLI for Azure DevOps Services and Team Foundation Server",
   "repository": {
     "type": "git",
@@ -21,7 +21,7 @@
     "build:tests": "tsc -p tsconfig.tests.json",
     "postbuild:tests": "ncp tests/build-samples _tests/build-samples && ncp tests/extension-samples _tests/extension-samples",
     "test": "npm run build:tests && mocha \"_tests/tests/**/*.js\"",
-    "test:ci": "npm run build:tests && mocha \"_tests/tests/**/*.js\" --reporter xunit --reporter-options output=test-results.xml",
+    "test:ci": "npm run build:tests && mocha \"_tests/tests/**/*.js\" --reporter mocha-multi-reporters --reporter-options configFile=.mocha-multi-reporters.json",
     "test:commandline": "npm run build:tests && mocha \"_tests/tests/commandline.js\"",
     "test:server-integration": "npm run build:tests && mocha \"_tests/tests/server-integration-*.js\"",
     "test:focused-login": "npm run build:tests && mocha \"_tests/tests/focused-login-test.js\"",
@@ -79,12 +79,13 @@
     "@types/xml2js": "0.0.27",
     "adm-zip": "^0.5.16",
     "mocha": "^10.2.0",
+    "mocha-multi-reporters": "^1.5.1",
     "ncp": "^2.0.0",
     "rimraf": "^2.6.1",
     "typescript": "^4.9.5"
   },
   "engines": {
-    "node": ">=16.0.0"
+    "node": ">=20.0.0"
   },
   "author": "Microsoft Corporation",
   "contributors": [