浏览代码

Non-serializable warning (#2921)

* fixed non-serializable values in selectors.spec.ts, navigation warning in Table.spec.tsx,

* added await for userEvent in Table.spec.tsx,

Co-authored-by: davitbejanyan <dbejanyan@provectus.com>
David 2 年之前
父节点
当前提交
97aa114696

+ 4 - 4
kafka-ui-react-app/src/components/common/NewTable/__test__/Table.spec.tsx

@@ -144,11 +144,11 @@ describe('Table', () => {
       expect(screen.getByRole('link', { name: 'lorem' })).toBeInTheDocument();
     });
 
-    it('link click stops propagation', () => {
+    it('link click stops propagation', async () => {
       const onRowClick = jest.fn();
       renderComponent({ onRowClick });
       const link = screen.getByRole('link', { name: 'lorem' });
-      userEvent.click(link);
+      await userEvent.click(link);
       expect(onRowClick).not.toHaveBeenCalled();
     });
   });
@@ -273,9 +273,9 @@ describe('Table', () => {
       const th = screen.getByRole('columnheader', { name: 'Text' });
       expect(th).toBeInTheDocument();
 
-      let rows = [];
+      let rows = screen.getAllByRole('row');
       // Check initial sort order by text column is descending
-      rows = screen.getAllByRole('row');
+
       expect(rows[4].textContent?.indexOf('dolor')).toBeGreaterThan(-1);
       expect(rows[3].textContent?.indexOf('ipsum')).toBeGreaterThan(-1);
       expect(rows[2].textContent?.indexOf('lorem')).toBeGreaterThan(-1);

+ 10 - 2
kafka-ui-react-app/src/redux/reducers/topicMessages/__test__/selectors.spec.ts

@@ -9,6 +9,10 @@ import {
 
 import { topicMessagePayload, topicMessagesMetaPayload } from './fixtures';
 
+const newTopicMessagePayload = {
+  ...topicMessagePayload,
+  timestamp: topicMessagePayload.timestamp.toString(),
+};
 describe('TopicMessages selectors', () => {
   describe('Initial state', () => {
     it('returns empty message array', () => {
@@ -28,14 +32,18 @@ describe('TopicMessages selectors', () => {
 
   describe('state', () => {
     beforeAll(() => {
-      store.dispatch(addTopicMessage({ message: topicMessagePayload }));
+      store.dispatch(
+        addTopicMessage({
+          message: newTopicMessagePayload,
+        })
+      );
       store.dispatch(updateTopicMessagesPhase('consuming'));
       store.dispatch(updateTopicMessagesMeta(topicMessagesMetaPayload));
     });
 
     it('returns messages', () => {
       expect(selectors.getTopicMessges(store.getState())).toEqual([
-        topicMessagePayload,
+        newTopicMessagePayload,
       ]);
     });