Skip to content

docs(migrate): a Playwright Locator in expect from e2e is a plain value - #1361

Merged
okwasniewski merged 1 commit into
tester-army:mainfrom
amoscicki:docs/migrate-playwright-locator
Oct 25, 2026
Merged

okwasniewski merged 1 commit into
tester-army:mainfrom
amoscicki:docs/migrate-playwright-locator

Conversation

@amoscicki

Copy link
Copy Markdown
Contributor

What and why

  • During a migration, a page.getByRole(...) left over from Playwright and passed to expect from e2e compiles to a value expectation, so toBeVisible() fails with TS2339 ... on type 'ValueExpectation<Locator>', which doesn't say what is wrong.
  • One row in the guide's Assertions table: rewrite the query on screen.

Issue

None (docs).

Verified

Ran it locally: yes

  • apps/testbed with its tsconfig.json and the built packages: await expect(playwrightLocator).toBeVisible() gives error TS2339: Property 'toBeVisible' does not exist on type 'ValueExpectation<Locator>'.

Checklist

  • Changeset added (pnpm changeset) for any change to a published package, or the change is testbed, docs, or CI only.
  • Breaking change: title carries !, the changeset body names what breaks and what replaces it, and the deprecation warning landed a window earlier (stability policy in CONTRIBUTING.md).
  • Docs updated in the same PR: the docs/**/*.mdx page for the behavior, and skills/e2e/ if the skill describes it.
  • Contract change: emitted .d.ts reviewed, tests/types/sdk-types.ts updated; wire change edits the schema and both fixtures.
  • Engine contract change (e2e/engine): changesets for e2e, @e2e-dev/web, and @e2e-dev/mobile.

Written with AI help (Claude Opus 5.5, OpenAI Sol 6.1, DeepSeek V4.1 Flash); the runs above are real.

@mintlify

mintlify Bot commented Oct 24, 2026

Copy link
Copy Markdown

Preview approval required

This pull request comes from a fork, so Mintlify won't build a preview until someone with write access approves commit b081c2c.

Approve preview

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 1 file

Heads up: you’ve reached your flex budget. Increase your flex budget or wait for usage to reset.

View guided diff | Re-trigger cubic

@okwasniewski okwasniewski left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

thanks

@okwasniewski
okwasniewski merged commit f6d0679 into tester-army:main Oct 25, 2026
3 checks passed
@amoscicki
amoscicki deleted the docs/migrate-playwright-locator branch October 26, 2026 09:45
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.

2 participants