RI-6231 - eslint fixes

Author: KIvanow

redisinsight/ui/src/pages/browser/modules/key-details/components/rejson-details/rejson-dynamic-types/RejsonDynamicTypes.spec.tsx

@@ -6,7 +6,7 @@ import RejsonDynamicTypes from './RejsonDynamicTypes'
 
 const mockedProps = mock<DynamicTypesProps>()
 
-const mockedDownloadedSimpleArray = "[1, 2, 3]"
+const mockedDownloadedSimpleArray = '[1, 2, 3]'
 
 describe('RejsonDynamicTypes Component', () => {
   it('renders correctly simple downloaded JSON', () => {

redisinsight/ui/src/pages/browser/modules/key-details/components/rejson-details/rejson-scalar/RejsonScalar.spec.tsx

@@ -103,17 +103,17 @@ describe('JSONScalar', () => {
   })
 
   it('should render BigInt value when root', () => {
-    render(<RejsonScalar 
+    render(<RejsonScalar
       {...instance(mockedProps)}
-      isRoot={true}
+      isRoot
       value={BigInt('1188950299261208742')}
     />)
 
     expect(screen.getByText('1188950299261208742')).toBeInTheDocument()
   })
 
   it('should render BigInt value when not root', () => {
-    render(<RejsonScalar 
+    render(<RejsonScalar
       {...instance(mockedProps)}
       isRoot={false}
       value={BigInt('1188950299261208742')}
@@ -123,19 +123,19 @@ describe('JSONScalar', () => {
   })
 
   it('should render regular number without n suffix', () => {
-    render(<RejsonScalar 
+    render(<RejsonScalar
       {...instance(mockedProps)}
-      isRoot={true}
+      isRoot
       value={123}
     />)
 
     expect(screen.getByText('123')).toBeInTheDocument()
   })
 
   it('should render string value with quotes', () => {
-    render(<RejsonScalar 
+    render(<RejsonScalar
       {...instance(mockedProps)}
-      isRoot={true}
+      isRoot
       value="test"
     />)
 

redisinsight/ui/src/pages/browser/modules/key-details/components/rejson-details/utils/utils.spec.ts

@@ -82,48 +82,48 @@ describe('JSONUtils', () => {
 
   describe('JSON Parsing Utils', () => {
     const bigintAsString = '1188950299261208742'
-  
+
     describe('parseValue', () => {
       it('should handle non-string values', () => {
         expect(parseValue(123)).toBe(123)
         expect(parseValue(null)).toBe(null)
         expect(parseValue(undefined)).toBe(undefined)
       })
-  
+
       it('should parse typed integer values', () => {
         const result = parseValue(bigintAsString, 'integer')
         expect(typeof result).toBe('bigint')
         expect(result.toString()).toBe(bigintAsString)
       })
-  
+
       it('should parse regular numbers as numbers, not bigints', () => {
         const result = parseValue('42', 'integer')
         expect(typeof result).toBe('number')
         expect(result).toBe(42)
       })
-  
+
       it('should handle string type with quotes', () => {
         expect(parseValue('"test"', 'string')).toBe('test')
         expect(parseValue('test', 'string')).toBe('test')
       })
-  
+
       it('should parse boolean values', () => {
         expect(parseValue('true', 'boolean')).toBe(true)
         expect(parseValue('false', 'boolean')).toBe(false)
       })
-  
+
       it('should parse null values', () => {
         expect(parseValue('null', 'null')).toBe(null)
       })
-  
+
       it('should parse JSON objects without type', () => {
         const input = `{"value": ${bigintAsString}, "text": "test"}`
         const result = parseValue(input)
         expect(typeof result.value).toBe('bigint')
         expect(result.value.toString()).toBe(bigintAsString)
         expect(result.text).toBe('test')
       })
-  
+
       it('should parse JSON arrays without type', () => {
         const input = `[${bigintAsString}, "test"]`
         const result = parseValue(input)
@@ -132,32 +132,32 @@ describe('JSONUtils', () => {
         expect(result[1]).toBe('test')
       })
     })
-  
+
     describe('parseJsonData', () => {
       it('should handle null or undefined data', () => {
         expect(parseJsonData(null)).toBe(null)
         expect(parseJsonData(undefined)).toBe(undefined)
       })
-  
+
       it('should parse array of typed values', () => {
         const input = [
           { type: 'string', value: '"John"' },
           { type: 'integer', value: bigintAsString }
         ]
         const result = parseJsonData(input)
-        
+
         expect(result[0].value).toBe('John')
         expect(typeof result[1].value).toBe('bigint')
         expect(result[1].value.toString()).toBe(bigintAsString)
       })
-  
+
       it('should preserve non-typed array items', () => {
         const input = [
           { value: '"John"' },
           { someOtherProp: 'test' }
         ]
         const result = parseJsonData(input)
-        
+
         expect(result[0].value).toBe('"John"')
         expect(result[1].someOtherProp).toBe('test')
       })

redisinsight/ui/src/pages/browser/modules/key-details/components/rejson-details/utils/utils.ts

@@ -63,7 +63,7 @@ export const getBrackets = (type: string, position: 'start' | 'end' = 'start') =
 
 export const isValidKey = (key: string): boolean => /^"([^"\\]|\\.)*"$/.test(key)
 
-const JSONParser = JSONBigInt({ 
+const JSONParser = JSONBigInt({
   useNativeBigInt: true,
   strict: false
 })
@@ -88,7 +88,7 @@ export const parseValue = (value: any, type?: string): any => {
           return null
         case 'string':
           if (value.startsWith('"') && value.endsWith('"')) {
-            value = value.slice(1, -1)
+            return value.slice(1, -1)
           }
           return value
         default:
@@ -97,10 +97,10 @@ export const parseValue = (value: any, type?: string): any => {
     }
 
     const parsed = JSONParser.parse(value)
-    
+
     if (typeof parsed === 'object' && parsed !== null) {
       if (Array.isArray(parsed)) {
-        return parsed.map(val => parseValue(val))
+        return parsed.map((val) => parseValue(val))
       }
       const result: { [key: string]: any } = {}
       Object.entries(parsed).forEach(([key, val]) => {
@@ -121,11 +121,11 @@ export const parseJsonData = (data: any) => {
   try {
     if (data && Array.isArray(data)) {
       return data.map((item: { type?: string; value?: any }) => ({
-      ...item,
-      value: item.type && item.value ? parseValue(item.value, item.type) : item.value
-      }))        
+        ...item,
+        value: item.type && item.value ? parseValue(item.value, item.type) : item.value
+      }))
     }
-    
+
     return parseValue(data)
   } catch (e) {
     return data