Skip to content

Coding Standards: replace isset() ternary with null coalescing #41689

Coding Standards: replace isset() ternary with null coalescing

Coding Standards: replace isset() ternary with null coalescing #41689

Triggered via pull request May 11, 2025 00:57
Status Success
Total duration 6m 11s
Artifacts 2

end-to-end-tests.yml

on: pull_request
Matrix: e2e-tests
Slack Notifications  /  Prepare notifications
Slack Notifications / Prepare notifications
Slack Notifications  /  Cancelled notifications
Slack Notifications / Cancelled notifications
Slack Notifications  /  Failure notifications
Slack Notifications / Failure notifications
Slack Notifications  /  Fixed notifications
Slack Notifications / Fixed notifications
Slack Notifications  /  Success notifications
Slack Notifications / Success notifications
Failed workflow tasks
0s
Failed workflow tasks
Fit to window
Zoom out
Zoom in

Annotations

4 errors and 2 notices
[chromium] › tests/e2e/specs/media-upload.test.js:11:5 › Test dismissing failed upload works correctly: tests/e2e/specs/media-upload.test.js#L14
1) [chromium] › tests/e2e/specs/media-upload.test.js:11:5 › Test dismissing failed upload works correctly Retry #1 ─────────────────────────────────────────────────────────────────────────────────────── Error: Not logged in 12 | // Log in before visiting admin page. 13 | await requestUtils.login(); > 14 | await admin.visitAdminPage( '/media-new.php' ); | ^ 15 | 16 | // It takes a moment for the multi-file uploader to become available. 17 | await page.waitForLoadState('load'); at Admin.visitAdminPage (/home/runner/work/wordpress-develop/wordpress-develop/node_modules/@wordpress/e2e-test-utils-playwright/src/admin/visit-admin-page.ts:36:9) at /home/runner/work/wordpress-develop/wordpress-develop/tests/e2e/specs/media-upload.test.js:14:2
[chromium] › tests/e2e/specs/media-upload.test.js:11:5 › Test dismissing failed upload works correctly: tests/e2e/specs/media-upload.test.js#L14
1) [chromium] › tests/e2e/specs/media-upload.test.js:11:5 › Test dismissing failed upload works correctly Error: Not logged in 12 | // Log in before visiting admin page. 13 | await requestUtils.login(); > 14 | await admin.visitAdminPage( '/media-new.php' ); | ^ 15 | 16 | // It takes a moment for the multi-file uploader to become available. 17 | await page.waitForLoadState('load'); at Admin.visitAdminPage (/home/runner/work/wordpress-develop/wordpress-develop/node_modules/@wordpress/e2e-test-utils-playwright/src/admin/visit-admin-page.ts:36:9) at /home/runner/work/wordpress-develop/wordpress-develop/tests/e2e/specs/media-upload.test.js:14:2
[chromium] › tests/e2e/specs/media-upload.test.js:11:5 › Test dismissing failed upload works correctly: tests/e2e/specs/media-upload.test.js#L14
2) [chromium] › tests/e2e/specs/media-upload.test.js:11:5 › Test dismissing failed upload works correctly Error: Not logged in 12 | // Log in before visiting admin page. 13 | await requestUtils.login(); > 14 | await admin.visitAdminPage( '/media-new.php' ); | ^ 15 | 16 | // It takes a moment for the multi-file uploader to become available. 17 | await page.waitForLoadState('load'); at Admin.visitAdminPage (/home/runner/work/wordpress-develop/wordpress-develop/node_modules/@wordpress/e2e-test-utils-playwright/src/admin/visit-admin-page.ts:36:9) at /home/runner/work/wordpress-develop/wordpress-develop/tests/e2e/specs/media-upload.test.js:14:2
[chromium] › tests/e2e/specs/install.test.js:34:6 › WordPress installation process › should install WordPress with pre-existing database credentials: tests/e2e/specs/install.test.js#L40
1) [chromium] › tests/e2e/specs/install.test.js:34:6 › WordPress installation process › should install WordPress with pre-existing database credentials Error: should redirect to the installation page Timed out 5000ms waiting for expect(locator).toHaveURL(expected) Locator: locator(':root') Expected pattern: /wp-admin\/install\.php$/ Received string: "http://localhost:8889/" Call log: - should redirect to the installation page with timeout 5000ms - waiting for locator(':root') 9 × locator resolved to <html lang="en-US">…</html> - unexpected value "http://localhost:8889/" 38 | page, 39 | 'should redirect to the installation page' > 40 | ).toHaveURL( /wp-admin\/install\.php$/ ); | ^ 41 | 42 | await expect( 43 | page.getByText( /WordPress database error/ ), at /home/runner/work/wordpress-develop/wordpress-develop/tests/e2e/specs/install.test.js:40:5
🎭 Playwright Run Summary
1 flaky [chromium] › tests/e2e/specs/media-upload.test.js:11:5 › Test dismissing failed upload works correctly 24 passed (43.7s)
🎭 Playwright Run Summary
2 flaky [chromium] › tests/e2e/specs/install.test.js:34:6 › WordPress installation process › should install WordPress with pre-existing database credentials [chromium] › tests/e2e/specs/media-upload.test.js:11:5 › Test dismissing failed upload works correctly 23 passed (44.8s)

Artifacts

Produced during runtime
Name Size Digest
failures-artifacts-14950752014
606 KB
sha256:18435c9cea5193d1e4898cb47f082756f83a682daa1be7bd7b4584c054f365d4
failures-artifacts-SCRIPT_DEBUG-14950752014
341 KB
sha256:863f6136b4dcf8ff4d3419424cd4b3dcaa724dd9b7be27bb2913f95c0062109c