Skip to content

Fix pytest.PytestRemovedIn9Warning / Marks applied to fixtures#246

Merged
m-philipps merged 1 commit intodevelopfrom
fix_pytest_depr
Jan 29, 2024
Merged

Fix pytest.PytestRemovedIn9Warning / Marks applied to fixtures#246
m-philipps merged 1 commit intodevelopfrom
fix_pytest_depr

Conversation

@dweindl
Copy link
Copy Markdown
Member

@dweindl dweindl commented Jan 29, 2024

Fixes

_________________ ERROR collecting tests/test_visualization.py _________________
tests/test_visualization.py:112: in <module>
    def visu_file_Fujita_empty():
.tox/unit/lib/python3.9/site-packages/_pytest/mark/structures.py:357: in __call__
    store_mark(func, self.mark)
.tox/unit/lib/python3.9/site-packages/_pytest/mark/structures.py:422: in store_mark
    warnings.warn(MARKED_FIXTURE, stacklevel=2)
E   pytest.PytestRemovedIn9Warning: Marks applied to fixtures have no effect
E   See docs: https://docs.pytest.org/en/stable/deprecations.html#applying-a-mark-to-a-fixture-function

https://github.com/PEtab-dev/libpetab-python/actions/runs/7691314124/job/20956434711

Fixes

```
_________________ ERROR collecting tests/test_visualization.py _________________
tests/test_visualization.py:112: in <module>
    def visu_file_Fujita_empty():
.tox/unit/lib/python3.9/site-packages/_pytest/mark/structures.py:357: in __call__
    store_mark(func, self.mark)
.tox/unit/lib/python3.9/site-packages/_pytest/mark/structures.py:422: in store_mark
    warnings.warn(MARKED_FIXTURE, stacklevel=2)
E   pytest.PytestRemovedIn9Warning: Marks applied to fixtures have no effect
E   See docs: https://docs.pytest.org/en/stable/deprecations.html#applying-a-mark-to-a-fixture-function
```
@dweindl dweindl requested a review from m-philipps as a code owner January 29, 2024 09:54
Copy link
Copy Markdown
Collaborator

@m-philipps m-philipps left a comment

Choose a reason for hiding this comment

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

Thank you!

@m-philipps m-philipps merged commit 7e24e14 into develop Jan 29, 2024
@m-philipps m-philipps deleted the fix_pytest_depr branch January 29, 2024 15:01
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