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 '