Merge pull request #1413 from RedisInsight/fe/feature/RI-3401_code-coverage

#RI-3401 - add tests, fix some paths to interfaces

Author: rsergeenko

redisinsight/ui/src/components/database-list-modules/DatabaseListModules.spec.tsx

@@ -2,12 +2,12 @@ import React from 'react'
 import { instance, mock } from 'ts-mockito'
 import { RedisDefaultModules, DATABASE_LIST_MODULES_TEXT } from 'uiSrc/slices/interfaces'
 import { fireEvent, render, act } from 'uiSrc/utils/test-utils'
-import { RedisModuleDto } from 'apiSrc/modules/instances/dto/database-instance.dto'
+import { AdditionalRedisModule } from 'apiSrc/modules/database/models/additional.redis.module'
 import DatabaseListModules, { Props } from './DatabaseListModules'
 
 const mockedProps = mock<Props>()
 
-const modulesMock: RedisModuleDto[] = [
+const modulesMock: AdditionalRedisModule[] = [
   { name: RedisDefaultModules.AI },
   { name: RedisDefaultModules.Bloom },
   { name: RedisDefaultModules.Gears },

redisinsight/ui/src/components/database-list-modules/DatabaseListModules.tsx

@@ -28,13 +28,13 @@ import RedisTimeSeriesLight from 'uiSrc/assets/img/modules/RedisTimeSeriesLight.
 import RedisTimeSeriesDark from 'uiSrc/assets/img/modules/RedisTimeSeriesDark.svg'
 import UnknownLight from 'uiSrc/assets/img/modules/UnknownLight.svg'
 import UnknownDark from 'uiSrc/assets/img/modules/UnknownDark.svg'
-import { RedisModuleDto } from 'apiSrc/modules/instances/dto/database-instance.dto'
+import { AdditionalRedisModule } from 'apiSrc/modules/database/models/additional.redis.module'
 
 import styles from './styles.module.scss'
 
 export interface Props {
   content?: JSX.Element
-  modules: RedisModuleDto[]
+  modules: AdditionalRedisModule[]
   inCircle?: boolean
   dark?: boolean
   highlight?: boolean

redisinsight/ui/src/components/database-overview/DatabaseOverview.tsx

@@ -13,7 +13,7 @@ import RediStackLightMin from 'uiSrc/assets/img/modules/redistack/RediStackLight
 import RediStackLightLogo from 'uiSrc/assets/img/modules/redistack/RedisStackLogoLight.svg'
 import RediStackDarkLogo from 'uiSrc/assets/img/modules/redistack/RedisStackLogoDark.svg'
 
-import { RedisModuleDto } from 'apiSrc/modules/instances/dto/database-instance.dto'
+import { AdditionalRedisModule } from 'apiSrc/modules/database/models/additional.redis.module'
 import { getResolutionLimits } from './utils/resolutionHelper'
 import { IMetric } from './components/OverviewMetrics'
 import DatabaseListModules from '../database-list-modules/DatabaseListModules'
@@ -23,7 +23,7 @@ import styles from './styles.module.scss'
 interface Props {
   windowDimensions: number
   metrics?: Array<IMetric>
-  modules?: Array<RedisModuleDto>
+  modules?: Array<AdditionalRedisModule>
   isRediStack?: boolean
 }
 

redisinsight/ui/src/components/database-overview/components/MoreInfoPopover/MoreInfoPopover.tsx

@@ -5,7 +5,7 @@ import parse from 'html-react-parser'
 import { DATABASE_LIST_MODULES_TEXT } from 'uiSrc/slices/interfaces'
 
 import { getModule, truncateText } from 'uiSrc/utils'
-import { RedisModuleDto } from 'apiSrc/modules/instances/dto/database-instance.dto'
+import { AdditionalRedisModule } from 'apiSrc/modules/database/models/additional.redis.module'
 
 import { IMetric } from '../OverviewMetrics'
 
@@ -16,7 +16,7 @@ const ModulesInfoText = 'More information about Redis modules can be found <a cl
 
 interface IProps {
   metrics: Array<IMetric>,
-  modules: Array<RedisModuleDto>
+  modules: Array<AdditionalRedisModule>
 }
 
 const MoreInfoPopover = ({ metrics, modules }: IProps) => {

redisinsight/ui/src/components/global-subscriptions/CommonAppSubscription/CommonAppSubscription.spec.tsx

@@ -3,13 +3,13 @@ import { cloneDeep } from 'lodash'
 import React from 'react'
 import MockedSocket from 'socket.io-mock'
 import socketIO from 'socket.io-client'
-import { NotificationsDto } from 'apiSrc/modules/notification/dto'
 import { NotificationEvent } from 'uiSrc/constants/notifications'
 import { setNewNotificationReceived, setLastReceivedNotification } from 'uiSrc/slices/app/notifications'
 import { setIsConnected } from 'uiSrc/slices/app/socket-connection'
 import { NotificationType } from 'uiSrc/slices/interfaces'
 import { cleanup, mockedStore, render } from 'uiSrc/utils/test-utils'
 import { SocketEvent } from 'uiSrc/constants'
+import { NotificationsDto } from 'apiSrc/modules/notification/dto'
 
 import CommonAppSubscription from './CommonAppSubscription'
 

redisinsight/ui/src/components/json-viewer/JSONViewer.spec.tsx

@@ -0,0 +1,21 @@
+import React from 'react'
+import { render, screen } from 'uiSrc/utils/test-utils'
+
+import JSONViewer from './JSONViewer'
+
+describe('JSONViewer', () => {
+  it('should render proper json', () => {
+    const jsx = JSONViewer({ value: JSON.stringify({}) })
+    render(jsx.value as React.ReactElement)
+
+    expect(jsx.isValid).toBeTruthy()
+    expect(screen.queryByTestId('value-as-json')).toBeInTheDocument()
+  })
+
+  it('should not render invalid json', () => {
+    const jsx = JSONViewer({ value: 'zxc' })
+
+    expect(jsx.value).toEqual('zxc')
+    expect(jsx.isValid).toBeFalsy()
+  })
+})

redisinsight/ui/src/components/json-viewer/JSONViewer.tsx

@@ -18,7 +18,7 @@ const JSONViewer = (props: Props) => {
 
     return {
       value: (
-        <div className={cx(styles.jsonViewer, { [styles.collapsed]: !expanded })} data-tesid="value-as-json">
+        <div className={cx(styles.jsonViewer, { [styles.collapsed]: !expanded })} data-testid="value-as-json">
           <JSONPretty json={value} space={space} />
         </div>
       ),

redisinsight/ui/src/components/notifications/components/EncryptionErrorContent.spec.tsx

@@ -1,6 +1,6 @@
 import { cloneDeep } from 'lodash'
 import React from 'react'
-import { cleanup, mockedStore, render } from 'uiSrc/utils/test-utils'
+import { cleanup, mockedStore, render, fireEvent, screen } from 'uiSrc/utils/test-utils'
 import EncryptionErrorContent from './EncryptionErrorContent'
 
 let store: typeof mockedStore
@@ -22,4 +22,12 @@ describe('EncryptionErrorContent', () => {
   it('should render', () => {
     expect(render(<EncryptionErrorContent />)).toBeTruthy()
   })
+
+  it('should call onClose', () => {
+    const onClose = jest.fn()
+    render(<EncryptionErrorContent onClose={onClose} />)
+    fireEvent.click(screen.getByTestId('toast-action-btn'))
+
+    expect(onClose).toBeCalled()
+  })
 })

redisinsight/ui/src/electron/utils/ipcCheckUpdates.ts

@@ -4,8 +4,8 @@ import { omit } from 'lodash'
 import { sendEventTelemetry, TelemetryEvent } from 'uiSrc/telemetry'
 import { setElectronInfo, setReleaseNotesViewed } from 'uiSrc/slices/app/info'
 import { addMessageNotification } from 'uiSrc/slices/app/notifications'
-import { GetServerInfoResponse } from 'apiSrc/dto/server.dto'
 import successMessages from 'uiSrc/components/notifications/success-messages'
+import { GetServerInfoResponse } from 'apiSrc/modules/server/dto/server.dto'
 import { ElectronStorageItem, IpcEvent } from '../constants'
 
 export const ipcCheckUpdates = async (serverInfo: GetServerInfoResponse, dispatch: Dispatch<any>) => {

redisinsight/ui/src/mocks/handlers/analytics/clusterDetailsHandlers.ts

@@ -1,9 +1,9 @@
-import { DatabaseInstanceResponse } from 'apiSrc/modules/instances/dto/database-instance.dto'
 import { rest, RestHandler } from 'msw'
-import { ClusterDetails, HealthStatus, NodeRole } from 'apiSrc/modules/cluster-monitor/models'
 import { ApiEndpoints } from 'uiSrc/constants'
 import { getUrl } from 'uiSrc/utils'
 import { getMswURL } from 'uiSrc/utils/test-utils'
+import { ClusterDetails, HealthStatus, NodeRole } from 'apiSrc/modules/cluster-monitor/models'
+import { Database as DatabaseInstanceResponse } from 'apiSrc/modules/database/models/database'
 
 export const INSTANCE_ID_MOCK = 'instanceId'