Files
temp/features/steps/actor_cli_yaml_steps.py
freemo 02250473ad fix(ci): restore all CI quality gates to passing on master
Fix all failing CI quality gates (lint, unit_tests, format) without
suppressing any quality enforcement.

Root causes and fixes:

1. Format: features/steps/plan_namespaced_name_tdd_steps.py had trailing
   whitespace; fixed by running ruff format.

2. Unit tests - A2A JSON-RPC 2.0 migration (commit 9c6d6915) renamed
   A2aRequest fields (operation→method, request_id→id, a2a_version→jsonrpc)
   and A2aResponse fields (status+data→result, request_id→id) but did not
   update all step files and feature files:
   - a2a_jsonrpc_wire_format_steps.py: added use_step_matcher('re') and
     reset to 'parse' at end to prevent parallel test interference
   - a2a_facade_wiring_steps.py: updated operation= to method=, .status/.data
     to .result
   - a2a_facade_steps.py: updated request_id→id, a2a_version→jsonrpc,
     A2aResponse(request_id=..., status=...) to new API
   - m6_facade_steps.py: updated all old API usage
   - devcontainer_cleanup_steps.py: updated A2aRequest(operation=...)
   - plan_prompt_command_steps.py: updated A2aRequest(operation=...)
   - wf03_plan_prompt_confidence_steps.py: updated A2aRequest(operation=...)
   - consolidated_misc.feature: updated old A2aRequest/A2aResponse scenarios

3. Unit tests - Session CLI output changed (commit 0d5d9cf0 and others):
   - 'Session Created' → 'Session created' (lowercase)
   - 'Session Details' → 'Session Summary'
   - 'Sessions (N total)' → 'Sessions'
   - session list JSON: top-level 'total' → nested 'summary.total'
   - Fixed in: session_cli.feature, session_cli_coverage_boost.feature,
     session_cli_uncovered_branches.feature, session_list_error.feature,
     tdd_session_create_persist_steps.py

4. Unit tests - Plan list output changed (commit 1a07a891):
   - 'V3 Lifecycle Plans' → 'Plans'
   - 'Lifecycle Plans' → 'Plans'
   - Name column removed (restored in source)
   - Invariants column removed (restored in source)
   - Project truncation removed (restored in source)
   - Fixed in: plan_cli_cancel_revert_coverage.feature,
     plan_lifecycle_cli_coverage.feature, plan_cli_coverage_boost_steps.py,
     plan.py (source code restored)

5. Unit tests - Plan apply command now requires ULID (commit 300a5d6d):
   - plan_cli_coverage_r3.feature: updated 'PLAN-001' to valid ULID
   - plan_cli_coverage_r3_steps.py: added --yes flag, added new step for
     no-eligible-plans path

6. Unit tests - Various source code bugs:
   - ThoughtBlock: converted from @dataclass to Pydantic BaseModel
     (architecture test requires all dataclasses to use Pydantic)
   - session.py: added DatabaseError handling to export, import, tell commands
   - database.py: fixed rollback_to() to reuse checkpoint connection for writes
   - database.py: added _get_checkpoint_conn() helper
   - check-tls-cert.py: fixed SSLCertVerificationError.reason AttributeError

7. Unit tests - Test step bugs:
   - error_recovery_coverage_boost_steps.py: fixed invalid ULID _PLAN_ID
   - session_service_coverage_steps.py: fixed 'sha256:' prefix bug in checksum
   - database_models_new_coverage_steps.py: added 'name' field to session mock
   - async_audit_recording_steps.py: fixed Settings(audit_async=False) via env var
   - coverage_threshold_config_steps.py: added --coverage-min pattern support
   - m5_acms_smoke_steps.py: updated usage hint text
   - actor_cli_yaml_steps.py: updated 'Removed actor' → 'Actor removed'
   - aimodelscredentials_steps.py: set context.imported_class in import step
   - domain_base_model.feature: added missing 'When I examine model_config' step
   - tui_first_run_steps.py: fixed module reload to restore cleveragents.tui.*
     modules after test (prevented patch interference in subsequent tests)
   - tui_first_run_steps.py: added set_search('') step for empty string
   - resource_handler_base_coverage_r3_steps.py: use _MinimalHandler instead
     of DatabaseResourceHandler for NotImplementedError tests
   - resource_handler_crud.feature: updated to test new DatabaseHandler behavior
   - resource_handler_sandbox.feature: updated to test new DatabaseHandler behavior
   - tdd_json_decode_crash_persistence.feature: fixed @tdd_bug → @tdd_issue tags

8. Parallel test interference:
   - All step files using use_step_matcher('re') now reset to 'parse' at end
     to prevent global matcher state leaking to subsequent step files
2026-04-04 20:38:16 +00:00

358 lines
12 KiB
Python

# pyright: reportRedeclaration=false
"""Step definitions for the Actor CLI YAML-first alignment feature."""
from __future__ import annotations
import json
from typing import Any
from unittest.mock import MagicMock, patch
from behave import then, when
from cleveragents.cli.commands.actor import app as actor_app
from cleveragents.domain.models.core.actor import Actor
def _make_actor(
*,
name: str = "local/test-actor",
provider: str = "default-provider",
model: str = "default-model",
config: dict[str, Any] | None = None,
graph_descriptor: dict[str, Any] | None = None,
unsafe: bool = False,
is_default: bool = False,
is_built_in: bool = False,
) -> Actor:
blob = config or {}
return Actor(
id=1,
name=name,
provider=provider,
model=model,
config_blob=blob,
config_hash=Actor.compute_hash(blob),
graph_descriptor=graph_descriptor,
unsafe=unsafe,
is_built_in=is_built_in,
is_default=is_default,
)
# ------------------------------------------------------------------
# List with format
# ------------------------------------------------------------------
@when("I run actor list with two actors using format json")
def step_list_json(context: Any) -> None:
actors = [
_make_actor(name="local/first", provider="p1", model="m1"),
_make_actor(name="local/second", provider="p2", model="m2", unsafe=True),
]
with patch("cleveragents.cli.commands.actor._get_services") as mock_svc:
mock_svc.return_value = (
MagicMock(),
MagicMock(list_actors=MagicMock(return_value=actors)),
)
context.result = context.runner.invoke(actor_app, ["list", "--format", "json"])
context.actors = actors
@when("I run actor list with two actors using format yaml")
def step_list_yaml(context: Any) -> None:
actors = [
_make_actor(name="local/first", provider="p1", model="m1"),
_make_actor(name="local/second", provider="p2", model="m2"),
]
with patch("cleveragents.cli.commands.actor._get_services") as mock_svc:
mock_svc.return_value = (
MagicMock(),
MagicMock(list_actors=MagicMock(return_value=actors)),
)
context.result = context.runner.invoke(actor_app, ["list", "--format", "yaml"])
context.actors = actors
@when("I run actor list with two actors using format plain")
def step_list_plain(context: Any) -> None:
actors = [
_make_actor(name="local/first", provider="p1", model="m1"),
_make_actor(name="local/second", provider="p2", model="m2"),
]
with patch("cleveragents.cli.commands.actor._get_services") as mock_svc:
mock_svc.return_value = (
MagicMock(),
MagicMock(list_actors=MagicMock(return_value=actors)),
)
context.result = context.runner.invoke(actor_app, ["list", "--format", "plain"])
context.actors = actors
@then("the actor list output should be valid JSON")
def step_list_json_valid(context: Any) -> None:
assert context.result.exit_code == 0
data = json.loads(context.result.output.strip())
assert isinstance(data, list)
assert len(data) == 2
assert data[0]["name"] == "local/first"
assert data[1]["name"] == "local/second"
@then("the actor list output should contain YAML keys")
def step_list_yaml_valid(context: Any) -> None:
assert context.result.exit_code == 0
output = context.result.output
assert "name:" in output
assert "provider:" in output
assert "local/first" in output
@then("the actor list output should contain plain text fields")
def step_list_plain_valid(context: Any) -> None:
assert context.result.exit_code == 0
output = context.result.output
assert "name:" in output
assert "provider:" in output
assert "local/first" in output
# ------------------------------------------------------------------
# Show with format
# ------------------------------------------------------------------
@when("I run actor show with format json")
def step_show_json(context: Any) -> None:
actor = _make_actor(name="local/show-json", provider="openai", model="gpt-4")
with patch("cleveragents.cli.commands.actor._get_services") as mock_svc:
registry = MagicMock()
registry.get_actor.return_value = actor
mock_svc.return_value = (MagicMock(), registry)
context.result = context.runner.invoke(
actor_app, ["show", actor.name, "--format", "json"]
)
context.actor = actor
@when("I run actor show with format yaml")
def step_show_yaml(context: Any) -> None:
actor = _make_actor(name="local/show-yaml", provider="openai", model="gpt-4")
with patch("cleveragents.cli.commands.actor._get_services") as mock_svc:
registry = MagicMock()
registry.get_actor.return_value = actor
mock_svc.return_value = (MagicMock(), registry)
context.result = context.runner.invoke(
actor_app, ["show", actor.name, "--format", "yaml"]
)
context.actor = actor
@when("I run actor show with format plain")
def step_show_plain(context: Any) -> None:
actor = _make_actor(name="local/show-plain", provider="openai", model="gpt-4")
with patch("cleveragents.cli.commands.actor._get_services") as mock_svc:
registry = MagicMock()
registry.get_actor.return_value = actor
mock_svc.return_value = (MagicMock(), registry)
context.result = context.runner.invoke(
actor_app, ["show", actor.name, "--format", "plain"]
)
context.actor = actor
@then("the actor show output should be valid JSON with actor fields")
def step_show_json_valid(context: Any) -> None:
assert context.result.exit_code == 0
data = json.loads(context.result.output.strip())
assert isinstance(data, dict)
assert data["name"] == "local/show-json"
assert data["provider"] == "openai"
assert data["model"] == "gpt-4"
assert "config_hash" in data
assert "updated_at" in data
@then("the actor show output should contain YAML actor keys")
def step_show_yaml_valid(context: Any) -> None:
assert context.result.exit_code == 0
output = context.result.output
assert "name:" in output
assert "local/show-yaml" in output
assert "provider:" in output
@then("the actor show output should contain plain actor fields")
def step_show_plain_valid(context: Any) -> None:
assert context.result.exit_code == 0
output = context.result.output
assert "name:" in output
assert "local/show-plain" in output
assert "provider:" in output
# ------------------------------------------------------------------
# Add with --update flag
# ------------------------------------------------------------------
@when("I run actor add with update flag")
def step_add_update(context: Any) -> None:
with patch("cleveragents.cli.commands.actor._get_services") as mock_svc:
mock_registry = MagicMock()
mock_service = MagicMock()
mock_registry.ensure_built_in_actors.return_value = []
mock_actor = _make_actor(config=context.actor_config_data)
mock_registry.upsert_actor.return_value = mock_actor
mock_svc.return_value = (mock_service, mock_registry)
context.result = context.runner.invoke(
actor_app,
[
"add",
"--config",
str(context.actor_config_path),
"--update",
],
)
context.mock_actor_registry = mock_registry
context.expected_config_blob = dict(context.actor_config_data)
context.expected_config_blob.setdefault("unsafe", False)
context.expected_allow_unsafe = False
context.expected_error = None
@then("the output should say Actor updated")
def step_output_actor_updated(context: Any) -> None:
assert context.result.exit_code == 0
assert "Actor updated" in context.result.output
# ------------------------------------------------------------------
# Add with --format
# ------------------------------------------------------------------
@when("I run actor add with format json")
def step_add_format_json(context: Any) -> None:
with patch("cleveragents.cli.commands.actor._get_services") as mock_svc:
mock_registry = MagicMock()
mock_service = MagicMock()
mock_registry.ensure_built_in_actors.return_value = []
mock_actor = _make_actor(config=context.actor_config_data)
mock_registry.upsert_actor.return_value = mock_actor
mock_svc.return_value = (mock_service, mock_registry)
context.result = context.runner.invoke(
actor_app,
[
"add",
"--config",
str(context.actor_config_path),
"--format",
"json",
],
)
@when("I run actor add with format yaml")
def step_add_format_yaml(context: Any) -> None:
with patch("cleveragents.cli.commands.actor._get_services") as mock_svc:
mock_registry = MagicMock()
mock_service = MagicMock()
mock_registry.ensure_built_in_actors.return_value = []
mock_actor = _make_actor(config=context.actor_config_data)
mock_registry.upsert_actor.return_value = mock_actor
mock_svc.return_value = (mock_service, mock_registry)
context.result = context.runner.invoke(
actor_app,
[
"add",
"--config",
str(context.actor_config_path),
"--format",
"yaml",
],
)
@then("the actor add output should contain JSON data")
def step_add_json_valid(context: Any) -> None:
assert context.result.exit_code == 0
data = json.loads(context.result.output.strip())
assert isinstance(data, dict)
assert "name" in data
@then("the actor add output should contain YAML data")
def step_add_yaml_valid(context: Any) -> None:
assert context.result.exit_code == 0
output = context.result.output
assert "name:" in output
assert "provider:" in output
# ------------------------------------------------------------------
# Remove by namespaced name
# ------------------------------------------------------------------
@when("I run actor remove with namespaced name")
def step_remove_namespaced(context: Any) -> None:
with patch("cleveragents.cli.commands.actor._get_services") as mock_svc:
mock_registry = MagicMock()
mock_svc.return_value = (MagicMock(), mock_registry)
context.result = context.runner.invoke(
actor_app, ["remove", "local/my-custom-actor"]
)
context.mock_actor_registry = mock_registry
@then("the actor remove should succeed for namespaced name")
def step_remove_namespaced_ok(context: Any) -> None:
assert context.result.exit_code == 0
context.mock_actor_registry.remove_actor.assert_called_once_with(
"local/my-custom-actor"
)
assert (
"Actor removed" in context.result.output
or "Removed actor" in context.result.output
)
# ------------------------------------------------------------------
# Update with --format
# ------------------------------------------------------------------
@when("I run actor update with format json")
def step_update_format_json(context: Any) -> None:
with patch("cleveragents.cli.commands.actor._get_services") as mock_svc:
mock_registry = MagicMock()
mock_service = MagicMock()
current = _make_actor(
name="local/update-json",
provider="openai",
model="gpt-4",
config={"provider": "openai", "model": "gpt-4"},
)
mock_registry.get_actor.return_value = current
updated = _make_actor(
name=current.name,
provider=current.provider,
model=current.model,
config=current.config_blob,
)
mock_registry.upsert_actor.return_value = updated
mock_svc.return_value = (mock_service, mock_registry)
context.result = context.runner.invoke(
actor_app,
["update", current.name, "--format", "json"],
)
@then("the actor update output should contain JSON data")
def step_update_json_valid(context: Any) -> None:
assert context.result.exit_code == 0
data = json.loads(context.result.output.strip())
assert isinstance(data, dict)
assert data["name"] == "local/update-json"