From f0760a6416a64e7da5be2d040b1f4e90214d6b5c Mon Sep 17 00:00:00 2001
From: dongmucat <1127093059@qq.com>
Date: Wed, 29 Apr 2026 10:05:51 +0800
Subject: [PATCH] fix(e2e): resolve strict mode violation in subscription tests

Use .first() for h1 heading locator since skill detail page
renders two h1 elements (page title + README heading).
Add page-ready wait in second test before locating Subscribe button.
---
 web/e2e/skill-subscription.spec.ts | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/web/e2e/skill-subscription.spec.ts b/web/e2e/skill-subscription.spec.ts
index 595c1fe0..90594b89 100644
--- a/web/e2e/skill-subscription.spec.ts
+++ b/web/e2e/skill-subscription.spec.ts
@@ -19,7 +19,7 @@ test.describe('Skill Subscription (Real API)', () => {
 
       await page.goto(`/space/${namespace.slug}/${skill.slug}`)
 
-      await expect(page.getByRole('heading', { level: 1 })).toBeVisible()
+      await expect(page.getByRole('heading', { level: 1 }).first()).toBeVisible()
 
       const subscribeButton = page.getByRole('button', { name: /Subscribe/ })
       await expect(subscribeButton).toBeVisible()
@@ -64,6 +64,8 @@ test.describe('Skill Subscription (Real API)', () => {
 
       await page.goto(`/space/${namespace.slug}/${skill.slug}`)
 
+      await expect(page.getByRole('heading', { level: 1 }).first()).toBeVisible()
+
       const subscribeButton = page.getByRole('button', { name: /Subscribe/ })
       await expect(subscribeButton).toBeVisible()
       await subscribeButton.click()