fix: do not panic! on ErrorMap::Panic

csgui · csgui · commit 09f06c9a9797 · 2024-10-11T18:13:19.000+01:00
diff --git a/clarity/src/vm/clarity_wasm.rs b/clarity/src/vm/clarity_wasm.rs
@@ -7810,7 +7810,9 @@ mod error_mapping {
                 Error::Runtime(RuntimeErrorType::UnwrapFailure, Some(Vec::new()))
             }
             ErrorMap::Panic => {
-                panic!("An error has been detected in the code")
+                // TODO: see issue: #531
+                // This RuntimeErrorType::UnwrapFailure need to have a proper context.
+                Error::Runtime(RuntimeErrorType::UnwrapFailure, Some(Vec::new()))
             }
             // TODO: UInt(42) value below is just a placeholder.
             // It should be replaced by the current "thrown-value" when clarity-wasm issue #385 is resolved.

Original file line number	Diff line number	Diff line change
`@@ -7810,7 +7810,9 @@ mod error_mapping {`
`7810`	`7810`	`Error::Runtime(RuntimeErrorType::UnwrapFailure, Some(Vec::new()))`
`7811`	`7811`	`}`
`7812`	`7812`	`ErrorMap::Panic => {`
`7813`		`- panic!("An error has been detected in the code")`
	`7813`	`+ // TODO: see issue: #531`
	`7814`	`+ // This RuntimeErrorType::UnwrapFailure need to have a proper context.`
	`7815`	`+ Error::Runtime(RuntimeErrorType::UnwrapFailure, Some(Vec::new()))`
`7814`	`7816`	`}`
`7815`	`7817`	`// TODO: UInt(42) value below is just a placeholder.`
`7816`	`7818`	`// It should be replaced by the current "thrown-value" when clarity-wasm issue #385 is resolved.`