Skip to content

gh-129269: Exclude everything in sys.path in test_coverage_ignore #130133

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed

Conversation

colesbury
Copy link
Contributor

@colesbury colesbury commented Feb 14, 2025

The test_trace.test_coverage_ignore test would fail if you had setuptools installed, such as in ~/.local/lib/python3.xxx/site-packages/. Ignore everything in sys.path when running the test.

…ore`

The `test_trace.test_coverage_ignore` test would fail if you had
`setuptools` installed, such as in `~/.local/lib/python3.xxx/site-packages/`.
Ignore everything in `sys.path` when running the test.
@colesbury colesbury added tests Tests in the Lib/test dir skip news 🔨 test-with-refleak-buildbots Test PR w/ refleak buildbots; report in status section labels Feb 14, 2025
@bedevere-bot
Copy link

🤖 New build scheduled with the buildbot fleet by @colesbury for commit d4efaf5 🤖

Results will be shown at:

https://buildbot.python.org/all/#/grid?branch=refs%2Fpull%2F30617%2Fmerge

If you want to schedule another build, you need to add the 🔨 test-with-refleak-buildbots label again.

@bedevere-bot bedevere-bot removed the 🔨 test-with-refleak-buildbots Test PR w/ refleak buildbots; report in status section label Feb 14, 2025
@colesbury colesbury marked this pull request as ready for review February 14, 2025 20:21
@colesbury colesbury requested a review from Yhg1s February 14, 2025 20:22
Copy link
Member

@hugovk hugovk left a comment

Choose a reason for hiding this comment

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

Was about to look into this, glad I searched the tracker first :)

This is reproducible on 3.12 as well, let's backport.

@colesbury
Copy link
Contributor Author

Ugh, GitHub merged the PR into main, but hasn't marked the PR as merged:

35925e9

colesbury added a commit to colesbury/cpython that referenced this pull request Feb 20, 2025
…age_ignore` (pythongh-130133)

The `test_trace.test_coverage_ignore` test would fail if you had
`setuptools` installed, such as in `~/.local/lib/python3.xxx/site-packages/`.
Ignore everything in `sys.path` when running the test.
(cherry picked from commit 35925e9)

Co-authored-by: Sam Gross <[email protected]>
@bedevere-app
Copy link

bedevere-app bot commented Feb 20, 2025

GH-130357 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.13 bugs and security fixes label Feb 20, 2025
@bedevere-app
Copy link

bedevere-app bot commented Feb 20, 2025

GH-130358 is a backport of this pull request to the 3.12 branch.

@bedevere-app bedevere-app bot removed the needs backport to 3.12 only security fixes label Feb 20, 2025
@colesbury
Copy link
Contributor Author

This is merged.

@colesbury colesbury closed this Feb 20, 2025
colesbury added a commit that referenced this pull request Feb 20, 2025
…nore` (gh-130133) (gh-130358)

The `test_trace.test_coverage_ignore` test would fail if you had
`setuptools` installed, such as in `~/.local/lib/python3.xxx/site-packages/`.
Ignore everything in `sys.path` when running the test.
(cherry picked from commit 35925e9)
pull bot pushed a commit to TechPenguineer/cpython that referenced this pull request Feb 20, 2025
…ore` (pythongh-130133)

The `test_trace.test_coverage_ignore` test would fail if you had
`setuptools` installed, such as in `~/.local/lib/python3.xxx/site-packages/`.
Ignore everything in `sys.path` when running the test.
colesbury added a commit that referenced this pull request Feb 20, 2025
…nore` (gh-130133) (gh-130357)

The `test_trace.test_coverage_ignore` test would fail if you had
`setuptools` installed, such as in `~/.local/lib/python3.xxx/site-packages/`.
Ignore everything in `sys.path` when running the test.
(cherry picked from commit 35925e9)
@colesbury colesbury deleted the gh-129269-test-coverage-ignore branch February 28, 2025 15:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting merge skip news tests Tests in the Lib/test dir
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants