increase test coverage, mock ocean.versions.get()

client/__mocks__/ocean-mock.ts

@@ -31,6 +31,34 @@ const oceanMock = {
                     }
                 }
             }
+        },
+        versions: {
+            get: jest.fn(() =>
+                Promise.resolve({
+                    squid: {
+                        name: 'Squid-js',
+                        status: 'Working'
+                    },
+                    aquarius: {
+                        name: 'Aquarius',
+                        status: 'Working'
+                    },
+                    brizo: {
+                        name: 'Brizo',
+                        network: 'Nile',
+                        status: 'Working',
+                        contracts: {
+                            hello: 'hello',
+                            hello2: 'hello2'
+                        }
+                    },
+                    status: {
+                        ok: true,
+                        network: true,
+                        contracts: true
+                    }
+                })
+            )
         }
     }
 }

client/src/components/molecules/VersionNumbers/VersionNumber.test.tsx

@@ -0,0 +1,23 @@
+import React from 'react'
+import { render } from '@testing-library/react'
+import VersionNumber from './VersionNumber'
+
+describe('VersionNumber', () => {
+    it('renders without crashing', () => {
+        const { container } = render(<VersionNumber name="Commons" />)
+        expect(container.firstChild).toBeInTheDocument()
+    })
+
+    it('renders with all props set', () => {
+        const { container } = render(
+            <VersionNumber
+                name="Commons"
+                version="6.6.6"
+                network="Nile"
+                commit="xxxxxxxxxxx"
+            />
+        )
+        expect(container.firstChild).toBeInTheDocument()
+        expect(container.firstChild).toHaveTextContent('6.6.6')
+    })
+})

client/src/components/molecules/VersionNumbers/VersionStatus.test.tsx

@@ -0,0 +1,23 @@
+import React from 'react'
+import { render } from '@testing-library/react'
+import VersionStatus from './VersionStatus'
+
+describe('VersionStatus', () => {
+    it('renders without crashing', () => {
+        const { container } = render(
+            <VersionStatus
+                status={{ ok: false, contracts: false, network: false }}
+            />
+        )
+        expect(container.firstChild).toBeInTheDocument()
+    })
+
+    it('renders true states', () => {
+        const { container } = render(
+            <VersionStatus
+                status={{ ok: true, contracts: false, network: false }}
+            />
+        )
+        expect(container.firstChild).toBeInTheDocument()
+    })
+})

client/src/components/molecules/VersionNumbers/VersionTable.test.tsx

@@ -0,0 +1,40 @@
+import React from 'react'
+import { render } from '@testing-library/react'
+import { VersionTableContracts } from './VersionTable'
+
+describe('VersionTableContracts', () => {
+    it('renders without crashing', () => {
+        const { container } = render(
+            <VersionTableContracts
+                contracts={{ hello: 'hello', hello2: 'hello2' }}
+                network="nile"
+                keeperVersion="6.6.6"
+            />
+        )
+        expect(container.firstChild).toBeInTheDocument()
+    })
+
+    it('renders correct Submarine links', () => {
+        const { container, rerender } = render(
+            <VersionTableContracts
+                contracts={{ hello: 'hello', hello2: 'hello2' }}
+                network="duero"
+                keeperVersion="6.6.6"
+            />
+        )
+        expect(container.querySelector('tr:last-child a').href).toMatch(
+            /submarine.duero.dev-ocean/
+        )
+
+        rerender(
+            <VersionTableContracts
+                contracts={{ hello: 'hello', hello2: 'hello2' }}
+                network="pacific"
+                keeperVersion="6.6.6"
+            />
+        )
+        expect(container.querySelector('tr:last-child a').href).toMatch(
+            /submarine.pacific.dev-ocean/
+        )
+    })
+})

client/src/components/molecules/VersionNumbers/VersionTableRow.tsx

@@ -38,7 +38,7 @@ const VersionTableRow = ({ value }: { value: any }) => {
                             value.name || value.software
                         )}`}
                     >
-                        <strong>{value.name}</strong>
+                        <strong>{value.name || value.software}</strong>
                     </a>
                 </td>
                 <td>

client/src/components/molecules/VersionNumbers/index.test.tsx

@@ -1,48 +1,20 @@
 import React from 'react'
-import { render, waitForElement } from '@testing-library/react'
+import { render } from '@testing-library/react'
 import mockAxios from 'jest-mock-axios'
 import { StateMock } from '@react-mock/state'
-import { version as versionSquid } from '@oceanprotocol/squid/package.json'
+import VersionNumbers from '.'
-import VersionNumbers, { commonsVersion } from '.'
+
+import { User } from '../../../context'
+import { userMockConnected } from '../../../../__mocks__/user-mock'
 
 afterEach(() => {
     mockAxios.reset()
 })
 
-const stateMock = {
-    commons: {
-        name: 'Commons',
-        version: commonsVersion
-    },
-    squid: {
-        name: 'Squid-js',
-        status: 'Loading'
-    },
-    aquarius: {
-        name: 'Aquarius',
-        status: 'Loading'
-    },
-    brizo: {
-        name: 'Brizo',
-        network: 'Nile',
-        status: 'Loading'
-    },
-    faucet: {
-        name: 'Faucet',
-        version: '',
-        status: 'Loading'
-    },
-    status: {
-        ok: false,
-        network: false,
-        contracts: false
-    }
-}
-
 const stateMockIncomplete = {
     commons: {
         name: 'Commons',
-        version: commonsVersion
+        version: undefined
     },
     squid: {
         name: 'Squid-js',
@@ -87,9 +59,9 @@ const mockResponseFaulty = {
 describe('VersionNumbers', () => {
     it('renders without crashing', () => {
         const { container } = render(
-            <StateMock state={stateMock}>
+            <User.Provider value={userMockConnected}>
                 <VersionNumbers />
-            </StateMock>
+            </User.Provider>
         )
         mockAxios.mockResponse(mockResponse)
         expect(mockAxios.get).toHaveBeenCalled()
@@ -98,9 +70,11 @@ describe('VersionNumbers', () => {
 
     it('renders without proper component response', () => {
         const { container } = render(
-            <StateMock state={stateMockIncomplete}>
+            <User.Provider value={userMockConnected}>
-                <VersionNumbers />
+                <StateMock state={stateMockIncomplete}>
-            </StateMock>
+                    <VersionNumbers />
+                </StateMock>
+            </User.Provider>
         )
         mockAxios.mockResponse(mockResponseFaulty)
         expect(mockAxios.get).toHaveBeenCalled()