Skip to content
This repository was archived by the owner on Aug 26, 2022. It is now read-only.

Bug 1308322: Upgrade pytest and most plugins#4210

Merged
jwhitlock merged 2 commits into
mdn:masterfrom
jwhitlock:just_pytest_1308322
May 10, 2017
Merged

Bug 1308322: Upgrade pytest and most plugins#4210
jwhitlock merged 2 commits into
mdn:masterfrom
jwhitlock:just_pytest_1308322

Conversation

@jwhitlock

@jwhitlock jwhitlock commented May 2, 2017

Copy link
Copy Markdown
Contributor

Convert requirements/test.txt to include comments and use requirements/constraints.txt. Then update py.test and most plugins

  • pytest 2.8.7 -> 3.0.7: Breaking changes release, remove and rewrite features, better Py3 and unicode support, xfail(strict=True), deprecate yield_fixture, more assert rewriting
  • pytest-rerunfailures 1.0.2 -> 2.1.0: pytest 3.x support
  • pytest-xdist 1.15.0 -> 1.16.0: New scheuduler hook
  • pytest-cov 2.2.1 -> 2.4.0: xdist fixes
  • pytest-django 2.9.1 -> 3.1.2: pytest 3 support, expanded Django support, public database setup API, mailoutbox fixture
  • coverage 4.0.3 -> 4.3.4: Improved branch analysis, improved HTML report, smarter combine, tox.ini config support.
  • py 1.4.31 -> 1.4.33: Bug fixes

Django tests and coverage worked for me. I was able to run the functional tests locally with Chrome. This is the same as #4195, except that I've omitted the Selenium upgrades.

@codecov-io

codecov-io commented May 2, 2017

Copy link
Copy Markdown

Codecov Report

Merging #4210 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #4210   +/-   ##
=======================================
  Coverage   86.31%   86.31%           
=======================================
  Files         148      148           
  Lines        9105     9105           
  Branches     1222     1222           
=======================================
  Hits         7859     7859           
  Misses       1005     1005           
  Partials      241      241

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 624ad69...f045837. Read the comment docs.

@jwhitlock jwhitlock force-pushed the just_pytest_1308322 branch from 913b366 to 1b9d467 Compare May 2, 2017 17:18
@jwhitlock

Copy link
Copy Markdown
Contributor Author

I meant to push this branch to the mozilla repo, so Jenkins would run the integration tests. I duplicated it, and Jenkins is happy. I'm removing the groovy script from this PR. I might use this accidental pattern in the future - the main code changes in a PR against my repo, and add a commit for the groovy script if I want Jenkins to run it, and keep temporary groovy scripts out of merges.

@jwhitlock jwhitlock force-pushed the just_pytest_1308322 branch from 1b9d467 to 2c48292 Compare May 2, 2017 17:28
@jwhitlock jwhitlock force-pushed the just_pytest_1308322 branch 2 times, most recently from 3f692e1 to 5fe7206 Compare May 5, 2017 17:56
@jwhitlock

Copy link
Copy Markdown
Contributor Author

Force-push to rebase w/ current requirements, and add in coverage 4.3.4 as promised.

jwhitlock added 2 commits May 8, 2017 10:39
* Add a line for the "why" of each requirement
* Move requirements of requirements to constraints.txt
* Add TODOs for requirements that appear in multiple files at different
  versions.
* pytest 2.8.7 -> 3.0.7: Breaking changes release, remove and rewrite
  features, better Py3i and unicode support, xfail(strict=True),
  deprecate yield_fixture, more assert rewriting
* pytest-rerunfailures 1.0.2 -> 2.1.0: pytest 3.x support
* pytest-xdist 1.15.0 -> 1.16.0: New scheuduler hook
* pytest-cov 2.2.1 -> 2.4.0: xdist fixes
* pytest-django 2.9.1 -> 3.1.2: pytest 3 support, expanded Django
  support, public database setup API, mailoutbox fixture
* coverage 4.0.3 -> 4.3.4: Improved branch analysis, improved HTML
  report, smarter combine, tox.ini config support.
* py 1.4.31 -> 1.4.33: Bug fixes
@jwhitlock jwhitlock force-pushed the just_pytest_1308322 branch from 5fe7206 to f045837 Compare May 8, 2017 15:42
@jwhitlock

Copy link
Copy Markdown
Contributor Author

Force-push, pytest-django is back out of requirements/test.txt

@jwhitlock jwhitlock merged commit e8450a9 into mdn:master May 10, 2017
@jwhitlock jwhitlock deleted the just_pytest_1308322 branch May 10, 2017 14:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants