Merge pull request #16 from peterhuene/fix-ci

Peter Huene · web-flow · commit 7fc10e99acea · 2020-06-09T18:16:50.000-07:00
Update expected trap message for backtrace.
diff --git a/.github/workflows/main.yml b/.github/workflows/main.yml
@@ -13,6 +13,7 @@ jobs:
     name: Test .NET embedding of Wasmtime
     runs-on: ${{ matrix.os }}
     strategy:
+      fail-fast: false
       matrix:
         build: [linux-debug, linux-release, macos-debug, macos-release, windows-debug, windows-release]
         include:
diff --git a/tests/FunctionThunkingTests.cs b/tests/FunctionThunkingTests.cs
@@ -66,7 +66,9 @@ public void ItPropagatesExceptionsToCallersViaTraps()
             action
                 .Should()
                 .Throw<TrapException>()
-                .WithMessage(THROW_MESSAGE);
+                // Ideally this should contain a check for the backtrace
+                // See: https://github.com/bytecodealliance/wasmtime/issues/1845
+                .WithMessage(THROW_MESSAGE + "*");
         }
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -66,7 +66,9 @@ public void ItPropagatesExceptionsToCallersViaTraps()`
`66`	`66`	`action`
`67`	`67`	`.Should()`
`68`	`68`	`.Throw<TrapException>()`
`69`		`- .WithMessage(THROW_MESSAGE);`
	`69`	`+ // Ideally this should contain a check for the backtrace`
	`70`	`+ // See: https://github.com/bytecodealliance/wasmtime/issues/1845`
	`71`	`+ .WithMessage(THROW_MESSAGE + "*");`
`70`	`72`	`}`
`71`	`73`	`}`
`72`	`74`	`}`