open-dust/cairo-foundry

skip hint is no more functional after cairo-rs update

ptisserand opened this issue · 2 comments

After updating cairo-rs, it seems that "skip" hint is no more functionnal.

Here is the output of cairo-foundry test:

$ ./target/debug/cairo-foundry test
Error: binary '/home/robin/cairo_venv/bin/cairo-compile' failed to compile './test_cairo_contracts/test_invalid_program.cairo:18:17: Unknown identifier '__main__.balance.read'.
    let (res) = balance.read();
                ^**********^
'
Error: An ASSERT_EQ instruction failed: 50 != 55.
Error: An ASSERT_EQ instruction failed: 3 != 2.
Running tests in file ./test_cairo_hints/test_mock_call.cairo
[OK] test_mock_call (543.961µs)


Error: Got an exception while executing a hint: Custom Hint Error: skip

Maybe it's related to the following info in the changelog:
VirtualMachineErrors produced by HintProcessor::execute_hint() will be wrapped in a VirtualMachineError::Hint error containing their hint_index
https://github.com/lambdaclass/cairo-rs/pull/673/files

Look like it is now. Thanks @ptisserand