Skip to content

Update pytest to 9.0.3#375

Closed
oxo42 wants to merge 9 commits intomainfrom
update-pytest
Closed

Update pytest to 9.0.3#375
oxo42 wants to merge 9 commits intomainfrom
update-pytest

Conversation

@oxo42
Copy link
Copy Markdown
Contributor

@oxo42 oxo42 commented Apr 27, 2026

Validate that CI passes

oxo42 and others added 9 commits April 27, 2026 08:22
…nization

Pygments 2.20 emits the inter-token space before an underlined identifier
as its own Whitespace token, wrapping it in default-color ANSI codes
(\x1b[37m \x1b[39;49;00m). Update the expected literal accordingly.
typeguard 4.x renders typing.List[str] as list[str] in error messages.
Update assertRaisesRegex pattern in
raises_TypeCheckError_when_returning_coroutine_instance to match.
Allows `pytest pytest-testslide/tests` to run from the repo root without
needing to pre-set PYTHONPATH. Previously only the Makefile's pytest_tests
target worked, since it exported PYTHONPATH=pytest-testslide:. — the bare
command failed because the inner pytester run could not import
pytest_testslide or testslide (neither is installed in the venv; both are
sourced in-tree).
mypy cannot narrow Union[Mock, StrictMock] through isinstance against a
runtime-looked-up class (mock_class comes from MOCK_TEMPLATE_EXTRACTORS
iteration), so the call to extract_mock_template was rejected. Add an
explicit cast to express what the isinstance check guarantees.
- Drop unused `import sys` from testslide/bdd/lib.py and
  testslide/core/__init__.py (F401).
- Suppress F824 on `global` statements in mock_callable.py that declare
  names which are read but not assigned in scope. Collapsed multi-line
  global declarations onto one line so the noqa comment can be attached.
Bump pytest requirement to >=9.0.3
@meta-cla meta-cla Bot added the CLA Signed Do not delete this pull request or issue due to inactivity. label Apr 27, 2026
@coveralls
Copy link
Copy Markdown

Coverage Report for CI Build 25010161216

Warning

No base build found for commit afaa281 on main.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 92.17%

Details

  • Patch coverage: 2 of 2 lines across 1 file are fully covered (100%).

Uncovered Changes

No uncovered changes found.

Coverage Regressions

Requires a base build to compare against. How to fix this →


Coverage Stats

Coverage Status
Relevant Lines: 3014
Covered Lines: 2778
Line Coverage: 92.17%
Coverage Strength: 0.92 hits per line

💛 - Coveralls

@meta-codesync
Copy link
Copy Markdown

meta-codesync Bot commented Apr 28, 2026

@oxo42 has imported this pull request. If you are a Meta employee, you can view this in D102792375.

@meta-codesync
Copy link
Copy Markdown

meta-codesync Bot commented Apr 28, 2026

@oxo42 merged this pull request in 2943b90.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed Do not delete this pull request or issue due to inactivity. Merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants