mirror of
https://github.com/apache/arrow.git
synced 2026-10-29 00:56:31 +08:00
Guja
eb19b7fce7
### Rationale for this change Resolves #49046 `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. * GitHub Issue: #49046 Authored-by: Guja <127162872+GujaLomsadze@users.noreply.github.com> Signed-off-by: Raúl Cumplido <raulcumplido@gmail.com>