GujaLomsadze opened a new pull request, #50640: URL: https://github.com/apache/arrow/pull/50640
### Rationale for this change `python/scripts/test_imports.py` and `python/scripts/test_leak.py` are leftovers from an older test layout. They are not referenced by CI, packaging or the pytest suite, and are not documented anywhere. Their `test_`-prefixed names have also caused accidental pytest collection in the past (ARROW-1033). This follows the suggestion on the issue to remove them. Deletes `python/scripts/test_imports.py` and `python/scripts/test_leak.py`. The other two scripts in that directory are untouched and still in use: `run_emscripten_tests.py` (invoked from `ci/scripts/python_test_emscripten.sh`) and `update_stub_docstrings.py`. The issue description raised the option of keeping `test_leak.py` as a documented snippet for checking leaks. This PR removes it, following the later comment. Happy to restore and document it under the developer docs instead if reviewers prefer. ### Are these changes tested? No new tests, this only removes files that nothing references. I grepped `ci/`, `dev/`, `python/setup.py`, `python/pyproject.toml` and `python/MANIFEST.in` for references and found none. The only remaining mention in the repository is a historical `CHANGELOG.md` entry for ARROW-1033, left as is. ### Are there any user-facing changes? No. ### AI usage disclosure I used Claude Code to grep the repository for remaining references to the two scripts and to help draft this description. The change itself is a two file deletion, and I reviewed both the search results and the diff. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
