Skip to content

Add e2e test for loading settings in site editor preload spec #75661

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.

Sign up for GitHub

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

Merged
ramonjd merged 7 commits into from
Feb 18, 2026
Merged

Add e2e test for loading settings in site editor preload spec #75661

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
67366ec
Add e2e test for loading settings in site editor preload spec
ramonjd Feb 18, 2026
f1e1926
Update draggable block e2e tests to reflect new background color for …
ramonjd Feb 18, 2026
0342bc7
support older WP versions
ramonjd Feb 18, 2026
02e9dd1
Update font-library.spec.js to skip test temporarily
ramonjd Feb 18, 2026
5256194
Added link to pr
ramonjd Feb 18, 2026
3bf1f7a
missed a spot
ramonjd Feb 18, 2026
ccd10f1
Fix e2e test for Privacy Policy row by ensuring the first matching ro…
ramonjd Feb 18, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion test/e2e/specs/admin/font-library.spec.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,9 @@ test.describe( 'Font Library', () => {
);
} );

test( 'should allow user to add and remove multiple local font files', async ( {
// Temporarily skipped: font-library wp-admin page throws a fatal 500 error (possibly route related).
// See https://github.com/WordPress/gutenberg/pull/75661#issuecomment-3919166528
test.skip( 'should allow user to add and remove multiple local font files', async ( {
page,
editor,
admin,
Expand Down
12 changes: 9 additions & 3 deletions test/e2e/specs/editor/various/draggable-blocks.spec.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -395,7 +395,10 @@ test.describe( 'Draggable block', () => {
await expect(
rowAppender,
'Dragging over the button block appender should show the blue background'
).toHaveCSS( 'background-color', 'rgb(0, 124, 186)' );
).toHaveCSS(
'background-color',
/rgb\(0, 124, 186\)|rgb\(56, 88, 233\)/
);

const { width: rowWidth } = await rowBlock.boundingBox();
await dragOver( rowBlock, { position: { x: rowWidth - 10 } } );
Expand All @@ -404,7 +407,10 @@ test.describe( 'Draggable block', () => {
await expect(
rowAppender,
'Dragging over the empty group block but outside the appender should still show the blue background'
).toHaveCSS( 'background-color', 'rgb(0, 124, 186)' );
).toHaveCSS(
'background-color',
/rgb\(0, 124, 186\)|rgb\(56, 88, 233\)/
);

await drop();
await expect( rowAppender ).toBeHidden();
Expand Down Expand Up @@ -435,7 +441,7 @@ test.describe( 'Draggable block', () => {
// This is technically an implementation detail but easier to test in this case.
await expect( columnAppender ).toHaveCSS(
'background-color',
'rgb(0, 124, 186)'
/rgb\(0, 124, 186\)|rgb\(56, 88, 233\)/
);

await drop();
Expand Down
4 changes: 3 additions & 1 deletion test/e2e/specs/site-editor/page-list.spec.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -149,7 +149,7 @@
name: 'Edit Status & Visibility',
} )
.locator( '..' );
await expect( statusAndVisibility ).toContainText(

Check failure on line 152 in test/e2e/specs/site-editor/page-list.spec.js

View workflow job for this annotation

GitHub Actions / Playwright - 6

[chromium] › test/e2e/specs/site-editor/page-list.spec.js:374:5 › Page List › Quick Edit Mode › should initialize 

  1) [chromium] › test/e2e/specs/site-editor/page-list.spec.js:374:5 › Page List › Quick Edit Mode › should initialize, edit, and update statusVisibility field correctly 
    Error: expect(locator).toContainText(expected) failed

    Locator: getByRole('button', { name: 'Edit Status & Visibility' }).locator('..')
    Expected substring: "Published"
    Received string:    "Status & VisibilityDraft"
    Timeout: 5000ms

    Call log:
      - Expect "toContainText" with timeout 5000ms
      - waiting for getByRole('button', { name: 'Edit Status & Visibility' }).locator('..')
        9 × locator resolved to <div class="dataforms-layouts-panel__field-trigger dataforms-layouts-panel__field-trigger--label-side">…</div>
          - unexpected value "Status & VisibilityDraft"


      150 | 						} )
      151 | 						.locator( '..' );
    > 152 | 					await expect( statusAndVisibility ).toContainText(
          | 					                                    ^
      153 | 						'Published'
      154 | 					);
      155 | 				},
        at assertInitialState (/home/runner/work/gutenberg/gutenberg/test/e2e/specs/site-editor/page-list.spec.js:152:42)
        at /home/runner/work/gutenberg/gutenberg/test/e2e/specs/site-editor/page-list.spec.js:377:12
'Published'
);
},
Expand Down Expand Up @@ -361,7 +361,9 @@
await page.getByRole( 'menuitemradio', { name: 'Table' } ).click();

// Trigger Quick Edit action on Privacy Policy row
const row = page.getByRole( 'row', { name: /Privacy Policy/ } );
const row = page
.getByRole( 'row', { name: /Privacy Policy/ } )
.first();
await row.getByRole( 'button', { name: 'Quick Edit' } ).click();
} );

Expand Down
1 change: 1 addition & 0 deletions test/e2e/specs/site-editor/preload.spec.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -51,6 +51,7 @@ test.describe( 'Preload', () => {
'/wp-abilities/v1/abilities?per_page=100&context=edit',
// Seems to be coming from `enableComplementaryArea`.
'/wp/v2/users/me',
'/wp/v2/settings',
] );
} );
} );
Loading