Skip to content

Commit

Permalink
Merge pull request #2495 from tgodzik/ignore-test
Browse files Browse the repository at this point in the history
chore: Ignore test on linux for the time being
  • Loading branch information
tgodzik authored Nov 5, 2024
2 parents 750f592 + 8bd28c1 commit 4ffadcb
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 1 deletion.
8 changes: 8 additions & 0 deletions frontend/src/test/scala/bloop/bsp/BspBaseSuite.scala
Original file line number Diff line number Diff line change
Expand Up @@ -620,6 +620,14 @@ abstract class BspBaseSuite extends BaseSuite with BspClientTest {
}
}

def testMac(name: String)(fun: => Any): Unit = {
if (isMac) {
super.test(name)(fun)
} else {
super.ignore(name, "DISABLED")(fun)
}
}

private final lazy val tempDir = Files.createTempDirectory("temp-sockets")
tempDir.toFile.deleteOnExit()

Expand Down
2 changes: 1 addition & 1 deletion frontend/src/test/scala/bloop/bsp/BspCompileSpec.scala
Original file line number Diff line number Diff line change
Expand Up @@ -172,7 +172,7 @@ class BspCompileSpec(
}
}

testNonWindows(
testMac(
"create orphan client classes directory and make sure loading a BSP session cleans it up"
) {
TestUtil.withinWorkspace { workspace =>
Expand Down
1 change: 1 addition & 0 deletions frontend/src/test/scala/bloop/testing/BaseSuite.scala
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@ import utest.ufansi.Str
abstract class BaseSuite extends TestSuite with BloopHelpers {
val pprint = _root_.pprint.PPrinter.BlackWhite
def isWindows: Boolean = bloop.util.CrossPlatform.isWindows
def isMac: Boolean = bloop.util.CrossPlatform.isMac
def isAppveyor: Boolean = "True" == System.getenv("APPVEYOR")
def beforeAll(): Unit = ()
def afterAll(): Unit = ()
Expand Down

0 comments on commit 4ffadcb

Please sign in to comment.