From 5939729c45c68e23be4013ab75b628405c425dab Mon Sep 17 00:00:00 2001 From: Ayke van Laethem Date: Wed, 6 Mar 2019 10:45:34 +0100 Subject: [PATCH] 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. --- main_test.go | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/main_test.go b/main_test.go index 1eb6199d..80832288 100644 --- a/main_test.go +++ b/main_test.go @@ -81,16 +81,16 @@ func TestCompiler(t *testing.T) { runTest(path, tmpdir, "aarch64--linux-gnu", t) }) } - } - t.Log("running tests for WebAssembly...") - for _, path := range matches { - if path == "testdata/gc.go" { - continue // known to fail + t.Log("running tests for WebAssembly...") + for _, path := range matches { + if path == "testdata/gc.go" { + continue // known to fail + } + t.Run(path, func(t *testing.T) { + runTest(path, tmpdir, "wasm", t) + }) } - t.Run(path, func(t *testing.T) { - runTest(path, tmpdir, "wasm", t) - }) } }