wp-media / wp-rocket-e2e

Playwright E2E testing repo

GNU General Public License v3.0

0 stars 0 forks source link

Closes #70 Add scenario for pages without LCP/ATF #76

Closed Khadreal closed 5 months ago

Khadreal commented 6 months ago

Description

Fixes #70

Documentation

Technical documentation

This PR add checks for pages with no LCP/ATF and check the DB to assert the result are as expected.

Type of change

[x] Enhancement (non-breaking change which improves an existing functionality).

New dependencies

N/A

Risks

List possible performance & security issues or risks, and explain how they have been mitigated.

Checklists

Feature validation

[ ] I validated all the Acceptance Criteria. If possible, provide sreenshots or videos.
[ ] I triggered all changed lines of code at least once without new errors/warnings/notices.
[ ] I implemented built-in tests to cover the new/changed code.

Code style

[x] I wrote self-explanatory code about what it does.
[x] I wrote comments to explain why it does it.
[x] I named variables and functions explicitely.
[x] I did not introduce unecessary complexity.