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.

Sign up for GitHub

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

deps(testing) - consolidate testing frameworks across the project #819

Merged
merged 8 commits into from
Nov 11, 2023
Merged

deps(testing) - consolidate testing frameworks across the project #819

Show file tree
Hide file tree
Changes from 5 commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
399ac31
deps(testing) - consolidate testing frameworks across the project
ctrlc03 Nov 7, 2023
259c14b
deps(testing) - allow more timeout during MaciState tests to ensure t…
ctrlc03 Nov 8, 2023
1329572
deps(testing) - allow more timeout across all tests to ensure they ha…
ctrlc03 Nov 8, 2023
e14bebe
deps(mocha) - fix integration tests by changing filename and ensuring…
ctrlc03 Nov 9, 2023
8a14264
deps(mocha) - fix integration tests errors due to chai eq vs deep.eq …
ctrlc03 Nov 10, 2023
f2a8abf
Remove mention of Jest from README
samajammin Nov 10, 2023
e576c84
Add info on Mocha and Chai
samajammin Nov 10, 2023
f44ff8e
Update test instructions
samajammin Nov 11, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion .github/workflows/reusable-e2e.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ jobs:
id: check_changes
run: |
CHANGED_FILES=${{ steps.get-changed-files.outputs.all }}
if echo "$CHANGED_FILES" | grep -q "circuit" && ! echo "$CHANGED_FILES" | grep -q ".github/workflows/circuit-build.yml"; then
if echo "$CHANGED_FILES" | grep -q "\.circom"; then
Copy link
Member

Choose a reason for hiding this comment

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

What's this change about?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Oh yea sorry for not mentioning in the description! Basically this change is so that the e2e test only run when the actual circom files change, not any files in the circuits folder as it was until this commit. I was planning to introduce this with another PR but got carried away by not wanting to wait on circuits compilation for this PR

Copy link
Member

Choose a reason for hiding this comment

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

Awesome. Great catch.

echo "CHANGED=true" >> $GITHUB_ENV
echo "Circuits have changes."
else
Expand Down
29 changes: 0 additions & 29 deletions circuits/jest.config.js
View file
Open in desktop

This file was deleted.

Loading
Loading