Skip to content

Commit 5db5edc

Browse files
committed
fix: py tests
1 parent 8f280a7 commit 5db5edc

File tree

2 files changed

+11
-10
lines changed

2 files changed

+11
-10
lines changed

python/tests/unit/test_generation_program.py

Lines changed: 9 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -374,10 +374,10 @@ async def test_forward_retry_logic(self, call_method, generation_program):
374374
# Mock the generation_program to raise AdapterParseError
375375
side_effect = [
376376
AdapterParseError(
377-
"Parse error 1", CairoCodeGeneration(), "", "test response", None
377+
"Parse error 1", CairoCodeGeneration, "", "test response", None
378378
),
379379
AdapterParseError(
380-
"Parse error 2", CairoCodeGeneration(), "", "test response", None
380+
"Parse error 2", CairoCodeGeneration, "", "test response", None
381381
),
382382
dspy.Prediction(answer="Success"),
383383
]
@@ -399,16 +399,16 @@ async def test_forward_max_retries_exceeded(self, call_method, generation_progra
399399
# Mock the generation_program to always raise AdapterParseError
400400
side_effect = [
401401
AdapterParseError(
402-
"Parse error", CairoCodeGeneration(), "", "test response", None
402+
"Parse error", CairoCodeGeneration, "", "test response", None
403403
),
404404
AdapterParseError(
405-
"Parse error", CairoCodeGeneration(), "", "test response", None
405+
"Parse error", CairoCodeGeneration, "", "test response", None
406406
),
407407
AdapterParseError(
408-
"Parse error", CairoCodeGeneration(), "", "test response", None
408+
"Parse error", CairoCodeGeneration, "", "test response", None
409409
),
410410
AdapterParseError(
411-
"Parse error", CairoCodeGeneration(), "", "test response", None
411+
"Parse error", CairoCodeGeneration, "", "test response", None
412412
),
413413
]
414414
getattr(generation_program.generation_program, call_method).side_effect = side_effect
@@ -446,13 +446,13 @@ async def test_should_extract_code_before_raising(self, generation_program, call
446446
# Mock the generation_program to raise AdapterParseError
447447
side_effect = [
448448
AdapterParseError(
449-
"Parse error", CairoCodeGeneration(), "", "test response", None
449+
"Parse error", CairoCodeGeneration, "", "test response", None
450450
),
451451
AdapterParseError(
452-
"Parse error", CairoCodeGeneration(), "", "test response", None
452+
"Parse error", CairoCodeGeneration, "", "test response", None
453453
),
454454
AdapterParseError(
455-
"Parse error", CairoCodeGeneration(), "```cairo\nfn main() {}\n```", "test response", None
455+
"Parse error", CairoCodeGeneration, "```cairo\nfn main() {}\n```", "test response", None
456456
),
457457
]
458458
generation_program.generation_program.aforward.side_effect = side_effect

python/tests/unit/test_rag_pipeline.py

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@
88
from unittest.mock import AsyncMock, Mock, patch
99

1010
import pytest
11+
import dspy
1112

1213
from cairo_coder.core.rag_pipeline import (
1314
RagPipeline,
@@ -123,7 +124,7 @@ def mock_mcp_generation_program(self):
123124
124125
Storage variables use #[storage] attribute.
125126
"""
126-
program.forward.return_value = mock_res
127+
program.forward.return_value = dspy.Prediction(answer=mock_res)
127128
return program
128129

129130
@pytest.fixture

0 commit comments

Comments
 (0)