Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CI][Python] python_test_emscripten fails with missing PARQUET_TEST_DATA environment variable #43905

Closed
raulcd opened this issue Sep 2, 2024 · 1 comment

Comments

@raulcd
Copy link
Member

raulcd commented Sep 2, 2024

Describe the bug, including details regarding any error messages, version, and platform.

The test-conda-python-emscripten is failing with the following error:


    @pytest.fixture(scope='module')
    def parquet_test_datadir():
        result = os.environ.get('PARQUET_TEST_DATA')
        if not result:
>           raise RuntimeError('Please point the PARQUET_TEST_DATA environment '
                               'variable to the test data directory')
E           RuntimeError: Please point the PARQUET_TEST_DATA environment variable to the test data directory

This has been failing since this check was introduced in: #41580

Component(s)

Continuous Integration, Python

@kou
Copy link
Member

kou commented Sep 3, 2024

We already have an issue for this: #43868

@raulcd raulcd closed this as not planned Won't fix, can't repro, duplicate, stale Sep 3, 2024
raulcd added a commit to raulcd/arrow that referenced this issue Sep 17, 2024
kou pushed a commit that referenced this issue Sep 18, 2024
…on emscripten (#43906)

### Rationale for this change

The following PR:
- #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]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants