@linktr.ee/messaging-react 4.0.0-rc-1786383858 → 4.0.0-rc-1786386505

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@linktr.ee/messaging-react",
-  "version": "4.0.0-rc-1786383858",
+  "version": "4.0.0-rc-1786386505",
   "description": "React messaging components built on messaging-core for web applications",
   "type": "module",
   "main": "dist/index.cjs",
@@ -50,7 +50,7 @@
     "chromatic:local": "yarn storybook:build && chromatic"
   },
   "dependencies": {
-    "@linktr.ee/messaging-core": "2.4.0-rc-1786383858",
+    "@linktr.ee/messaging-core": "2.4.0-rc-1786386505",
     "@linktr.ee/messaging-taxonomy": "^0.9.0",
     "@phosphor-icons/react": "^2.1.10"
   },

package/src/components/CustomMessage/BroadcastContext.test.ts

@@ -1,6 +1,12 @@
 import { describe, expect, it, vi } from 'vitest'
 
-import { buildBroadcastsByOutboundId } from './BroadcastContext'
+import { renderHook } from '../../test/utils'
+import type { BroadcastResolver } from '../../types'
+
+import {
+  buildBroadcastsByOutboundId,
+  useBroadcastsByOutboundId,
+} from './BroadcastContext'
 
 describe('buildBroadcastsByOutboundId', () => {
   it('returns undefined when no resolver is provided', () => {
@@ -12,6 +18,16 @@ describe('buildBroadcastsByOutboundId', () => {
     ).toBeUndefined()
   })
 
+  it('returns undefined without invoking the resolver when there are no messages', () => {
+    const resolveBroadcast = vi.fn()
+
+    expect(
+      buildBroadcastsByOutboundId(undefined, resolveBroadcast)
+    ).toBeUndefined()
+    expect(buildBroadcastsByOutboundId([], resolveBroadcast)).toBeUndefined()
+    expect(resolveBroadcast).not.toHaveBeenCalled()
+  })
+
   it('resolves each unique outbound id once', () => {
     const resolveBroadcast = vi.fn((outboundId: string) => ({
       id: `broadcast-for-${outboundId}`,
@@ -50,3 +66,75 @@ describe('buildBroadcastsByOutboundId', () => {
     ).toBeUndefined()
   })
 })
+
+describe('useBroadcastsByOutboundId', () => {
+  const messages = [{ metadata: { outbound_id: 'outbound-1' } }]
+
+  const renderWithResolver = (resolver: BroadcastResolver) =>
+    renderHook(
+      ({ resolveBroadcast }: { resolveBroadcast: BroadcastResolver }) =>
+        useBroadcastsByOutboundId(messages, resolveBroadcast),
+      { initialProps: { resolveBroadcast: resolver } }
+    )
+
+  it('publishes the new name when a broadcast is renamed under the same outbound id', () => {
+    const { result, rerender } = renderWithResolver(() => ({
+      id: 'broadcast-1',
+      name: 'July 9th Class absentee',
+    }))
+
+    expect(result.current).toEqual({
+      'outbound-1': { id: 'broadcast-1', name: 'July 9th Class absentee' },
+    })
+
+    rerender({
+      resolveBroadcast: () => ({
+        id: 'broadcast-1',
+        name: 'July 9th Class waitlist',
+      }),
+    })
+
+    expect(result.current).toEqual({
+      'outbound-1': { id: 'broadcast-1', name: 'July 9th Class waitlist' },
+    })
+  })
+
+  it('publishes the new broadcast id when an outbound record repoints to a same-named broadcast', () => {
+    const { result, rerender } = renderWithResolver(() => ({
+      id: 'broadcast-1',
+      name: 'July 9th Class absentee',
+    }))
+
+    expect(result.current).toEqual({
+      'outbound-1': { id: 'broadcast-1', name: 'July 9th Class absentee' },
+    })
+
+    rerender({
+      resolveBroadcast: () => ({
+        id: 'broadcast-2',
+        name: 'July 9th Class absentee',
+      }),
+    })
+
+    expect(result.current).toEqual({
+      'outbound-1': { id: 'broadcast-2', name: 'July 9th Class absentee' },
+    })
+  })
+
+  it('keeps the published map referentially stable when the resolved values are unchanged', () => {
+    const { result, rerender } = renderWithResolver(() => ({
+      id: 'broadcast-1',
+      name: 'July 9th Class absentee',
+    }))
+    const firstPublished = result.current
+
+    rerender({
+      resolveBroadcast: () => ({
+        id: 'broadcast-1',
+        name: 'July 9th Class absentee',
+      }),
+    })
+
+    expect(result.current).toBe(firstPublished)
+  })
+})