kakarot-ssj
kakarot-ssj copied to clipboard
dev: replace `unwrap` with `expect`
Replace all occurrences of unwrap with expect, making it a lot easier to debug during integration testing. Having unwrap in the codebase causes the VM to crash with Result::unwrap failed., whereas expect returns the error message passed to expect.
hey @greged93 may i try to work on this one? Should i find by myself which felt252 error message to give as input to expect function? thank you.
@TAdev0 Could you wait a bit until we scope this issue with more precision? For now we didn't identify which parts could really benefit from this.
Sure, thank you!
@greged93 @enitrat is better debugging with expect during integration tests still needed? if yes did you scope which parts need better debugging experience?
for now SSJ is not on the priority list, so not a lot of stuff going on here