fix tests

Author: rsergeenko

redisinsight/ui/src/pages/browser/BrowserPage.test.tsx

@@ -5,7 +5,7 @@ import { useSelector } from 'react-redux'
 import { render, screen, fireEvent, mockedStore, cleanup, act, waitForEuiToolTipVisible } from 'uiSrc/utils/test-utils'
 import { KeyTypes } from 'uiSrc/constants'
 import { RootState } from 'uiSrc/slices/store'
-import { toggleBrowserFullScreen } from 'uiSrc/slices/browser/keys'
+import { setSelectedKeyRefreshDisabled, toggleBrowserFullScreen } from 'uiSrc/slices/browser/keys'
 import BrowserPage from './BrowserPage'
 import KeyList, { Props as KeyListProps } from './components/key-list/KeyList'
 
@@ -198,7 +198,7 @@ describe('KeyDetailsWrapper', () => {
     expect(queryByTestId('apply-btn')).toBeInTheDocument()
     expect(queryByTestId('apply-btn')).toBeDisabled()
 
-    expect(store.getActions()).toEqual([...afterRenderActions])
+    expect(store.getActions()).toEqual([...afterRenderActions, setSelectedKeyRefreshDisabled(true)])
 
     await act(async () => {
       fireEvent.mouseOver(screen.getByTestId('apply-btn'))
@@ -259,7 +259,7 @@ describe('KeyDetailsWrapper', () => {
     expect(queryByTestId('apply-btn')).toBeInTheDocument()
     expect(queryByTestId('apply-btn')).toBeDisabled()
 
-    expect(store.getActions()).toEqual([...afterRenderActions])
+    expect(store.getActions()).toEqual([...afterRenderActions, setSelectedKeyRefreshDisabled(true)])
   })
 
   it('Verify that user cannot save key value (List) with unprintable characters', () => {
@@ -313,7 +313,7 @@ describe('KeyDetailsWrapper', () => {
     expect(queryByTestId('apply-btn')).toBeInTheDocument()
     expect(queryByTestId('apply-btn')).toBeDisabled()
 
-    expect(store.getActions()).toEqual([...afterRenderActions])
+    expect(store.getActions()).toEqual([...afterRenderActions, setSelectedKeyRefreshDisabled(true)])
   })
 })
 

redisinsight/ui/src/pages/browser/modules/key-details/components/hash-details/hash-details-table/HashDetailsTable.spec.tsx

@@ -143,12 +143,14 @@ describe('HashDetailsTable', () => {
     })
   })
 
-  it('should disable refresh after click on edit', () => {
+  it('should disable refresh after click on edit', async () => {
     render(<HashDetailsTable {...instance(mockedProps)} />)
 
     const afterRenderActions = [...store.getActions()]
 
-    fireEvent.click(screen.getAllByTestId(/edit-hash-button/)[0])
+    await act(() => {
+      fireEvent.click(screen.getByTestId('edit-hash-button-2'))
+    })
 
     expect(store.getActions()).toEqual([
       ...afterRenderActions,

redisinsight/ui/src/pages/browser/modules/key-details/components/list-details/list-details-table/ListDetailsTable.spec.tsx

@@ -145,7 +145,7 @@ describe('ListDetailsTable', () => {
     const afterRenderActions = [...store.getActions()]
 
     await act(() => {
-      fireEvent.click(screen.getAllByTestId(/edit-list-button/)[0])
+      fireEvent.click(screen.getByTestId('edit-list-button-0'))
     })
 
     expect(store.getActions()).toEqual([

redisinsight/ui/src/pages/browser/modules/key-details/components/string-details/StringDetails.spec.tsx

@@ -1,8 +1,7 @@
 import React from 'react'
 import { instance, mock } from 'ts-mockito'
 import { cloneDeep } from 'lodash'
-import { fireEvent } from '@testing-library/react'
-import { cleanup, mockedStore, render, screen } from 'uiSrc/utils/test-utils'
+import { cleanup, mockedStore, render, screen, fireEvent, act } from 'uiSrc/utils/test-utils'
 import { stringDataSelector, stringSelector } from 'uiSrc/slices/browser/string'
 import { setSelectedKeyRefreshDisabled } from 'uiSrc/slices/browser/keys'
 import { Props, StringDetails } from './StringDetails'
@@ -98,11 +97,14 @@ describe('StringDetails', () => {
     expect(queryByTestId('edit-key-value-btn')).toBeInTheDocument()
   })
 
-  it('should disable refresh when editing', () => {
+  it('should disable refresh when editing', async () => {
     render(<StringDetails {...mockedProps} />)
     const afterRenderActions = [...store.getActions()]
 
-    fireEvent.click(screen.getByTestId(`${EDIT_VALUE_BTN_TEST_ID}`))
+    await act(() => {
+      fireEvent.click(screen.getByTestId(`${EDIT_VALUE_BTN_TEST_ID}`))
+    })
+
     expect(store.getActions()).toEqual([
       ...afterRenderActions,
       setSelectedKeyRefreshDisabled(true)

redisinsight/ui/src/pages/browser/modules/key-details/components/zset-details/zset-details-table/ZSetDetailsTable.spec.tsx

@@ -113,7 +113,7 @@ describe('ZSetDetailsTable', () => {
     const afterRenderActions = [...store.getActions()]
 
     await act(() => {
-      fireEvent.click(screen.getByTestId(/zset-edit-button-3/))
+      fireEvent.click(screen.getAllByTestId(/zset-edit-button/)[0])
     })
 
     expect(store.getActions()).toEqual([

redisinsight/ui/src/slices/browser/keys.ts

@@ -179,9 +179,9 @@ const keysSlice = createSlice({
       state.selectedKey = {
         ...state.selectedKey,
         loading: false,
+        isRefreshDisabled: false,
         data: {
           ...payload,
-          isRefreshDisabled: false,
           nameString: bufferToString(payload.name),
         },
       }

redisinsight/ui/src/slices/tests/browser/keys.spec.ts

@@ -383,6 +383,7 @@ describe('keys slice', () => {
         selectedKey: {
           ...initialState.selectedKey,
           loading: false,
+          isRefreshDisabled: false,
           data: {
             ...data,
           },