Skip to content

[WIP] debug/fix race with _ensure_supporting_files#4880

Closed
blueyed wants to merge 4 commits into
pytest-dev:featuresfrom
blueyed:assert-_ensure_supporting_files
Closed

[WIP] debug/fix race with _ensure_supporting_files#4880
blueyed wants to merge 4 commits into
pytest-dev:featuresfrom
blueyed:assert-_ensure_supporting_files

Conversation

@blueyed

@blueyed blueyed commented Mar 4, 2019

Copy link
Copy Markdown
Contributor

Related to / because of #4872.

@codecov

codecov Bot commented Mar 4, 2019

Copy link
Copy Markdown

Codecov Report

Merging #4880 into features will increase coverage by 23.98%.
The diff coverage is 100%.

Impacted file tree graph

@@              Coverage Diff              @@
##           features    #4880       +/-   ##
=============================================
+ Coverage     71.68%   95.66%   +23.98%     
=============================================
  Files            58      113       +55     
  Lines         12156    25260    +13104     
  Branches       2071     2504      +433     
=============================================
+ Hits           8714    24165    +15451     
+ Misses         2818      775     -2043     
+ Partials        624      320      -304
Flag Coverage Δ
#linux 95.48% <100%> (?)
#osx 93.32% <100%> (?)
#windows 93.6% <100%> (?)
Impacted Files Coverage Δ
testing/test_cacheprovider.py 99.72% <ø> (ø)
src/_pytest/cacheprovider.py 97.63% <100%> (+40.41%) ⬆️
src/_pytest/deprecated.py 100% <0%> (ø) ⬆️
src/_pytest/hookspec.py 100% <0%> (ø) ⬆️
testing/test_warnings.py 98.9% <0%> (ø)
testing/test_pastebin.py 100% <0%> (ø)
testing/test_argcomplete.py 68.11% <0%> (ø)
testing/python/setup_plan.py 100% <0%> (ø)
testing/test_helpconfig.py 97.29% <0%> (ø)
testing/logging/test_fixture.py 100% <0%> (ø)
... and 98 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4668ee0...8b97601. Read the comment docs.

blueyed added 4 commits March 5, 2019 23:13
Coverage changes by Codecov indicate that this might not be the case:

https://codecov.io/gh/pytest-dev/pytest/compare/f3f6cb20933c2e1e0377d5297eb35877e8781102...83558a0ba3a737752deb7d0b1c78a0ef0ec0843c/changes

This is mainly for debugging this first, since it appears to be flaky
(e.g. due to xdist).
This reverts commit bb662a9.
@blueyed blueyed force-pushed the assert-_ensure_supporting_files branch from 6576671 to 8b97601 Compare March 5, 2019 22:13
@blueyed

blueyed commented Mar 6, 2019

Copy link
Copy Markdown
Contributor Author

Looks like there is no race here.

@blueyed blueyed closed this Mar 6, 2019
@blueyed blueyed deleted the assert-_ensure_supporting_files branch March 6, 2019 16:44
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.

1 participant