Skip to content
This repository was archived by the owner on Jul 22, 2024. It is now read-only.

ERC20 test with cairo2 #767

Merged
merged 17 commits into from
Jul 5, 2023
Merged

ERC20 test with cairo2 #767

merged 17 commits into from
Jul 5, 2023

Conversation

SantiagoPittella
Copy link
Collaborator

Add a test for ERC20 contract with Cairo2

Description

Adds:

  • ERC20 contract with modifications.
    • Instead of using for max_supply I replaced it with felt252 due to a lack of support on our side.
  • A contract that deploys the ERC20 contract.
  • A tests that deploys 2 accounts, and uses the deploy contract to deploy to ERC20 with a new token and a token supply for one of the previous accounts, then transfers a certain amount of tokens into the other account. Finally, checks the balances.

Checklist

  • Linked to Github Issue
  • Unit tests added
  • Integration tests added.
  • This change requires new documentation.
    • Documentation has been added/updated.

Copy link
Contributor

@fmoletta fmoletta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great!

SantiagoPittella and others added 5 commits July 5, 2023 14:42
Co-authored-by: Matías Ignacio González <[email protected]>
Co-authored-by: Matías Ignacio González <[email protected]>
Co-authored-by: Matías Ignacio González <[email protected]>
@codecov-commenter
Copy link

codecov-commenter commented Jul 5, 2023

Codecov Report

Merging #767 (efda09e) into main (8f40853) will increase coverage by 0.13%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #767      +/-   ##
==========================================
+ Coverage   92.50%   92.64%   +0.13%     
==========================================
  Files          50       51       +1     
  Lines       11899    12110     +211     
==========================================
+ Hits        11007    11219     +212     
+ Misses        892      891       -1     
Impacted Files Coverage Δ
src/testing/mod.rs 100.00% <ø> (ø)
src/lib.rs 98.71% <100.00%> (+<0.01%) ⬆️
src/testing/erc20.rs 100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

@SantiagoPittella SantiagoPittella added this pull request to the merge queue Jul 5, 2023
Merged via the queue into main with commit 384f459 Jul 5, 2023
@juanbono juanbono deleted the erc20-cairo2 branch July 31, 2023 23:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants