Title: We don't run `test_lazy_imports` · Issue #145334 · python/cpython · GitHub
Open Graph Title: We don't run `test_lazy_imports` · Issue #145334 · python/cpython
X Title: We don't run `test_lazy_imports` · Issue #145334 · python/cpython
Description: Bug report Bug description: This is Lib.test.test_import: Lib/test/test_import ├── __init__.py ├── __main__.py ├── data └── test_lazy_imports.py Our test finder doesn't see test_lazy_imports: #144852 (comment). python -m test does not ru...
Open Graph Description: Bug report Bug description: This is Lib.test.test_import: Lib/test/test_import ├── __init__.py ├── __main__.py ├── data └── test_lazy_imports.py Our test finder doesn't see test_lazy_imports: #1448...
X Description: Bug report Bug description: This is Lib.test.test_import: Lib/test/test_import ├── __init__.py ├── __main__.py ├── data └── test_lazy_imports.py Our test finder doesn't see test_lazy_imports: #...
Opengraph URL: https://github.com/python/cpython/issues/145334
X: @github
Domain: github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"We don't run `test_lazy_imports`","articleBody":"# Bug report\n\n### Bug description:\n\nThis is `Lib.test.test_import`:\n\n```\nLib/test/test_import\n├── __init__.py\n├── __main__.py\n├── data\n└── test_lazy_imports.py\n```\n\nOur test finder doesn't see `test_lazy_imports`: https://github.com/python/cpython/pull/144852#issuecomment-3974999139.\n\n`python -m test` does not run `test_lazy_imports` at all.\n\nWe could try to keep regular import test package with lazy import test package by:\n1) Updating [`Lib.test.libregrtest.findtests.SPLITTESTDIRS`](https://github.com/python/cpython/blob/d2f709de1e827c1865532f9fe2dd33fde88057d8/Lib/test/libregrtest/findtests.py#L14-L34) to include `test_import`.\n2) Moving `Lib/test/test_import/__init__.py` to `Lib/test/test_import/test_import.py` .\n3) Creating a new `Lib/test/test_import/__init__.py` that defines `load_tests` hook that calls `load_package_tests`.\n\nIt is visible by now that this is a can of worms. `Lib.test.test_import` uses relative imports itself, so moving `Lib.test.test_import` to `Lib.test.test_import.test_import` breaks them. This means we'd need to massage `Lib.test.test_import.test_import` to work again, which means we'd need to modify old tests -- which I think on its own disqualifies this approach.\n\nWe can't have `Lib.test.test_import` and `Lib.test.test_import.test_lazy_imports` at the same time set up in a maintainable way, because it would involve having manual registration of test suites in `test_import` (or maybe in `test_lazy_imports`?). Perhaps we can do this cheaply, but I assume this makes these tests less self-contained and complicates the setup.\n\nI propose we move `Lib/test/test_import/test_lazy_imports.py` (+data) into `Lib/test/test_lazy_import/__init__.py` (+data). Unless there is any other cost-effective solution to this that I've missed?\n\nCC @pablogsal @DinoV @brittanyrey @Yhg1s @vstinner @sobolevn \n\n### CPython versions tested on:\n\nCPython main branch\n\n### Operating systems tested on:\n\n_No response_\n\n\u003c!-- gh-linked-prs --\u003e\n### Linked PRs\n* gh-145336\n\u003c!-- /gh-linked-prs --\u003e\n","author":{"url":"https://github.com/johnslavik","@type":"Person","name":"johnslavik"},"datePublished":"2026-02-27T23:17:08.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":1},"url":"https://github.com/145334/cpython/issues/145334"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:953f09d8-6953-ac34-8d8b-8e54448db2be |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | BFFA:2373EB:6AB902B:95868CD:6A505281 |
| html-safe-nonce | 9e0a44909f042ce1a890845db9ec1b07be80ce08ec3042cda20620b0783e1227 |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJCRkZBOjIzNzNFQjo2QUI5MDJCOjk1ODY4Q0Q6NkE1MDUyODEiLCJ2aXNpdG9yX2lkIjoiNjUzMTM2MDM5MjYzNjIxNzYxIiwicmVnaW9uX2VkZ2UiOiJpYWQiLCJyZWdpb25fcmVuZGVyIjoiaWFkIn0= |
| visitor-hmac | ec6c71b789b663e273cd42bb45aa014365cf89e30a2bf0e54b0079cdf994cf51 |
| hovercard-subject-tag | issue:4003261115 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/python/cpython/145334/issue_layout |
| twitter:image | https://opengraph.githubassets.com/7bcf13690e94f34b20080128da79e69983e642d5ffcc4d0798209c0c4f39d534/python/cpython/issues/145334 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/7bcf13690e94f34b20080128da79e69983e642d5ffcc4d0798209c0c4f39d534/python/cpython/issues/145334 |
| og:image:alt | Bug report Bug description: This is Lib.test.test_import: Lib/test/test_import ├── __init__.py ├── __main__.py ├── data └── test_lazy_imports.py Our test finder doesn't see test_lazy_imports: #1448... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | johnslavik |
| hostname | github.com |
| expected-hostname | github.com |
| None | d6dc8294eb500fa36bbc57472d61fe87c522f9c3c1d64f70f4926f66a66a7efb |
| turbo-cache-control | no-preview |
| go-import | github.com/python/cpython git https://github.com/python/cpython.git |
| octolytics-dimension-user_id | 1525981 |
| octolytics-dimension-user_login | python |
| octolytics-dimension-repository_id | 81598961 |
| octolytics-dimension-repository_nwo | python/cpython |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 81598961 |
| octolytics-dimension-repository_network_root_nwo | python/cpython |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | 5741a45fe07e5f377d0d4c524a92e3056acc51bb |
| ui-target | full |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width