search

ChainSafe / blst-ts

Typescript wrapper for https://github.com/supranational/blst native bindings, a highly performant BLS12-381 signature library
Other
18 stars 13 forks source link

test(specs): aggregate and eth_aggregate_pubkeys #93

Closed matthewkeil closed 1 year ago

matthewkeil commented 1 year ago

This PR is related to #88 and covers the spec testing for aggregatePublicKeys and aggregateSignatures

note: spec eth_aggregate_pubkeys_x40_pubkey fails. This test is fixed in a subsequent PR

Inclusions

How to test

NOTE: to build and test copy the blst folder into rebuild/deps/blst. This will go away when we merge but for now node-gyp gets heartburn when building deps in folder above the binding.gyp file

cd rebuild
yarn
yarn download-spec-tests
yarn test:spec
CLAassistant commented 1 year ago

CLA assistant check
All committers have signed the CLA.

matthewkeil commented 1 year ago

@dapplion this is the test that is failing. It can be found here:

spec-tests/tests/general/altair/bls/eth_aggregate_pubkeys/small/eth_aggregate_pubkeys_x41_pubkey

https://github.com/ethereum/consensus-spec-tests/blob/master/tests/general/altair/bls/eth_aggregate_pubkeys/small/eth_aggregate_pubkeys_x40_pubkey/data.yaml

input: ['0x400000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000']
output: null