Skip to content
This repository has been archived by the owner on Nov 11, 2024. It is now read-only.

Commit

Permalink
docs: update the comment for the syscall to actual behavior
Browse files Browse the repository at this point in the history
  • Loading branch information
glihm committed Nov 5, 2024
1 parent c9d86e4 commit 4bbfe89
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions crates/contracts/src/contract/components/upgradeable.cairo
Original file line number Diff line number Diff line change
Expand Up @@ -50,9 +50,10 @@ pub mod upgradeable_cpt {
);
assert(new_class_hash.is_non_zero(), Errors::INVALID_CLASS);

// Seems like the match doesn't catch the error if the entrypoint is
// not found. This is observed on public network like Sepolia.
// On the cairo runner, behavior seems to be as expected.
// Currently - any syscall that fails on starknet - fails the transaction, and it won't
// be included in any block.
// The test runner does not simulate this, but instead simulates the future behavior
// when errors can be recovered from.
match starknet::syscalls::library_call_syscall(
new_class_hash, selector!("dojo_name"), [].span(),
) {
Expand Down

0 comments on commit 4bbfe89

Please sign in to comment.