Skip to content

Commit

Permalink
Fix flaky ccpa monitoring tests (#955)
Browse files Browse the repository at this point in the history
* remove probably redundant reload

* The reload is now triggered by onConsentChange so we no longer need this before checking the ads

* temporarily running every 2 minutes on CODE

* Revert monitoring duration increase for CODE
  • Loading branch information
sookburt committed Apr 3, 2024
1 parent aeee56d commit 56c7197
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion monitoring/src/check-page/ccpa.ts
Original file line number Diff line number Diff line change
Expand Up @@ -80,7 +80,6 @@ const checkPages = async (config: Config, url: string, nextUrl: string) => {
await checkCMPIsOnPage(page);
await clickDoNotSellMyInfo(config, page);
await checkCMPIsNotVisible(page);
await reloadPage(page);
await checkTopAdHasLoaded(page);

if (nextUrl) {
Expand Down

1 comment on commit 56c7197

@github-actions
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Coverage report

St.
Category Percentage Covered / Total
🟢 Statements 91.04% 254/279
🟢 Branches 81.67% 98/120
🟢 Functions 88.57% 62/70
🟢 Lines 90.74% 245/270

Test suite run success

331 tests passing in 16 suites.

Report generated by 🧪jest coverage report action from 56c7197

Please sign in to comment.