Skip to content

Add ECDSA secp256k1 verification Solidity tests #3048

@Savio-Sou

Description

@Savio-Sou

Context

Now that basic Solidity verifier tests have been reintroduced with #2997, it would be great to expand the tests to cover more barretenberg features exposed through Noir to better guarantee they work end-to-end from a Noir app dev perspective.

Problem

Out of all features, I would suggest prioritizing the testing of ECDSA secp256k1 verification given issues reported with the particular function:

Adding the test for it asap should help provide further insights on how to debug the existing issues.

Objectives

(Open to discussion as I am in lack of barretenberg's technical context to judge if these make sense.)

Add tests that:

  1. Generate a ECDSA secp256k1 verification proof
  2. Verify the proof natively
  3. Generate a corresponding Solidity verifier
  4. Verify the proof via the Solidity verifier

Using:

  1. Native barretenberg
  2. bb binary
  3. bb.js

Metadata

Metadata

Assignees

Labels

C-barretenbergComponent: barretenberg cryptography library

Type

No type
No fields configured for issues without a type.

Projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions