Skip to content

wrap_session: restore old behavior for initstate=1#5028

Merged
blueyed merged 1 commit intopytest-dev:featuresfrom
blueyed:fix-wrap_session-exit-code
Apr 3, 2019
Merged

wrap_session: restore old behavior for initstate=1#5028
blueyed merged 1 commit intopytest-dev:featuresfrom
blueyed:fix-wrap_session-exit-code

Conversation

@blueyed
Copy link
Contributor

@blueyed blueyed commented Apr 3, 2019

The condition for initstate <= 2 was always True, after #4145.

/cc @cacoze

No changelog.

@blueyed blueyed force-pushed the fix-wrap_session-exit-code branch from a6ffbdd to 2950339 Compare April 3, 2019 02:22
@codecov
Copy link

codecov bot commented Apr 3, 2019

Codecov Report

Merging #5028 into features will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##           features    #5028      +/-   ##
============================================
+ Coverage     96.06%   96.06%   +<.01%     
============================================
  Files           114      114              
  Lines         25749    25757       +8     
  Branches       2550     2551       +1     
============================================
+ Hits          24736    24744       +8     
  Misses          704      704              
  Partials        309      309
Impacted Files Coverage Δ
testing/test_pdb.py 99.18% <100%> (ø) ⬆️
src/_pytest/main.py 96.04% <100%> (ø) ⬆️
testing/test_runner.py 96.96% <100%> (+0.04%) ⬆️

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 15ea5ce...cc90bcc. Read the comment docs.

@blueyed blueyed force-pushed the fix-wrap_session-exit-code branch from 2950339 to cc90bcc Compare April 3, 2019 03:09
@blueyed blueyed merged commit a65edf6 into pytest-dev:features Apr 3, 2019
@blueyed blueyed deleted the fix-wrap_session-exit-code branch April 3, 2019 22:39
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