Skip to content

Commit 5939729

Browse files
aykevldeadprogram
authored andcommitted
main: only run WebAssembly tests on Linux
The WebAssembly target is not yet considered stable in LLVM 7, but has been enabled in the Debian builds so tests can run on Debian. However, the Homebrew builds don't have it enabled which results in test failures. Temporarily run WebAssembly tests only on Linux to fix this. This can be reverted after a switch to LLVM 8, which has WebAssembly enabled by default.
1 parent c7b91da commit 5939729

File tree

1 file changed

+8
-8
lines changed

1 file changed

+8
-8
lines changed

main_test.go

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -81,16 +81,16 @@ func TestCompiler(t *testing.T) {
8181
runTest(path, tmpdir, "aarch64--linux-gnu", t)
8282
})
8383
}
84-
}
8584

86-
t.Log("running tests for WebAssembly...")
87-
for _, path := range matches {
88-
if path == "testdata/gc.go" {
89-
continue // known to fail
85+
t.Log("running tests for WebAssembly...")
86+
for _, path := range matches {
87+
if path == "testdata/gc.go" {
88+
continue // known to fail
89+
}
90+
t.Run(path, func(t *testing.T) {
91+
runTest(path, tmpdir, "wasm", t)
92+
})
9093
}
91-
t.Run(path, func(t *testing.T) {
92-
runTest(path, tmpdir, "wasm", t)
93-
})
9494
}
9595
}
9696

0 commit comments

Comments
 (0)