Skip to content

chore: retire legacy curve mapping#1082

Open
a1denvalu3 wants to merge 1 commit into
mainfrom
fix/remove-legacy-h2c
Open

chore: retire legacy curve mapping#1082
a1denvalu3 wants to merge 1 commit into
mainfrom
fix/remove-legacy-h2c

Conversation

@a1denvalu3

Copy link
Copy Markdown
Collaborator

Summary

  • remove the obsolete curve-mapping compatibility path
  • use the current mapping consistently in wallet and proof verification
  • add regression coverage for mapping aliases

Testing

  • poetry run pytest tests/test_crypto.py tests/mint/test_mint_verification.py -q
  • make check

@codecov

codecov Bot commented Jul 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.49%. Comparing base (a935eb9) to head (49eef78).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1082      +/-   ##
==========================================
- Coverage   74.54%   74.49%   -0.05%     
==========================================
  Files         115      115              
  Lines       13195    13159      -36     
==========================================
- Hits         9836     9803      -33     
+ Misses       3359     3356       -3     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

@a1denvalu3 a1denvalu3 requested a review from callebtc July 14, 2026 17:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

1 participant