From 044a2801b9c1261cf2a0d50c26cce82300aacacc Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E5=8D=83=E4=B9=98=E5=A6=8D=20=28Xiaoyaner=29?= Date: Thu, 5 Nov 2026 15:37:17 +0800 Subject: [PATCH] test(e2e): follow two-step publish confirmation in publish flow MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The publish page now opens a confirmation dialog from "Next: confirm"; drive that step before clicking "Confirm Publish". Signed-off-by: 千乘妍 (Xiaoyaner) --- web/e2e/publish-flow-ui.spec.ts | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/web/e2e/publish-flow-ui.spec.ts b/web/e2e/publish-flow-ui.spec.ts index 6cb0cf40d..c827fa11d 100644 --- a/web/e2e/publish-flow-ui.spec.ts +++ b/web/e2e/publish-flow-ui.spec.ts @@ -49,7 +49,13 @@ test.describe('Publish Flow UI (Real API)', () => { // Scope this regression test to the existing ZIP picker. await page.locator('input[type="file"][accept*=".zip"]').setInputFiles(packagePath) await expect(page.getByText(path.basename(packagePath))).toBeVisible() - const confirmButton = page.getByRole('button', { name: 'Confirm Publish' }) + // Publishing is a two-step flow: review the form, then confirm in the dialog. + const nextButton = page.getByRole('button', { name: 'Next: confirm' }) + await expect(nextButton).toBeEnabled() + await nextButton.click() + const confirmDialog = page.getByRole('dialog') + await expect(confirmDialog).toContainText(`@${namespace.slug}/${skillName}`) + const confirmButton = confirmDialog.getByRole('button', { name: 'Confirm Publish' }) await expect(confirmButton).toBeEnabled() const publishResponsePromise = page.waitForResponse( (response) =>