mirror of
https://github.com/iflytek/skillhub.git
synced 2026-09-11 22:51:04 +00:00
230b915194
Add an "Update" button to each skill card on the My Skills dashboard page. Clicking it navigates to the Publish page with the skill's namespace and visibility pre-selected, reducing manual steps when re-publishing a skill package. - Add visibility field to SkillSummaryResponse so the list API exposes each skill's current visibility setting - Add publish-prefill module to normalize and validate URL search params - Wire TanStack Router validateSearch on the publish route - Add E2E tests covering the prefill flow and invalid-param fallback
57 lines
2.3 KiB
TypeScript
57 lines
2.3 KiB
TypeScript
import { expect, test } from '@playwright/test'
|
|
import { setEnglishLocale } from './helpers/auth-fixtures'
|
|
import { registerSession } from './helpers/session'
|
|
import { E2eTestDataBuilder } from './helpers/test-data-builder'
|
|
|
|
test.describe('My Skills Update Prefill (Real API)', () => {
|
|
test.beforeEach(async ({ page }, testInfo) => {
|
|
await setEnglishLocale(page)
|
|
await registerSession(page, testInfo)
|
|
})
|
|
|
|
test('opens publish page with namespace and visibility prefilled from my skills card', async ({ page }, testInfo) => {
|
|
const builder = new E2eTestDataBuilder(page, testInfo)
|
|
await builder.init()
|
|
|
|
try {
|
|
const namespace = await builder.ensureWritableNamespace()
|
|
const skillName = `update-ui-${Date.now().toString(36)}`
|
|
await builder.publishSkill(namespace.slug, { name: skillName })
|
|
|
|
await page.goto('/dashboard/skills')
|
|
await expect(page.getByRole('heading', { name: 'My Skills' })).toBeVisible()
|
|
|
|
const skillCard = page.locator('.group').filter({
|
|
has: page.getByRole('heading', { name: skillName, exact: true }),
|
|
}).first()
|
|
await expect(skillCard).toBeVisible()
|
|
await skillCard.getByRole('button', { name: 'Update' }).click()
|
|
|
|
await expect(page).toHaveURL(/\/dashboard\/publish/)
|
|
|
|
const currentUrl = new URL(page.url())
|
|
expect(currentUrl.searchParams.get('namespace')).toBe(namespace.slug)
|
|
expect(currentUrl.searchParams.get('visibility')).toBe('PUBLIC')
|
|
await expect(page.locator('#namespace')).toContainText(`@${namespace.slug}`)
|
|
await expect(page.locator('#visibility')).toContainText('Public')
|
|
} finally {
|
|
await builder.cleanup()
|
|
}
|
|
})
|
|
|
|
test('falls back to public visibility when publish search params are invalid', async ({ page }, testInfo) => {
|
|
const builder = new E2eTestDataBuilder(page, testInfo)
|
|
await builder.init()
|
|
|
|
try {
|
|
const namespace = await builder.ensureWritableNamespace()
|
|
|
|
await page.goto(`/dashboard/publish?namespace=${encodeURIComponent(namespace.slug)}&visibility=internal`)
|
|
await expect(page.getByRole('heading', { name: 'Publish Skill' })).toBeVisible()
|
|
await expect(page.locator('#namespace')).toContainText(`@${namespace.slug}`)
|
|
await expect(page.locator('#visibility')).toContainText('Public')
|
|
} finally {
|
|
await builder.cleanup()
|
|
}
|
|
})
|
|
})
|