Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrate duplicating block test to playwright #43171

Conversation

pooja-muchandikar
Copy link
Contributor

What?

Part of #38851.
Migrate duplicating-blocks.test.js to its Playwright version.

Why?

Part of #38851.

How?

See MIGRATION.md for migration steps.

Testing Instructions

Run npm run test-e2e:playwright test/e2e/specs/editor/various/duplicating-blocks.spec.js

@pooja-muchandikar
Copy link
Contributor Author

Hi @kevin940726, @talldan 👋

I hope you are doing great!

I have raised another migration PR, please could anyone of you help me in reviewing this PR?

Thanks!!

@talldan talldan added the [Type] Automated Testing Testing infrastructure changes impacting the execution of end-to-end (E2E) and/or unit tests. label Aug 25, 2022
Copy link
Contributor

@talldan talldan left a comment

Choose a reason for hiding this comment

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

Looks good, thank you!

@talldan talldan merged commit 33fa163 into WordPress:trunk Aug 25, 2022
@github-actions github-actions bot added this to the Gutenberg 14.1 milestone Aug 25, 2022
@pooja-muchandikar pooja-muchandikar deleted the add/playwright-duplicating-block-test branch August 25, 2022 04:35
@cbravobernal cbravobernal added the [Type] Code Quality Issues or PRs that relate to code quality label Sep 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[Type] Automated Testing Testing infrastructure changes impacting the execution of end-to-end (E2E) and/or unit tests. [Type] Code Quality Issues or PRs that relate to code quality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants