Merge pull request #3664 from RedisInsight/bugfix/release-small-fixes

Bugfix/release small fixes

Author: mariasergeenko

redisinsight/ui/src/pages/rdi/statistics/StatisticsPage.spec.tsx

@@ -161,7 +161,7 @@ describe('StatisticsPage', () => {
     })
   })
 
-  it('should call proper telemetry event when refresh is clicked for data streams section', () => {
+  xit('should call proper telemetry event when refresh is clicked for data streams section', () => {
     render(<StatisticsPage />)
 
     fireEvent.click(screen.getByTestId('data-streams-refresh-btn'))
@@ -175,7 +175,7 @@ describe('StatisticsPage', () => {
     })
   })
 
-  it('should call proper telemetry event when refresh is clicked for clients section', () => {
+  xit('should call proper telemetry event when refresh is clicked for clients section', () => {
     render(<StatisticsPage />)
 
     fireEvent.click(screen.getByTestId('clients-refresh-btn'))
@@ -228,7 +228,7 @@ describe('StatisticsPage', () => {
     })
   })
 
-  it('should call proper telemetry event when auto refresh is enabled for data streams section', async () => {
+  xit('should call proper telemetry event when auto refresh is enabled for data streams section', async () => {
     render(<StatisticsPage />)
 
     const testid = 'data-streams'
@@ -247,7 +247,7 @@ describe('StatisticsPage', () => {
     })
   })
 
-  it('should call proper telemetry event when auto refresh is disabled for data streams section', async () => {
+  xit('should call proper telemetry event when auto refresh is disabled for data streams section', async () => {
     render(<StatisticsPage />)
 
     const testid = 'data-streams'
@@ -267,7 +267,7 @@ describe('StatisticsPage', () => {
     })
   })
 
-  it('should call proper telemetry event when auto refresh is enabled for clients section', async () => {
+  xit('should call proper telemetry event when auto refresh is enabled for clients section', async () => {
     render(<StatisticsPage />)
 
     const testid = 'clients'
@@ -286,7 +286,7 @@ describe('StatisticsPage', () => {
     })
   })
 
-  it('should call proper telemetry event when auto refresh is disabled for clients section', async () => {
+  xit('should call proper telemetry event when auto refresh is disabled for clients section', async () => {
     render(<StatisticsPage />)
 
     const testid = 'clients'

redisinsight/ui/src/pages/rdi/statistics/clients/Clients.tsx

@@ -70,6 +70,7 @@ const Clients = ({ data, loading, onRefresh, onRefreshClicked, onChangeAutoRefre
       <Accordion
         id="clients"
         title="Clients"
+        hideAutoRefresh
         loading={loading}
         onRefresh={onRefresh}
         onRefreshClicked={onRefreshClicked}

redisinsight/ui/src/pages/rdi/statistics/data-streams/DataStreams.tsx

@@ -101,6 +101,7 @@ const DataStreams = ({ data, loading, onRefresh, onRefreshClicked, onChangeAutoR
       <Accordion
         id="data-streams"
         title="Data streams overview"
+        hideAutoRefresh
         loading={loading}
         onRefresh={onRefresh}
         onRefreshClicked={onRefreshClicked}

redisinsight/ui/src/pages/rdi/statistics/target-connections/TargetConnections.tsx

@@ -1,7 +1,7 @@
 import { EuiBasicTableColumn, EuiIcon, EuiToolTip } from '@elastic/eui'
 import React from 'react'
 
-import { IConnections } from 'uiSrc/slices/interfaces'
+import { IConnections, StatisticsConnectionStatus } from 'uiSrc/slices/interfaces'
 import { formatLongName } from 'uiSrc/utils'
 import Accordion from '../components/accordion'
 import Panel from '../components/panel'
@@ -22,7 +22,7 @@ const columns: EuiBasicTableColumn<ConnectionData>[] = [
     field: 'status',
     width: '80px',
     render: (status: string) =>
-      (status === 'good' ? (
+      (status === StatisticsConnectionStatus.connected ? (
         <EuiIcon type="dot" color="var(--buttonSuccessColor)" />
       ) : (
         <EuiIcon type="alert" color="danger" />

redisinsight/ui/src/slices/interfaces/rdi.ts

@@ -51,11 +51,16 @@ export interface IRdiPipelineStrategies {
   data: IRdiPipelineStrategy[]
 }
 
+export enum StatisticsConnectionStatus {
+  notYetUsed = 'not yet used',
+  connected = 'connected'
+}
+
 export interface IConnections {
   [key: string]: {
     host: string
     port: number
-    status: string
+    status: StatisticsConnectionStatus
     type: string
     database: string
     user: string