Skip to content

tests(app): e2e tests part 67#16406

Merged
adamdotdevin merged 1 commit intoanomalyco:devfrom
neriousy:tests/e2e-test-propaganda
Mar 6, 2026
Merged

tests(app): e2e tests part 67#16406
adamdotdevin merged 1 commit intoanomalyco:devfrom
neriousy:tests/e2e-test-propaganda

Conversation

@neriousy
Copy link
Contributor

@neriousy neriousy commented Mar 6, 2026

Issue for this PR

Closes #67 NOTHING THESE ARE THESES

Type of change

  • Bug fix
  • New feature
  • Refactor / code improvement
  • Documentation

What does this PR do?

ADDS TESTS

If you paste a large clearly AI generated description here your PR may be IGNORED or CLOSED!

How did you verify your code works?

RUN TESTS

Screenshots / recordings

If this is a UI change, please include a screenshot or recording.

Checklist

  • I have tested my changes locally
  • I have not included unrelated changes in this PR

If you do not follow this template your PR will be automatically rejected.

@neriousy neriousy requested a review from adamdotdevin as a code owner March 6, 2026 22:50
Copilot AI review requested due to automatic review settings March 6, 2026 22:50
@github-actions
Copy link
Contributor

github-actions bot commented Mar 6, 2026

Hey! Your PR title tests(app): e2e tests part 67 doesn't follow conventional commit format.

Please update it to start with one of:

  • feat: or feat(scope): new feature
  • fix: or fix(scope): bug fix
  • docs: or docs(scope): documentation changes
  • chore: or chore(scope): maintenance tasks
  • refactor: or refactor(scope): code refactoring
  • test: or test(scope): adding or updating tests

Where scope is the package name (e.g., app, desktop, opencode).

See CONTRIBUTING.md for details.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds new Playwright e2e coverage for terminal tab behavior and multiple prompt/composer flows, plus a small e2e utility for addressing workspace-persisted localStorage state.

Changes:

  • Add terminal tab e2e tests for buffer persistence across tab switches and active-tab close fallback behavior.
  • Add prompt e2e tests for /share + /unshare, shell-mode command execution, and prompt/shell history navigation.
  • Add a workspacePersistKey() helper to compute workspace-scoped persisted storage keys used by the app.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
packages/app/e2e/utils.ts Adds workspacePersistKey() for computing workspace persisted localStorage keys in e2e.
packages/app/e2e/terminal/terminal-tabs.spec.ts New terminal tab persistence + close-fallback e2e tests.
packages/app/e2e/prompt/prompt-slash-share.spec.ts New e2e test ensuring /share and /unshare update share state.
packages/app/e2e/prompt/prompt-shell.spec.ts New e2e test verifying shell mode runs commands from the project directory.
packages/app/e2e/prompt/prompt-history.spec.ts New e2e tests for restoring drafts and keeping shell history separate.
packages/app/e2e/prompt/prompt-async.spec.ts Extends existing async prompt coverage with a test for restoring composer input on failed send.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@adamdotdevin adamdotdevin merged commit c4fd677 into anomalyco:dev Mar 6, 2026
15 of 16 checks passed
qdddddd pushed a commit to qdddddd/opencode that referenced this pull request Mar 7, 2026
manno23 pushed a commit to manno23/opencode that referenced this pull request Mar 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[docs] how to use typescript-go lsp

3 participants