Skip to content

tests: clarify/document/harden acceptance tests#4123

Merged
blueyed merged 2 commits intopytest-dev:masterfrom
blueyed:clarify
Oct 13, 2018
Merged

tests: clarify/document/harden acceptance tests#4123
blueyed merged 2 commits intopytest-dev:masterfrom
blueyed:clarify

Conversation

@blueyed
Copy link
Contributor

@blueyed blueyed commented Oct 12, 2018

Followup of / replaces #4106.

# └── test_bar.py

def join_pythonpath(*dirs):
def append_pythonpath(*dirs):
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is actually prepending dirs to the PYTHONPATH right?

# └── test_world.py

def join_pythonpath(*dirs):
def append_pythonpath(*dirs):
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is actually prepending dirs to the PYTHONPATH right?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

True (also above). Will rename it.

"*lib/foo/bar/test_bar.py::test_bar*PASSED*",
"*lib/foo/bar/test_bar.py::test_other*PASSED*",
"*lib/foo/bar/test_bar.py::test_bar PASSED*",
"*lib/foo/bar/test_bar.py::test_other PASSED*",
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With #4108 this will change to:

lib/foo/bar/test_bar.py::test_bar <- local/lib/foo/bar/test_bar.py PASSED [ 50%]
lib/foo/bar/test_bar.py::test_other <- local/lib/foo/bar/test_bar.py PASSED [100%]

@coveralls
Copy link

coveralls commented Oct 12, 2018

Coverage Status

Coverage increased (+0.02%) to 93.802% when pulling 6bf4692 on blueyed:clarify into eabf15b on pytest-dev:master.

Copy link
Member

@RonnyPfannschmidt RonnyPfannschmidt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

great work at finding those enhancement points

please take a look at integrating the features of monkey patch more neatly

@codecov
Copy link

codecov bot commented Oct 13, 2018

Codecov Report

Merging #4123 into master will increase coverage by 0.04%.
The diff coverage is 81.81%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #4123      +/-   ##
==========================================
+ Coverage   94.44%   94.49%   +0.04%     
==========================================
  Files         109      109              
  Lines       23813    23803      -10     
  Branches     2349     2346       -3     
==========================================
+ Hits        22491    22493       +2     
+ Misses       1010     1003       -7     
+ Partials      312      307       -5
Flag Coverage Δ
#linux 94.34% <81.81%> (+0.04%) ⬆️
#nobyte 0% <0%> (ø) ⬆️
#numpy 28.16% <0%> (+0.08%) ⬆️
#pexpect 0% <0%> (ø) ⬆️
#py27 92.66% <81.81%> (+0.04%) ⬆️
#py34 92.12% <81.81%> (+0.03%) ⬆️
#py35 92.13% <81.81%> (+0.03%) ⬆️
#py36 92.7% <81.81%> (+0.04%) ⬆️
#py37 92.33% <81.81%> (+0.03%) ⬆️
#trial 31.39% <0%> (+0.08%) ⬆️
#windows 93.8% <81.81%> (ø) ⬆️
#xdist 18.58% <0%> (+0.03%) ⬆️
Impacted Files Coverage Δ
testing/acceptance_test.py 98.2% <81.81%> (+0.39%) ⬆️
src/_pytest/terminal.py 91.62% <0%> (+1.74%) ⬆️

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 eabf15b...6bf4692. Read the comment docs.

@blueyed blueyed merged commit c27c8f4 into pytest-dev:master Oct 13, 2018
@blueyed blueyed deleted the clarify branch October 13, 2018 16:26
@blueyed
Copy link
Contributor Author

blueyed commented Oct 13, 2018

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