From fe39c8f5518a8b57723d457a8048cc03715bd389 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ra=C3=BAl=20Cumplido?= Date: Wed, 18 Sep 2024 03:18:28 +0200 Subject: [PATCH] GH-43868: [CI][Python] Skip test that requires PARQUET_TEST_DATA env on emscripten (#43906) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ### Rationale for this change The following PR: - https://github.com/apache/arrow/pull/41580 Made mandatory for a test the requirement to have `PARQUET_TEST_DATA` env defined. This is currently not available from `python_test_emscripten.sh` as we require to mount the filesystem for both Node and ChromeDriver. ### What changes are included in this PR? Skip the test that requires `PARQUET_TEST_DATA` for emscripten. ### Are these changes tested? Via archery ### Are there any user-facing changes? No * GitHub Issue: #43905 * GitHub Issue: #43868 Lead-authored-by: Raúl Cumplido Co-authored-by: Joris Van den Bossche Signed-off-by: Sutou Kouhei --- python/pyarrow/tests/parquet/conftest.py | 3 +++ 1 file changed, 3 insertions(+) diff --git a/python/pyarrow/tests/parquet/conftest.py b/python/pyarrow/tests/parquet/conftest.py index 80605e973cda8..94b3058fa02c8 100644 --- a/python/pyarrow/tests/parquet/conftest.py +++ b/python/pyarrow/tests/parquet/conftest.py @@ -17,6 +17,7 @@ import os import pathlib +import sys import pytest @@ -30,6 +31,8 @@ def datadir(base_datadir): @pytest.fixture(scope='module') def parquet_test_datadir(): + if sys.platform == 'emscripten': + pytest.skip("needs PARQUET_TEST_DATA files access") result = os.environ.get('PARQUET_TEST_DATA') if not result: raise RuntimeError('Please point the PARQUET_TEST_DATA environment '