Compare commits

..

1 Commits

Author SHA1 Message Date
peptiderepo
5a859e6d1a fix(setup): retry login up to 3× with 90s timeout and 15s backoff
Hostinger rate-limits GitHub runner IPs on wp-login.php, causing
TimeoutError (30s) during the auth setup step. This has produced 6
deploy-gate failures over 2 days across multiple repos.

Changes:
- Raise navigation timeout from 30s to 90s per attempt
- Wrap the full login sequence in a retry loop (up to 3 attempts)
- Create a fresh BrowserContext per attempt so stale cookies / throttle
  state cannot carry over between retries
- Wait 15s between attempts to let transient throttle windows clear
- Log each attempt number and outcome clearly (console.log + console.error)
  so any future failure is diagnosable from the CI log without a local repro
- On final failure, surface a hint pointing to the Hostinger throttle cause

Assertion behaviour: unchanged. The #wpadminbar check and storageState
write are identical to before — retries only affect getting past wp-login.

No changes to smoke test assertions, playwright.config.ts, or workflow files.

Agent-Session: e2e-login-flake-20260612
2026-06-12 02:32:14 +00:00
3 changed files with 1 additions and 187 deletions

View File

@@ -1,35 +0,0 @@
# Changelog
All notable changes to the peptide-e2e smoke suite will be documented here.
## [1.2.0] - 2026-06-12
### Added
- **Dossier page smoke test** (`03-admin-plugins.spec.ts` — `WP Admin — PRAutoBlogger Article Dossier`).
Navigates to `/wp-admin/admin.php?page=prautoblogger-dossier&post_id=<N>`, asserts HTTP 200,
`#prab-dossier` container visible, body does NOT contain "Invalid plugin page" or "Fatal error".
Guards against: menu-ordering regression (wrong hookname → wp_die 404), PHP fatals on dossier
load, missing dossier container, and stage-section regression.
Defensive strategy: tries canonical staging post IDs 925/930 first, then falls back to the
most recent REST post, then falls back to post_id=0 (graceful empty state). Stage-section
assertions are conditional on a run record existing. Loud test.info() annotations on fallback
paths. New in PRAutoBlogger v0.19.2 / v0.19.3.
## [1.1.0] - 2026-06-12
### Added
- **Board page smoke test** (`03-admin-plugins.spec.ts` — `WP Admin — PRAutoBlogger board page`).
Navigates to `/wp-admin/admin.php?page=prautoblogger-board`, asserts HTTP 200,
`#prab-board` container visible, and body does NOT contain "Invalid plugin page".
Guards against the v0.19.1 regression class: Board submenu registered before parent
menu existed (both at admin_menu priority 10) → WordPress fallback hookname →
wp_die 404. Now in the deploy-gate smoke suite so this failure class is un-shippable.
Note: cannot be validated against prod until v0.19.1 of PRAutoBlogger deploys.
Spec compiles and lists cleanly under `npx playwright test --list`.
## [1.0.0] - 2026-06-11
Initial release. 21 tests across 5 specs: homepage, reconstitution calculator,
peptide search, admin plugins, REST API.

View File

