test: add more cases

Author: g-saracca

src/sections/file/file-labels/FileLabels.tsx

@@ -9,7 +9,7 @@ const VARIANT_BY_LABEL_TYPE: Record<FileLabelType, 'secondary' | 'info'> = {
 
 export function FileLabels({ labels }: { labels: FileLabel[] }) {
   return (
-    <div className={styles.container}>
+    <div className={styles.container} data-testid="file-labels">
       {labels.map((label, index) => (
         <Badge key={`${label.value}-${index}`} variant={VARIANT_BY_LABEL_TYPE[label.type]}>
           {label.value}

tests/component/sections/collection/collection-items-panel/dataset-card/DatasetCard.spec.tsx

@@ -2,6 +2,7 @@ import { DatasetCard } from '@/sections/collection/collection-items-panel/items-
 import { DatasetItemTypePreviewMother } from '@tests/component/dataset/domain/models/DatasetItemTypePreviewMother'
 import { DateHelper } from '@/shared/helpers/DateHelper'
 import styles from '@/sections/collection/collection-items-panel/items-list/dataset-card/DatasetCard.module.scss'
+import { Route } from '@/sections/Route.enum'
 
 describe('DatasetCard', () => {
   it('should render the card', () => {
@@ -55,4 +56,32 @@ describe('DatasetCard', () => {
       .parent()
       .should('have.class', styles['deaccesioned'])
   })
+
+  describe('Parent Collection Link', () => {
+    it('should render it if parentCollectionAlias is not the one where the dataset card is being shown', () => {
+      const dataset = DatasetItemTypePreviewMother.create({
+        parentCollectionAlias: 'parent-collection-alias',
+        parentCollectionName: 'Parent Collection Name'
+      })
+      cy.customMount(
+        <DatasetCard datasetPreview={dataset} parentCollectionAlias="another-collection-alias" />
+      )
+
+      cy.findByText('Parent Collection Name')
+        .should('exist')
+        .should('have.attr', 'href', `${Route.COLLECTIONS_BASE}/parent-collection-alias`)
+    })
+
+    it('should not render it if parentCollectionAlias is the same as the one where the dataset card is being shown', () => {
+      const dataset = DatasetItemTypePreviewMother.create({
+        parentCollectionAlias: 'parent-collection-alias',
+        parentCollectionName: 'Parent Collection Name'
+      })
+      cy.customMount(
+        <DatasetCard datasetPreview={dataset} parentCollectionAlias="parent-collection-alias" />
+      )
+
+      cy.findByText('Parent Collection Name').should('not.exist')
+    })
+  })
 })

tests/component/sections/collection/collection-items-panel/dataset-card/DatasetCardHeader.spec.tsx

@@ -39,4 +39,44 @@ describe('DatasetCardHeader', () => {
       .should('exist')
       .should('have.attr', 'href', `/datasets?persistentId=${dataset.persistentId}&version=DRAFT`)
   })
+
+  it('should sort the publication statuses by name', () => {
+    const dataset = DatasetItemTypePreviewMother.create({
+      publicationStatuses: [
+        PublicationStatus.Unpublished,
+        PublicationStatus.Draft,
+        PublicationStatus.InReview
+      ]
+    })
+    cy.customMount(
+      <DatasetCardHeader
+        persistentId={dataset.persistentId}
+        version={dataset.version}
+        publicationStatuses={dataset.publicationStatuses}
+      />
+    )
+
+    cy.findAllByText(PublicationStatus.Draft).should('exist')
+    cy.findAllByText(PublicationStatus.InReview).should('exist')
+    cy.findAllByText(PublicationStatus.Unpublished).should('exist')
+
+    cy.get('.badge').then((badges) => {
+      expect(badges[0].textContent).to.equal(PublicationStatus.Draft)
+      expect(badges[1].textContent).to.equal(PublicationStatus.InReview)
+      expect(badges[2].textContent).to.equal(PublicationStatus.Unpublished)
+    })
+  })
+
+  it('should filter out and dont render the Published status badge', () => {
+    const dataset = DatasetItemTypePreviewMother.create()
+    cy.customMount(
+      <DatasetCardHeader
+        persistentId={dataset.persistentId}
+        version={dataset.version}
+        publicationStatuses={[PublicationStatus.Published]}
+      />
+    )
+
+    cy.findByText(PublicationStatus.Published).should('not.exist')
+  })
 })

tests/component/sections/collection/collection-items-panel/file-card/FileCard.spec.tsx

@@ -67,4 +67,20 @@ describe('FileCard', () => {
       .should('have.attr', 'href')
       .and('include', 'version=DRAFT')
   })
+
+  it('should not show any tag if the file has no tags', () => {
+    const filePreview = FileItemTypePreviewMother.create({ tags: [] })
+    cy.customMount(<FileCard filePreview={filePreview} />)
+    cy.findByTestId('file-labels').children().should('have.length', 0)
+  })
+
+  it('should default to 0 variables and 0 observations if file is tabular and they are not present', () => {
+    const filePreview = FileItemTypePreviewMother.create({
+      fileType: 'Tab-Delimited',
+      variables: undefined,
+      observations: undefined
+    })
+    cy.customMount(<FileCard filePreview={filePreview} />)
+    cy.contains('0 variables, 0 observations').should('exist')
+  })
 })