Skip to content

gh-155727: Ignore internal imports in test_attribute_completion - #155730

Merged
serhiy-storchaka merged 1 commit into
python:mainfrom
serhiy-storchaka:test-pyrepl-completion-imports
Aug 14, 2026
Merged

gh-155727: Ignore internal imports in test_attribute_completion#155730
serhiy-storchaka merged 1 commit into
python:mainfrom
serhiy-storchaka:test-pyrepl-completion-imports

Conversation

@serhiy-storchaka

@serhiy-storchaka serhiy-storchaka commented Aug 13, 2026

Copy link
Copy Markdown
Member

Ignore _pyrepl submodules when comparing the modules imported while the reader runs.

Reader.run_hooks() imports _pyrepl._threading_handler lazily, and the
test counted it as a module imported by the completer.

@vstinner vstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM. The change fix the test as expected.

@serhiy-storchaka
serhiy-storchaka merged commit 94c6066 into python:main Aug 14, 2026
60 checks passed
@miss-islington-app

Copy link
Copy Markdown

Thanks @serhiy-storchaka for the PR 🌮🎉.. I'm working now to backport this PR to: 3.15.
🐍🍒⛏🤖

@serhiy-storchaka
serhiy-storchaka deleted the test-pyrepl-completion-imports branch August 14, 2026 04:40
@bedevere-app

bedevere-app Bot commented Aug 14, 2026

Copy link
Copy Markdown

GH-155769 is a backport of this pull request to the 3.15 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.15 pre-release feature fixes, bugs and security fixes label Aug 14, 2026
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