Skip to content

Refactor(e2e): Convert remaining utilities to TS - #84917

Merged
manzoorwanijk merged 4 commits into
WordPress:trunkfrom
USERSATOSHI:refactor/playwright-utils-typescript
Oct 23, 2026
Merged

manzoorwanijk merged 4 commits into
WordPress:trunkfrom
USERSATOSHI:refactor/playwright-utils-typescript

Conversation

@USERSATOSHI

Copy link
Copy Markdown
Contributor

What?

See #67691.

Convert set-browser-viewport.js and request-utils/widgets.js in @wordpress/e2e-test-utils-playwright to TypeScript.

Why?

Complete the package's TypeScript source migration.

How?

Add explicit types, convert existing comments to TSDoc, preserve legacy viewport type aliases, and remove redundant TypeScript configuration overrides. No runtime changes.

Testing Instructions

Run npm run build, npm run typecheck, and npm run lint:js.

Then run:

npm run test:e2e -- widgets/ editor/various/navigable-toolbar.spec.js editor/plugins/plugins-api.spec.js --project=chromium

Use of AI Tools

OpenAI Codex assisted with testing, and validation. I reviewed and updated the changes myself.

@github-actions github-actions Bot added the [Tool] E2E Test Utils /packages/e2e-test-utils-playwright label Oct 16, 2026
@github-actions

github-actions Bot commented Oct 16, 2026 •

Copy link
Copy Markdown

🤖 PR meta 🤖

Preview in WordPress Playground

🎉 Props

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: USERSATOSHI <tusharbharti@git.wordpress.org>
Co-authored-by: manzoorwanijk <manzoorwanijk@git.wordpress.org>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

Updated as activity occurs, without notifying anyone named here. Add the props-bot label to refresh.

@USERSATOSHI
USERSATOSHI marked this pull request as ready for review October 17, 2026 10:22
@manzoorwanijk manzoorwanijk added the [Type] Code Quality Issues or PRs that relate to code quality label Oct 21, 2026
Comment thread packages/e2e-test-utils-playwright/src/page-utils/types.ts Outdated

@manzoorwanijk manzoorwanijk 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.

This branch needs to be updated from trunk.

@USERSATOSHI
USERSATOSHI force-pushed the refactor/playwright-utils-typescript branch from 11a496a to 806da31 Compare October 23, 2026 11:46
@manzoorwanijk manzoorwanijk added the props-bot Manually triggers Props Bot to ensure the list of props is up to date. label Oct 23, 2026
@github-actions github-actions Bot removed the props-bot Manually triggers Props Bot to ensure the list of props is up to date. label Oct 23, 2026
@manzoorwanijk
manzoorwanijk enabled auto-merge (squash) October 23, 2026 11:48
@manzoorwanijk
manzoorwanijk merged commit 66d64ab into WordPress:trunk Oct 23, 2026
78 of 79 checks passed
@github-actions github-actions Bot added this to the Gutenberg 24.4 milestone Oct 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Tool] E2E Test Utils /packages/e2e-test-utils-playwright [Type] Code Quality Issues or PRs that relate to code quality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants