Skip to content

Commit

Permalink
Update tested GraalPy version in CI (#4867)
Browse files Browse the repository at this point in the history
* Add GraalPy 24.1 version to CI

* Skip failing test on GraalPy
  • Loading branch information
msimacek authored Feb 7, 2025
1 parent 903afcd commit 8ab39d2
Show file tree
Hide file tree
Showing 3 changed files with 11 additions and 0 deletions.
4 changes: 4 additions & 0 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,10 @@ jobs:
- name: Install nox
run: python -m pip install --upgrade pip && pip install nox

- if: inputs.python-version == 'graalpy24.1'
name: Install GraalPy virtualenv (only GraalPy 24.1)
run: python -m pip install 'git+https://github.com/oracle/graalpython#egg=graalpy_virtualenv_seeder&subdirectory=graalpy_virtualenv_seeder'

- name: Install Rust toolchain
uses: dtolnay/rust-toolchain@master
with:
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -255,6 +255,7 @@ jobs:
"pypy3.9",
"pypy3.10",
"graalpy24.0",
"graalpy24.1",
]
platform:
[
Expand Down
6 changes: 6 additions & 0 deletions pytests/tests/test_comparisons.py
Original file line number Diff line number Diff line change
@@ -1,5 +1,6 @@
from typing import Type, Union

import sys
import pytest
from pyo3_pytests.comparisons import (
Eq,
Expand Down Expand Up @@ -28,6 +29,11 @@ def __ne__(self, other: Self) -> bool:
return NotImplemented


@pytest.mark.skipif(
sys.implementation.name == "graalpy"
and __graalpython__.get_graalvm_version().startswith("24.1"), # noqa: F821
reason="Bug in GraalPy 24.1",
)
@pytest.mark.parametrize(
"ty", (Eq, EqDerived, PyEq), ids=("rust", "rust-derived", "python")
)
Expand Down

0 comments on commit 8ab39d2

Please sign in to comment.