Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: added ContractCaller solidity example contract (#534) #535

Merged
merged 1 commit into from
Nov 7, 2023

Conversation

quiet-node
Copy link
Member

Description:

  • added ContractCaller solidity example contract to cover

    • call(g, a, v, in, insize, out, outsize)
    • staticcall(g, a, in, insize, out, outsize)
    • callcode(g, a, v, in, insize, out, outsize)
    • delegatecall(g, a, in, insize, out, outsize)
  • added unit tests to cover ContractCaller contract

Related issue(s):

Fixes #534

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

@github-actions
Copy link

github-actions bot commented Oct 26, 2023

Test Results

  20 files  +  6  102 suites  +38   12m 37s ⏱️ + 4m 54s
199 tests +  3  189 ✔️ +  1  6 💤 ±0  4 +2 
290 runs  +92  278 ✔️ +88  6 💤 ±0  6 +4 

For more details on these failures, see this check.

Results for commit 3105bda. ± Comparison against base commit e7dd7b7.

This pull request removes 1 and adds 4 tests. Note that renamed tests count towards both.
"before all" hook for "should test that hollow account is created and the amount of non-fungible tokens is correctly transferred via precompile" ‑ HIP583 Test Suite - Ethereum Transfer TX via Precompile Non-Fungible Token Test "before all" hook for "should test that hollow account is created and the amount of non-fungible tokens is correctly transferred via precompile"
"before all" hook for "should NOT be able to use transferFrom on fungible tokens without approval" ‑ TokenTransferContract Test Suite "before all" hook for "should NOT be able to use transferFrom on fungible tokens without approval"
should test that can make non-fungible token transfer via precompile from hollow account to another ‑ HIP583 Test Suite - Ethereum Transfer TX via Precompile Non-Fungible Token Test should test that can make non-fungible token transfer via precompile from hollow account to another
should test that hollow account is created and the amount of non-fungible tokens is correctly transferred via precompile ‑ HIP583 Test Suite - Ethereum Transfer TX via Precompile Non-Fungible Token Test should test that hollow account is created and the amount of non-fungible tokens is correctly transferred via precompile
should test that second transfer non-fungible tokens via precompile to the hollow account is successful ‑ HIP583 Test Suite - Ethereum Transfer TX via Precompile Non-Fungible Token Test should test that second transfer non-fungible tokens via precompile to the hollow account is successful

♻️ This comment has been updated with latest results.

Copy link
Collaborator

@Nana-EC Nana-EC left a comment

Choose a reason for hiding this comment

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

LG

@quiet-node quiet-node merged commit ce381d4 into main Nov 7, 2023
20 of 21 checks passed
@quiet-node quiet-node deleted the 534-Contract-Caller branch November 7, 2023 18:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request EVM evm related P2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Solidity Assembly Yul support] Contract Caller
2 participants