Skip to content

Fix hatch test scripts#409

Merged
JP-Ellis merged 3 commits intomasterfrom
chore/fix-hatch-script
Oct 11, 2023
Merged

Fix hatch test scripts#409
JP-Ellis merged 3 commits intomasterfrom
chore/fix-hatch-script

Conversation

@JP-Ellis
Copy link
Copy Markdown
Contributor

@JP-Ellis JP-Ellis commented Oct 6, 2023

📝 Summary

Commits describe the individual changes:

  • fix(test): ignore internal deprecation warnings

  • chore(test): add pytest options in root

    To help ensure there is a unified experience when running any/all tests,
    we need to define this in the root directory. The rest of the
    example-specific conftest.py can remain in examples/conftest.py.

    Running pytest in the root directory will now correctly execute all
    tests, including the examples.

  • chore: fix hatch test scripts

    Hatch scripts allow for additional arguments to be specified through the
    {args:<default>} placeholder. Unfortunately, this was not
    automatically enabled.

    In doing this, I have also simplified the scripts by moving some common
    pytest options into pytest's own addopts option.

🚨 Breaking Changes

🔥 Motivation

🔨 Test Plan

🔗 Related issues/PRs

Hatch scripts allow for additional arguments to be specified through the
`{args:<default>}` placeholder. Unfortunately, this was not
automatically enabled.

In doing this, I have also simplified the scripts by moving some common
pytest options into pytest's own `addopts` option.

Signed-off-by: JP-Ellis <josh@jpellis.me>
To help ensure there is a unified experience when running any/all tests,
we need to define this in the root directory. The rest of the
example-specific `conftest.py` can remain in `examples/conftest.py`.

Running `pytest` in the root directory will now correctly execute _all_
tests, including the examples.

Signed-off-by: JP-Ellis <josh@jpellis.me>
Signed-off-by: JP-Ellis <josh@jpellis.me>
@JP-Ellis JP-Ellis requested a review from YOU54F October 6, 2023 05:36
@JP-Ellis JP-Ellis self-assigned this Oct 6, 2023
@JP-Ellis JP-Ellis enabled auto-merge (rebase) October 6, 2023 06:32
@JP-Ellis JP-Ellis merged commit aa69a20 into master Oct 11, 2023
@mefellows mefellows deleted the chore/fix-hatch-script branch October 11, 2023 02:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Completed

Development

Successfully merging this pull request may close these issues.

Update hatch run test to support arguments

2 participants