@@ -1,6 +1,6 @@
{
"name": "peptide-e2e",
"version": "1.2.0",
"version": "1.0.0",
"description": "Playwright smoke tests for peptiderepo.com (staging + production)",
"private": true,
"scripts": {

View File

@@ -94,44 +94,6 @@ test.describe('WP Admin — PRAutoBlogger settings page', () => {
});
});
test.describe('WP Admin — PRAutoBlogger board page', () => {
/**
* Smoke test: Board submenu page must load without 404 or "Invalid plugin page".
*
* This test guards against the v0.19.1 regression class: Board submenu was
* registered before the parent top-level menu page existed (both at admin_menu
* priority 10), causing WordPress to fall back to the `admin_page_*` hookname.
* At request time WP recomputed the hookname, found nothing, and called
* wp_die("Invalid plugin page"). The fix (v0.19.1) moves the board hook to
* priority 11 so the parent menu slot is populated first.
*
* Selector sourced from templates/admin/board-page.php: `<div id="prab-board" ...>`.
*
* NOTE: This test cannot be validated against prod until v0.19.1 is deployed.
* It is included in the deploy-gate smoke suite so the bug class becomes
* un-shippable in future. The spec compiles and lists cleanly; the test will
* run correctly once the fix is live.
*/
test('board page loads and board container is present', async ({ page }) => {
// Navigate to the Board submenu page.
const res = await page.goto(`${ADMIN}/admin.php?page=prautoblogger-board`);
// Must be HTTP 200 — not wp_die 404.
expect(res?.status()).toBe(200);
// wp-admin chrome must render (confirms we're authenticated and in admin).
await expect(page.locator('#wpbody')).toBeVisible();
// Body must NOT contain the wp_die "Invalid plugin page" error string.
const body = await page.textContent('body');
expect(body).not.toContain('Invalid plugin page');
// Board container div must be present (from templates/admin/board-page.php).
// selector: <div id="prab-board" class="prab-board" ...>
await expect(page.locator('#prab-board')).toBeVisible({ timeout: 10_000 });
});
});
test.describe('WP Admin — Peptide Repo Core admin', () => {
test('PR Core menu item exists in wp-admin', async ({ page }) => {
await page.goto(`${ADMIN}/`);
@@ -154,116 +116,3 @@ test.describe('WP Admin — Peptide Repo Core admin', () => {
await expect(page.locator('form')).toBeVisible();
});
});
test.describe('WP Admin — PRAutoBlogger Article Dossier', () => {
/**
* Smoke test: Article Dossier page must load without "Invalid plugin page" error.
*
* The dossier is a hidden-submenu page registered at admin_menu priority 12
* (after board p11, after parent p10). It is link-accessed via
* admin.php?page=prautoblogger-dossier&post_id=<int>.
*
* This test guards against:
* 1. Menu-ordering regression (wrong hookname → wp_die 404).
* 2. PHP fatal errors on dossier load (missing require, fatal in assembler).
* 3. Missing dossier container div (template regression).
* 4. Stage-section regression (stage markup absent when run data exists).
*
* Defensive strategy: the test first queries the WP REST API for a post
* published by PRAutoBlogger (_prautoblogger_generated=1). If a post is found,
* it opens its dossier and asserts the full set. If no matching post exists
* on this environment (staging/prod may differ), the test opens the dossier
* with post_id=0 and asserts the graceful "no run record" empty state — which
* still validates page load, no fatal, no "Invalid plugin page".
*
* Selector sourced from templates/admin/dossier-page.php:
* <div id="prab-dossier" class="prab-dossier-wrap" ...>
*
* Posts 925/930 are the canonical staging test posts (v0.18.3 generation run).
* They are tried first; REST fallback handles environments where they don't exist.
*/
test('dossier page loads and dossier container is present', async ({ page, request }) => {
const ADMIN = '/wp-admin';
const CANDIDATE_POST_IDS = [925, 930]; // staging canonical posts from v0.18.3
// Strategy 1: try known staging post IDs
let targetPostId: number | null = null;
for (const pid of CANDIDATE_POST_IDS) {
const apiRes = await request.get(`/wp-json/wp/v2/posts/${pid}`);
if (apiRes.ok()) {
targetPostId = pid;
console.log(`[dossier smoke] Found candidate post id=${pid} via REST API`);
break;
}
}
// Strategy 2: query for any PRAutoBlogger-generated post via REST
if (targetPostId === null) {
const listRes = await request.get('/wp-json/wp/v2/posts?per_page=5&orderby=date&order=desc');
if (listRes.ok()) {
const posts: Array<{ id: number; meta?: Record<string, unknown> }> = await listRes.json();
if (posts.length > 0) {
targetPostId = posts[0].id;
console.log(`[dossier smoke] Fallback: using most recent post id=${targetPostId}`);
}
}
}
// Navigate to dossier — use targetPostId if found, else post_id=0 for graceful empty state
const postIdParam = targetPostId !== null ? targetPostId : 0;
const dossierUrl = `${ADMIN}/admin.php?page=prautoblogger-dossier&post_id=${postIdParam}`;
if (targetPostId === null) {
// Loud annotation — the dossier will render empty state, not full dossier
test.info().annotations.push({
type: 'warning',
description:
'[dossier smoke] No PRAutoBlogger post found on this environment. ' +
'Asserting graceful empty state (post_id=0). ' +
'Full dossier assertions (stage sections) require a post with a run record.',
});
}
const res = await page.goto(dossierUrl);
// Must be HTTP 200 — not wp_die 404 (menu-ordering regression guard).
expect(res?.status()).toBe(200);
// wp-admin chrome must render (confirms auth session is valid).
await expect(page.locator('#wpbody')).toBeVisible();
// Body must NOT contain the wp_die "Invalid plugin page" error string.
const body = await page.textContent('body');
expect(body).not.toContain('Invalid plugin page');
expect(body).not.toContain('Fatal error');
// Dossier container div must be present in all cases (full run OR empty state).
// Selector from templates/admin/dossier-page.php: <div id="prab-dossier" ...>
await expect(page.locator('#prab-dossier')).toBeVisible({ timeout: 10_000 });
// Stage sections — only assert if we have a post with (likely) a run record.
if (targetPostId !== null) {
// At least one stage section must be present IF the post has a run record.
// The dossier may still show empty state if this post pre-dates the substrate.
// We assert conditionally: if any stage section exists, it has the right class.
const stageSections = page.locator('.prab-stage-section');
const stageCount = await stageSections.count();
if (stageCount > 0) {
// Dossier has stage data — verify stage sections render without error
console.log(`[dossier smoke] Found ${stageCount} stage sections on post id=${targetPostId}`);
await expect(stageSections.first()).toBeVisible();
} else {
// Post exists but has no run stages — graceful empty state (pre-substrate post)
test.info().annotations.push({
type: 'info',
description:
`[dossier smoke] Post id=${targetPostId} exists but has no stage sections. ` +
'This is the expected graceful state for posts published before v0.18.0.',
});
// The dossier container is already verified above; this is not a failure.
}
}
});
});