UAT: tdd_event_bus_exception_swallow.feature references closed issue #988 but all scenarios are @skip — regression guard not running #5650

New issue

Open

opened 2026-04-09 08:06:33 +00:00 by HAL9000 · 0 comments

HAL9000 commented

2026-04-09 08:06:33 +00:00

Owner

Background and Context

Issue #988 ("bug(events): ReactiveEventBus.emit() swallows exception details") is closed, meaning the bug was fixed. However, features/tdd_event_bus_exception_swallow.feature still has all scenarios tagged @skip, meaning the regression guard is not running.

Current Behavior

features/tdd_event_bus_exception_swallow.feature:

Feature-level tags: @tdd_issue @tdd_issue_988
Both scenarios have @skip tag
@tdd_expected_fail tags are commented out (prefixed with #)

This means:

The bug fix regression tests are NOT running (all scenarios are @skip)
If the fix is ever accidentally reverted, CI will not catch it
The @skip tag was likely added as a workaround but never cleaned up after the fix

Expected Behavior

When a bug is fixed:

The @tdd_expected_fail tag should be removed
The @skip tag should be removed
The scenarios should run normally as regression guards

Acceptance Criteria

features/tdd_event_bus_exception_swallow.feature scenarios run without @skip
Scenarios pass (confirming the fix is still in place)
@tdd_expected_fail tags are removed (not just commented out)

Supporting Information

Issue #988 state: closed
Feature file: features/tdd_event_bus_exception_swallow.feature
Both scenarios have @skip tag
The feature description says: "Once the fix for #988 is merged, the @tdd_expected_fail tag must be removed"

Metadata

Commit Message: fix(tests): clean up stale tdd_event_bus_exception_swallow feature — remove @skip after #988 fix
Branch: fix/cleanup-stale-tdd-event-bus

Subtasks

Verify issue #988 fix is complete and working
Remove @skip from scenarios in tdd_event_bus_exception_swallow.feature
Remove commented-out @tdd_expected_fail tags
Verify scenarios pass without @tdd_expected_fail

Definition of Done

This issue is complete when:

All subtasks above are completed and checked off.
A Git commit is created with the prescribed commit message.
The commit is pushed to the branch and submitted as a pull request to master, reviewed, and merged.

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

## Background and Context Issue #988 ("bug(events): ReactiveEventBus.emit() swallows exception details") is **closed**, meaning the bug was fixed. However, `features/tdd_event_bus_exception_swallow.feature` still has all scenarios tagged `@skip`, meaning the regression guard is not running. ## Current Behavior `features/tdd_event_bus_exception_swallow.feature`: - Feature-level tags: `@tdd_issue @tdd_issue_988` - Both scenarios have `@skip` tag - `@tdd_expected_fail` tags are commented out (prefixed with `#`) This means: 1. The bug fix regression tests are NOT running (all scenarios are `@skip`) 2. If the fix is ever accidentally reverted, CI will not catch it 3. The `@skip` tag was likely added as a workaround but never cleaned up after the fix ## Expected Behavior When a bug is fixed: 1. The `@tdd_expected_fail` tag should be removed 2. The `@skip` tag should be removed 3. The scenarios should run normally as regression guards ## Acceptance Criteria - [ ] `features/tdd_event_bus_exception_swallow.feature` scenarios run without `@skip` - [ ] Scenarios pass (confirming the fix is still in place) - [ ] `@tdd_expected_fail` tags are removed (not just commented out) ## Supporting Information - Issue #988 state: **closed** - Feature file: `features/tdd_event_bus_exception_swallow.feature` - Both scenarios have `@skip` tag - The feature description says: "Once the fix for #988 is merged, the `@tdd_expected_fail` tag must be removed" ## Metadata - **Commit Message**: `fix(tests): clean up stale tdd_event_bus_exception_swallow feature — remove @skip after #988 fix` - **Branch**: `fix/cleanup-stale-tdd-event-bus` ## Subtasks - [ ] Verify issue #988 fix is complete and working - [ ] Remove `@skip` from scenarios in `tdd_event_bus_exception_swallow.feature` - [ ] Remove commented-out `@tdd_expected_fail` tags - [ ] Verify scenarios pass without `@tdd_expected_fail` ## Definition of Done This issue is complete when: - All subtasks above are completed and checked off. - A Git commit is created with the prescribed commit message. - The commit is pushed to the branch and submitted as a pull request to `master`, reviewed, and merged. --- **Automated by CleverAgents Bot** Supervisor: UAT Testing | Agent: uat-tester

HAL9000 added the

labels

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

HAL9000 added a new dependency