[Wasm] Don't write .wat files in box tests

skuzmich · Space · commit 813b600555fb · 2022-05-13T11:20:13.000Z
These files take space on CI, but they are not validated or processed
automatically in any way.

Disabled by default, enabled via kotlin.wasm.debugMode=1
diff --git a/compiler/ir/backend.wasm/src/org/jetbrains/kotlin/backend/wasm/compiler.kt b/compiler/ir/backend.wasm/src/org/jetbrains/kotlin/backend/wasm/compiler.kt
@@ -168,9 +168,12 @@ fun writeCompilationResult(
     dir: File,
     loaderKind: WasmLoaderKind,
     fileNameBase: String = "index",
+    writeWat: Boolean = true,
 ) {
     dir.mkdirs()
-    File(dir, "$fileNameBase.wat").writeText(result.wat)
+    if (writeWat) {
+        File(dir, "$fileNameBase.wat").writeText(result.wat)
+    }
     File(dir, "$fileNameBase.wasm").writeBytes(result.wasm)
     val jsWithLoader = generateJsWasmLoader(loaderKind, "./$fileNameBase.wasm", result.js)
     File(dir, "$fileNameBase.js").writeText(jsWithLoader)
diff --git a/js/js.tests/test/org/jetbrains/kotlin/js/testOld/BasicWasmBoxTest.kt b/js/js.tests/test/org/jetbrains/kotlin/js/testOld/BasicWasmBoxTest.kt
@@ -176,7 +176,7 @@ abstract class BasicWasmBoxTest(
                     println(" ------ $name Test file://$path/test.js")
                 }
 
-                writeCompilationResult(res, dir, WasmLoaderKind.D8)
+                writeCompilationResult(res, dir, WasmLoaderKind.D8, writeWat = debugMode >= DebugMode.DEBUG)
                 File(dir, "test.js").writeText(testJs)
                 ExternalTool(System.getProperty("javascript.engine.path.V8"))
                     .run(

Original file line number	Diff line number	Diff line change
`@@ -176,7 +176,7 @@ abstract class BasicWasmBoxTest(`
`176`	`176`	`println(" ------ $name Test file://$path/test.js")`
`177`	`177`	`}`
`178`	`178`
`179`		`- writeCompilationResult(res, dir, WasmLoaderKind.D8)`
	`179`	`+ writeCompilationResult(res, dir, WasmLoaderKind.D8, writeWat = debugMode >= DebugMode.DEBUG)`
`180`	`180`	`File(dir, "test.js").writeText(testJs)`
`181`	`181`	`ExternalTool(System.getProperty("javascript.engine.path.V8"))`
`182`	`182`	`.run(`