Skip to content

Commit 6ffe942

Browse files
authored
Merge pull request #1969 from CosmWasm/1918-panic-test
Fix panic test
2 parents 04a0e52 + 2be2cf4 commit 6ffe942

File tree

2 files changed

+7
-1
lines changed

2 files changed

+7
-1
lines changed

README.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,11 @@ This code is compiled into Wasm bytecode as part of the smart contract.
7575
some given source code, allowing a
7676
[similar contract verification algorithm](https://medium.com/coinmonks/how-to-verify-and-publish-on-etherscan-52cf25312945)
7777
as Etherscan.
78+
79+
Building locally instead of using the docker image can
80+
[leak some information about the directory structure of your system](https://github.com/CosmWasm/cosmwasm/issues/1918)
81+
and makes the build non-reproducible.
82+
7883
- [serde-json-wasm](https://github.com/CosmWasm/serde-json-wasm) - A custom json
7984
library, forked from `serde-json-core`. This provides an interface similar to
8085
`serde-json`, but without any floating-point instructions (non-deterministic)

contracts/hackatom/tests/integration.rs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -453,7 +453,8 @@ fn execute_panic() {
453453
match execute_res.unwrap_err() {
454454
VmError::RuntimeErr { msg, .. } => {
455455
assert!(
456-
msg.contains("Aborted: panicked at 'This page intentionally faulted'"),
456+
msg.contains("Aborted: panicked")
457+
&& msg.contains("This page intentionally faulted"),
457458
"Must contain panic message"
458459
);
459460
assert!(msg.contains("contract.rs:"), "Must contain file and line");

0 commit comments

Comments
 (0)