Skip to content

test(tui): cover unknown tool lifecycle errors#3711

Merged
Hmbown merged 1 commit into
Hmbown:mainfrom
cyq1017:codex/codewhale-2886-tool-lifecycle-error
Jun 28, 2026
Merged

test(tui): cover unknown tool lifecycle errors#3711
Hmbown merged 1 commit into
Hmbown:mainfrom
cyq1017:codex/codewhale-2886-tool-lifecycle-error

Conversation

@cyq1017

@cyq1017 cyq1017 commented Jun 28, 2026

Copy link
Copy Markdown
Contributor

Refs #2886

Problem

#2886 tracks remaining tool lifecycle acceptance coverage after the existing happy-path harness. One uncovered case is an LLM requesting a tool that is not available.

Change

  • Rename the tool lifecycle feature file so it can hold happy and non-happy paths.
  • Add a Gherkin scenario for unknown tool calls.
  • Assert the public running/error lifecycle events, the tool error sent back to the mocked LLM, and the final assistant output.

Verification

  • cargo test -p codewhale-tui --test tool_lifecycle_acceptance --locked -- --nocapture
  • cargo fmt --all --check
  • git diff HEAD --check

@cyq1017 cyq1017 marked this pull request as ready for review June 28, 2026 05:56
@cyq1017 cyq1017 requested a review from Hmbown as a code owner June 28, 2026 05:56
@Hmbown Hmbown merged commit ad1211f into Hmbown:main Jun 28, 2026
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants