Skip to content
This repository has been archived by the owner on Dec 15, 2023. It is now read-only.

Enable using default_entrypoint by switching to invoke_raw #78

Merged
merged 11 commits into from
Apr 13, 2022
Merged

Enable using default_entrypoint by switching to invoke_raw #78

merged 11 commits into from
Apr 13, 2022

Conversation

FabijanC
Copy link
Collaborator

@FabijanC FabijanC commented Apr 12, 2022 •
edited
Loading

Usage related changes

Development related changes

  • Use invoke_raw for invokes and calls.
    • This also made the adapt utilities in adapt.py redundant.
  • Introduce decorator (wrapper) which runs devnet in background and prints its log in case of AssertionError:
    • Not applied to all tests, can be expanded in the future.

Checklist:

  • Performed a self-review of the code
  • Rebased to the base branch
  • Documented the changes - None required.
  • Updated the tests - None required.

@FabijanC FabijanC force-pushed the default-entrypoint branch from c6b5be3 to 6cfac85 Compare April 12, 2022 12:43
@FabijanC FabijanC requested a review from badurinantun April 12, 2022 12:43
@FabijanC FabijanC marked this pull request as ready for review April 12, 2022 12:45
badurinantun
badurinantun reviewed Apr 12, 2022
View reviewed changes
test/util.py Outdated Show resolved Hide resolved
starknet_devnet/transaction_wrapper.py Outdated Show resolved Hide resolved
starknet_devnet/contract_wrapper.py Outdated Show resolved Hide resolved
@FabijanC FabijanC merged commit 45667c8 into master Apr 13, 2022
@FabijanC FabijanC deleted the default-entrypoint branch April 13, 2022 08:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@badurinantun badurinantun badurinantun approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Deploy argent-x wallet contract
2 participants
@FabijanC @badurinantun