changes

Author: shuklahy

src/test-info/test-info.ts

@@ -124,9 +124,9 @@ export class BuildTargetTestCaseInfo extends TestCaseInfo {
       }
     }
 
-    // Add the IDE tag (--define flag) to additionalBazelParams
+    // Additional Bazel params is not supported in BSP coverage for now
     const ideTag = currentRun.getIdeTag()
-    if (ideTag && ideTag.trim().length > 0) {
+    if (ideTag && !bazelParams.coverage) {
       if (bazelParams.additionalBazelParams) {
         bazelParams.additionalBazelParams += ` ${ideTag}`
       } else {

src/test/suite/run-factory.test.ts

@@ -134,7 +134,7 @@ suite('Test Runner Factory', () => {
         sampleToken
       )
       assert.ok(detectIdeClientStub.calledOnce)
-      assert.equal(runTracker.getIdeTag(), '--define=ide_client=test-ide')
+      assert.equal(runTracker.getIdeTag(), '--test_env=IDE_CLIENT=test-ide')
     } finally {
       detectIdeClientStub.restore()
     }

src/test/suite/runner.test.ts

@@ -238,7 +238,7 @@ suite('Test Runner', () => {
       assert.ok(!callArgs[1].data.coverage)
       assert.strictEqual(
         callArgs[1].data.additionalBazelParams,
-        '--my_flag_1 --my_flag_2 --define=ide_client=cursor'
+        '--my_flag_1 --my_flag_2 --test_env=IDE_CLIENT=cursor'
       )
       assert.strictEqual(callArgs[1].dataKind, TestParamsDataKind.BazelTest)
     }

src/test/suite/test-info.test.ts

@@ -207,7 +207,7 @@ suite('TestInfo', () => {
       const currentRun = sandbox.createStubInstance(TestRunTracker)
       sandbox.stub(currentRun, 'originName').get(() => 'sample')
       currentRun.getRunProfileKind.returns(vscode.TestRunProfileKind.Run)
-      currentRun.getIdeTag.returns('--define=ide_client=cursor')
+      currentRun.getIdeTag.returns('--test_env=IDE_CLIENT=cursor')
 
       const result = testInfo.prepareTestRunParams(currentRun)
       assert.deepStrictEqual(result, {
@@ -223,7 +223,7 @@ suite('TestInfo', () => {
         dataKind: TestParamsDataKind.BazelTest,
         data: {
           coverage: false,
-          additionalBazelParams: '--define=ide_client=cursor',
+          additionalBazelParams: '--test_env=IDE_CLIENT=cursor',
         },
       })
     })