X Tutup
Skip to content

Convert test_[ab]* files to pytest.#7916

Merged
phobson merged 2 commits intomatplotlib:masterfrom
QuLogic:pytest-ab
Jan 23, 2017
Merged

Convert test_[ab]* files to pytest.#7916
phobson merged 2 commits intomatplotlib:masterfrom
QuLogic:pytest-ab

Conversation

@QuLogic
Copy link
Copy Markdown
Member

@QuLogic QuLogic commented Jan 22, 2017

Pretty small set of changes in most of these. Note that test_backends_* is in #7863.

@QuLogic QuLogic added this to the 2.1 (next point release) milestone Jan 22, 2017
@codecov-io
Copy link
Copy Markdown

codecov-io commented Jan 22, 2017

Current coverage is 62.29% (diff: 100%)

Merging #7916 into master will increase coverage by 0.11%

@@             master      #7916   diff @@
==========================================
  Files           174        177      +3   
  Lines         56121      57417   +1296   
  Methods           0          0           
  Messages          0          0           
  Branches          0          0           
==========================================
+ Hits          34897      35770    +873   
- Misses        21224      21647    +423   
  Partials          0          0           

Powered by Codecov. Last update 21c8157...675424c

@QuLogic
Copy link
Copy Markdown
Member Author

QuLogic commented Jan 22, 2017

Added some flake8 cleanup like in the other PRs; it's mostly just deleting unused imports.

Copy link
Copy Markdown
Member

@NelleV NelleV left a comment

Choose a reason for hiding this comment

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

LGTM

@NelleV NelleV changed the title Convert test_[ab]* files to pytest. [MRG+1] Convert test_[ab]* files to pytest. Jan 22, 2017
@phobson phobson merged commit e9d0892 into matplotlib:master Jan 23, 2017
@phobson phobson changed the title [MRG+1] Convert test_[ab]* files to pytest. Convert test_[ab]* files to pytest. Jan 23, 2017
@QuLogic QuLogic deleted the pytest-ab branch January 23, 2017 04:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

X Tutup