Skip to content

Commit

Permalink
fix: Fix working dir bug causing stdlib-tests to not run. (#2495)
Browse files Browse the repository at this point in the history
Please provide a paragraph or two giving a summary of the change,
including relevant motivation and context.

# 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.
- [ ] I have reviewed my diff in github, line by line and removed
unexpected formatting changes, testing logs, or commented-out code.
- [ ] 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).
  • Loading branch information
charlielye authored Sep 25, 2023
1 parent 57a2f10 commit 6b3402c
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 2 deletions.
3 changes: 2 additions & 1 deletion barretenberg/cpp/.rebuild_patterns
Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
^barretenberg/.*\\.(cpp|cc|cxx|c\\+\\+|h|hpp|hxx|h\\+\\+|c|h|inl|inc|ipp|tpp|cmake)$
^barretenberg/.*CMakeLists\\.txt$
^barretenberg/.*Dockerfile.*$
^barretenberg/.*Dockerfile.*$
^barretenberg/scripts/
3 changes: 2 additions & 1 deletion barretenberg/cpp/scripts/run_tests
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,8 @@ IMAGE_URI=$(calculate_image_uri $REPOSITORY)
retry docker pull $IMAGE_URI

# If there is a file in the scripts directory named $TESTS, those are the tests to run.
if [ -f "$(query_manifest projectDir $REPOSITORY)/scripts/$TESTS" ]; then
cd $(query_manifest projectDir $REPOSITORY)/scripts
if [ -f "$TESTS" ]; then
TESTS=$(cat $TESTS | tr '\n' ' ')
fi

Expand Down

0 comments on commit 6b3402c

Please sign in to comment.