Skip to content

Conversation

@ImTei
Copy link
Collaborator

@ImTei ImTei commented Feb 20, 2024

Description

Added fuzz tests for syscall implementation. based on cannon's fuzz_evm_test.go
New tests check if the syscall implementations(fast, slow, evm) work as expected.
This PR does not cover every syscalls yet.

  • sys_read
    • FuzzStateHintRead
    • FuzzStatePreimageRead
  • sys_write
    • FuzzStateHintWrite
    • FuzzStatePreimageWrite
  • Unsupported syscalls
    • TestStateSyscallUnsupported
  • No-op (ignored) syscalls
    • FuzzStateSyscallNoop
  • sys_exit, sys_exitgroup
    • FuzzStateSyscallExit

@ImTei ImTei requested a review from pcw109550 February 20, 2024 12:06
@ImTei ImTei changed the title Syscall test & fuzz Syscall fuzz test Feb 20, 2024
@ImTei ImTei requested a review from pcw109550 February 23, 2024 06:14
@ImTei ImTei mentioned this pull request Mar 6, 2024
@pcw109550 pcw109550 mentioned this pull request Mar 12, 2024
@ImTei ImTei force-pushed the tip/fuzz_evm_test branch from 716929f to 552103e Compare March 12, 2024 08:11
Copy link
Contributor

@pcw109550 pcw109550 left a comment

Choose a reason for hiding this comment

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

Left some suggestions.

@ImTei ImTei enabled auto-merge March 14, 2024 07:26
@ImTei ImTei disabled auto-merge March 18, 2024 01:30
@ImTei ImTei enabled auto-merge March 18, 2024 01:31
@ImTei ImTei added this pull request to the merge queue Mar 18, 2024
Merged via the queue into master with commit 8d7f523 Mar 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants