Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix issue #2920 #2964

Merged
merged 8 commits into from
Nov 29, 2017
Merged

fix issue #2920 #2964

merged 8 commits into from
Nov 29, 2017

Conversation

rpuntaie
Copy link

See issue #2920 for details.

@nicoddemus
Copy link
Member

@rpuntaie thanks for tackling this!

We need to add a new test for it though. I suggest writing a test in test_config.py or acceptance_test.py; take a look at other tests using the testdir fixture, it should be straightforward to prepare an integration test which triggers the bug.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 92.688% when pulling 4a840a7 on rpuntaie:master into 4d2f05e on pytest-dev:master.

@rpuntaie
Copy link
Author

rpuntaie commented Nov 28, 2017

test_addopts_before_initini fails, but I think that has no relation to my changes. The changed test test_plugin_preparse_prevents_setuptools_loading fails without my fix and it succeeds with my fix.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 92.688% when pulling 4a840a7 on rpuntaie:master into 4d2f05e on pytest-dev:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 92.688% when pulling 3dbac17 on rpuntaie:master into 4d2f05e on pytest-dev:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 92.688% when pulling 833f33f on rpuntaie:master into 4d2f05e on pytest-dev:master.

@nicoddemus nicoddemus mentioned this pull request Nov 28, 2017
4 tasks
Make it a parametrized test to ensure all the mocked mechanisms in
the test actually work as expected when we *do not* use "-p no:"
@nicoddemus
Copy link
Member

Thanks @rpuntaie! I verified locally that your test reproduces the problem and also improved it a bit to ensure all the mechanisms that we are mocking in that test work as expected when we don't use the -p no: flag.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 92.688% when pulling 5f1a733 on rpuntaie:master into 4d2f05e on pytest-dev:master.

@The-Compiler
Copy link
Member

Thanks @rpuntaie and @nicoddemus!

@The-Compiler The-Compiler merged commit 88ed1ab into pytest-dev:master Nov 29, 2017
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.

4 participants