@linktr.ee/messaging-react 4.4.7 → 4.4.8-rc-1788251276

package/src/components/MessageAttachment/MessageAttachment.behavior.test.tsx

@@ -1,6 +1,6 @@
 import { describe, expect, it, vi } from 'vitest'
 
-import { fireEvent, renderWithProviders, screen } from '../../test/utils'
+import { userEvent, renderWithProviders, screen } from '../../test/utils'
 
 import { triggerDownload } from './_shared/triggerDownload'
 import { bubbleVariantForState, bubbleGroupPositionFromStream } from './types'
@@ -14,7 +14,8 @@ vi.mock('./_shared/triggerDownload', () => ({
 const mockedDownload = vi.mocked(triggerDownload)
 
 describe('MessageAttachment behavior', () => {
-  it('renders file rows with precedence, fallback names, and cancellation', () => {
+  it('renders file rows with precedence, fallback names, and cancellation', async () => {
+    const user = userEvent.setup()
     const onClick = vi.fn(() => false)
     const { rerender } = renderWithProviders(
       <MessageAttachment.File.Sent
@@ -32,14 +33,14 @@ describe('MessageAttachment behavior', () => {
     )
     expect(screen.getByText('first.pdf')).toBeInTheDocument()
     expect(screen.getByText('second.pdf')).toBeInTheDocument()
-    fireEvent.click(screen.getByRole('button', { name: 'Download first.pdf' }))
+    await user.click(screen.getByRole('button', { name: 'Download first.pdf' }))
     expect(onClick).toHaveBeenCalledWith(0)
     expect(mockedDownload).not.toHaveBeenCalled()
 
     rerender(
       <MessageAttachment.File.Sent src="https://cdn.example.com/fallback.txt" />
     )
-    fireEvent.click(
+    await user.click(
       screen.getByRole('button', { name: 'Download fallback.txt' })
     )
     expect(mockedDownload).toHaveBeenCalledWith(
@@ -48,10 +49,12 @@ describe('MessageAttachment behavior', () => {
     )
   })
 
-  it('renders no file when inputs are absent and supports composer dismissal', () => {
+  it('renders no file when inputs are absent and supports composer dismissal', async () => {
+    const user = userEvent.setup()
     const { container } = renderWithProviders(
       <MessageAttachment.File.Received />
     )
+    // eslint-disable-next-line testing-library/no-node-access -- the null-rendering DOM contract has no semantic query.
     expect(container.firstChild).toBeNull()
     const onDismiss = vi.fn()
     renderWithProviders(
@@ -60,11 +63,12 @@ describe('MessageAttachment behavior', () => {
         onDismiss={onDismiss}
       />
     )
-    fireEvent.click(screen.getByRole('button', { name: 'Dismiss attachment' }))
+    await user.click(screen.getByRole('button', { name: 'Dismiss attachment' }))
     expect(onDismiss).toHaveBeenCalledOnce()
   })
 
-  it('renders image stacks, forwards loading, and supports click cancellation', () => {
+  it('renders image stacks, forwards loading, and supports click cancellation', async () => {
+    const user = userEvent.setup()
     const onClick = vi.fn(() => false)
     renderWithProviders(
       <MessageAttachment.Image.Sent
@@ -79,12 +83,13 @@ describe('MessageAttachment behavior', () => {
     )
     expect(screen.getAllByRole('img')[0]).toHaveAttribute('loading', 'lazy')
     expect(screen.getAllByRole('img')).toHaveLength(2)
-    fireEvent.click(screen.getByRole('button', { name: 'Open image 2 of 2' }))
+    await user.click(screen.getByRole('button', { name: 'Open image 2 of 2' }))
     expect(onClick).toHaveBeenCalledWith(1)
     expect(screen.queryByTestId('image-viewer')).not.toHaveAttribute('open')
   })
 
-  it('opens images at the clicked index and suffixes a shared filename', () => {
+  it('opens images at the clicked index and suffixes a shared filename', async () => {
+    const user = userEvent.setup()
     renderWithProviders(
       <MessageAttachment.Image.Sent
         filename="photos.jpg"
@@ -94,7 +99,7 @@ describe('MessageAttachment behavior', () => {
         ]}
       />
     )
-    fireEvent.click(screen.getByRole('button', { name: 'Open image 2 of 2' }))
+    await user.click(screen.getByRole('button', { name: 'Open image 2 of 2' }))
     expect(screen.getByTestId('image-viewer')).toHaveAttribute(
       'aria-label',
       'photos.jpg (2)'
@@ -105,7 +110,8 @@ describe('MessageAttachment behavior', () => {
     )
   })
 
-  it('renders video posters and posterless first-frame previews', () => {
+  it('renders video posters and posterless first-frame previews', async () => {
+    const user = userEvent.setup()
     const onDismiss = vi.fn()
     const { rerender } = renderWithProviders(
       <MessageAttachment.Video.Composer
@@ -119,7 +125,7 @@ describe('MessageAttachment behavior', () => {
     expect(
       screen.getByRole('img', { name: 'Video 1 thumbnail' })
     ).toHaveAttribute('src', '/poster.jpg')
-    fireEvent.click(screen.getByRole('button', { name: 'Dismiss attachment' }))
+    await user.click(screen.getByRole('button', { name: 'Dismiss attachment' }))
     expect(onDismiss).toHaveBeenCalledOnce()
     rerender(
       <MessageAttachment.Video.Sent src="https://cdn.example.com/video.mp4" />
@@ -172,7 +178,8 @@ describe('MessageAttachment behavior', () => {
     expect(screen.queryByLabelText('Video 1 thumbnail')).not.toBeInTheDocument()
   })
 
-  it('supports PDF titles, download actions, dismiss, and viewer index', () => {
+  it('supports PDF titles, download actions, dismiss, and viewer index', async () => {
+    const user = userEvent.setup()
     const onDismiss = vi.fn()
     renderWithProviders(
       <MessageAttachment.Pdf.Composer
@@ -182,7 +189,7 @@ describe('MessageAttachment behavior', () => {
       />
     )
     expect(screen.getByText('Quarterly report')).toBeInTheDocument()
-    fireEvent.click(screen.getByRole('button', { name: 'Dismiss attachment' }))
+    await user.click(screen.getByRole('button', { name: 'Dismiss attachment' }))
     expect(onDismiss).toHaveBeenCalledOnce()
 
     renderWithProviders(
@@ -193,7 +200,7 @@ describe('MessageAttachment behavior', () => {
         ]}
       />
     )
-    fireEvent.click(screen.getByRole('button', { name: 'Open two.pdf' }))
+    await user.click(screen.getByRole('button', { name: 'Open two.pdf' }))
     expect(screen.getAllByTestId('pdf-viewer').at(-1)).toHaveAttribute(
       'aria-label',
       'two.pdf'