fix: Track the version of the test framework (#1608)

Signed-off-by: Sheng Chen <[email protected]>

Author: jdneo

java-extension/com.microsoft.java.test.target/com.microsoft.java.test.tp.target

@@ -9,11 +9,11 @@
             <unit id="org.eclipse.jdt.source.feature.group" version="0.0.0"/>
             <unit id="org.eclipse.sdk.feature.group" version="0.0.0"/>
             <unit id="org.mockito.mockito-core" version="0.0.0"/>
-            <repository location="https://download.eclipse.org/eclipse/updates/4.29/"/>
+            <repository location="https://download.eclipse.org/eclipse/updates/4.28/R-4.28-202306050440/"/>
         </location>
         <location includeAllPlatforms="false" includeConfigurePhase="false" includeMode="planner" includeSource="true" type="InstallableUnit">
             <unit id="org.eclipse.xtext.xbase.lib" version="0.0.0"/>
-            <repository location="https://download.eclipse.org/releases/2023-06/"/>
+            <repository location="https://download.eclipse.org/releases/2023-03/"/>
         </location>
         <location includeAllPlatforms="false" includeConfigurePhase="false" includeMode="planner" includeSource="true" type="InstallableUnit">
            <repository location="https://download.eclipse.org/lsp4j/updates/releases/0.21.0/"/>

package.json

@@ -55,22 +55,22 @@
     "main": "./main.js",
     "contributes": {
         "javaExtensions": [
-            "./server/junit-jupiter-api_5.10.0.jar",
-            "./server/junit-jupiter-engine_5.10.0.jar",
-            "./server/junit-jupiter-migrationsupport_5.10.0.jar",
-            "./server/junit-jupiter-params_5.10.0.jar",
-            "./server/junit-platform-commons_1.10.0.jar",
-            "./server/junit-platform-engine_1.10.0.jar",
-            "./server/junit-platform-launcher_1.10.0.jar",
-            "./server/junit-platform-runner_1.10.0.jar",
-            "./server/junit-platform-suite-api_1.10.0.jar",
-            "./server/junit-platform-suite-commons_1.10.0.jar",
-            "./server/junit-platform-suite-engine_1.10.0.jar",
-            "./server/junit-vintage-engine_5.10.0.jar",
+            "./server/junit-jupiter-api_5.9.3.jar",
+            "./server/junit-jupiter-engine_5.9.3.jar",
+            "./server/junit-jupiter-migrationsupport_5.9.3.jar",
+            "./server/junit-jupiter-params_5.9.3.jar",
+            "./server/junit-platform-commons_1.9.3.jar",
+            "./server/junit-platform-engine_1.9.3.jar",
+            "./server/junit-platform-launcher_1.9.3.jar",
+            "./server/junit-platform-runner_1.9.3.jar",
+            "./server/junit-platform-suite-api_1.9.3.jar",
+            "./server/junit-platform-suite-commons_1.9.3.jar",
+            "./server/junit-platform-suite-engine_1.9.3.jar",
+            "./server/junit-vintage-engine_5.9.3.jar",
             "./server/org.apiguardian.api_1.1.2.jar",
             "./server/org.eclipse.jdt.junit4.runtime_1.3.0.v20220609-1843.jar",
             "./server/org.eclipse.jdt.junit5.runtime_1.1.100.v20220907-0450.jar",
-            "./server/org.opentest4j_1.3.0.jar",
+            "./server/org.opentest4j_1.2.0.jar",
             "./server/com.microsoft.java.test.plugin-0.39.1.jar"
         ],
         "viewsWelcome": [

src/controller/testController.ts

@@ -2,6 +2,7 @@
 // Licensed under the MIT license.
 
 import * as _ from 'lodash';
+import * as path from 'path';
 import { CancellationToken, DebugConfiguration, Disposable, FileSystemWatcher, RelativePattern, TestController, TestItem, TestRun, TestRunProfileKind, TestRunRequest, tests, TestTag, Uri, window, workspace, WorkspaceFolder } from 'vscode';
 import { instrumentOperation, sendError, sendInfo } from 'vscode-extension-telemetry-wrapper';
 import { refreshExplorer } from '../commands/testExplorerCommands';
@@ -215,9 +216,6 @@ export const runTests: (request: TestRunRequest, option: IRunOption) => any = in
                             testRun: run,
                             workspaceFolder,
                         };
-                        sendInfo(operationId, {
-                            testFramework: TestKind[testContext.kind],
-                        });
                         const runner: BaseRunner | undefined = getRunnerByContext(testContext);
                         if (!runner) {
                             window.showErrorMessage(`Failed to get suitable runner for the test kind: ${testContext.kind}.`);
@@ -228,6 +226,7 @@ export const runTests: (request: TestRunRequest, option: IRunOption) => any = in
                             const resolvedConfiguration: DebugConfiguration = mergeConfigurations(option.launchConfiguration, config) ?? await resolveLaunchConfigurationForRunner(runner, testContext, config);
                             resolvedConfiguration.__progressId = option.progressReporter?.getId();
                             delegatedToDebugger = true;
+                            trackTestFrameworkVersion(testContext.kind, resolvedConfiguration.classPaths, resolvedConfiguration.modulePaths);
                             await runner.run(resolvedConfiguration, token, option.progressReporter);
                         } catch(error) {
                             window.showErrorMessage(error.message || 'Failed to run tests.');
@@ -543,6 +542,36 @@ function getRunnerByContext(testContext: IRunTestContext): BaseRunner | undefine
     }
 }
 
+function trackTestFrameworkVersion(testKind: TestKind, classpaths: string[], modulepaths: string[]) {
+    let artifactPattern: RegExp;
+    switch (testKind) {
+        case TestKind.JUnit:
+            artifactPattern = /junit-(\d+\.\d+\.\d+(-[a-zA-Z\d]+)?).jar/;
+            break;
+        case TestKind.JUnit5:
+            artifactPattern = /junit-jupiter-api-(\d+\.\d+\.\d+(-[a-zA-Z\d]+)?).jar/;
+            break;
+        case TestKind.TestNG:
+            artifactPattern = /testng-(\d+\.\d+\.\d+(-[a-zA-Z\d]+)?).jar/;
+            break;
+        default:
+            return;
+    }
+    let version: string = 'unknown';
+    for (const entry of [...classpaths, ...modulepaths]) {
+        const fileName: string = path.basename(entry);
+        const match: RegExpMatchArray | null = artifactPattern.exec(fileName);
+        if (match) {
+            version = match[1];
+            break;
+        }
+    }
+    sendInfo('', {
+        testFramework: testKind,
+        version,
+    });
+}
+
 interface IRunOption {
     isDebug: boolean;
     progressReporter?: IProgressReporter;