FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

gh-143955: Prevent schema drift false-positives in asyncio tools tests by johnslavik · Pull Request #148525 · python/cpython · GitHub

Repository navigation

gh-143955: Prevent schema drift false-positives in asyncio tools tests - #148525

Merged
pablogsal merged 1 commit into
python:mainfrom
johnslavik:gh-143955-refactor-test-tools-structs
Apr 14, 2026
Merged

pablogsal merged 1 commit into
python:mainfrom
johnslavik:gh-143955-refactor-test-tools-structs

Conversation

johnslavik commented Apr 13, 2026 •
edited
Loading

Copy link
Copy Markdown
Member

Replace namedtuple reimplementations with factory helpers that wrap the real _remote_debugging structseq types. This prevents silent drift between test stubs and production structs, as previously occurred in gh-142394 (see gh-143952).

Replace namedtuple reimplementations with factory helpers that wrap
the real _remote_debugging structseq types. This prevents silent
drift between test stubs and production structs, as previously
occurred in pythongh-142394/pythongh-143952.

Copy link
Copy Markdown
Member Author

I don't expect a schema change in 3.14 so let's not backport it

johnslavik changed the title gh-143955: Use _remote_debugging structs directly in test_tools gh-143955: Prevent schema drift false-positives in asyncio tools tests Apr 13, 2026
pablogsal merged commit 21da9d7 into python:main Apr 14, 2026
54 checks passed
ljfp pushed a commit to ljfp/cpython that referenced this pull request Apr 25, 2026
johnslavik deleted the gh-143955-refactor-test-tools-structs branch July 21, 2026 01:54
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip news tests Tests in the Lib/test dir

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL