bugfix: Also ignore test case in CodeLensLspSuite

scalameta · Oct 19, 2024 · 97cc557 · 97cc557
1 parent 0e3ada9
commit 97cc557
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 5 deletions.
diff --git a/tests/unit/src/main/scala/tests/BaseCodeLensLspSuite.scala b/tests/unit/src/main/scala/tests/BaseCodeLensLspSuite.scala
@@ -82,7 +82,7 @@ abstract class BaseCodeLensLspSuite(
 
   protected def testRunShellCommand(
       name: String,
-      javaHome: Option[String] = None,
+      javaHome: => Option[String] = None,
   ): Unit =
     test(name) {
       def javaHomeString(javaHome: String) = {

diff --git a/tests/unit/src/test/scala/tests/CodeLensLspSuite.scala b/tests/unit/src/test/scala/tests/CodeLensLspSuite.scala
@@ -329,10 +329,11 @@ class CodeLensLspSuite extends BaseCodeLensLspSuite("codeLenses") {
         |""".stripMargin
   )
 
-  testRunShellCommand(
-    "run-shell-command-old-java",
-    Some(JvmManager.create().get("8").toString()),
-  )
+  if (!isMacOS)
+    testRunShellCommand(
+      "run-shell-command-old-java",
+      Some(JvmManager.create().get("8").toString()),
+    )
   testRunShellCommand("run-shell-command")
   testRunShellCommand("run shell command")