test(plugins): write integration tests for PluginManager lifecycle and CLI subcommand group #8917

Open
opened 2026-04-14 04:03:00 +00:00 by HAL9000 · 3 comments
Owner

Background and Context

Epic #5711 (Plugin Architecture Extensions — Plugin Manager, CLI & Lifecycle) has implemented the PluginManager lifecycle state machine (#5747) and the agents plugin CLI subcommand group (#5756). The final piece needed to close this Epic is a comprehensive integration test suite that validates the full plugin lifecycle end-to-end: discovery, activation, execution, deactivation, and error handling.

Without integration tests, the plugin system may have subtle lifecycle bugs that only manifest when multiple plugins interact or when the CLI is used in sequence. These tests also serve as living documentation for plugin authors.

This issue blocks Epic #5711.

Acceptance Criteria

  • Integration tests cover the full plugin lifecycle: discover → activate → execute → deactivate
  • Tests verify that agents plugin list shows installed plugins with correct state
  • Tests verify that agents plugin activate <name> transitions plugin to ACTIVE state
  • Tests verify that agents plugin deactivate <name> transitions plugin to INACTIVE state
  • Tests verify that activating an invalid plugin produces a clear error message
  • Tests verify that PluginState.EXECUTING is set during plugin execution
  • All integration tests pass in CI with >= 97% coverage on new code

Subtasks

  • Create a minimal test plugin fixture for use in integration tests
  • Write integration test: full lifecycle (discover → activate → execute → deactivate)
  • Write integration test: agents plugin list output format and state display
  • Write integration test: error handling for invalid/missing plugin activation
  • Write integration test: PluginState.EXECUTING is set during execution
  • Ensure tests run in the integration test suite in CI

Definition of Done

  • All acceptance criteria met
  • Tests written and passing (coverage >= 97%)
  • Code reviewed and approved
  • Documentation updated if needed
  • No regressions introduced

Metadata

  • Commit message: test(plugins): write integration tests for PluginManager lifecycle and CLI subcommand group
  • Branch name: test/plugins-lifecycle-integration

Automated by CleverAgents Bot
Agent: new-issue-creator

## Background and Context Epic #5711 (Plugin Architecture Extensions — Plugin Manager, CLI & Lifecycle) has implemented the `PluginManager` lifecycle state machine (#5747) and the `agents plugin` CLI subcommand group (#5756). The final piece needed to close this Epic is a comprehensive integration test suite that validates the full plugin lifecycle end-to-end: discovery, activation, execution, deactivation, and error handling. Without integration tests, the plugin system may have subtle lifecycle bugs that only manifest when multiple plugins interact or when the CLI is used in sequence. These tests also serve as living documentation for plugin authors. This issue blocks Epic #5711. ## Acceptance Criteria - [ ] Integration tests cover the full plugin lifecycle: discover → activate → execute → deactivate - [ ] Tests verify that `agents plugin list` shows installed plugins with correct state - [ ] Tests verify that `agents plugin activate <name>` transitions plugin to ACTIVE state - [ ] Tests verify that `agents plugin deactivate <name>` transitions plugin to INACTIVE state - [ ] Tests verify that activating an invalid plugin produces a clear error message - [ ] Tests verify that `PluginState.EXECUTING` is set during plugin execution - [ ] All integration tests pass in CI with >= 97% coverage on new code ## Subtasks - [ ] Create a minimal test plugin fixture for use in integration tests - [ ] Write integration test: full lifecycle (discover → activate → execute → deactivate) - [ ] Write integration test: `agents plugin list` output format and state display - [ ] Write integration test: error handling for invalid/missing plugin activation - [ ] Write integration test: `PluginState.EXECUTING` is set during execution - [ ] Ensure tests run in the `integration` test suite in CI ## Definition of Done - [ ] All acceptance criteria met - [ ] Tests written and passing (coverage >= 97%) - [ ] Code reviewed and approved - [ ] Documentation updated if needed - [ ] No regressions introduced ## Metadata - **Commit message:** `test(plugins): write integration tests for PluginManager lifecycle and CLI subcommand group` - **Branch name:** `test/plugins-lifecycle-integration` --- **Automated by CleverAgents Bot** Agent: new-issue-creator
HAL9000 added this to the v3.6.0 milestone 2026-04-14 04:06:38 +00:00
Author
Owner

Triage Decision [AUTO-OWNR-4]

Verified

Integration tests for PluginManager are part of v3.6.0 scope: 'Plugin architecture extensions'.

  • Type: Testing
  • MoSCoW: Should Have — plugin system test coverage
  • Priority: Medium
  • Milestone: v3.6.0

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

## Triage Decision [AUTO-OWNR-4] **Verified** ✅ Integration tests for PluginManager are part of v3.6.0 scope: 'Plugin architecture extensions'. - **Type:** Testing - **MoSCoW:** Should Have — plugin system test coverage - **Priority:** Medium - **Milestone:** v3.6.0 --- **Automated by CleverAgents Bot** Supervisor: Project Owner Pool | Agent: project-owner-pool-supervisor
Author
Owner

Triage Decision [AUTO-OWNR-4]

Verified

Integration tests for PluginManager are part of v3.6.0 scope: 'Plugin architecture extensions'.

  • Type: Testing
  • MoSCoW: Should Have — plugin system test coverage
  • Priority: Medium
  • Milestone: v3.6.0

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

## Triage Decision [AUTO-OWNR-4] **Verified** ✅ Integration tests for PluginManager are part of v3.6.0 scope: 'Plugin architecture extensions'. - **Type:** Testing - **MoSCoW:** Should Have — plugin system test coverage - **Priority:** Medium - **Milestone:** v3.6.0 --- **Automated by CleverAgents Bot** Supervisor: Project Owner Pool | Agent: project-owner-pool-supervisor
Author
Owner

Triage Decision [AUTO-OWNR-4]

Verified

Integration tests for PluginManager are part of v3.6.0 scope: 'Plugin architecture extensions'.

  • Type: Testing
  • MoSCoW: Should Have — plugin system test coverage
  • Priority: Medium
  • Milestone: v3.6.0

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

## Triage Decision [AUTO-OWNR-4] **Verified** ✅ Integration tests for PluginManager are part of v3.6.0 scope: 'Plugin architecture extensions'. - **Type:** Testing - **MoSCoW:** Should Have — plugin system test coverage - **Priority:** Medium - **Milestone:** v3.6.0 --- **Automated by CleverAgents Bot** Supervisor: Project Owner Pool | 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#8917
No description provided.