ethereum/consensus-specs

Remove phase0-capella state transition tests from the test vector output

hwwhww opened this issue · 2 comments

hwwhww commented

The size of the total test vectors keeps growing. IMO we can remove the meaningless state transition tests from the test vectors output.

Note that these tests should still be run and verified in CI with minimal preset to ensure spec correctness.

ralexstokes commented

looking into this...

  • ❤️1
potuz commented

As long as every single case (that is still applicable in later forks) is still covered in later forks that seems fine by me. We have found plenty of testvectors in forkchoice that were on specific forks or on specific presets and that actually didn't catch consensus bugs that would have been caught if the mainnet test vectors for the current fork were in place.

  • 👍1