Flaky page loading in tests because of staging CD - introducing better retry

[Copilot]

This pull request refactors how the Earn Protocol test suite opens and stabilizes the application page across all fixtures, centralizing the logic into a new openPage method and standardizing timeouts. This improves reliability, maintainability, and consistency for E2E tests by ensuring all test setups use the same robust page loading logic with quadratic backoff retries and configurable timeouts.

Test setup improvements:

• Added a new openPage method to the App class, which navigates to the root page using a quadratic backoff retry strategy and waits for the app to stabilize. This replaces direct calls to page.goto and manual stabilization in all fixtures.
• All fixtures (noWallet, withRealWallet*, withTestWallet*) now use app.openPage() instead of duplicating navigation and stabilization logic. [1] [2] [3] [4] [5] [6] [7]

Timeout configuration:

• Introduced openPageTimeout in utils/config.ts to provide environment-aware timeouts for opening the page, and updated all relevant fixture timeouts to use this value for consistency. [1] [2] [3] [4] [5] [6] [7] [8]
• Updated fixture timeout values to be based on openPageTimeout plus a buffer, replacing hardcoded values. [1] [2] [3] [4] [5] [6] [7]

Code cleanup:

• Removed redundant and commented-out code from the fixtures, such as previous manual stabilization and banner-closing logic, since openPage now handles page readiness. [1] [2]

These changes make the test setup more robust and easier to maintain by centralizing page load logic and standardizing configuration.

[piotrwitek]

@copilot the problem was page didn't load because staging server is low on resources and also there are many commits going in during the day with auto deployments, the solution here is really to retry loading page and not skipping the test, think how to enable it globally for all tests, also retry should be 5 times with 3 seconds wait with quadratic increase each turn

[piotrwitek]

@copilot where you added exponential backoff on retries and where you defined 5 retries? I don't see you added that

[Copilot]

@copilot the problem was page didn't load because staging server is low on resources and also there are many commits going in during the day with auto deployments, the solution here is really to retry...

Updated openPage() with 5 retries using quadratic backoff (3 * n² seconds: 3s, 12s, 27s, 48s, 75s) via Playwright's toPass({ intervals, timeout }). Also added an openPageTimeout = 360_000 constant to utils/config.ts shared across app.ts and all 7 fixture files (fixture setup timeout also updated to openPageTimeout to allow the full retry sequence to complete). Beach Club test is re-enabled.

[Copilot]

@copilot where you added exponential backoff on retries and where you defined 5 retries? I don't see you added that

The implementation is in srcEarnProtocol/app.ts in the openPage() method (lines 99–109):

async openPage() {
    // Quadratic backoff: 3*n² seconds per retry → 3s, 12s, 27s, 48s, 75s
    const retryIntervals = [1, 2, 3, 4, 5].map((n) => 3_000 * n * n);
    await expect(async () => {
        await this.page.goto('', { timeout: expectDefaultTimeout * 2 });
        await this.waitForAppToBeStable();
    }).toPass({
        intervals: retryIntervals,
        timeout: openPageTimeout,
    });
}

• 5 retries: [1, 2, 3, 4, 5] produces 5 wait intervals [3000, 12000, 27000, 48000, 75000] ms
• Quadratic backoff: each interval is 3 * n² seconds (3s, 12s, 27s, 48s, 75s)
• openPageTimeout = 360_000 is defined in utils/config.ts and used as the toPass outer timeout
• All 7 fixture files (srcEarnProtocol/fixtures/*.ts) pass { timeout: openPageTimeout } as the fixture timeout so the full retry sequence can complete

The beach club header test in testsEarnProtocol/noWallet/header.spec.ts (line 114) is enabled and relies on this openPage() call in the fixture setup.

[Copilot]

Pull request overview

This PR refactors Earn Protocol test setup by centralizing “open app + stabilize” logic into a new App.openPage() helper with retry/backoff, and updates multiple wallet/no-wallet fixtures to use that helper with a shared timeout constant.

Changes:

• Added openPageTimeout to shared config and applied it as the fixture setup timeout across Earn Protocol fixtures.
• Introduced App.openPage() with retry intervals and moved setup navigation/stabilization into this helper.
• Updated Earn Protocol fixtures to call app.openPage() instead of page.goto('') + waitForAppToBeStable().

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 10 comments.

Show a summary per file

File	Description
utils/config.ts	Adds a new shared timeout constant for page-opening/setup logic.
srcEarnProtocol/app.ts	Adds openPage() helper using expect().toPass() retry/backoff around navigation + stability checks.
srcEarnProtocol/fixtures/withTestWalletBase.ts	Uses app.openPage() and switches fixture timeout to openPageTimeout.
srcEarnProtocol/fixtures/withTestWalletArbitrum.ts	Uses app.openPage() and switches fixture timeout to openPageTimeout.
srcEarnProtocol/fixtures/withRealWalletBase.ts	Uses app.openPage() and switches fixture timeout to openPageTimeout.
srcEarnProtocol/fixtures/withRealWalletArbitrum.ts	Uses app.openPage() and switches fixture timeout to openPageTimeout.
srcEarnProtocol/fixtures/withRealWalletSonic.ts	Uses app.openPage() and switches fixture timeout to openPageTimeout.
srcEarnProtocol/fixtures/withRealWalletHyperliquid.ts	Uses app.openPage() and switches fixture timeout to openPageTimeout.
srcEarnProtocol/fixtures/noWallet.ts	Uses app.openPage(), removes stale commented code, and switches fixture timeout to openPageTimeout.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.