Don't throw on failure to install C# extension for tests

We skip those tests in ADO anyway.

Author: andyleejordan

test/features/DebugSession.test.ts

@@ -414,7 +414,13 @@ describe("DebugSessionFeature", () => {
 
             const config = await debugSessionFeature.resolveDebugConfigurationWithSubstitutedVariables(undefined, attachConfig);
 
-            assert.deepStrictEqual(config!.dotnetAttachConfig, foundDotnetConfig);
+            // This config will only be present if the C# extension is installed.
+            if (extensions.getExtension("ms-dotnettools.csharp")) {
+                assert.ok(config);
+                assert.deepStrictEqual(config.dotnetAttachConfig, foundDotnetConfig);
+            } else {
+                assert.ok(!config);
+            }
         });
     });
 

test/runTests.ts

@@ -105,9 +105,10 @@ function InstallExtension(vscodeExePath: string, extensionIdOrVSIXPath: string):
     });
 
     if (installResult.status !== 0) {
-        console.error(installResult.stderr);
-        throw new Error(`Failed to install extension: ${installResult.stderr}`);
+        console.error(`Failed to install extension: ${installResult.stderr}`);
+        console.log("Binary Module Tests will fail if not skipped!");
     }
+    
     return installResult.stdout;
 }
 

test/runTestsInner.ts

@@ -35,10 +35,6 @@ function runTestsInner(testsRoot: string): Promise<void> {
     }
 
     const mocha = new Mocha(config);
-    // if (process.env.TF_BUILD) {
-    //     console.log("Detected Azure DevOps, disabling color output as ANSI escapes do not make Azure Devops happy.");
-    //     config.color = false;
-    // }
 
     // Test if files is empty
     const files = globSync(config.spec, { cwd: rootDir });