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

ci: require new fixtures be committed #28

Closed
wants to merge 2 commits into from
Closed

ci: require new fixtures be committed #28

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
707e33c
fuzz: generate firedancer-compatible fixtures
buffalojoec Nov 25, 2024
919834e
ci: require new fixtures be committed
buffalojoec Nov 25, 2024
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
15 changes: 15 additions & 0 deletions .github/workflows/main.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -95,6 +95,9 @@ jobs:
name: Test Programs
runs-on: ubuntu-latest
needs: format_and_lint_programs
env:
EJECT_FUZZ_FIXTURES_FD: "fuzz/blob"
EJECT_FUZZ_FIXTURES_JSON_FD: "fuzz/json"
steps:
- name: Git Checkout
uses: actions/checkout@v4
Expand All @@ -106,8 +109,20 @@ jobs:
cargo-cache-fallback-key: cargo-programs
solana: true

- name: Clear Fixtures
run: rm -rf program/fuzz

- name: Test Programs
run: pnpm programs:test

- name: Check Working Directory
run: |
if [ -n "$(git status --porcelain)" ]; then
test -z "$(git status --porcelain)"
echo "Generated test fixtures have changed. Please set the `EJECT_FUZZ_FIXTURES_FD` and `EJECT_FUZZ_FIXTURES_JSON_FD`, \
remove the existing fixtures, run the tests, and commit the updated fixtures.";
exit 1;
fi

bench_program_compute_units:
name: Benchmark Program Compute Units
Expand Down
Loading
Loading