From 9476d4c64711e4c53f9aab8fd86bbcc9842bd484 Mon Sep 17 00:00:00 2001 From: Svyatoslav Scherbina Date: Fri, 19 Nov 2021 11:54:46 +0000 Subject: [PATCH] Native: use Gradle exec for jsinterop invocation in jsinterop_math test The previous implementation didn't log anything, including stdout, stderr and the command line, and didn't detect non-zero exit code, making it hard to investigate the test failures. --- kotlin-native/backend.native/tests/build.gradle | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/kotlin-native/backend.native/tests/build.gradle b/kotlin-native/backend.native/tests/build.gradle index e2ba0643703..c2ce651f64b 100644 --- a/kotlin-native/backend.native/tests/build.gradle +++ b/kotlin-native/backend.native/tests/build.gradle @@ -5013,7 +5013,9 @@ standaloneTest("jsinterop_math") { def jsinteropScript = isWindows() ? "jsinterop.bat" : "jsinterop" def jsinterop = "$kotlinNativeDist/bin/$jsinteropScript" // TODO: We probably need a NativeInteropPlugin for jsinterop? - "$jsinterop -pkg kotlinx.interop.wasm.math -o $buildDir/jsmath -target wasm32".execute().waitFor() + exec { + commandLine jsinterop, "-pkg", "kotlinx.interop.wasm.math", "-o", "$buildDir/jsmath", "-target", "wasm32" + } } dependsOn ':kotlin-native:wasm32PlatformLibs'