Skip to content

test: cover agents empty-state create action - #52

Closed
kingrubic wants to merge 1 commit into
aadorian:masterfrom
kingrubic:test-agents-empty-create-agent
Closed

test: cover agents empty-state create action#52
kingrubic wants to merge 1 commit into
aadorian:masterfrom
kingrubic:test-agents-empty-create-agent

Conversation

@kingrubic

Copy link
Copy Markdown
Contributor

Summary

  • add a regression test that the Agents empty-state CTA runs opencode-walkthrough.createAgent
  • inject the agent-list runner into AgentsProvider so the empty state can be tested without shelling out to the real CLI
  • keep the production command path unchanged

Fixes #12.

Tests

  • npm run validate
  • npm test

@kingrubic
kingrubic requested a review from aadorian as a code owner June 26, 2026 01:16
aadorian added a commit that referenced this pull request Jun 26, 2026
Thanks @kingrubic for PR #52.

Co-authored-by: Cursor <cursoragent@cursor.com>
@aadorian

Copy link
Copy Markdown
Owner

Merged to master — thank you, @kingrubic!

Resolved the merge conflict with #53 and landed your changes in:

The empty-state CTA now has a regression test with an injectable AgentsProvider runner. Welcome aboard, and thanks for the contribution!

@aadorian aadorian closed this Jun 26, 2026
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.

Agents empty-state CTA runs List Agents instead of Create Agent

2 participants