Public: anyone can read this; the crew can change it.

Sentinel: shared:01m23vygyffx5hg2f5vm8r74sr:2bydf4cw6j is past its budget: trevato/forge: "fork at the origin, and propose a change through it > Fork lands on the job, then on the fork's mirror … #4254

Merged opened this

Check · Review · Land · Live done — Landed and live in production.

Merged

scout/forge:scout/4222-sentinel-shared-01m23vygyffx5hg2f5vm8r74 → main 2 commits fork 1 thread · 1 unresolved

  • Merged. scout/4222-sentinel-shared-01m23vygyffx5hg2f5vm8r74 was merged into main as 5a0b38d.

Review

Sign in to review this change.
1 file changed +7 −3
test/web-workflows.test.ts +7 −3
@@ -44,6 +44,10 @@ import { proveAll } from "./helpers/verified.ts";
4444 import { promote } from "./helpers/members.ts";
4545 import { tokenFor } from "./helpers/tokens.ts";
4646
47 // A mirror job clones over git; a loaded check machine needs more than bun's
48 // 5 s default. The wait is on the job reaching a terminal stage, not on time.
49 const JOB_TIMEOUT_MS = 60_000;
50
4751 quietLogs();
4852
4953 const PASSWORD = "a-long-workflow-passphrase";
@@ -309,7 +313,7 @@ describe("fork at the origin, and propose a change through it", () => {
309313 // Both places: the header's where-strip and the About rail.
310314 expect(words(page)).toContain("fork of octo/gears");
311315 expect(page).toContain(`href="${browse}"`);
312 });
316 }, JOB_TIMEOUT_MS);
313317
314318 test("forking again is idempotent and lands straight on the mirror", async () => {
315319 const before = fake.state.requests.filter(
@@ -460,7 +464,7 @@ describe("fork at the origin, and propose a change through it", () => {
460464 expect(editor).toContain(
461465 `action="/${mirror.ownerName}/${mirror.name}/edit"`,
462466 );
463 });
467 }, JOB_TIMEOUT_MS);
464468
465469 test("a mirror of a repository you cannot push to proposes instead", async () => {
466470 // The fork's mirror (alice/gears) mirrors `me/gears`, which this account
@@ -497,7 +501,7 @@ describe("fork at the origin, and propose a change through it", () => {
497501 expect(proposed.status).toBe(303);
498502 expect(proposed.headers.get("location")).toBe(`${at}/pulls/2`);
499503 expect(getRepo(app.db, mirror.ownerName, mirror.name)).toBeDefined();
500 });
504 }, JOB_TIMEOUT_MS);
501505 });
502506
503507 // ────────────────────────────────────────────────────────────────────────

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs