Skip to content
This repository has been archived by the owner on May 25, 2022. It is now read-only.

Fix pytest warnings related to explicit use of test fixture #407

Merged
merged 1 commit into from
Oct 4, 2018

Conversation

drdavella
Copy link
Contributor

@drdavella drdavella commented Oct 1, 2018

This gets rid of several warnings that were occurring when running the tests:

/Users/user/miniconda3/envs/ah-helpers/lib/python3.7/site-packages/_pytest/python.py:198: RemovedInPytest4Warning: Fixture "testpackage" called directly. Fixtures are not meant to be called directly, are created automatically when test functions request them as parameters. See https://docs.pytest.org/en/latest/fixture.html for more information.

@astropy-bot
Copy link

astropy-bot bot commented Oct 1, 2018

Hi there @drdavella 👋 - thanks for the pull request! I'm just a friendly 🤖 that checks for issues related to the changelog and making sure that this pull request is milestoned and labeled correctly. This is mainly intended for the maintainers, so if you are not a maintainer you can ignore this, and a maintainer will let you know if any action is required on your part 😃.

Everything looks good from my point of view! 👍

If there are any issues with this message, please report them here.

@bsipocz bsipocz added this to the v2.0.8 milestone Oct 1, 2018
@astrofrog astrofrog merged commit 01878b6 into astropy:master Oct 4, 2018
bsipocz pushed a commit that referenced this pull request Oct 10, 2018
Fix pytest warnings related to explicit use of test fixture
bsipocz pushed a commit that referenced this pull request Oct 10, 2018
Fix pytest warnings related to explicit use of test fixture
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants