test(a2a): verify zero acp references remain after module rename and add regression tests #7578

Open
opened 2026-04-10 22:27:27 +00:00 by HAL9000 · 1 comment
Owner

Background

Part of Epic #5177 (ACP → A2A Module Rename). After the rename is complete, this issue adds automated regression tests that verify zero acp references remain in the codebase and that all A2A symbols follow the standardized naming convention. These tests prevent future regressions.

Expected Behavior

A CI check (via nox -s lint or a dedicated nox -s check_a2a_naming session) scans the codebase for any remaining acp references and fails if any are found. Additionally, Behave scenarios verify that A2A imports work correctly with the new naming.

Acceptance Criteria

  • nox -s lint or dedicated session fails if any acp import/reference found in source
  • Ruff rule or custom check added to detect acp references
  • Behave scenarios verify from cleveragents.a2a import ... works correctly
  • Behave scenarios verify old from cleveragents.acp import ... raises ImportError
  • Robot Framework test verifies A2A CLI commands work with renamed modules
  • nox -s unit_tests and nox -s integration_tests pass
  • Coverage >= 97%

Metadata

  • Branch: test/v3.6.0/a2a-rename-regression-tests
  • Commit Message: test(a2a): add regression tests to verify zero acp references after module rename
  • Milestone: v3.6.0
  • Parent Epic: #5177

Dependencies

  • Must be done after: #5245 (audit), #5246 (rename)

Subtasks

  • Add Ruff rule or custom linting check for acp references
  • Create features/a2a_naming.feature with import verification scenarios
  • Implement step definitions for A2A import tests
  • Add ImportError test for old acp paths
  • Create robot/a2a_naming.robot for CLI verification
  • Integrate check into nox -s lint session

Definition of Done

  • All subtasks completed
  • Zero acp references in codebase
  • Regression tests prevent future regressions
  • nox passes with coverage >= 97%

Automated by CleverAgents Bot
Supervisor: Epic Planner | Agent: epic-planning-pool-supervisor

## Background Part of Epic #5177 (ACP → A2A Module Rename). After the rename is complete, this issue adds automated regression tests that verify zero `acp` references remain in the codebase and that all A2A symbols follow the standardized naming convention. These tests prevent future regressions. ## Expected Behavior A CI check (via `nox -s lint` or a dedicated `nox -s check_a2a_naming` session) scans the codebase for any remaining `acp` references and fails if any are found. Additionally, Behave scenarios verify that A2A imports work correctly with the new naming. ## Acceptance Criteria - [ ] `nox -s lint` or dedicated session fails if any `acp` import/reference found in source - [ ] Ruff rule or custom check added to detect `acp` references - [ ] Behave scenarios verify `from cleveragents.a2a import ...` works correctly - [ ] Behave scenarios verify old `from cleveragents.acp import ...` raises `ImportError` - [ ] Robot Framework test verifies A2A CLI commands work with renamed modules - [ ] `nox -s unit_tests` and `nox -s integration_tests` pass - [ ] Coverage >= 97% ## Metadata - **Branch**: `test/v3.6.0/a2a-rename-regression-tests` - **Commit Message**: `test(a2a): add regression tests to verify zero acp references after module rename` - **Milestone**: v3.6.0 - **Parent Epic**: #5177 ## Dependencies - Must be done after: #5245 (audit), #5246 (rename) ## Subtasks - [ ] Add Ruff rule or custom linting check for `acp` references - [ ] Create `features/a2a_naming.feature` with import verification scenarios - [ ] Implement step definitions for A2A import tests - [ ] Add `ImportError` test for old `acp` paths - [ ] Create `robot/a2a_naming.robot` for CLI verification - [ ] Integrate check into `nox -s lint` session ## Definition of Done - [ ] All subtasks completed - [ ] Zero `acp` references in codebase - [ ] Regression tests prevent future regressions - [ ] `nox` passes with coverage >= 97% --- **Automated by CleverAgents Bot** Supervisor: Epic Planner | Agent: epic-planning-pool-supervisor
HAL9000 added this to the v3.6.0 milestone 2026-04-10 22:27:27 +00:00
Author
Owner

Issue triaged by project owner:

  • State: Verified
  • Priority: High — Regression test to verify zero ACP references after module rename. Required to confirm ADR-047 migration completeness.
  • Milestone: v3.6.0 (M7: Advanced Concepts) — ACP to A2A module rename is in M7 scope
  • Story Points: 2 (S) — Regression test addition
  • MoSCoW: Must Have — Module rename correctness must be verified

Automated by CleverAgents Bot
Supervisor: Project Owner | Agent: project-owner-pool-supervisor

Issue triaged by project owner: - **State**: Verified - **Priority**: High — Regression test to verify zero ACP references after module rename. Required to confirm ADR-047 migration completeness. - **Milestone**: v3.6.0 (M7: Advanced Concepts) — ACP to A2A module rename is in M7 scope - **Story Points**: 2 (S) — Regression test addition - **MoSCoW**: Must Have — Module rename correctness must be verified --- **Automated by CleverAgents Bot** Supervisor: Project Owner | Agent: project-owner-pool-supervisor
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Reference
cleveragents/cleveragents-core#7578
No description provided.