fix(backend/copilot): reformat assert statements to pass black lint

This commit is contained in:
majdyz
2026-04-15 15:05:46 +07:00
parent c68c7d7560
commit e0a8bc5224
2 changed files with 24 additions and 24 deletions

View File

@@ -237,9 +237,9 @@ class TestPromptSupplement:
for tool_name, tool in TOOL_REGISTRY.items():
if not tool.is_available:
continue
assert f"`{tool_name}`" in docs, (
f"Tool '{tool_name}' missing from baseline supplement"
)
assert (
f"`{tool_name}`" in docs
), f"Tool '{tool_name}' missing from baseline supplement"
def test_pause_task_scheduled_before_transcript_upload(self):
"""Pause is scheduled as a background task before transcript upload begins.

View File

@@ -193,9 +193,9 @@ class TestRunAgentToolSchema:
"""dry_run must be in the run_agent LLM schema so the LLM can request
per-call dry runs in normal sessions."""
params = cast(dict[str, Any], schema["function"].get("parameters", {}))
assert "dry_run" in params.get("properties", {}), (
"dry_run must be exposed in the run_agent LLM schema"
)
assert "dry_run" in params.get(
"properties", {}
), "dry_run must be exposed in the run_agent LLM schema"
assert "dry_run" not in params.get("required", [])
def test_wait_for_result_in_schema(self, schema: ChatCompletionToolParam):
@@ -226,9 +226,9 @@ class TestRunBlockToolSchema:
"""dry_run must NOT be in the run_block LLM schema — it is session-level."""
params = cast(dict[str, Any], schema["function"].get("parameters", {}))
props = params.get("properties", {})
assert "dry_run" not in props, (
"dry_run must not be exposed in the run_block LLM schema"
)
assert (
"dry_run" not in props
), "dry_run must not be exposed in the run_block LLM schema"
assert "dry_run" not in params.get("required", [])
def test_block_id_and_input_data_are_required(
@@ -265,9 +265,9 @@ class TestRunAgentInputModel:
from backend.copilot.tools.run_agent import RunAgentTool
tool = RunAgentTool()
assert "dry_run" in tool.parameters.get("properties", {}), (
"dry_run must be exposed in the LLM tool schema"
)
assert "dry_run" in tool.parameters.get(
"properties", {}
), "dry_run must be exposed in the LLM tool schema"
def test_dry_run_accepts_true(self):
model = RunAgentInput(username_agent_slug="user/agent", dry_run=True)
@@ -341,9 +341,9 @@ class TestGuideWorkflowOrdering:
"""Step 7 (Save/create_agent) must appear before step 8 (Dry-run)."""
create_pos = guide_content.index("create_agent")
dry_run_pos = guide_content.index("dry_run=True")
assert create_pos < dry_run_pos, (
"create_agent must appear before dry_run=True in the workflow"
)
assert (
create_pos < dry_run_pos
), "create_agent must appear before dry_run=True in the workflow"
def test_dry_run_before_inspect_in_verification_section(self, guide_content: str):
"""In the verification loop section, Dry-run step must come before
@@ -352,9 +352,9 @@ class TestGuideWorkflowOrdering:
section = guide_content[section_start:]
dry_run_pos = section.index("**Dry-run**")
inspect_pos = section.index("**Inspect")
assert dry_run_pos < inspect_pos, (
"Dry-run step must come before Inspect & fix in the verification loop"
)
assert (
dry_run_pos < inspect_pos
), "Dry-run step must come before Inspect & fix in the verification loop"
def test_fix_before_repeat_in_verification_section(self, guide_content: str):
"""The Fix step must come before the Repeat step."""
@@ -377,9 +377,9 @@ class TestGuideWorkflowOrdering:
section = guide_content[section_start:]
# The section should contain numbered items 1 through 5
for step_num in range(1, 6):
assert f"{step_num}. " in section, (
f"Missing numbered step {step_num} in verification workflow"
)
assert (
f"{step_num}. " in section
), f"Missing numbered step {step_num} in verification workflow"
def test_workflow_steps_are_in_numbered_order(self, guide_content: str):
"""The main workflow steps (1-9) must appear in ascending order."""
@@ -395,9 +395,9 @@ class TestGuideWorkflowOrdering:
if match:
step_positions.append((step_num, match.start()))
# Verify at least steps 1-9 are present and in order
assert len(step_positions) >= 9, (
f"Expected 9 workflow steps, found {len(step_positions)}"
)
assert (
len(step_positions) >= 9
), f"Expected 9 workflow steps, found {len(step_positions)}"
for i in range(1, len(step_positions)):
prev_num, prev_pos = step_positions[i - 1]
curr_num, curr_pos = step_positions[i]