Circuit-Forge/snipe
6
0
Fork 0
Code Issues 13 Pull requests 1 Projects Releases 2 Packages Wiki Activity Actions

fix(ci): skip cf-orch tests when circuitforge_orch is not installed
Some checks failed
Mirror / mirror (push) Has been cancelled
Details
CI / Python tests (push) Has been cancelled
Details
CI / Frontend typecheck + tests (push) Has been cancelled
Details

Browse source 
The 4 photo-analysis-orch tests require the internal circuitforge_orch
package which is not available in GitHub CI. Guard them with a skipif
mark checked at import time via importlib.util.find_spec.
This commit is contained in:
pyr0ball 2026-07-06 02:57:39 -07:00
parent ad687fabfd
commit 72ab3ac177
1 changed files with 11 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
tests/test_tasks/test_runner.py
View file

@ -1,6 +1,7 @@
"""Tests for snipe background task runner.""" """Tests for snipe background task runner."""
from __future__ import annotations from __future__ import annotations
import importlib.util
import json import json
import sqlite3 import sqlite3
from pathlib import Path from pathlib import Path
@ -8,6 +9,12 @@ from unittest.mock import MagicMock, patch
import pytest import pytest
_cforch_available = importlib.util.find_spec("circuitforge_orch") is not None
requires_cforch = pytest.mark.skipif(
not _cforch_available,
reason="circuitforge_orch not installed",
)
from app.tasks.runner import ( from app.tasks.runner import (
LLM_TASK_TYPES, LLM_TASK_TYPES,
VRAM_BUDGETS, VRAM_BUDGETS,
@ -172,6 +179,7 @@ def _make_orch_client_mock(vision_json: str) -> MagicMock:
return client return client
@requires_cforch
def test_run_task_photo_analysis_orch_success(tmp_db: Path): def test_run_task_photo_analysis_orch_success(tmp_db: Path):
"""Cloud path: CFOrchClient.task_allocate is used when GPU_SERVER_URL is set.""" """Cloud path: CFOrchClient.task_allocate is used when GPU_SERVER_URL is set."""
task_id, _ = insert_task(tmp_db, "trust_photo_analysis", job_id=1, params=_PARAMS) task_id, _ = insert_task(tmp_db, "trust_photo_analysis", job_id=1, params=_PARAMS)
@ -207,6 +215,7 @@ def test_run_task_photo_analysis_orch_success(tmp_db: Path):
assert parsed["authenticity_signal"] == "genuine_product_photo" assert parsed["authenticity_signal"] == "genuine_product_photo"
@requires_cforch
def test_run_task_photo_analysis_orch_uses_image_assessment_task(tmp_db: Path): def test_run_task_photo_analysis_orch_uses_image_assessment_task(tmp_db: Path):
"""task_allocate must be called with product='snipe', task='image_assessment'.""" """task_allocate must be called with product='snipe', task='image_assessment'."""
task_id, _ = insert_task(tmp_db, "trust_photo_analysis", job_id=1, params=_PARAMS) task_id, _ = insert_task(tmp_db, "trust_photo_analysis", job_id=1, params=_PARAMS)
@ -232,6 +241,7 @@ def test_run_task_photo_analysis_orch_uses_image_assessment_task(tmp_db: Path):
client_instance.task_allocate.assert_called_once_with("snipe", "image_assessment") client_instance.task_allocate.assert_called_once_with("snipe", "image_assessment")
@requires_cforch
def test_run_task_photo_analysis_orch_sends_image_url_content(tmp_db: Path): def test_run_task_photo_analysis_orch_sends_image_url_content(tmp_db: Path):
"""Vision payload must include image_url content block with data URI.""" """Vision payload must include image_url content block with data URI."""
task_id, _ = insert_task(tmp_db, "trust_photo_analysis", job_id=1, params=_PARAMS) task_id, _ = insert_task(tmp_db, "trust_photo_analysis", job_id=1, params=_PARAMS)
@ -268,6 +278,7 @@ def test_run_task_photo_analysis_orch_sends_image_url_content(tmp_db: Path):
assert url.startswith("data:image/jpeg;base64,"), f"Unexpected image URL format: {url[:40]}" assert url.startswith("data:image/jpeg;base64,"), f"Unexpected image URL format: {url[:40]}"
@requires_cforch
def test_run_task_photo_analysis_orch_task_not_found_falls_back(tmp_db: Path): def test_run_task_photo_analysis_orch_task_not_found_falls_back(tmp_db: Path):
"""TaskNotFound from cf-orch → graceful fallback to local LLMRouter.""" """TaskNotFound from cf-orch → graceful fallback to local LLMRouter."""
from circuitforge_orch.client import TaskNotFound from circuitforge_orch.client import TaskNotFound