fix(e2e): the lint pass renamed Playwright fixtures, which killed the suite (#349)

Playwright resolves a fixture BY NAME from the destructured parameter, so
renaming an unused one to `_page` asks for a fixture that does not exist. The
e2e leg dies before a single test runs and the report contains no test
entries at all.

Introduced by the pass that let the linter see tests/. The linter was right
that the parameters were unused; underscore-prefixing is the fix for an
ordinary unused argument and the wrong fix for a Playwright fixture, because
the name is the lookup key.

Same defect as buildiq#642, found by a fleet sweep.

Co-authored-by: Conduction Release Bot <release-bot@conduction.nl>

Author: rubenvdlinde

tests/e2e/jobs.spec.ts

@@ -299,9 +299,7 @@ test.describe("background jobs", () => {
 	});
 
 	// --- audit retention prune job ----------------------------------------
-	test("the prune job removes entries older than the retention window", async ({
-		page: _page,
-	}) => {
+	test("the prune job removes entries older than the retention window", async () => {
 		// Seed one clearly-old audit row and one recent row.
 		await sqlExec(
 			`INSERT INTO oc_app_versions_audit (actor_uid, app_id, operation, status, created_at) VALUES ('system','prunetest','install','success', '${tsOffset(-400)}')`,