Skip to content

Commit

Permalink
chore(ci): Update acir tests to reflect compilation based off of pack…
Browse files Browse the repository at this point in the history
…age name (#1405)

ACIR tests are failing and needed a refresh on the Noir side. However,
there has been a change where compilation is specified off of the
package name in the `Nargo.toml` rather than a specified file name such
as `main.json`

# Checklist:
Remove the checklist to signal you've completed it. Enable auto-merge if
the PR is ready to merge.
- [] If the pull request requires a cryptography review (e.g.
cryptographic algorithm implementations) I have added the 'crypto' tag.
- [X] I have reviewed my diff in github, line by line and removed
unexpected formatting changes, testing logs, or commented-out code.
- [X] Every change is related to the PR description.
- [ ] I have
[linked](https://docs.github.com/en/issues/tracking-your-work-with-issues/linking-a-pull-request-to-an-issue)
this pull request to relevant issues (if any exist).

---------

Co-authored-by: Tom French <[email protected]>
  • Loading branch information
2 people authored and ludamad committed Aug 8, 2023
1 parent 441e972 commit 151e816
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions acir_tests/run_acir_tests.sh
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ function test() {
return
fi

if [[ ! -f ./$1/target/main.json || ! -f ./$1/target/witness.tr ]]; then
if [[ ! -f ./$1/target/$dir_name.json || ! -f ./$1/target/witness.tr ]]; then
echo -e "\033[33mSKIPPED\033[0m (uncompiled)"
return
fi
Expand All @@ -65,9 +65,9 @@ function test() {

set +e
if [ -n "$VERBOSE" ]; then
$BB prove_and_verify -v -c $CRS_PATH
$BB prove_and_verify -v -c $CRS_PATH -j ./target/$dir_name.json
else
$BB prove_and_verify -c $CRS_PATH > /dev/null 2>&1
$BB prove_and_verify -c $CRS_PATH -j ./target/$dir_name.json > /dev/null 2>&1
fi
result=$?
set -e
Expand Down

0 comments on commit 151e816

Please sign in to comment.