Skip to content

Commit

Permalink
Migrate Starknet Account Preset tests (#1069)
Browse files Browse the repository at this point in the history
* Update test utils and helpers

* Update signature tests

* Update account tests

* Update dual account tests

* Run linter

* Run linter

* Fix review issues

* Update ignore reason messages

* Run linter

* Support eth account tests changes

* Run linter

* Improve setup functions, remove unused imports

* Remove unnecessary accept_ownership step, make use of serialized_sign fn

* Migrate Starknet Account Preset tests, make some code improvements

* Run linter

* Remove unnecessary clone

* Update the comment for assert_entrypoint_not_found_error function

* Minor review fixes
  • Loading branch information
immrsd authored Jul 29, 2024
1 parent bc316ca commit 2e0f3c6
Show file tree
Hide file tree
Showing 9 changed files with 275 additions and 237 deletions.
2 changes: 1 addition & 1 deletion src/tests/account/ethereum/test_eth_account.cairo
Original file line number Diff line number Diff line change
Expand Up @@ -316,7 +316,7 @@ fn test_multicall() {
to: erc20.contract_address, selector: selectors::transfer, calldata: calldata2.span()
};

// Bundle calls and exeute
// Bundle calls and execute
calls.append(call1);
calls.append(call2);
let ret = account.__execute__(calls);
Expand Down
2 changes: 1 addition & 1 deletion src/tests/account/starknet/test_account.cairo
Original file line number Diff line number Diff line change
Expand Up @@ -308,7 +308,7 @@ fn test_multicall() {
to: erc20.contract_address, selector: selectors::transfer, calldata: calldata2.span()
};

// Bundle calls and exeute
// Bundle calls and execute
let calls = array![call1, call2];
let ret = account.__execute__(calls);

Expand Down
2 changes: 1 addition & 1 deletion src/tests/presets.cairo
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
// mod test_account;
mod test_account;
mod test_erc1155;
mod test_erc20;
mod test_erc721;
Expand Down
Loading

0 comments on commit 2e0f3c6

Please sign in to comment.