Skip to content

test(e2e): cover Canvas plugin conversation - #1066

Draft
saurya wants to merge 1 commit into
mainfrom
e2e/ohe-canvas-plugin-conversation
Draft

test(e2e): cover Canvas plugin conversation#1066
saurya wants to merge 1 commit into
mainfrom
e2e/ohe-canvas-plugin-conversation

Conversation

@saurya

@saurya saurya commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Migration

Ports https://github.com/OpenHands/saas-deploy/pull/575 from OpenHands/saas-deploy so the test ships with the OpenHands-Cloud chart release it validates.

This pull request is based directly on main, which includes the foundational Playwright harness from OpenHands-Cloud #1048.

Original scenario contract

Summary

  • add exactly one Playwright spec containing one authenticated OHE Canvas test
  • select a configured plugin through stable picker, card, toggle, and count test IDs
  • start a conversation with the selected plugin and a UUID-based unique prompt
  • apply e2e_tests/fixtures/auth.json at file level because Chromium clears project storage state

Validation

  • npm run lint (blocked only by 15 pre-existing lint/format errors in shared e2e files; this PR does not alter them)
  • npx eslint tests/ohe-canvas-plugin-conversation.spec.ts
  • npx prettier --check tests/ohe-canvas-plugin-conversation.spec.ts
  • npx playwright test tests/ohe-canvas-plugin-conversation.spec.ts --list --project=chromium

Required environment

  • BASE_URL: authenticated OHE deployment base URL
  • OHE_E2E_PLUGIN_NAME: exact plugin name available in the member-visible plugin catalog
  • generated e2e_tests/fixtures/auth.json for an authenticated OHE member with a launchable agent profile

The test fails helpfully if the required plugin is unavailable and never silently skips.

Cleanup

The test waits for asynchronous provisioning to reach a non-task Canvas conversation route, then deletes the provisioned app conversation in finally through /api/v1/app-conversations/{id}. Cleanup failure fails the test.

Live-fixture status

Not run live because this workspace has no authenticated OHE fixture or configured OHE_E2E_PLUGIN_NAME. Static discovery and focused lint/format checks pass.

This pull request was created by an AI agent (OpenHands) on behalf of the user.

Port validation

  • Exactly one changed file: e2e_tests/tests/ohe-canvas-plugin-conversation.spec.ts.
  • Exactly one Playwright test in the spec.
  • Focused ESLint and Prettier checks passed.
  • tsc --noEmit passed against the Cloud harness.
  • Chromium Playwright --list discovery passed.
  • Live execution still requires the explicit authentication and TEST_* contracts documented above.

AI disclosure: This pull request was migrated by an AI agent (OpenHands) on behalf of the user.

@saurya
saurya deleted the branch main August 12, 2026 15:16
@saurya saurya closed this Aug 12, 2026
@github-actions github-actions Bot added the type: test Adding or fixing tests label Aug 12, 2026
@saurya saurya reopened this Aug 12, 2026
@saurya
saurya changed the base branch from test/e2e-playwright-harness to main August 12, 2026 15:18
@saurya
saurya force-pushed the e2e/ohe-canvas-plugin-conversation branch from 1947f8c to 83a5a8b Compare August 12, 2026 15:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: test Adding or fixing tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant