Skip to content

Commit a48a3cd

Browse files
mikejhillCopilot
andcommitted
fix: remove unused import and fix integration test socket blocking
- Remove unused json import from validate_examples.py (ruff F401) - Replace --force-enable-socket with -p no:socket for integration tests to fully disable pytest-socket plugin when running against real API - Update README integration test command accordingly Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 59f2070 commit a48a3cd

File tree

3 files changed

+2
-3
lines changed

3 files changed

+2
-3
lines changed

.github/workflows/integration-tests.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,6 +31,6 @@ jobs:
3131
ROUVY_TEST_PASSWORD: ${{ secrets.ROUVY_TEST_PASSWORD }}
3232
run: >-
3333
uv run pytest tests/integration/ -m integration
34-
--force-enable-socket
34+
-p no:socket
3535
-v --tb=short
3636
${{ inputs.test_filter && format('-k "{0}"', inputs.test_filter) || '' }}

README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -259,7 +259,7 @@ uv run ruff format . # Format
259259
```bash
260260
export ROUVY_TEST_EMAIL="email@example.com"
261261
export ROUVY_TEST_PASSWORD="password"
262-
uv run pytest tests/integration/ -m integration -v
262+
uv run pytest tests/integration/ -m integration -p no:socket -v
263263
```
264264

265265
**GitHub Actions:** Trigger the **Integration Tests** workflow manually from

docs-src/scripts/validate_examples.py

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@
1313

1414
from __future__ import annotations
1515

16-
import json
1716
import sys
1817
from pathlib import Path
1918

0 commit comments

Comments
 (0)