Skip to content

Conversation

nicoddemus
Copy link
Member

No description provided.

nicoddemus and others added 30 commits February 19, 2019 08:08
…features

Merge master into features (including fix from 4816)
AppVeyor: use xdist with pypy, drop pluggymaster
Require funcsigs>=1.0 on Python 2.7
Change -p so it is possible to early load setuptools plugins
Move import of  _format_explanation in _pytest.assertion.rewrite
--pdbcls: improve validation, and allow for "mod:attr.class"
It is only called with empty/new dirs since 0385c27.
cacheprovider: _ensure_supporting_files: remove unused branches
…header-4875

Show testpaths option in the header if it has been used for collection
Co-Authored-By: nicoddemus <nicoddemus@gmail.com>
…eanup-1895

getfixturevalue does not correctly declare dependency with the calling fixture
@codecov
Copy link

codecov bot commented Mar 29, 2019

Codecov Report

Merging #5014 into master will increase coverage by 0.36%.
The diff coverage is 97.85%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #5014      +/-   ##
==========================================
+ Coverage   95.69%   96.06%   +0.36%     
==========================================
  Files         113      114       +1     
  Lines       25178    25749     +571     
  Branches     2498     2548      +50     
==========================================
+ Hits        24095    24735     +640     
+ Misses        765      704      -61     
+ Partials      318      310       -8
Impacted Files Coverage Δ
src/_pytest/_code/source.py 91.58% <ø> (ø) ⬆️
src/_pytest/warnings.py 88.88% <ø> (ø) ⬆️
src/_pytest/mark/structures.py 92.61% <ø> (ø) ⬆️
src/_pytest/helpconfig.py 97.29% <ø> (ø) ⬆️
src/_pytest/outcomes.py 96.15% <ø> (ø) ⬆️
testing/test_junitxml.py 97.8% <ø> (ø) ⬆️
testing/test_terminal.py 99.82% <100%> (ø) ⬆️
testing/deprecated_test.py 100% <100%> (ø) ⬆️
src/_pytest/stepwise.py 96.66% <100%> (+0.51%) ⬆️
testing/test_modimport.py 100% <100%> (+18.18%) ⬆️
... and 51 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 2812c08...4621638. Read the comment docs.

Co-Authored-By: nicoddemus <nicoddemus@gmail.com>
@blueyed
Copy link
Contributor

blueyed commented Mar 30, 2019

Disabled Code Climate's "Pull request status updates" now.
The last time I wanted to do this it reported an actual good issue in a single PR, but this here is not very helpful.

@nicoddemus
Copy link
Member Author

OK sounds good.

Other than that, if there are no other issues, please approve. 👍

@blueyed
Copy link
Contributor

blueyed commented Mar 30, 2019

@nicoddemus
Haven't looked in detail really, would appreciate if somebody else approves it.

@nicoddemus nicoddemus changed the base branch from master to features March 30, 2019 11:52
@nicoddemus
Copy link
Member Author

I have changed the target to features temporarily so it is easier to review. After we get approvals I will change back the target to master and merge. 👍

def test_equals(self, a, b):
> assert a == b
E assert 1 == 2
E -1
Copy link
Member Author

Choose a reason for hiding this comment

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

Those changes are actually correct, they were a recent fix by how "verbose" was handled internally by some plugins (-q was being treated as -v in some cases.

Copy link
Member

@Zac-HD Zac-HD left a comment

Choose a reason for hiding this comment

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

Looks good to me 👍

@nicoddemus nicoddemus changed the base branch from features to master March 31, 2019 21:14
@nicoddemus nicoddemus merged commit 49d690d into pytest-dev:master Mar 31, 2019
@nicoddemus nicoddemus deleted the release-4.4.0 branch March 31, 2019 21:14
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.

7 participants