Skip to content

Conversation

@jakejackson1
Copy link
Member

@jakejackson1 jakejackson1 commented Aug 21, 2025

Description

Convert testcafe E2E tests into playwright E2E tests

Testing instructions

Screenshots

Checklist:

  • I've tested the code.
  • My code is easy to read, follow, and understand
  • My code follows the accessibility standards.
  • My code has proper inline documentation / docblocks.

Additional Comments

@jakejackson1 jakejackson1 mentioned this pull request Aug 21, 2025
4 tasks
@codecov
Copy link

codecov bot commented Aug 21, 2025

Codecov Report

❌ Patch coverage is 90.00000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 92.90%. Comparing base (4bc6ea4) to head (8aafcdf).
⚠️ Report is 58 commits behind head on hot-patch.

Files with missing lines Patch % Lines
src/assets/js/react/sagas/fontManager.js 50.00% 2 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff              @@
##           hot-patch    #1625      +/-   ##
=============================================
+ Coverage      92.72%   92.90%   +0.17%     
=============================================
  Files             58       58              
  Lines           1526     1508      -18     
  Branches         420      415       -5     
=============================================
- Hits            1415     1401      -14     
+ Misses           103      102       -1     
+ Partials           8        5       -3     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

When the PDF tmp directory is moved to a shared folder – like the top-level `tmp` folder – permission errors can occur. Add extra error handling for this use-case.
Guard against arrays when checking `page` and `subview` URL params in admin area
The EE_Front_Controller::get_request runs on the `parse_request` hook with a priority of 1. For whatever reason, this hook also removes itself when it runs. Under specific circumstances, WordPress will skip hooks on priority 10 due to this removal.

To workaround this bug this PR moves our own `parse_request` hooks to priority 0.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants