cleveragents/cleveragents-core

Fork 3

UAT: `tdd_a2a_sdk_dependency.feature` has stale `@skip` tags — A2A SDK is now installed and all scenarios pass #5583

New issue

Open

opened 2026-04-09 07:40:30 +00:00 by HAL9000 · 1 comment

HAL9000 commented

2026-04-09 07:40:30 +00:00

Owner

Bug Report

Summary

features/tdd_a2a_sdk_dependency.feature has all three scenarios tagged with @skip (and commented-out @tdd_expected_fail tags). These scenarios were originally skipped because the a2a-sdk package was not yet declared as a project dependency. However, a2a-sdk>=0.3.0 has since been added to pyproject.toml and the SDK is installed (version 0.3.25). All three scenarios now pass when run without the @skip tag.

Evidence

Current state of tdd_a2a_sdk_dependency.feature:

# @tdd_issue @tdd_issue_2922 @tdd_expected_fail @tdd_issue_4178 @skip
@skip
Feature: A2A Python SDK is a declared project dependency
  ...
  @skip
  Scenario: a2a-sdk is listed in project dependencies
  ...
  @skip
  Scenario: a2a module is importable as a project dependency
  ...
  @skip
  Scenario: a2a SDK provides the A2AClient class

Verified working:

# a2a-sdk is in pyproject.toml
import importlib.metadata
version = importlib.metadata.version('a2a-sdk')  # Returns '0.3.25'

# a2a module is importable
import a2a  # Works fine

# A2AClient is available
from a2a.client import A2AClient  # Works fine

pyproject.toml:

"a2a-sdk>=0.3.0",  # A2A Python SDK — required transport for local (stdio) and server (HTTP) modes (ADR-047)

Impact

These scenarios are never run, providing false confidence that the SDK dependency is tested
The @skip tags are stale and should be removed so the tests run in CI
The feature file has misleading commented-out @tdd_expected_fail tags that suggest the tests are expected to fail, when they actually pass

Fix Required

Remove all @skip tags from features/tdd_a2a_sdk_dependency.feature. The feature-level @skip tag and all scenario-level @skip tags should be removed. The commented-out @tdd_issue and @tdd_expected_fail tags can be removed as well since the issues are resolved.

Spec Reference

v3.6.0 Deliverable #1: ACP → A2A module rename and symbol standardization
ADR-047: A2A Standard Adoption — requires A2A Python SDK

Metadata

Commit message: test(a2a): remove stale @skip tags from tdd_a2a_sdk_dependency feature
Branch name: fix/m7-remove-stale-skip-tags-a2a-sdk

Subtasks

Remove @skip tag from Feature declaration
Remove @skip tags from all three Scenario declarations
Remove commented-out @tdd_issue and @tdd_expected_fail tags
Verify all three scenarios pass with nox -e unit_tests

Definition of Done

All three scenarios in tdd_a2a_sdk_dependency.feature run and pass in CI
No @skip tags remain in the file

Automated by CleverAgents Bot
Supervisor: UAT Testing | Agent: uat-tester

## Bug Report ### Summary `features/tdd_a2a_sdk_dependency.feature` has all three scenarios tagged with `@skip` (and commented-out `@tdd_expected_fail` tags). These scenarios were originally skipped because the `a2a-sdk` package was not yet declared as a project dependency. However, `a2a-sdk>=0.3.0` has since been added to `pyproject.toml` and the SDK is installed (version 0.3.25). All three scenarios now pass when run without the `@skip` tag. ### Evidence **Current state of `tdd_a2a_sdk_dependency.feature`:** ```gherkin # @tdd_issue @tdd_issue_2922 @tdd_expected_fail @tdd_issue_4178 @skip @skip Feature: A2A Python SDK is a declared project dependency ... @skip Scenario: a2a-sdk is listed in project dependencies ... @skip Scenario: a2a module is importable as a project dependency ... @skip Scenario: a2a SDK provides the A2AClient class ``` **Verified working:** ```python # a2a-sdk is in pyproject.toml import importlib.metadata version = importlib.metadata.version('a2a-sdk') # Returns '0.3.25' # a2a module is importable import a2a # Works fine # A2AClient is available from a2a.client import A2AClient # Works fine ``` **pyproject.toml:** ```toml "a2a-sdk>=0.3.0", # A2A Python SDK — required transport for local (stdio) and server (HTTP) modes (ADR-047) ``` ### Impact - These scenarios are never run, providing false confidence that the SDK dependency is tested - The `@skip` tags are stale and should be removed so the tests run in CI - The feature file has misleading commented-out `@tdd_expected_fail` tags that suggest the tests are expected to fail, when they actually pass ### Fix Required Remove all `@skip` tags from `features/tdd_a2a_sdk_dependency.feature`. The feature-level `@skip` tag and all scenario-level `@skip` tags should be removed. The commented-out `@tdd_issue` and `@tdd_expected_fail` tags can be removed as well since the issues are resolved. ### Spec Reference - v3.6.0 Deliverable #1: ACP → A2A module rename and symbol standardization - ADR-047: A2A Standard Adoption — requires A2A Python SDK ### Metadata - **Commit message**: `test(a2a): remove stale @skip tags from tdd_a2a_sdk_dependency feature` - **Branch name**: `fix/m7-remove-stale-skip-tags-a2a-sdk` ## Subtasks - [ ] Remove `@skip` tag from Feature declaration - [ ] Remove `@skip` tags from all three Scenario declarations - [ ] Remove commented-out `@tdd_issue` and `@tdd_expected_fail` tags - [ ] Verify all three scenarios pass with `nox -e unit_tests` ## Definition of Done - All three scenarios in `tdd_a2a_sdk_dependency.feature` run and pass in CI - No `@skip` tags remain in the file --- **Automated by CleverAgents Bot** Supervisor: UAT Testing | Agent: uat-tester

HAL9000 added the

labels

2026-04-09 07:41:05 +00:00

HAL9000 added this to the v3.2.0 milestone

2026-04-09 07:45:13 +00:00

HAL9000 commented

2026-04-09 07:46:20 +00:00

Author

Owner

Label compliance fix applied:

Added missing labels and/or milestone to bring issue into compliance with CONTRIBUTING.md

Automated by CleverAgents Bot
Supervisor: Backlog Grooming | Agent: backlog-groomer

Label compliance fix applied: - Added missing labels and/or milestone to bring issue into compliance with CONTRIBUTING.md --- **Automated by CleverAgents Bot** Supervisor: Backlog Grooming | Agent: backlog-groomer

HAL9000 added

and removed

labels

2026-04-09 07:46:23 +00:00

HAL9000 referenced this issue

2026-04-09 07:47:09 +00:00