Skip to content

test(e2e): cover Skills and Plugins navigation - #1069

Draft
saurya wants to merge 1 commit into
mainfrom
e2e/ohe-customize-skills-plugins
Draft

test(e2e): cover Skills and Plugins navigation#1069
saurya wants to merge 1 commit into
mainfrom
e2e/ohe-customize-skills-plugins

Conversation

@saurya

@saurya saurya commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Migration

Ports https://github.com/OpenHands/saas-deploy/pull/577 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 navigation test
  • enter Customize from /canvas using sidebar-skills-link, verify the Canvas extensions navigation, then open the cloud Skills and Plugins link
  • assert the current OHE /settings/skills route and skills-settings-screen test ID
  • 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-customize-skills-plugins.spec.ts
  • npx prettier --check tests/ohe-customize-skills-plugins.spec.ts
  • npx playwright test tests/ohe-customize-skills-plugins.spec.ts --list --project=chromium

Required environment

  • BASE_URL: authenticated OHE deployment base URL; the test fails with a helpful error when unset
  • generated e2e_tests/fixtures/auth.json for an authenticated OHE member

Cleanup

Read-only navigation scenario; it creates no server-side state and requires no cleanup.

Live-fixture status

Not run live because this workspace has no authenticated OHE BASE_URL/auth fixture. 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-customize-skills-plugins.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.

@github-actions github-actions Bot added the type: test Adding or fixing tests label Aug 12, 2026
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