Skip to content

manually run can pass args - #378

Merged
njbrake merged 1 commit into
mainfrom
brake/args
Sep 4, 2025
Merged

njbrake merged 1 commit into
mainfrom
brake/args

Conversation

@njbrake

@njbrake njbrake commented Sep 4, 2025

Copy link
Copy Markdown
Contributor

Description

Sometime you only make a change for one provider and you want to run the integration test just on that feature or provider.

PR Type

🆕 New Feature 🚦 Infrastructure ## Relevant issues

Checklist

  • I have added unit tests that prove my fix/feature works
  • New and existing tests pass locally
  • Documentation was updated where necessary
  • I have read and followed the contribution guidelines```

@njbrake
njbrake requested a review from daavoo September 4, 2025 13:34
@codecov

codecov Bot commented Sep 4, 2025 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
see 24 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@njbrake
njbrake merged commit a63d749 into main Sep 4, 2025
11 checks passed
@njbrake
njbrake deleted the brake/args branch September 4, 2025 13:48
Comment on lines +73 to +78
run: |
if [ -n "${{ inputs.filter }}" ]; then
pytest tests/integration -v -n auto --cov --cov-report=xml -k "${{ inputs.filter }}"
else
pytest tests/integration -v -n auto --cov --cov-report=xml
fi

@daavoo daavoo Sep 4, 2025 •

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Probably doesn't matter but I think this would also work because empty string matches everything

Suggested change
run: |
if [ -n "${{ inputs.filter }}" ]; then
pytest tests/integration -v -n auto --cov --cov-report=xml -k "${{ inputs.filter }}"
else
pytest tests/integration -v -n auto --cov --cov-report=xml
fi
run: |
pytest tests/integration -v -n auto --cov --cov-report=xml -k "${{ inputs.filter }}"

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.

2 participants