Skip to content

Commit 6090f2e

Browse files
committed
fix(ci): correct test imports and disable PyPy (scikit-learn incompatible)
1 parent 19146a1 commit 6090f2e

16 files changed

+18
-53
lines changed

.github/workflows/ci.yml

Lines changed: 2 additions & 36 deletions
Original file line numberDiff line numberDiff line change
@@ -134,42 +134,8 @@ jobs:
134134
retention-days: 1
135135

136136
# ============================================================================
137-
# PyPy Compatibility Testing
138-
# ============================================================================
139-
test-pypy:
140-
name: Test (PyPy ${{ matrix.python-version }})
141-
needs: [pre-commit, lint-type-check]
142-
runs-on: ubuntu-latest
143-
strategy:
144-
fail-fast: false
145-
matrix:
146-
python-version: [pypy-3.10]
147-
148-
steps:
149-
- uses: actions/checkout@v6
150-
151-
- name: Set up PyPy ${{ matrix.python-version }}
152-
uses: actions/setup-python@v6
153-
with:
154-
python-version: ${{ matrix.python-version }}
155-
156-
- name: Cache pip
157-
uses: actions/cache@v5
158-
with:
159-
path: ~/.cache/pip
160-
key: pypy-${{ matrix.python-version }}-pip-${{ hashFiles('**/pyproject.toml') }}
161-
restore-keys: |
162-
pypy-${{ matrix.python-version }}-pip-
163-
164-
- name: Install dependencies
165-
run: |
166-
python -m pip install --upgrade pip
167-
pip install -e .
168-
pip install pytest pytest-cov pytest-asyncio
169-
170-
- name: Run tests
171-
run: pytest tests/ -v
172-
137+
# NOTE: PyPy testing disabled - scikit-learn doesn't support PyPy
138+
# (scipy build fails due to OpenBLAS dependency issues)
173139
# ============================================================================
174140
# LAYER 3: Quality analysis (parallel, non-blocking on main)
175141
# ============================================================================

tests/integration/conftest.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,8 @@
99

1010
import pytest
1111
import pytest_asyncio
12-
from arbitrium import Arbitrium
1312

13+
from arbitrium_core import Arbitrium
1414
from arbitrium_core.adapters.llm.registry import ProviderRegistry
1515
from arbitrium_core.ports.llm import BaseModel, ModelResponse
1616

tests/integration/test_e2e_anonymizer.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
"""End-to-end tests for model anonymization."""
22

33
import pytest
4-
from arbitrium import Arbitrium
54

5+
from arbitrium_core import Arbitrium
66
from arbitrium_core.domain.tournament.anonymizer import ModelAnonymizer
77
from tests.integration.conftest import MockModel
88

tests/integration/test_e2e_config.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,8 @@
55

66
import pytest
77
import yaml
8-
from arbitrium import Arbitrium
98

9+
from arbitrium_core import Arbitrium
1010
from arbitrium_core.adapters.config.loader import validate_config
1111
from arbitrium_core.application.bootstrap import health_check_models
1212
from arbitrium_core.domain.errors import ConfigurationError

tests/integration/test_e2e_error_handling.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
"""End-to-end tests for error handling and edge cases."""
22

33
import pytest
4-
from arbitrium import Arbitrium
54

5+
from arbitrium_core import Arbitrium
66
from arbitrium_core.application.bootstrap import health_check_models
77
from tests.integration.conftest import MockModel
88

tests/integration/test_e2e_factory.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -331,7 +331,7 @@ async def test_factory_used_in_arbitrium_initialization(
331331
self,
332332
basic_config: dict,
333333
) -> None:
334-
from arbitrium import Arbitrium
334+
from arbitrium_core import Arbitrium
335335

336336
arbitrium = await Arbitrium.from_settings(
337337
settings=basic_config,
@@ -346,7 +346,7 @@ async def test_factory_creates_correct_number_of_models(
346346
self,
347347
tmp_output_dir,
348348
) -> None:
349-
from arbitrium import Arbitrium
349+
from arbitrium_core import Arbitrium
350350

351351
config = {
352352
"models": {

tests/integration/test_e2e_failures.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
"""End-to-end tests for tournament failure scenarios and error paths."""
22

33
import pytest
4-
from arbitrium import Arbitrium
54

5+
from arbitrium_core import Arbitrium
66
from tests.integration.conftest import MockModel
77

88

tests/integration/test_e2e_judge_mode.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
"""End-to-end tests for judge mode and evaluation edge cases."""
22

33
import pytest
4-
from arbitrium import Arbitrium
54

5+
from arbitrium_core import Arbitrium
66
from tests.integration.conftest import MockModel
77

88

tests/integration/test_e2e_knowledge_bank.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
"""End-to-end tests for Knowledge Bank functionality."""
22

33
import pytest
4-
from arbitrium import Arbitrium
54

5+
from arbitrium_core import Arbitrium
66
from tests.integration.conftest import MockModel
77

88

tests/integration/test_e2e_simple.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,8 @@
11
"""Simple E2E tests that don't require full tournament execution."""
22

33
import pytest
4-
from arbitrium import Arbitrium
54

5+
from arbitrium_core import Arbitrium
66
from tests.integration.conftest import MockModel
77

88

0 commit comments

Comments
 (0)