Fixing the tests and issue with react-test-renderer

Author: vishalnarkhede

jest-setup.js

@@ -1,5 +1,19 @@
 import { registerNativeHandlers } from './src/native';
 
+// eslint-disable-next-line no-underscore-dangle
+const _consoleE = console.error;
+console.error = (e) => {
+  // For some reason following error gets rased from waitFor function.
+  // Source of the error is in react-test-renderer, although cause of the
+  // issue is unknown. Until we figure out the proper way to fix this issue,
+  // following hack should work. We just suppress this particular error.
+  // It should be safe since the error is only raised in dev environment
+  // https://github.com/facebook/react/blob/b683c07ccce340b9d687683d5dd7347a4c866787/packages/react-dom/src/test-utils/ReactTestUtilsAct.js#L121
+  if (e.indexOf('You called act(async () => ...) without await') === -1) {
+    _consoleE(e);
+  }
+};
+
 registerNativeHandlers({
   NetInfo: {
     addEventListener: () => {},

src/components/Avatar/__tests__/Avatar.test.js

@@ -1,5 +1,5 @@
 import React from 'react';
-import { getNodeText, render, waitFor } from '@testing-library/react-native';
+import { render, waitFor } from '@testing-library/react-native';
 
 import Avatar from '../Avatar';
 
@@ -49,7 +49,7 @@ describe('Avatar', () => {
     await waitFor(() => {
       expect(queryByTestId('avatar-image')).toBeFalsy();
       expect(queryByTestId('avatar-text')).toBeTruthy();
-      expect(getNodeText(getByTestId('avatar-text'))).toBe('TU');
+      expect(getByTestId('avatar-text')).toHaveTextContent('TU');
     });
   });
 
@@ -61,7 +61,7 @@ describe('Avatar', () => {
     await waitFor(() => {
       expect(queryByTestId('avatar-image')).toBeFalsy();
       expect(queryByTestId('avatar-text')).toBeTruthy();
-      expect(getNodeText(getByTestId('avatar-text'))).toBe('TU');
+      expect(getByTestId('avatar-text')).toHaveTextContent('TU');
     });
   });
 });

src/components/ChannelPreview/__tests__/ChannelPreview.test.js

@@ -1,10 +1,5 @@
 import React from 'react';
-import {
-  act,
-  getNodeText,
-  render,
-  waitFor,
-} from '@testing-library/react-native';
+import { act, render, waitFor } from '@testing-library/react-native';
 
 import {
   dispatchMessageDeletedEvent,
@@ -87,13 +82,15 @@ describe('ChannelPreview', () => {
     const { getByTestId } = render(getComponent());
 
     await waitFor(() => getByTestId('channel-id'));
-    expect(getNodeText(getByTestId('unread-count'))).toBe('20');
-    await act(async () => {
-      await dispatchMessageReadEvent(chatClient, clientUser, channel);
+
+    expect(getByTestId('unread-count')).toHaveTextContent('20');
+
+    act(() => {
+      dispatchMessageReadEvent(chatClient, clientUser, channel);
     });
 
     await waitFor(() => {
-      expect(getNodeText(getByTestId('unread-count'))).toBe('0');
+      expect(getByTestId('unread-count')).toHaveTextContent('0');
     });
   });
 
@@ -117,12 +114,12 @@ describe('ChannelPreview', () => {
         user: clientUser,
       });
 
-      await act(async () => {
-        await dispatcher(chatClient, message, channel);
+      act(() => {
+        dispatcher(chatClient, message, channel);
       });
 
       await waitFor(() => {
-        expect(getNodeText(getByTestId('last-event-message'))).toBe(
+        expect(getByTestId('last-event-message')).toHaveTextContent(
           message.text,
         );
       });
@@ -143,12 +140,12 @@ describe('ChannelPreview', () => {
 
     channel.countUnread = () => 10;
 
-    await act(async () => {
-      await dispatchMessageNewEvent(chatClient, message, channel);
+    act(() => {
+      dispatchMessageNewEvent(chatClient, message, channel);
     });
 
     await waitFor(() => {
-      expect(getNodeText(getByTestId('unread-count'))).toBe('10');
+      expect(getByTestId('unread-count')).toHaveTextContent('10');
     });
   });
 });

src/utils/__tests__/Streami18n.test.js

@@ -57,7 +57,7 @@ const customDayjsLocaleConfig = {
 };
 
 describe('Streami18n instance - default', () => {
-  const streami18nOptions = {};
+  const streami18nOptions = { logger: () => {} };
   const streami18n = new Streami18n(streami18nOptions);
 
   it('should provide default english translator', async () => {