Closed Bug 1732795 Opened 3 years ago Closed 2 years ago

Bump pytest version

Categories

(Testing :: Python Test, enhancement)

enhancement

Tracking

(firefox101 fixed)

RESOLVED FIXED
101 Branch
Tracking Status
firefox101 --- fixed

People

(Reporter: mhentges, Assigned: mhentges)

References

Details

Attachments

(4 files, 1 obsolete file)

Our pytest==4.6.6 is almost three years old.
Newer versions have new features/APIs, slightly better performance, and a cool "warnings" display that makes failures caused by misuse easier to spot.

Assignee: nobody → mhentges
Status: NEW → ASSIGNED
Component: Mach Core → Python Test
Product: Firefox Build System → Testing

A bunch of modern packages (pytest, twisted, automat) all need attrs==19.2.0 (or newer).
We could bump attrs all the way to the modern 21.4.0 version, but I'd like to defer that upgrade risk, since there's a
lot of backwards-incompatible changes and deprecations. So, lightly bump it to 19.2.0.

Comment on attachment 9257964 [details]
WIP: Bug 1732795: Bump attrs to be compatible with new pytest

Revision D135178 was moved to bug 1748737. Setting attachment 9257964 [details] to obsolete.

Attachment #9257964 - Attachment is obsolete: true

Following the pytest "deprecations and removals" docs [1], this patch:

  • Replaces @pytest.yield_fixture with @pytest.fixture.
  • Replaces .funcargnames with .fixturenames.
  • Uses Pathlib parameter instead of the py.path.local one in
    associated pytest_* hooks.
  • Replaces --strict with --strict-markers

[1] https://docs.pytest.org/en/latest/deprecations.html

Depends on D138608

coverage has native code, so the vendored version was only used as
source code from which the actual package could be built.
Since its always used in a context where we can pip install over the
network, let's do that. This cleans up our tree a bit and allows us to
leverage the coverage wheels.

Depends on D138815

Depends on D138816

Depends on: 1755516
Pushed by mhentges@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/d0f3bf6ad3a3
Resolve upcoming pytest deprecations r=webdriver-reviewers,ahal,whimboo
https://hg.mozilla.org/integration/autoland/rev/0bfaaa811421
Install `coverage` using pip r=ahal
Created web-platform-tests PR https://github.com/web-platform-tests/wpt/pull/32902 for changes under testing/web-platform/tests
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 99 Branch
Status: RESOLVED → REOPENED
Keywords: leave-open
Resolution: FIXED → ---

3.6.2 was 3 years old. There's been many helpful features and
ergonomics improvements in the years since.

Depends on D140258

Regressions: 1763615

Can this be closed now?

Flags: needinfo?(mhentges)

Good call, yep!

Status: REOPENED → RESOLVED
Closed: 2 years ago2 years ago
Flags: needinfo?(mhentges)
Resolution: --- → FIXED
Target Milestone: 99 Branch → 101 Branch
Regressions: 1764466
Upstream PR was closed without merging
Upstream PR merged by moz-wptsync-bot
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: