diff --git a/tests/release-smoke/docker-auth-onboarding.spec.ts b/tests/release-smoke/docker-auth-onboarding.spec.ts index 031d5862d9..650c24a2ea 100644 --- a/tests/release-smoke/docker-auth-onboarding.spec.ts +++ b/tests/release-smoke/docker-auth-onboarding.spec.ts @@ -10,7 +10,6 @@ const ADMIN_PASSWORD = "paperclip-smoke-password"; const COMPANY_NAME = `Release-Smoke-${Date.now()}`; -const MISSION = "Ship a reliable release smoke suite for Paperclip."; const AGENT_NAME = "CEO"; // Seeded by the wizard's launch step (DEFAULT_TASK_TITLE in // ui/src/components/OnboardingWizard.tsx). @@ -47,20 +46,12 @@ test.describe("Docker authenticated onboarding smoke", () => { await signIn(page); await openOnboarding(page); - // Step 1: name the company. + // Step 1: name the company. "Next" creates the company itself and routes + // straight to the agent step — onboarding no longer asks for the mission + // (it is collected later, in the tenant app), so there is no step 2. await page.locator('input[placeholder="Acme Corp"]').fill(COMPANY_NAME); await page.getByRole("button", { name: "Next" }).click(); - // Step 2: define the mission directly; confirming creates the company. - await expect( - page.locator("h3", { hasText: "Define your mission" }) - ).toBeVisible({ timeout: 10_000 }); - await page.getByRole("button", { name: "I know my mission" }).click(); - await page - .locator('textarea[placeholder="What is your team trying to achieve?"]') - .fill(MISSION); - await page.getByRole("button", { name: "Confirm mission" }).click(); - // Step 3: give the team lead a role, then a name. The role gates "Next". const roleSelect = page.locator("#onboarding-agent-role"); await expect(roleSelect).toBeVisible({ timeout: 20_000 }); @@ -111,25 +102,15 @@ test.describe("Docker authenticated onboarding smoke", () => { expect(ceoAgent!.role).toBe("ceo"); expect(ceoAgent!.adapterType).not.toBe("process"); + // Onboarding deliberately writes no goal: the mission is collected later + // in the tenant app, so a fresh company must come out of the wizard with + // an empty goal list rather than an unchosen one. const goalsRes = await page.request.get( `${baseUrl}/api/companies/${company!.id}/goals` ); expect(goalsRes.ok()).toBe(true); - const goals = (await goalsRes.json()) as Array<{ - id: string; - title: string; - level: string; - status: string; - }>; - expect(goals).toEqual( - expect.arrayContaining([ - expect.objectContaining({ - title: MISSION, - level: "company", - status: "active", - }), - ]) - ); + const goals = (await goalsRes.json()) as Array<{ id: string }>; + expect(goals).toEqual([]); const issuesRes = await page.request.get( `${baseUrl}/api/companies/${company!.id}/issues`