@linktr.ee/messaging-react 4.4.3-rc-1788180473 → 4.4.3-rc-1788180953

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@linktr.ee/messaging-react",
-  "version": "4.4.3-rc-1788180473",
+  "version": "4.4.3-rc-1788180953",
   "description": "React messaging components built on messaging-core for web applications",
   "type": "module",
   "main": "dist/index.cjs",
@@ -50,8 +50,8 @@
     "chromatic:local": "yarn storybook:build && chromatic"
   },
   "dependencies": {
-    "@linktr.ee/messaging-core": "2.6.0-rc-1788180473",
-    "@linktr.ee/messaging-taxonomy": "0.11.0-rc-1788180473",
+    "@linktr.ee/messaging-core": "2.6.0-rc-1788180953",
+    "@linktr.ee/messaging-taxonomy": "0.11.0-rc-1788180953",
     "@phosphor-icons/react": "^2.1.10"
   },
   "devDependencies": {

package/src/components/ChannelView.outbound-resolution.integration.test.tsx

@@ -1,4 +1,4 @@
-import { act, render, screen } from '@testing-library/react'
+import { render, screen, waitFor } from '@testing-library/react'
 import React, { useEffect } from 'react'
 import { Channel as StreamChannel, StreamChat } from 'stream-chat'
 import { Channel, Chat } from 'stream-chat-react'
@@ -75,42 +75,37 @@ describe('ChannelView outbound resolution', () => {
       <RowProbe>{messageNode}</RowProbe>
     )
 
-    let rerender!: ReturnType<typeof render>['rerender']
-    await act(async () => {
-      rerender = render(
-        <Chat client={client}>
-          <Channel channel={channel as StreamChannel}>
-            <ChannelView
-              channel={channel as StreamChannel}
-              resolveOutbound={firstResolver}
-              renderMessage={renderMessage}
-            />
-          </Channel>
-        </Chat>
-      ).rerender
-    })
+    const { rerender } = render(
+      <Chat client={client}>
+        <Channel channel={channel as StreamChannel}>
+          <ChannelView
+            channel={channel as StreamChannel}
+            resolveOutbound={firstResolver}
+            renderMessage={renderMessage}
+          />
+        </Channel>
+      </Chat>
+    )
 
+    await waitFor(() => expect(mounts).toHaveBeenCalledOnce())
     expect(
       screen.queryByTestId('sent-message-outbound-status')
     ).not.toBeInTheDocument()
-    expect(mounts).toHaveBeenCalledOnce()
 
-    act(() => {
-      rerender(
-        <Chat client={client}>
-          <Channel channel={channel as StreamChannel}>
-            <ChannelView
-              channel={channel as StreamChannel}
-              resolveOutbound={secondResolver}
-              renderMessage={renderMessage}
-            />
-          </Channel>
-        </Chat>
-      )
-    })
+    rerender(
+      <Chat client={client}>
+        <Channel channel={channel as StreamChannel}>
+          <ChannelView
+            channel={channel as StreamChannel}
+            resolveOutbound={secondResolver}
+            renderMessage={renderMessage}
+          />
+        </Channel>
+      </Chat>
+    )
 
     expect(
-      screen.getByTestId('sent-message-outbound-status')
+      await screen.findByTestId('sent-message-outbound-status')
     ).toHaveTextContent('July 9th Class absentee')
     expect(mounts).toHaveBeenCalledOnce()
     expect(unmounts).not.toHaveBeenCalled()

package/src/providers/MessagingProvider.test.tsx

@@ -157,21 +157,20 @@ describe('MessagingProvider', () => {
     // Re-render with a NEW inline serviceConfig object (same apiKey). The
     // service is created once per client lifetime, so a serviceConfig change
     // neither recreates the service nor tears down the live connection.
-    await act(async () => {
-      rerender(
-        <MessagingProvider
-          apiKey="mock-api-key"
-          user={user}
-          serviceConfig={{
-            fetchToken: async () => 'token',
-            createChannel: async () => ({ channelId: 'ch-2' }),
-          }}
-        >
-          <div />
-        </MessagingProvider>
-      )
-    })
+    rerender(
+      <MessagingProvider
+        apiKey="mock-api-key"
+        user={user}
+        serviceConfig={{
+          fetchToken: async () => 'token',
+          createChannel: async () => ({ channelId: 'ch-2' }),
+        }}
+      >
+        <div />
+      </MessagingProvider>
+    )
 
+    await waitFor(() => expect(connectUser).toHaveBeenCalledTimes(1))
     expect(disconnectUser).not.toHaveBeenCalled()
   })
 
@@ -203,19 +202,18 @@ describe('MessagingProvider', () => {
     // Toggling `debug` changes the internal debugLog identity. The client
     // cleanup must stay keyed on the client's lifetime, not on debugLog, so the
     // live connection is not torn down for a logging change.
-    await act(async () => {
-      rerender(
-        <MessagingProvider
-          apiKey="mock-api-key"
-          user={user}
-          serviceConfig={serviceConfig}
-          debug={true}
-        >
-          <div />
-        </MessagingProvider>
-      )
-    })
+    rerender(
+      <MessagingProvider
+        apiKey="mock-api-key"
+        user={user}
+        serviceConfig={serviceConfig}
+        debug={true}
+      >
+        <div />
+      </MessagingProvider>
+    )
 
+    await waitFor(() => expect(connectUser).toHaveBeenCalledTimes(1))
     expect(disconnectUser).not.toHaveBeenCalled()
   })
 
@@ -246,21 +244,19 @@ describe('MessagingProvider', () => {
     await waitFor(() => expect(connectUser).toHaveBeenCalledTimes(1))
     expect(connectUser).toHaveBeenLastCalledWith(guestUser)
 
-    await act(async () => {
-      rerender(
-        <MessagingProvider
-          key="authed"
-          apiKey="mock-api-key"
-          user={authedUser}
-          serviceConfig={{
-            fetchToken: async () => 'token',
-            createChannel: async () => ({ channelId: 'ch-1' }),
-          }}
-        >
-          <div />
-        </MessagingProvider>
-      )
-    })
+    rerender(
+      <MessagingProvider
+        key="authed"
+        apiKey="mock-api-key"
+        user={authedUser}
+        serviceConfig={{
+          fetchToken: async () => 'token',
+          createChannel: async () => ({ channelId: 'ch-1' }),
+        }}
+      >
+        <div />
+      </MessagingProvider>
+    )
 
     await waitFor(() => expect(disconnectUser).toHaveBeenCalled())
     await waitFor(() => expect(connectUser).toHaveBeenCalledTimes(2))