Increase coverage

webview/jest.config.js

@@ -4,7 +4,7 @@ module.exports = {
   collectCoverageFrom: [
     'src/**/*.{ts,tsx}',
     '!**/*.test.*',
-    '!src/stories/*',
+    '!src/stories/**/*.{ts,tsx}',
     '!src/test/*',
     '!src/shared/components/icons/*',
     '!src/util/wdyr.ts'

webview/setup-tests.js

@@ -1,12 +1,3 @@
-jest.mock('./src/util/styles', () => {
-  const actualModule = jest.requireActual('./src/util/styles')
-  return {
-    __esModule: true,
-    ...actualModule,
-    getThemeValue: jest.fn().mockImplementation(() => '#ffffff')
-  }
-})
-
 // eslint-disable-next-line no-global-assign
 window = {
   addEventListener: jest.fn(),

webview/src/plots/components/comparisonTable/ComparisonTable.test.tsx

@@ -479,6 +479,15 @@ describe('ComparisonTable', () => {
     it('should add and remove the style for the ghost header being dragged', () => {
       jest.useFakeTimers()
 
+      document.documentElement.style.setProperty(
+        ThemeProperty.ACCENT_COLOR,
+        '#ffffff'
+      )
+      document.documentElement.style.setProperty(
+        ThemeProperty.BACKGROUND_COLOR,
+        '#ffffff'
+      )
+
       renderTable()
 
       const [header] = getHeaders()

webview/src/util/array.test.ts

@@ -0,0 +1,41 @@
+import { changeOrderWithDraggedInfo, pushIf } from './array'
+
+describe('array', () => {
+  describe('pushIf', () => {
+    it('should not push to array if condiction is not met', () => {
+      const currentArray = [1, 2, 3]
+      pushIf(currentArray, false, [4])
+
+      expect(currentArray.length).toBe(3)
+    })
+
+    it('should push to array if condiction is met', () => {
+      const currentArray = [1, 2, 3]
+      pushIf(currentArray, true, [4])
+
+      expect(currentArray.length).toBe(4)
+    })
+  })
+
+  describe('changeOrderWithDraggedInfo', () => {
+    it('should return the same order if there are no draggedInfo', () => {
+      const currentOrder = ['1', '2', '3', '4']
+
+      expect(changeOrderWithDraggedInfo(currentOrder, undefined)).toBe(
+        currentOrder
+      )
+    })
+
+    it('should return the order with the dragged item id moved to the end', () => {
+      const currentOrder = ['1', '2', '3', '4']
+
+      expect(
+        changeOrderWithDraggedInfo(currentOrder, {
+          group: 'any',
+          itemId: '2',
+          itemIndex: '1'
+        })
+      ).toStrictEqual(['1', '3', '4', '2'])
+    })
+  })
+})

webview/src/util/ids.test.ts

@@ -0,0 +1,11 @@
+import { createIDWithPrefixAndIndex } from './ids'
+
+describe('ids', () => {
+  describe('createIDWithPrefixAndIndex', () => {
+    it('should create an id with a prefix and an index correctly', () => {
+      expect(createIDWithPrefixAndIndex('my-id', 42, 'this-isAPREFIX')).toBe(
+        'this-isAPREFIXmy-id_42'
+      )
+    })
+  })
+})

webview/src/util/styles.test.ts

@@ -0,0 +1,14 @@
+import { getThemeValue, ThemeProperty } from './styles'
+
+describe('styles', () => {
+  it('should get the theme value for the requested property when calling getThemeValue', () => {
+    const styleValue = 'blue'
+
+    document.documentElement.style.setProperty(
+      ThemeProperty.ACCENT_COLOR,
+      styleValue
+    )
+
+    expect(getThemeValue(ThemeProperty.ACCENT_COLOR)).toBe(styleValue)
+  })
+})