#RI-5348 - add tests, add handling errors

Author: rsergeenko

redisinsight/ui/src/pages/home/components/databases-list-component/databases-list/DatabasesList.tsx

@@ -56,9 +56,13 @@ function DatabasesList({
     if (columnsToHide?.length && containerTableRef.current) {
       const { offsetWidth } = containerTableRef.current
       const beforeAdjustHiddenCols = hiddenCols.current.size
-      const columnsResults = adjustColumns(columns, offsetWidth)
-      if (beforeAdjustHiddenCols !== hiddenCols.current.size) {
-        setColumns(columnsResults)
+      try {
+        const columnsResults = adjustColumns(columns, offsetWidth)
+        if (beforeAdjustHiddenCols !== hiddenCols.current.size) {
+          setColumns(columnsResults)
+        }
+      } catch (_) {
+        // ignore
       }
     }
   }, [width])

redisinsight/ui/src/pages/home/components/databases-list-component/databases-list/utils.spec.ts

@@ -0,0 +1,56 @@
+import { getColumnWidth, findColumn, hideColumn, MIN_COLUMN_WIDTH } from './utils'
+
+const getColumnWidthTests: any[] = [
+  ['10px', 10],
+  ['350px', 350],
+  ['100', MIN_COLUMN_WIDTH],
+  ['x', MIN_COLUMN_WIDTH],
+]
+
+describe('getColumnWidth', () => {
+  it.each(getColumnWidthTests)('for input: %s (input), should be output: %s',
+    (input, expected) => {
+      const result = getColumnWidth(input)
+      expect(result).toBe(expected)
+    })
+})
+
+const findColumnTests: any[] = [
+  [
+    [
+      [{ field: '1' }, { field: '2' }],
+      '1'
+    ],
+    { field: '1' }
+  ],
+  [
+    [
+      [{ field: '1' }, { field: '2' }],
+      '3'
+    ],
+    undefined
+  ]
+]
+
+describe('findColumn', () => {
+  it.each(findColumnTests)('for input: %s (input), should be output: %s',
+    (input, expected) => {
+      const result = findColumn(...input as [any, string])
+      expect(result).toEqual(expected)
+    })
+})
+
+const hideColumnTests: any[] = [
+  [
+    { field: '1' },
+    { field: '1', width: '0px', className: 'hiddenColumn' }
+  ],
+]
+
+describe('hideColumn', () => {
+  it.each(hideColumnTests)('for input: %s (input), should be output: %s',
+    (input, expected) => {
+      const result = hideColumn(input)
+      expect(result).toEqual(expected)
+    })
+})

redisinsight/ui/src/pages/home/components/databases-list-component/databases-list/utils.ts

@@ -13,6 +13,7 @@ const findColumn = (columns: EuiTableFieldDataColumnType<Instance>[], colName: s
   find(columns, ({ field }) => field === colName)
 
 export {
+  MIN_COLUMN_WIDTH,
   getColumnWidth,
   hideColumn,
   findColumn,