Skip to content

feat: Run all tests in workflow with --no-fail-fast flag#6

Merged
weswalla merged 1 commit into
mainfrom
cyrus/per-9-run-all-tests-in-workflow
Oct 19, 2025
Merged

feat: Run all tests in workflow with --no-fail-fast flag#6
weswalla merged 1 commit into
mainfrom
cyrus/per-9-run-all-tests-in-workflow

Conversation

@cyrusagent

@cyrusagent cyrusagent Bot commented Oct 19, 2025

Copy link
Copy Markdown

Summary

  • Updated the GitHub Actions workflow to use --no-fail-fast flag with cargo nextest run
  • This ensures all tests are executed even when some fail, providing complete test suite visibility

Context

Previously, the workflow would stop running tests after the first failure. The warning message indicated:

warning: 46/125 tests were not run due to test failure (run with --no-fail-fast to run all tests, or run with --max-fail)

Changes

  • Modified .github/workflows/rust-tests.yml line 37 to add the --no-fail-fast flag

Test plan

  • The workflow will run on this PR and demonstrate running all tests regardless of failures
  • Verify in the Actions tab that all 125 tests are attempted

Fixes PER-9

🤖 Generated with Claude Code

Update the GitHub Actions workflow to use the --no-fail-fast flag with cargo nextest.
This ensures all tests run even when some fail, allowing us to see the full test suite
results instead of stopping at the first failure.

Fixes PER-9

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@weswalla
weswalla merged commit 32bd92d into main Oct 19, 2025
1 check passed
@weswalla
weswalla deleted the cyrus/per-9-run-all-tests-in-workflow branch October 19, 2025 15:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant