Skip to content

Commit

Permalink
test: add tests for dynamic Dory commitment computation (#242)
Browse files Browse the repository at this point in the history
# Rationale for this change
A CPU implementation of the dynamic Dory commitment computation was
implemented in a previous commit:
543573f. This PR introduces tests for
the dynamic Dory commitment computation by reworking the tests in the
`dory_compute_commitments_test` module to match the updated dynamic Dory
structure.

# What changes are included in this PR?
- Tests are added for the dynamic Dory commitment computation.
- Dynamic Dory commitment computation tests are added to the CI
workflow.

# Are these changes tested?
Yes

---------

Co-authored-by: Jay White <[email protected]>
  • Loading branch information
jacobtrombetta and JayWhite2357 authored Oct 12, 2024
1 parent cbf5078 commit 4211dea
Show file tree
Hide file tree
Showing 3 changed files with 518 additions and 1 deletion.
4 changes: 3 additions & 1 deletion .github/workflows/lint-and-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -103,7 +103,9 @@ jobs:
- name: Dry run cargo test (proof-of-sql) (std feature only)
run: cargo test -p proof-of-sql --no-run --no-default-features --features="std"
- name: Run cargo test (proof primitives - Dory) (std feature only - i.e. not using blitzar)
run: cargo test proof_primitive::dory::dory_compute_commitments_test --no-default-features --features="std"
run: |
cargo test proof_primitive::dory::dory_compute_commitments_test --no-default-features --features="std" && \
cargo test proof_primitive::dory::dynamic_dory_compute_commitments_test --no-default-features --features="std"
- name: Run hello_world example (With Blitzar)
run: cargo run --example hello_world --features="test"
- name: Run hello_world example (Without Blitzar and With Rayon)
Expand Down
Loading

0 comments on commit 4211dea

Please sign in to comment.