Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Include additional tests to BLS bindings #384

Merged
merged 8 commits into from
Apr 12, 2023
Merged

Conversation

iquerejeta
Copy link
Collaborator

Including more unit tests to BLS bindings. This covers all tests included in the Plutus PR#5231, allowing the latter to reduce its test suite.

Similarly we expose the zero element of both Group1 and Group2.

@iquerejeta iquerejeta force-pushed the iquerejeta/bls-tests branch from c94aad8 to e189c31 Compare April 10, 2023 08:10
@iquerejeta iquerejeta marked this pull request as ready for review April 10, 2023 08:10
@iquerejeta iquerejeta requested review from lehins and tdammers April 10, 2023 08:10
Copy link
Collaborator

@lehins lehins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Better generator could be used, otherwise looks good.

cardano-crypto-tests/src/Test/Crypto/EllipticCurve.hs Outdated Show resolved Hide resolved
cardano-crypto-tests/src/Test/Crypto/EllipticCurve.hs Outdated Show resolved Hide resolved
cardano-crypto-tests/src/Test/Crypto/EllipticCurve.hs Outdated Show resolved Hide resolved
cardano-crypto-tests/src/Test/Crypto/EllipticCurve.hs Outdated Show resolved Hide resolved
cardano-crypto-tests/src/Test/Crypto/EllipticCurve.hs Outdated Show resolved Hide resolved
@iquerejeta iquerejeta requested a review from lehins April 11, 2023 14:53
Copy link
Collaborator

@lehins lehins left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@iquerejeta iquerejeta force-pushed the iquerejeta/bls-tests branch from d0c5efd to c985393 Compare April 12, 2023 11:10
@iquerejeta iquerejeta merged commit 2c99dea into master Apr 12, 2023
lehins added a commit that referenced this pull request May 18, 2023
* Include additional tests to BLS bindings

* Final verify fails with random values

* Add check that blsZero is infinity, and simplify code

* Generate random integers from larger space

Co-authored-by: Alexey Kuleshevich <alexey.kuleshevich@iohk.io>
lehins added a commit that referenced this pull request May 23, 2023
* Include additional tests to BLS bindings

* Final verify fails with random values

* Add check that blsZero is infinity, and simplify code

* Generate random integers from larger space

Co-authored-by: Alexey Kuleshevich <alexey.kuleshevich@iohk.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants