Skip to content

Conversation

@JonZeolla
Copy link
Member

Contributor Comments

This allows us to run task test -- not slow and it will run all the tests, but exclude the slow ones, as an example.

Pull Request Checklist

Thank you for submitting a contribution!

Please address the following items:

  • If you are adding a dependency, please explain how it was chosen.
  • If manual testing is needed in order to validate the changes, provide a testing plan and the expected results.
  • Validate that documentation is accurate and aligned to any project updates or additions.

Copy link

@ai-coding-guardrails ai-coding-guardrails bot left a comment

Choose a reason for hiding this comment

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

I've got 1 comment for you to consider

The PR title and description are not accurate. Here are my suggestions:

Title: chore(tests): allow test customization via cli

Description: # Contributor Comments

This PR enables flexible test execution by allowing CLI arguments to be passed to pytest through the task runner. Users can now run commands like task test -- not slow to exclude slow tests, or combine any pytest markers and expressions.

Changes Made

  • Modified Taskfile.yml configurations to support CLI_ARGS with existing test markers
  • Added @pytest.mark.slow markers to long-running tests for better categorization
  • Updated both main project and cookiecutter template configurations

Pull Request Checklist

Thank you for submitting a contribution!

Please address the following items:

  • If you are adding a dependency, please explain how it was chosen.
  • If manual testing is needed in order to validate the changes, provide a testing plan and the expected results.
  • Validate that documentation is accurate and aligned to any project updates or additions.

Are we missing a feature you'd like? Submit it here!

@JonZeolla JonZeolla merged commit 56aa100 into main Jul 16, 2025
7 checks passed
@JonZeolla JonZeolla deleted the add-slow-marker branch July 16, 2025 17:47
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