Merge pull request #1821 from ably/ECO-4890/fix-presence-hook

fix: add missing `ablyId` argument to  `useConnectionStateListener`

src/platform/react-hooks/src/hooks/usePresence.test.tsx

@@ -128,6 +128,30 @@ describe('usePresence', () => {
     });
   });
 
+  /** @nospec */
+  it('usePresence works without default client', async () => {
+    const updateListener = vi.fn();
+    ablyClient.channels.get(testChannelName).presence.subscribe('update', updateListener);
+
+    render(
+      <AblyProvider ablyId="otherClient" client={otherClient as unknown as Ably.RealtimeClient}>
+        <ChannelProvider channelName={testChannelName} ablyId="otherClient">
+          <UsePresenceComponentWithOtherClient />
+        </ChannelProvider>
+      </AblyProvider>,
+    );
+
+    await act(async () => {
+      const button = screen.getByText(/Update/i);
+      button.click();
+      await wait(2);
+    });
+
+    await waitFor(() => {
+      expect(updateListener).toHaveBeenCalledWith(expect.objectContaining({ data: 'baz' }));
+    });
+  });
+
   /** @nospec */
   it('handles channel errors', async () => {
     const onChannelError = vi.fn();
@@ -246,6 +270,22 @@ const UsePresenceComponentMultipleClients = () => {
   );
 };
 
+const UsePresenceComponentWithOtherClient = () => {
+  const { updateStatus } = usePresence({ channelName: testChannelName, ablyId: 'otherClient' }, 'bar');
+
+  return (
+    <>
+      <button
+        onClick={() => {
+          updateStatus('baz');
+        }}
+      >
+        Update
+      </button>
+    </>
+  );
+};
+
 interface UsePresenceStateErrorsComponentProps {
   onConnectionError?: (err: Ably.ErrorInfo) => unknown;
   onChannelError?: (err: Ably.ErrorInfo) => unknown;

src/platform/react-hooks/src/hooks/usePresence.ts

@@ -48,7 +48,7 @@ export function usePresence<T = any>(
   const [connectionState, setConnectionState] = useState(ably.connection.state);
   useConnectionStateListener((stateChange) => {
     setConnectionState(stateChange.current);
-  });
+  }, params.ablyId);
 
   // similar to connection states, we should only attempt to enter presence when in certain
   // channel states.