Skip to content

Remove warning of pytest for rewriting modules#143

Merged
vasukulkarni merged 1 commit intored-hat-storage:masterfrom
petr-balogh:fix-pystest-rewrite-warnings
Jun 4, 2019
Merged

Remove warning of pytest for rewriting modules#143
vasukulkarni merged 1 commit intored-hat-storage:masterfrom
petr-balogh:fix-pystest-rewrite-warnings

Conversation

@petr-balogh
Copy link
Copy Markdown
Member

Cause we are loading config and defaults from our run_ocsci.py wrapper
we need to add PYTEST_DONT_REWRITE msg to dosctring of modules where we
don't want to see warning.

Documentation:
https://docs.pytest.org/en/latest/reference.html#pytest-dont-rewrite-module-docstring

Reference where I found about this workarround is mentioned here:
pytest-dev/pytest-cov#148 (comment)

Cause we are loading config and defaults from our run_ocsci.py wrapper
we need to add PYTEST_DONT_REWRITE msg to dosctring of modules where we
don't want to see warning.

Documentation:
https://docs.pytest.org/en/latest/reference.html#pytest-dont-rewrite-module-docstring

Reference where I found about this workarround is mentioned here:
pytest-dev/pytest-cov#148 (comment)
@petr-balogh petr-balogh added the bug Something isn't working label Jun 1, 2019
@petr-balogh petr-balogh requested a review from a team as a code owner June 1, 2019 21:11
@petr-balogh petr-balogh self-assigned this Jun 1, 2019
@vasukulkarni vasukulkarni merged commit b8582d4 into red-hat-storage:master Jun 4, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants