Automattic / wp-calypso

The JavaScript and API powered WordPress.com
https://developer.wordpress.com
GNU General Public License v2.0
12.42k stars 1.99k forks source link

Flaky E2E: Other Jetpack Blocks (Atomic: wp-previous): Add and configure blocks in the editor: Related Posts: Configure the block #82072

Open worldomonation opened 1 year ago

worldomonation commented 1 year ago

Spec file

specs/blocks/blocks__jetpack-other.ts

TeamCity ID

No response

Logs

======= Failed test run #1 ==========
  locator.click: Timeout 10000ms exceeded.
  =========================== logs ===========================
  waiting for locator('body.block-editor-page').getByLabel('Display headline')
  ============================================================
  at RelatedPostsFlow.click [as configure] (/home/teamcity-3/buildAgent/work/c4a9d5b38c1dacde/packages/calypso-e2e/src/lib/blocks/block-flows/related-posts.ts:43:56)
      at Object.<anonymous> (/home/teamcity-3/buildAgent/work/c4a9d5b38c1dacde/test/e2e/specs/blocks/shared/block-smoke-testing.ts:75:7)
======= Failed test run #2 ==========
  locator.click: Timeout 10000ms exceeded.
  =========================== logs ===========================
  waiting for locator('body.block-editor-page').getByLabel('Display headline')
  ============================================================
  at RelatedPostsFlow.click [as configure] (/home/teamcity-3/buildAgent/work/c4a9d5b38c1dacde/packages/calypso-e2e/src/lib/blocks/block-flows/related-posts.ts:43:56)
      at Object.<anonymous> (/home/teamcity-3/buildAgent/work/c4a9d5b38c1dacde/test/e2e/specs/blocks/shared/block-smoke-testing.ts:75:7)

image

worldomonation commented 1 year ago

Turns out on a bunch of test sites, the Related Posts feature had to be activated in the editor as well.

image
worldomonation commented 1 year ago

After enabling the feature for all sites that were affected, I think we are in a good state.

fullofcaffeine commented 1 year ago

FYI, found the same issue in the Gutenberg test sites. I already activated the block for them, too. I'm curious on why that's needed now, though. 🤔 Thanks for creating this issue and documenting it in the failed test!