Merge pull request #1245 from tgodzik/support-theia

bugfix: Check if tests namespace is enabled
scalameta · Dec 5, 2022 · 4db148c · 4db148c
2 parents f4eafd6 + b2cede7
commit 4db148c
Showing 1 changed file with 5 additions and 1 deletion.
diff --git a/src/extension.ts b/src/extension.ts
@@ -28,6 +28,7 @@ import {
   ProviderResult,
   Hover,
   TextDocument,
+  tests as vscodeTextExplorer,
 } from "vscode";
 import {
   LanguageClient,
@@ -600,7 +601,10 @@ function launchMetals(
           "Test Explorer"
         );
 
-      const istTestManagerDisabled = getTestUI() === "Code Lenses";
+      // vscodeTextExplorer can be undefined e.g. for eclipse theia,
+      // see https://github.com/scalameta/metals-vscode/discussions/1244
+      const istTestManagerDisabled =
+        !vscodeTextExplorer || getTestUI() === "Code Lenses";
       const testManager = createTestManager(client, istTestManagerDisabled);
 
       const disableTestExplorer = workspace.onDidChangeConfiguration(() => {