gh-77476: Remove urllib HTTP test by vstinner · Pull Request #139502 · python/cpython
Navigation Menu
{{ message }}
- Notifications You must be signed in to change notification settings
- Fork 34.4k
Merged
vstinner merged 1 commit intopython:mainfrom
Oct 2, 2025Merged
gh-77476: Remove urllib HTTP test#139502
vstinner merged 1 commit intopython:mainfrom
gh-77476: Remove urllib HTTP test#139502
vstinner merged 1 commit intopython:mainfrom
Conversation
Copy link Copy Markdown
Member
Remove test_sites_no_connection_close() which is skipped since 2018 (commit 36d56ea).
Remove test_sites_no_connection_close() which is skipped since 2018 (commit 36d56ea).
vstinner
added
the
skip news
label
bedevere-app
bot
mentioned this pull request
Copy link Copy Markdown
Member Author
vstinner
commented
Oct 2, 2025
vstinner commented
Oct 2, 2025Test skipped since 2018, since commit 36d56ea:
commit 36d56ea826caffbeac0fc0c6d90248b80516e33c
Author: INADA Naoki <methane@users.noreply.github.com>
Date: Wed Apr 18 00:31:29 2018 +0900
bpo-33295: Skip test using missing external site (GH-6504)
`test_urllib2net.OtherNetworkTests.test_sites_no_connection_close`
used `http://www.imdb.com/` but it is moved to https so the test is
not valid anymore. Skip test for the moment to allow CI to proceed.
See #77476.
orsenthil approved these changes Oct 2, 2025
bedevere-app
bot
added
awaiting merge
and removed
awaiting core review
labels
vstinner
deleted the
remove_http_test
branch
bedevere-app
bot
removed
the
awaiting merge
label
Copy link Copy Markdown
Member Author
vstinner
commented
Oct 2, 2025
vstinner commented
Oct 2, 2025Merged, thanks for the review @orsenthil.
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