From bc4db190cef7d1fa91bea0d4c2c8679bb4e465a4 Mon Sep 17 00:00:00 2001
From: eleanorjboyd <eleanorboyd@microsoft.com>
Date: Fri, 10 Jan 2025 12:56:57 -0800
Subject: [PATCH] remove unneeded null check

---
 .../testing/testController/unittest/testDiscoveryAdapter.ts     | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/client/testing/testController/unittest/testDiscoveryAdapter.ts b/src/client/testing/testController/unittest/testDiscoveryAdapter.ts
index 2436ed62a2f9..9a014116de96 100644
--- a/src/client/testing/testController/unittest/testDiscoveryAdapter.ts
+++ b/src/client/testing/testController/unittest/testDiscoveryAdapter.ts
@@ -201,7 +201,7 @@ export class UnittestTestDiscoveryAdapter implements ITestDiscoveryAdapter {
                 // if the child has testIds then this is a run request
                 spawnOptions?.outputChannel?.append(MESSAGE_ON_TESTING_OUTPUT_MOVE);
 
-                if (code && code !== 0) {
+                if (code !== 0) {
                     // This occurs when we are running discovery
                     traceError(
                         `Subprocess exited unsuccessfully with exit code ${code} and signal ${signal} on workspace ${options.cwd}. Creating and sending error discovery payload \n`,