GeezerVison

@terryjreedy asked to not move IDLE tests:

• Move test sub-packages to Lib/test #54781 (comment)
• Move test sub-packages to Lib/test #54781 (comment)

I would prefer to not have IDLE as a special case and have all tests in Lib/test/. But @terryjreedy is the IDLE maintainer, and so I will follow his decision :-)

I created a PR to see in practice the consequences of moving Lib/idlelib/idle_test/ to Lib/test/test_idle/.

IMO such PR should not be backported since it has an impact on Linux distributions which put Lib/idlelib/idle_test/ in a separated "test" package, separated from the "python" package, to keep the "python" package smaller. At least, Fedora does that for "python3-test".

If the PR is not backported, we rely on Git to detect renamed/moved files to automate backports. In my experience, Git is quite good at that. We move files often, and Git always smoothly finds the old files and automatically patchs the correct file on backport.

If I understood correctly, IDLE is developed as an application separated from Python, but its code lives in Python Lib/idlelib/ directory. The code is the same, or almost the same, in 3.10, 3.11 and main branches. So I'm not sure how to deal with the minor different introduced by this PR. For example, 'idlelib.idle_test.test_help' is replaced with 'test.test_idle.test_help' in Lib/idlelib/help.py:

$ tail Lib/idlelib/help.py

if __name__ == '__main__':
    from unittest import main
    main('test.test_idle.test_help', verbosity=2, exit=False)

    from test.test_idle.htest import run
    run(show_idlehelp)

asyncio was developed as a third-party project (named "tulip") and distributed on PyPI. I recall that it was painful to synchronize the code between the 3rd party project and all supported branches in Python. So well, I'm not sure why should be done with IDLE. It's up to you Terry ;-)

cc @serhiy-storchaka who also seems to be actively maintaining IDLE.

Oh, I'm not sure how to copy this code from the removed test_idle.py:

if __name__ == '__main__':
    tk.NoDefaultRoot()
    unittest.main(exit=False)
    tk._support_default_root = True
    tk._default_root = None

Maybe put a setUpModule() method somewhere.

PR rebased to fix a merge conflict (remove unused imports).

When you're done making the requested changes, leave the comment: I have made the requested changes; please review again.

Sorry I lost track of the issue. I mostly wrote it to see how it can go. It seems like you want to do it differently. Feel free to do it your way and copy/paste anything from this PR. I close my PR.