[arrow] branch master updated: ARROW-17725: [CI][Python] Fix test collection in case of Arrow built without parquet (#14119)

[ap...@apache.org]

This is an automated email from the ASF dual-hosted git repository.

apitrou pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/arrow.git


The following commit(s) were added to refs/heads/master by this push:
     new bc8a608237 ARROW-17725: [CI][Python] Fix test collection in case of Arrow built without parquet (#14119)
bc8a608237 is described below

commit bc8a608237424fa0a575e5226dd2dc4508fd8333
Author: Raúl Cumplido <ra...@gmail.com>
AuthorDate: Wed Sep 14 14:26:11 2022 +0200

    ARROW-17725: [CI][Python] Fix test collection in case of Arrow built without parquet (#14119)
    
    As seen on the JIRA ticket our minimal builds were failing due to failures on test collection. The issue happened when Arrow was built without PARQUET.
    
    Authored-by: Raúl Cumplido <ra...@gmail.com>
    Signed-off-by: Antoine Pitrou <an...@python.org>
---
 python/pyarrow/tests/parquet/test_dataset.py | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/python/pyarrow/tests/parquet/test_dataset.py b/python/pyarrow/tests/parquet/test_dataset.py
index fe31fe7931..6fdc743541 100644
--- a/python/pyarrow/tests/parquet/test_dataset.py
+++ b/python/pyarrow/tests/parquet/test_dataset.py
@@ -563,8 +563,9 @@ def test_filters_invalid_column(tempdir, use_legacy_dataset):
                           pc.field('integers') < 3,
                           pc.field('nested', 'a') < 3,
                           pc.field('nested', 'b').cast(pa.int64()) < 3))
-@pytest.mark.parametrize("read", (pq.read_table, pq.read_pandas))
-def test_filters_read_table(tempdir, use_legacy_dataset, filters, read):
+@pytest.mark.parametrize("read_method", ("read_table", "read_pandas"))
+def test_filters_read_table(tempdir, use_legacy_dataset, filters, read_method):
+    read = getattr(pq, read_method)
     # test that filters keyword is passed through in read_table
     fs = LocalFileSystem._get_instance()
     base_path = tempdir