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

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


The following commit(s) were added to refs/heads/main by this push:
     new fe39c8f551 GH-43868: [CI][Python] Skip test that requires 
PARQUET_TEST_DATA env on emscripten (#43906)
fe39c8f551 is described below

commit fe39c8f5518a8b57723d457a8048cc03715bd389
Author: Raúl Cumplido <[email protected]>
AuthorDate: Wed Sep 18 03:18:28 2024 +0200

    GH-43868: [CI][Python] Skip test that requires PARQUET_TEST_DATA env on 
emscripten (#43906)
    
    ### 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 <[email protected]>
    Co-authored-by: Joris Van den Bossche <[email protected]>
    Signed-off-by: Sutou Kouhei <[email protected]>
---
 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 80605e973c..94b3058fa0 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 '

Reply via email to