fix: prevent querying thread draft when drafts are disabled

Author: MartinCupela

src/components/MessageInput/MessageInput.tsx

@@ -111,7 +111,12 @@ const MessageInputProvider = (props: PropsWithChildren<MessageInputProps>) => {
 
   useEffect(() => {
     const threadId = messageComposer.threadId;
-    if (!threadId || !messageComposer.channel || !messageComposer.compositionIsEmpty)
+    if (
+      !threadId ||
+      !messageComposer.channel ||
+      !messageComposer.compositionIsEmpty ||
+      !messageComposer.config.drafts.enabled
+    )
       return;
     // get draft data for legacy thead composer
     messageComposer.channel.getDraft({ parent_id: threadId }).then(({ draft }) => {

src/components/MessageInput/__tests__/ThreadMessageInput.test.js

@@ -197,4 +197,56 @@ describe('MessageInput in Thread', () => {
     });
     expect(screen.getByLabelText('Also send in channel')).toBeInTheDocument();
   });
+
+  describe('draft', () => {
+    it('is queried when drafts are enabled', async () => {
+      const { customChannel, customClient, getDraftSpy } = await setup();
+      await act(() => {
+        customClient.setMessageComposerSetupFunction(({ composer }) => {
+          composer.updateConfig({ drafts: { enabled: true } });
+        });
+      });
+      await renderComponent({
+        customChannel,
+        customClient,
+      });
+      expect(getDraftSpy).toHaveBeenCalledTimes(1);
+    });
+    it('prevents querying if composition is not empty', async () => {
+      const { customChannel, customClient, getDraftSpy } = await setup();
+      await act(() => {
+        customClient.setMessageComposerSetupFunction(({ composer }) => {
+          composer.updateConfig({ drafts: { enabled: true } });
+          composer.textComposer.setText('abc');
+        });
+      });
+      await renderComponent({
+        customChannel,
+        customClient,
+      });
+      expect(getDraftSpy).not.toHaveBeenCalled();
+    });
+    it('prevents querying if not rendered inside a thread', async () => {
+      const { customChannel, customClient, getDraftSpy } = await setup();
+      await act(() => {
+        customClient.setMessageComposerSetupFunction(({ composer }) => {
+          composer.updateConfig({ drafts: { enabled: true } });
+          composer.compositionContext = customChannel;
+        });
+      });
+      await renderComponent({
+        customChannel,
+        customClient,
+      });
+      expect(getDraftSpy).not.toHaveBeenCalled();
+    });
+    it('prevents querying if drafts are disabled (default)', async () => {
+      const { customChannel, customClient, getDraftSpy } = await setup();
+      await renderComponent({
+        customChannel,
+        customClient,
+      });
+      expect(getDraftSpy).not.toHaveBeenCalled();
+    });
+  });
 });