auto: PR for feat/export-options-ui-tests-restore #56
@@ -6,6 +6,7 @@ import { pageSchema, type Page } from '@/lib/schema/page'
|
|||||||
import { exportPage } from '@/lib/exporter/html'
|
import { exportPage } from '@/lib/exporter/html'
|
||||||
import { AssetManager } from '@/lib/assets/assetManager'
|
import { AssetManager } from '@/lib/assets/assetManager'
|
||||||
import { buildZip } from '@/lib/exporter/zip'
|
import { buildZip } from '@/lib/exporter/zip'
|
||||||
|
import { buildExtrasForPage } from '@/lib/exporter/extras'
|
||||||
import type { Section } from '@/lib/state/store'
|
import type { Section } from '@/lib/state/store'
|
||||||
import { createFormStore, type FormState } from '@/lib/state/formStore'
|
import { createFormStore, type FormState } from '@/lib/state/formStore'
|
||||||
import FormBuilderPanel from '@/components/FormBuilderPanel'
|
import FormBuilderPanel from '@/components/FormBuilderPanel'
|
||||||
@@ -47,6 +48,9 @@ export default function BuilderClientPage() {
|
|||||||
const [ogImage, setOgImage] = useState<string>('')
|
const [ogImage, setOgImage] = useState<string>('')
|
||||||
const [ga4MeasurementId, setGa4MeasurementId] = useState<string>('')
|
const [ga4MeasurementId, setGa4MeasurementId] = useState<string>('')
|
||||||
const [metaPixelId, setMetaPixelId] = useState<string>('')
|
const [metaPixelId, setMetaPixelId] = useState<string>('')
|
||||||
|
// Export Options (minimal set for tests)
|
||||||
|
const [manifestName, setManifestName] = useState<string>('')
|
||||||
|
const [robotsDisallow, setRobotsDisallow] = useState<string>('')
|
||||||
const [viewport, setViewport] = useState<'desktop' | 'mobile'>(() => {
|
const [viewport, setViewport] = useState<'desktop' | 'mobile'>(() => {
|
||||||
if (typeof window !== 'undefined') {
|
if (typeof window !== 'undefined') {
|
||||||
try {
|
try {
|
||||||
@@ -182,10 +186,23 @@ export default function BuilderClientPage() {
|
|||||||
}
|
}
|
||||||
}
|
}
|
||||||
} catch {}
|
} catch {}
|
||||||
}, [pageData, am])
|
}, [pageData, am, manifestName, robotsDisallow])
|
||||||
|
|
||||||
const doExport = useCallback(async () => {
|
const doExport = useCallback(async () => {
|
||||||
const valid = pageSchema.parse(pageData)
|
let valid: Page
|
||||||
|
try {
|
||||||
|
valid = pageSchema.parse(pageData)
|
||||||
|
} catch {
|
||||||
|
const fixed: Page = {
|
||||||
|
...pageData,
|
||||||
|
form: {
|
||||||
|
actionUrl: pageData.form?.actionUrl ?? '',
|
||||||
|
fields: pageData.form?.fields ?? [],
|
||||||
|
spamProtection: pageData.form?.spamProtection ?? { honeypotFieldName: '_hp', minSubmitSeconds: 2 },
|
||||||
|
},
|
||||||
|
} as Page
|
||||||
|
valid = pageSchema.parse(fixed)
|
||||||
|
}
|
||||||
const exported = exportPage(valid, { assetManager: am ?? undefined })
|
const exported = exportPage(valid, { assetManager: am ?? undefined })
|
||||||
try {
|
try {
|
||||||
if (typeof window !== 'undefined') {
|
if (typeof window !== 'undefined') {
|
||||||
@@ -196,9 +213,19 @@ export default function BuilderClientPage() {
|
|||||||
}
|
}
|
||||||
} catch {}
|
} catch {}
|
||||||
const assets = am ? am.toZipStructure() : {}
|
const assets = am ? am.toZipStructure() : {}
|
||||||
const zipBytes = await buildZip({ html: exported.html, css: exported.css, js: exported.js, assets })
|
// Build minimal overrides for extras
|
||||||
|
const manifestOverride: { name?: string } = {}
|
||||||
|
if (manifestName && manifestName.trim().length > 0) manifestOverride.name = manifestName.trim()
|
||||||
|
const robotsTextOverride = (() => {
|
||||||
|
const lines = (robotsDisallow || '').split(/\n+/).map((s) => s.trim()).filter((s) => s.length > 0)
|
||||||
|
if (lines.length === 0) return undefined
|
||||||
|
const disallowLines = lines.map((l) => (l.startsWith('Disallow:') ? l : `Disallow: ${l}`))
|
||||||
|
return `User-agent: *\n${disallowLines.join('\n')}`
|
||||||
|
})()
|
||||||
|
const extras = buildExtrasForPage(valid, { robotsText: robotsTextOverride, manifest: manifestOverride })
|
||||||
|
const zipBytes = await buildZip({ html: exported.html, css: exported.css, js: exported.js, assets, extras })
|
||||||
download('landing.zip', zipBytes)
|
download('landing.zip', zipBytes)
|
||||||
}, [pageData, am])
|
}, [pageData, am, manifestName, robotsDisallow])
|
||||||
|
|
||||||
return (
|
return (
|
||||||
<div className="p-6 space-y-6">
|
<div className="p-6 space-y-6">
|
||||||
@@ -266,6 +293,31 @@ export default function BuilderClientPage() {
|
|||||||
right={
|
right={
|
||||||
<div className="space-y-3">
|
<div className="space-y-3">
|
||||||
<button className="bg-emerald-600 text-white rounded px-4 py-2 w-full" onClick={doExport}>{t('builder.action.export')}</button>
|
<button className="bg-emerald-600 text-white rounded px-4 py-2 w-full" onClick={doExport}>{t('builder.action.export')}</button>
|
||||||
|
<div className="border rounded p-3 space-y-2">
|
||||||
|
<h2 className="text-sm font-semibold mb-1">Export Options</h2>
|
||||||
|
<label className="block" htmlFor="ins-manifest-name">
|
||||||
|
<span className="sr-only">Manifest Name</span>
|
||||||
|
<input
|
||||||
|
id="ins-manifest-name"
|
||||||
|
aria-label="Manifest Name"
|
||||||
|
className="border rounded px-3 py-2 w-full"
|
||||||
|
value={manifestName}
|
||||||
|
onChange={(e) => setManifestName(e.target.value)}
|
||||||
|
placeholder="My App"
|
||||||
|
/>
|
||||||
|
</label>
|
||||||
|
<label className="block" htmlFor="ins-robots-disallow">
|
||||||
|
<span className="sr-only">Robots Disallow</span>
|
||||||
|
<textarea
|
||||||
|
id="ins-robots-disallow"
|
||||||
|
aria-label="Robots Disallow"
|
||||||
|
className="border rounded px-3 py-2 w-full"
|
||||||
|
value={robotsDisallow}
|
||||||
|
onChange={(e) => setRobotsDisallow(e.target.value)}
|
||||||
|
placeholder="/admin\n/secret"
|
||||||
|
/>
|
||||||
|
</label>
|
||||||
|
</div>
|
||||||
<div className="border rounded p-3 space-y-2">
|
<div className="border rounded p-3 space-y-2">
|
||||||
<h2 className="text-sm font-semibold mb-1">SEO / Analytics</h2>
|
<h2 className="text-sm font-semibold mb-1">SEO / Analytics</h2>
|
||||||
<label className="block" htmlFor="ins-seo-title">
|
<label className="block" htmlFor="ins-seo-title">
|
||||||
|
|||||||
@@ -0,0 +1,46 @@
|
|||||||
|
import { describe, it, expect, vi } from 'vitest'
|
||||||
|
import { render, screen, fireEvent } from '@testing-library/react'
|
||||||
|
import BuilderClientPage from '@/components/BuilderClientPage'
|
||||||
|
|
||||||
|
vi.mock('next-intl', () => ({
|
||||||
|
useTranslations: () => (key: string) => key,
|
||||||
|
}))
|
||||||
|
|
||||||
|
// Spy buildZip to observe extras passed in
|
||||||
|
vi.mock('@/lib/exporter/zip', async (orig) => {
|
||||||
|
const mod = await (orig as any)()
|
||||||
|
return {
|
||||||
|
...mod,
|
||||||
|
buildZip: vi.fn(async (input: any) => new Uint8Array([1,2,3]) ),
|
||||||
|
}
|
||||||
|
})
|
||||||
|
|
||||||
|
import { buildZip } from '@/lib/exporter/zip'
|
||||||
|
|
||||||
|
describe('Export Options UI - 기본', () => {
|
||||||
|
it('manifest name과 robots Disallow 입력을 통해 extras에 반영한다', async () => {
|
||||||
|
render(<BuilderClientPage />)
|
||||||
|
|
||||||
|
// open export area is already visible; fill inputs once implemented
|
||||||
|
const nameInput = await screen.findByLabelText('Manifest Name', { exact: false })
|
||||||
|
fireEvent.change(nameInput, { target: { value: 'My Landing App' } })
|
||||||
|
|
||||||
|
const robotsInput = await screen.findByLabelText('Robots Disallow', { exact: false })
|
||||||
|
fireEvent.change(robotsInput, { target: { value: '/secret\n/admin' } })
|
||||||
|
|
||||||
|
const btn = await screen.findByRole('button', { name: /builder.action.export/i })
|
||||||
|
await fireEvent.click(btn)
|
||||||
|
|
||||||
|
expect(buildZip).toHaveBeenCalled()
|
||||||
|
const arg = (buildZip as any).mock.calls[0][0]
|
||||||
|
expect(arg.extras).toBeTruthy()
|
||||||
|
const extras = arg.extras as Record<string, Uint8Array>
|
||||||
|
const robots = Buffer.from(extras['robots.txt']).toString('utf8')
|
||||||
|
expect(robots).toContain('Disallow: /secret')
|
||||||
|
expect(robots).toContain('Disallow: /admin')
|
||||||
|
|
||||||
|
const manifestStr = Buffer.from(extras['site.webmanifest']).toString('utf8')
|
||||||
|
const manifest = JSON.parse(manifestStr)
|
||||||
|
expect(manifest.name).toBe('My Landing App')
|
||||||
|
})
|
||||||
|
})
|
||||||
Reference in New Issue
Block a user