Skip to content

[pull] forks/amsterdam from ethereum:forks/amsterdam#119

Merged
pull[bot] merged 1 commit intoLuckQuack:forks/amsterdamfrom
ethereum:forks/amsterdam
Mar 12, 2026
Merged

[pull] forks/amsterdam from ethereum:forks/amsterdam#119
pull[bot] merged 1 commit intoLuckQuack:forks/amsterdamfrom
ethereum:forks/amsterdam

Conversation

@pull
Copy link

@pull pull bot commented Mar 12, 2026

See Commits and Changes for more details.


Created by pull[bot] (v2.0.0-alpha.4)

Can you help keep this open source service alive? 💖 Please sponsor : )

* feat(tests): port stZeroKnowledge ECADD static tests

Port pointAddFiller.json and pointAddTruncFiller.json from
stZeroKnowledge into the existing test_ecadd.py test suite,
adding test vectors that were not already covered:

Valid: S1 doubling, S1+S1x2, S1+generator, truncated S1 input.
Invalid: valid+not-on-curve, not-on-curve+valid, both not-on-curve.

Also fix pre-existing bugs: missing commas in ported_from URL
lists (caused implicit string concatenation) and duplicate test
ID "Pplus1_2_plus_inf".

* fix: use ethereum/tests URLs instead of legacytests in ported_from

* fix(tests): add pr= link for #2477 to ecadd ported_from markers
@pull pull bot locked and limited conversation to collaborators Mar 12, 2026
@pull pull bot added the ⤵️ pull label Mar 12, 2026
@pull pull bot merged commit 348a423 into LuckQuack:forks/amsterdam Mar 12, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant