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

Prague test fixtures runner instructions #3387

Closed
wants to merge 3 commits into from

Conversation

jochem-brouwer
Copy link
Member

This PR is not intended to get merged, but for instructions only

This PR describes how to run Prague fixtures, taken from https://github.com/ethereum/execution-spec-tests

For instance, these 2537 tests: https://github.com/ethereum/execution-spec-tests/files/15099569/fixtures_develop.tar.gz

  • Download the fixtures
  • Put the blockchain tests in a directory inside ./packages/ethereum-tests (for instance EIP2537)
  • Now from ./packages/vm:
  • npm run test:blockchain -- --fork=Prague --dir=../EIP2537

For 2537:

1..1099
# tests 1099
# pass  935
# fail  164

@acolytec3 acolytec3 mentioned this pull request May 3, 2024
@acolytec3
Copy link
Contributor

I think we can safely close this one as I've incorporated these changes into #3393

@acolytec3 acolytec3 closed this May 3, 2024
@jochem-brouwer
Copy link
Member Author

jochem-brouwer commented May 3, 2024

In your branch the tests do not run out of the box for me, have to add

      {
        name: 'prague',
        block: null,
      }

To chains.ts (in mainnet)

Also have to add it to config.ts to add it to supported forks (line 104)

@acolytec3
Copy link
Contributor

🤔 lemme look again. I hope I didn't miss that in the updated branch I merged to master.

@jochem-brouwer
Copy link
Member Author

Never mind!! Forgot to pull latest master - woops, sorry!

@holgerd77 holgerd77 deleted the eip2537-fixtures branch July 11, 2024 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants