Skip to content

Conversation

@cte
Copy link
Collaborator

@cte cte commented May 27, 2025

Description

This test seems to flake a lot, but this prompt seems a bit more stable.


Important

Improves stability of "modes" e2e test by updating the prompt and simplifying test logic in modes.test.ts.

  • Test Stability:
    • Update prompt in modes.test.ts to "For each of architect, ask, and debug use the switch_mode tool to switch to that mode."
    • Simplifies test logic by removing unused variables and listeners.
  • Assertions:
    • Verifies modes array includes "architect", "ask", "debug" and has length 3.

This description was created by Ellipsis for 3c4f355. You can customize this summary. It will automatically update as commits are pushed.

@cte cte requested a review from mrubens as a code owner May 27, 2025 19:56
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label May 27, 2025
@cte cte merged commit 343f29b into main May 27, 2025
13 checks passed
@cte cte deleted the cte/less-flakey-e2e-test-prompt branch May 27, 2025 19:56
@github-project-automation github-project-automation bot moved this from Triage to Done in Roo Code Roadmap May 27, 2025
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap May 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:M This PR changes 30-99 lines, ignoring generated files.

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants