Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

We don't runtest_lazy_imports #145334

Closed
Closed
Labels
3.15new features, bugs and security fixesinfraCI, GitHub Actions, buildbots, Dependabot, etc.testsTests in the Lib/test dirtype-bugAn unexpected behavior, bug, or error
@johnslavik

Description

@johnslavik

Bug report

Bug description:

This isLib.test.test_import:

Lib/test/test_import├── __init__.py├── __main__.py├── data└── test_lazy_imports.py

Our test finder doesn't seetest_lazy_imports:#144852 (comment).

python -m test does not runtest_lazy_imports at all.

We could try to keep regular import test package with lazy import test package by:

  1. UpdatingLib.test.libregrtest.findtests.SPLITTESTDIRS to includetest_import.
  2. MovingLib/test/test_import/__init__.py toLib/test/test_import/test_import.py .
  3. Creating a newLib/test/test_import/__init__.py that definesload_tests hook that callsload_package_tests.

It is visible by now that this is a can of worms.Lib.test.test_import uses relative imports itself, so movingLib.test.test_import toLib.test.test_import.test_import breaks them. This means we'd need to massageLib.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.

We can't haveLib.test.test_import andLib.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 intest_import (or maybe intest_lazy_imports?). Perhaps we can do this cheaply, but I assume this makes these tests less self-contained and complicates the setup.

I propose we moveLib/test/test_import/test_lazy_imports.py (+data) intoLib/test/test_lazy_import/__init__.py (+data). Unless there is any other cost-effective solution to this that I've missed?

CC@pablogsal@DinoV@brittanyrey@Yhg1s@vstinner@sobolevn

CPython versions tested on:

CPython main branch

Operating systems tested on:

No response

Linked PRs

Metadata

Metadata

Assignees

No one assigned

    Labels

    3.15new features, bugs and security fixesinfraCI, GitHub Actions, buildbots, Dependabot, etc.testsTests in the Lib/test dirtype-bugAn unexpected behavior, bug, or error

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions


      [8]ページ先頭

      ©2009-2026 Movatter.jp