Merge pull request #111 from kubernetes-sigs/fix-testdat

make sure if there's no diff in ci

Author: k8s-ci-robot

.github/workflows/testdata.yaml

@@ -47,8 +47,12 @@ jobs:
 
       - name: "Set up wat2wasm"
         run: |  # Needed for testdata. wabt includes wat2wasm.
-          wabt_version=1.0.33
-          wabt_url=https://github.com/WebAssembly/wabt/releases/download/${wabt_version}/wabt-${wabt_version}-ubuntu.tar.gz
+          wabt_version=1.0.35
+          wabt_url=https://github.com/WebAssembly/wabt/releases/download/${wabt_version}/wabt-${wabt_version}-ubuntu-20.04.tar.gz
           curl -sSL ${wabt_url} | tar --strip-components 2 -C /usr/local/bin -xzf - wabt-${wabt_version}/bin/wat2wasm
 
-      - run: make testdata
+      - name: "generate testdata"
+        run: make testdata
+
+      - name: Check diff
+        run: git diff --exit-code -- .

Makefile

@@ -36,15 +36,13 @@ guest/.tinygo-target.json: scheduler/go.mod
 
 .PHONY: build-wat
 build-wat: $(wildcard scheduler/test/testdata/*/*.wat)
-	@for f in $^; do \
+	@set -e; for f in $^; do \
         wasm=$$(echo $$f | sed -e 's/\.wat/\.wasm/'); \
 		wat2wasm -o $$wasm --debug-names $$f; \
 	done
 
 .PHONY: testdata
-testdata:
-	@$(MAKE) build-tinygo
-	@$(MAKE) build-wat
+testdata: build-tinygo build-wat
 
 .PHONY: profile
 profile: examples/advanced/main-debug.wasm

scheduler/test/testdata/test/bind_from_global.wasm

@@ -9,7 +9,7 @@
 
   (func (export "postbind")
     (if (i32.eq (global.get $flag) (i32.const 1))
-      (unreachable)
+      (then unreachable)
     )
   )
 

scheduler/test/testdata/test/postbind_from_global.wasm

@@ -14,7 +14,7 @@
 
   (func (export "unreserve")
     (if (i32.eq (global.get $flag) (i32.const 1))
-      (unreachable)
+      (then unreachable)
     )
   )
 )