Sync public snapshot from freebuff-private

Source: CodebuffAI/freebuff-private@b3a2a67adce0339131bbd5e54290a2c334b50fc4

Author: github-actions[bot]

bun.lock

@@ -1,6 +1,202 @@
-import { describe, expect, test } from 'bun:test'
+import { beforeAll, describe, expect, test } from 'bun:test'
+import { createTestRenderer } from '@opentui/core/testing'
+import { createRoot, flushSync } from '@opentui/react'
+import React from 'react'
 
-import { getAdDisplayLabel, getInlineAdLayout } from '../ad-banner'
+import {
+  AD_CARD_HEIGHT,
+  AdCard,
+  getAdDisplayLabel,
+  getCardAdLayout,
+  getInlineAdLayout,
+} from '../ad-banner'
+import { initializeThemeStore } from '../../hooks/use-theme'
+
+beforeAll(() => {
+  initializeThemeStore()
+})
+
+describe('card ad layout', () => {
+  const ad = {
+    adText:
+      'Automate mobile UI testing with plain-English test steps and AI-powered execution.',
+    title: 'Test every release before you ship',
+    cta: 'Try free',
+    url: 'https://www.drizz.dev/ios',
+  }
+
+  test('renders the headline for the creative the console asks for', () => {
+    // The regression this whole function exists for: with a CTA and a landing
+    // URL both set, the headline used to reach neither the CTA fallback nor
+    // the destination label, so it rendered nowhere at all.
+    const layout = getCardAdLayout(ad, 78)
+
+    expect(layout.headline).toBe('Test every release before you ship')
+    expect(layout.ctaText).toBe('Try free')
+    expect(layout.labelText).toBe('drizz.dev')
+  })
+
+  test('gives up a description line to make room for the headline', () => {
+    expect(getCardAdLayout(ad, 78).descriptionLines).toBe(1)
+  })
+
+  test('keeps both description lines when the ad has no headline', () => {
+    const layout = getCardAdLayout({ ...ad, title: '' }, 78)
+
+    expect(layout.headline).toBe('')
+    expect(layout.descriptionLines).toBe(2)
+  })
+
+  test('the interior always sums to the reserved card height', () => {
+    // The landing screen subtracts AD_CARD_HEIGHT from the model picker's
+    // budget, so a layout that needs a row it was not given clips silently.
+    for (const title of ['A headline', '']) {
+      const layout = getCardAdLayout({ ...ad, title }, 78)
+      const headlineRows = layout.headline ? 1 : 0
+      const borderRows = 2
+      const ctaRow = 1
+
+      expect(borderRows + headlineRows + layout.descriptionLines + ctaRow).toBe(
+        AD_CARD_HEIGHT,
+      )
+    }
+  })
+
+  test('falls back to Learn more rather than reprinting the headline', () => {
+    // `ad.cta || ad.title` put the same string in two rows of a five-row card.
+    expect(getCardAdLayout({ ...ad, cta: '' }, 78).ctaText).toBe('Learn more')
+  })
+
+  test('drops the destination label when it would repeat the headline', () => {
+    // With no URL the label falls back to the title, which now has its own row.
+    const layout = getCardAdLayout({ ...ad, url: '' }, 78)
+
+    expect(layout.headline).toBe('Test every release before you ship')
+    expect(layout.labelText).toBe('')
+  })
+
+  test('keeps the Sponsored label when there is no headline and no URL', () => {
+    const layout = getCardAdLayout({ ...ad, title: '', url: '' }, 78)
+
+    expect(layout.labelText).toBe('Sponsored')
+  })
+
+  test('survives a provider that omits fields the type says are required', () => {
+    // `AdResponse` types these as required strings, but the Gravity provider
+    // casts `response.json()` instead of parsing it and copies `cta: raw.cta`
+    // with no default — while Carbon beside it writes `?? 'Learn more'`. A
+    // throw here is a throw inside AdCard's render on the landing screen.
+    const layout = getCardAdLayout(
+      {} as Parameters<typeof getCardAdLayout>[0],
+      78,
+    )
+
+    expect(layout.headline).toBe('')
+    expect(layout.description).toBe('')
+    expect(layout.ctaText).toBe('Learn more')
+    expect(layout.labelText).toBe('Sponsored')
+    expect(layout.descriptionLines).toBe(2)
+  })
+
+  test('falls back to Learn more when only the CTA is missing', () => {
+    const layout = getCardAdLayout(
+      { ...ad, cta: undefined } as unknown as Parameters<
+        typeof getCardAdLayout
+      >[0],
+      78,
+    )
+
+    expect(layout.ctaText).toBe('Learn more')
+    expect(layout.headline).toBe('Test every release before you ship')
+  })
+
+  test('truncates the headline to the interior width', () => {
+    const layout = getCardAdLayout(
+      {
+        ...ad,
+        title:
+          'A headline considerably longer than this narrow card could ever hold',
+      },
+      60,
+    )
+
+    expect(layout.headline.length).toBeLessThanOrEqual(60 - 8)
+    expect(layout.headline.endsWith('…')).toBe(true)
+  })
+})
+
+describe('card ad render', () => {
+  // getCardAdLayout being correct proves nothing on its own: the bug it fixes
+  // was that the JSX never referenced the title at all. This renders a real
+  // character frame, so a refactor that drops the headline row goes red here
+  // rather than shipping an advertiser a field that renders nowhere.
+  const ad = {
+    adText: 'Automate mobile UI testing with plain-English test steps.',
+    title: 'Test every release before you ship',
+    cta: 'Try free',
+    url: 'https://www.drizz.dev/ios',
+    favicon: '',
+    clickUrl: 'https://www.drizz.dev/ios?click=1',
+    impUrl: 'imp-1',
+  }
+
+  const renderCard = async (
+    overrides: Partial<typeof ad>,
+    width = 78,
+  ): Promise<string> => {
+    const setup = await createTestRenderer({ width, height: AD_CARD_HEIGHT })
+    const root = createRoot(setup.renderer)
+    flushSync(() => {
+      root.render(<AdCard ad={{ ...ad, ...overrides }} width={width} />)
+    })
+    try {
+      await setup.renderOnce()
+      return setup.captureCharFrame()
+    } finally {
+      flushSync(() => root.unmount())
+      setup.renderer.destroy()
+    }
+  }
+
+  test('draws the headline, the body, the CTA and the destination', async () => {
+    const frame = await renderCard({})
+
+    expect(frame).toContain('Test every release before you ship')
+    expect(frame).toContain('Automate mobile UI testing')
+    expect(frame).toContain('Try free')
+    expect(frame).toContain('drizz.dev')
+  })
+
+  test('still discloses itself as an ad', async () => {
+    expect(await renderCard({})).toContain('Ad')
+  })
+
+  test('does not print the headline twice when there is no CTA', async () => {
+    const frame = await renderCard({ cta: '' })
+
+    expect(frame).toContain('Test every release before you ship')
+    expect(frame).toContain('Learn more')
+  })
+
+  test('renders rather than throwing when the provider omits a CTA', async () => {
+    // The regression this guards: `ad.cta.trim()` threw inside render, and
+    // cli/src/components/error-boundary.tsx does not catch render errors, so
+    // one malformed Gravity creative took down the landing screen.
+    const frame = await renderCard({
+      cta: undefined,
+    } as Partial<typeof ad>)
+
+    expect(frame).toContain('Learn more')
+    expect(frame).toContain('Test every release before you ship')
+  })
+
+  test('renders a title-less ad without a blank first row', async () => {
+    const frame = await renderCard({ title: '' })
+
+    expect(frame).toContain('Automate mobile UI testing')
+    expect(frame).toContain('Ad')
+  })
+})
 
 describe('ad banner display label', () => {
   test('uses the display domain when the ad has a URL', () => {
@@ -69,7 +265,8 @@ describe('inline ad layout', () => {
   test('uses the full detail row when no destination domain is available', () => {
     const layout = getInlineAdLayout(
       {
-        adText: 'A Carbon ad whose tracked destination is intentionally hidden.',
+        adText:
+          'A Carbon ad whose tracked destination is intentionally hidden.',
         title: 'Example Sponsor',
         url: '',
       },