Merge pull request #810 from CSCfi/test

Merge test to stable v0.9.40-rc1
CSCfi · Apr 30, 2021 · fde3861 · fde3861
2 parents a057331 + 2f6f5f9
commit fde3861
Show file tree

Hide file tree

Showing 2 changed files with 52 additions and 3 deletions.
diff --git a/etsin_finder/frontend/__tests__/utils/cite.test.js b/etsin_finder/frontend/__tests__/utils/cite.test.js
@@ -42,6 +42,24 @@ const organizationDataset = {
   identifier: 'metax_identifier_for_this_dataset',
 }
 
+const publisherSuborganizationDataset = {
+  research_dataset: {
+    creator: [{ name: { en: 'Creator', fi: 'Creator' } }],
+    publisher: {
+      name: { en: 'Suborganization', fi: 'Joku aliorganisaatio' },
+      '@type': 'Organization',
+      is_part_of: {
+        name: { en: 'Top organization', fi: 'Pääorganisaatio' },
+        '@type': 'Organization',
+      },
+    },
+    issued: '2021-02-23',
+    title: { fi: 'Julkaisun nimi', en: 'Publication title' },
+    preferred_identifier: 'urn:nbn:fi:att:feedc0de',
+  },
+  identifier: 'metax_identifier_for_this_dataset',
+}
+
 const doiDataset = {
   research_dataset: {
     ...organizationDataset.research_dataset,
@@ -165,6 +183,12 @@ describe('Citation styles', () => {
           '(2021). Publication title. Publisher. http://urn.fi/urn:nbn:fi:att:feedc0de'
       )
     })
+
+    it('should render citation for dataset published by a suborganization', () => {
+      c(publisherSuborganizationDataset).should.eq(
+        'Creator. (2021). Publication title. Top organization, Suborganization. http://urn.fi/urn:nbn:fi:att:feedc0de'
+      )
+    })
   })
 
   describe('Chicago', () => {
@@ -222,6 +246,12 @@ describe('Citation styles', () => {
         'Eka, Tyyppi, Tyyppi Toka, Tyyppi Kolmas, Tyyppi Neljäs, Tyyppi Viides, Tyyppi Kuudes, Tyyppi Seitsemäs, et al. 2021. ”Publication title”. Publisher. http://urn.fi/urn:nbn:fi:att:feedc0de'
       )
     })
+
+    it('should render citation for dataset published by a suborganization', () => {
+      c(publisherSuborganizationDataset).should.eq(
+        'Creator. 2021. ”Publication title”. Top organization, Suborganization. http://urn.fi/urn:nbn:fi:att:feedc0de'
+      )
+    })
   })
 
   describe('MLA', () => {
@@ -279,6 +309,12 @@ describe('Citation styles', () => {
         'Eka, Tyyppi, et al. ”Publication title”. Publisher, 2021. http://urn.fi/urn:nbn:fi:att:feedc0de'
       )
     })
+
+    it('should render citation for dataset published by a suborganization', () => {
+      c(publisherSuborganizationDataset).should.eq(
+        'Creator. ”Publication title”. Top organization, Suborganization, 2021. http://urn.fi/urn:nbn:fi:att:feedc0de'
+      )
+    })
   })
 })
 
@@ -292,6 +328,10 @@ describe('Utils', () => {
       getNameInitials('Mauri Antero Numminen').should.eq('Numminen, M. A.')
     })
 
+    it('should ignore whitespace around name', () => {
+      getNameInitials('   Mauri Antero Numminen   ').should.eq('Numminen, M. A.')
+    })
+
     it('should leave multi-part last name unchanged', () => {
       getNameInitials('Johannes Diderik van der Waals').should.eq('van der Waals, J. D.')
     })

diff --git a/etsin_finder/frontend/js/components/dataset/citation/cite/utils.js b/etsin_finder/frontend/js/components/dataset/citation/cite/utils.js
@@ -22,7 +22,7 @@ const romanNumeralMatch = /^(((IX|IV|V)I{0,3})|I{1,3})$/
 const toInitial = name => `${name[0]}.`
 
 export const getNameParts = name => {
-  const parts = name.split(' ')
+  const parts = name.trim().split(' ')
   if (parts.length === 1) {
     return { first: [], last: [parts[0]], suffixes: [] }
   }
@@ -182,5 +182,14 @@ export const getIdentifier = dataset => {
   return identifier
 }
 
-export const getPublisher = (dataset, getTranslation) =>
-  getTranslation(dataset.research_dataset.publisher?.name)
+export const getPublisher = (dataset, getTranslation) => {
+  const pub = dataset.research_dataset.publisher
+  if (pub) {
+    const top = topOrg(pub)
+    if (top !== pub) {
+      return `${getTranslation(top.name)}, ${getTranslation(pub.name)}`
+    }
+    return getTranslation(pub.name)
+  }
+  return undefined
+}