crashlabs/huskies
0
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 26 Wiki Activity

storkit: accept 451_bug_chat_test_tsx_help_test_expects_removed_overlay_behavior

Browse Source
This commit is contained in:
dave
2026-04-02 10:11:49 +00:00
parent c51428414e
commit 46a89d481a
1 changed files with 0 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files
.storkit/work/5_done/451_bug_chat_test_tsx_help_test_expects_removed_overlay_behavior.md
-27
View File
@@ -1,27 +0,0 @@
---
name: "Chat.test.tsx /help test expects removed overlay behavior"
---
# Bug 451: Chat.test.tsx /help test expects removed overlay behavior
## Description
The test `AC: /help shows help overlay` in `Chat.test.tsx:1645` expects `/help` to show the `help-overlay` testid element. However, the `/help` intercept was removed from Chat.tsx and `help` was added to `knownCommands`, so `/help` now goes through `api.botCommand()` like other commands. The test needs to be updated to expect a `botCommand("help", ...)` call instead of the overlay. This is blocking gates on stories 449 and 450.
## How to Reproduce
Run `cd frontend && npm test` — the test `AC: /help shows help overlay` fails.
## Actual Result
Test fails: `findByTestId("help-overlay")` times out because the overlay is never rendered. `/help` is dispatched to the backend via `botCommand` instead.
## Expected Result
Test should pass by expecting `/help` to call `api.botCommand("help", ...)` and display the response in chat, consistent with the current code behavior.
## Acceptance Criteria
- [ ] The /help test in Chat.test.tsx is updated to expect botCommand dispatch
- [ ] All frontend tests pass
- [ ] HelpOverlay component and showHelp state can be removed from Chat.tsx if no longer used