Fix incorrect wat in tests (WebAssembly#6207)

The new wat parser is much more strict than the legacy wat parser; the latter
accepts all sorts of things that the spec does not allow. To ease an eventual
transition to using the new wat parser by default, update the tests to use the
standard text format in many places where they previously did not. We do not yet
have a way to prevent new errors from being introduced into the test suite, but
at least there will now be many fewer errors when it comes time to make the
switch.

Author: tlively

test/ctor-eval/bad-indirect-call.wast

@@ -4,7 +4,7 @@
   (data (i32.const 10) "waka waka waka waka waka")
   (table 1 1 funcref)
   (elem (i32.const 0) $call-indirect)
-  (export "test1" $test1)
+  (export "test1" (func $test1))
   (func $test1
     (call_indirect (type $v) (i32.const 1)) ;; unsafe to call, out of range
     (i32.store8 (i32.const 20) (i32.const 120))

test/ctor-eval/bad-indirect-call2.wast

@@ -5,7 +5,7 @@
   (import "env" "_abort" (func $_abort))
   (table 2 2 funcref)
   (elem (i32.const 0) $_abort $call-indirect)
-  (export "test1" $test1)
+  (export "test1" (func $test1))
   (func $test1
     (call_indirect (type $v) (i32.const 0)) ;; unsafe to call, imported
     (i32.store8 (i32.const 20) (i32.const 120))

test/ctor-eval/basics-flatten.wast

@@ -7,9 +7,9 @@
   (data (i32.const 20) "waka waka waka")
   (table 1 1 funcref)
   (elem (i32.const 0) $call-indirect)
-  (export "test1" $test1)
-  (export "test2" $test2)
-  (export "test3" $test3)
+  (export "test1" (func $test1))
+  (export "test2" (func $test2))
+  (export "test3" (func $test3))
   (func $test1
     (drop (i32.const 0)) ;; no work at all, really
     (call $safe-to-call) ;; safe to call

test/ctor-eval/basics.wast

@@ -4,9 +4,9 @@
   (data (i32.const 10) "waka waka waka waka waka")
   (table 1 1 funcref)
   (elem (i32.const 0) $call-indirect)
-  (export "test1" $test1)
-  (export "test2" $test2)
-  (export "test3" $test3)
+  (export "test1" (func $test1))
+  (export "test2" (func $test2))
+  (export "test3" (func $test3))
   (func $test1
     (drop (i32.const 0)) ;; no work at all, really
     (call $safe-to-call) ;; safe to call

test/ctor-eval/imported-global.wast

@@ -3,7 +3,7 @@
   (data (i32.const 10) "waka waka waka waka waka")
   ;; imports must not be used
   (import "env" "tempDoublePtr" (global $tempDoublePtr i32))
-  (export "test1" $test1)
+  (export "test1" (func $test1))
   (global $mine (mut i32) (global.get $tempDoublePtr)) ;; BAD, if used
   (func $test1
     (i32.store8 (i32.const 13) (i32.const 115)) ;; we never get here.

test/ctor-eval/indirect-call3.wast

@@ -5,7 +5,7 @@
   (import "env" "_abort" (func $_abort))
   (table 2 2 funcref)
   (elem (i32.const 0) $_abort $call-indirect)
-  (export "test1" $test1)
+  (export "test1" (func $test1))
   (func $test1
     (call_indirect (type $v) (i32.const 1)) ;; safe to call
     (i32.store8 (i32.const 20) (i32.const 120))

test/ctor-eval/just_some.wast

@@ -1,9 +1,9 @@
 (module
   (memory 256 256)
   (data (i32.const 10) "waka waka waka waka waka")
-  (export "test1" $test1)
-  (export "test2" $test2)
-  (export "test3" $test3)
+  (export "test1" (func $test1))
+  (export "test2" (func $test2))
+  (export "test3" (func $test3))
   (func $test1
     (i32.store8 (i32.const 12) (i32.const 115)) ;; a safe store, should alter memory
   )

test/ctor-eval/partial-locals-tee.wast

@@ -4,7 +4,7 @@
   (memory 256 256)
   (data (i32.const 10) "_________________")
 
-  (export "test1" $test1)
+  (export "test1" (func $test1))
 
   (func $test1
     (local $temp i32)

test/ctor-eval/partial-locals.wast

@@ -4,7 +4,7 @@
   (memory 256 256)
   (data (i32.const 10) "_________________")
 
-  (export "test1" $test1)
+  (export "test1" (func $test1))
 
   (global $sp (mut i32) (i32.const 100))
 

test/ctor-eval/partial-return.wast

@@ -4,7 +4,7 @@
   (memory 256 256)
   (data (i32.const 10) "_________________")
 
-  (export "test1" $test1)
+  (export "test1" (func $test1))
   (export "memory" (memory $0))
 
   (func $test1