Merge pull request #494 from IQSS/fix-develop-merge

fix: add missing import statement

packages/design-system/cypress.config.ts

@@ -19,7 +19,7 @@ export default defineConfig({
   },
   env: {
     codeCoverage: {
-      exclude: 'tests/**/*.*'
+      exclude: ['tests/**/*.*', '**/Tab.tsx']
     }
   }
 })

packages/design-system/tests/component/tabs/Tabs.spec.tsx

@@ -38,4 +38,22 @@ describe('Tabs', () => {
     cy.findByText('Tab 2').should('have.class', 'active')
     cy.findByText('Content 2').should('have.class', 'show').and('have.class', 'active')
   })
+
+  it('renders with a disabled tab', () => {
+    cy.mount(
+      <Tabs defaultActiveKey="key-1">
+        <Tabs.Tab eventKey="key-1" title="Tab 1">
+          Content 1
+        </Tabs.Tab>
+        <Tabs.Tab eventKey="key-2" title="Tab 2" disabled>
+          Content 2
+        </Tabs.Tab>
+        <Tabs.Tab eventKey="key-3" title="Tab 3">
+          Content 3
+        </Tabs.Tab>
+      </Tabs>
+    )
+
+    cy.findByText('Tab 2').should('have.attr', 'disabled')
+  })
 })

src/sections/account/api-token-section/ApiTokenSection.tsx

@@ -6,6 +6,7 @@ import styles from './ApiTokenSection.module.scss'
 export const ApiTokenSection = () => {
   const { t } = useTranslation('account', { keyPrefix: 'apiToken' })
 
+  // TODO: When we have the use cases we need to mock stub to unit test this with or without token
   const apiToken = '999fff-666rrr-this-is-not-a-real-token-123456'
   const expirationDate = '2025-09-04'
 

src/sections/layout/header/LoggedInHeaderActions.tsx

@@ -3,7 +3,7 @@ import { Link, useNavigate } from 'react-router-dom'
 import { Navbar } from '@iqss/dataverse-design-system'
 import { useGetCollectionUserPermissions } from '../../../shared/hooks/useGetCollectionUserPermissions'
 import { useSession } from '../../session/SessionContext'
-import { RouteWithParams } from '../../Route.enum'
+import { RouteWithParams, Route } from '../../Route.enum'
 import { User } from '../../../users/domain/models/User'
 import { CollectionRepository } from '../../../collection/domain/repositories/CollectionRepository'
 import { ROOT_COLLECTION_ALIAS } from '../../../collection/domain/models/Collection'

tests/component/sections/account/AccountHelper.spec.ts

@@ -0,0 +1,25 @@
+import { AccountHelper } from '../../../../src/sections/account/AccountHelper'
+
+describe('AccountHelper', () => {
+  describe('defineSelectedTabKey', () => {
+    it('should return the correct tab key when a tab query param is present in the URL', () => {
+      const searchParams = new URLSearchParams()
+      searchParams.set(
+        AccountHelper.ACCOUNT_PANEL_TAB_QUERY_KEY,
+        AccountHelper.ACCOUNT_PANEL_TABS_KEYS.notifications
+      )
+
+      const result = AccountHelper.defineSelectedTabKey(searchParams)
+
+      expect(result).to.equal(AccountHelper.ACCOUNT_PANEL_TABS_KEYS.notifications)
+    })
+
+    it('should return the my data tab key as default if the tab query param is not present in the URL', () => {
+      const searchParams = new URLSearchParams()
+
+      const result = AccountHelper.defineSelectedTabKey(searchParams)
+
+      expect(result).to.equal(AccountHelper.ACCOUNT_PANEL_TABS_KEYS.myData)
+    })
+  })
+})

tests/component/sections/dataset/dataset-files/files-table/files-info/file-info-cell/file-info-data/file-thumbnail/FileThumbnail.spec.tsx

@@ -10,7 +10,7 @@ describe('FileThumbnail', () => {
     cy.customMount(<FileThumbnail file={file} />)
 
     cy.findByAltText(file.name).should('exist')
-    cy.findByAltText(file.name).trigger('mouseover')
+    cy.findByAltText(file.name).trigger('mouseover', { force: true })
     cy.findByRole('tooltip').should('exist')
 
     cy.findByText('Restricted File Icon').should('not.exist')

tests/component/sections/homepage/SearchInput.spec.tsx

@@ -14,4 +14,15 @@ describe('SearchInput', () => {
     cy.findByLabelText('Clear search').click()
     cy.get('[aria-label="Search"]').should('have.value', '')
   })
+
+  it('should be able to click the submit button when the user enters a value or not', () => {
+    cy.customMount(<SearchInput />)
+    cy.get('[aria-label="Search"]').type('test')
+    cy.get('[aria-label="Search"]').should('have.value', 'test')
+    cy.get('[aria-label="Submit Search"]').click()
+
+    cy.get('[aria-label="Search"]').clear()
+    cy.get('[aria-label="Search"]').should('have.value', '')
+    cy.get('[aria-label="Submit Search"]').click()
+  })
 })

tests/e2e-integration/e2e/sections/homepage/Homepage.spec.tsx

@@ -0,0 +1,24 @@
+import { QueryParamKey } from '../../../../../src/sections/Route.enum'
+
+describe('Homepage', () => {
+  it('should navigate to the collections page with the search value encoded in the URL', () => {
+    const searchValue = 'John Doe'
+    cy.visit('/spa/')
+    cy.get('[aria-label="Search"]').type(searchValue)
+    cy.get('[aria-label="Submit Search"]').click()
+
+    const encodedSearchValue = encodeURIComponent(searchValue)
+
+    const searchParams = new URLSearchParams()
+    searchParams.set(QueryParamKey.QUERY, encodedSearchValue)
+
+    cy.url().should('include', `/collections?${searchParams.toString()}`)
+  })
+
+  it('navigates directly to the collection page when clicking the Browse Collections button', () => {
+    cy.visit('/spa/')
+    cy.findByRole('link', { name: 'Browse Collections' }).click()
+
+    cy.url().should('eq', `${Cypress.config().baseUrl as string}/spa/collections`)
+  })
+})