BLS tests: only keep the Ethereum specific tests

ethereum / consensus-spec-tests

Common tests for the Ethereum proof-of-stake consensus layer

MIT License

75 stars 24 forks source link

BLS tests: only keep the Ethereum specific tests #25

Open mratsim opened 3 years ago

mratsim commented 3 years ago

The current consensus-spec-tests tarballs (v1.1.2) still include the following tests:

aggregate
aggregate_verify
fast_aggregate_verify
sign
verify

Those were superseded by https://github.com/ethereum/bls12-381-tests.

Should we remove those and only keep ethereum specific tests in this repo?

eth_aggregate_pubkeys
eth_fast_aggregate_verify

cc @hwwhww @asanso

hwwhww commented 3 years ago

👍 on removing IETF APIs test cases from consensus-spec-tests. We only have to ensure that the test format is the same for reducing client teams' burden.