From fdeebed3da6d899d815d28ec5e092943a5f0d8cd Mon Sep 17 00:00:00 2001 From: Anas Khan <83116240+anxkhn@users.noreply.github.com> Date: Sun, 5 Jul 2026 02:16:43 +0530 Subject: [PATCH] MINOR: [Python] Assert selector file count on the s3fs branch in test_get_file_info_with_selector The recursive-selector count check on the fsspec S3 branch of test_get_file_info_with_selector was a bare expression (len(infos) == 4) with no assert, so the listing count was never verified for that filesystem. The non-recursive branch and the non-s3fs branch both already assert their counts. Add the missing assert so the recursive count is actually checked on the s3fs path. --- python/pyarrow/tests/test_fs.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/python/pyarrow/tests/test_fs.py b/python/pyarrow/tests/test_fs.py index 5bf1950c0654..e5caa1c88ddf 100644 --- a/python/pyarrow/tests/test_fs.py +++ b/python/pyarrow/tests/test_fs.py @@ -803,7 +803,7 @@ def test_get_file_info_with_selector(fs, pathfn): infos = fs.get_file_info(selector) if fs.type_name == "py::fsspec+('s3', 's3a')": # s3fs only lists directories if they are not empty - len(infos) == 4 + assert len(infos) == 4 else: assert len(infos) == 5