fix: call submitPublish() with VersionUpdateType.MAJOR when requiresM…

…ajorVersionUpdate=true
IQSS · Jan 6, 2025 · cb7403e · cb7403e
1 parent ad1f70a
commit cb7403e
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 2 deletions.
diff --git a/src/sections/dataset/publish-dataset/PublishDatasetModal.tsx b/src/sections/dataset/publish-dataset/PublishDatasetModal.tsx
@@ -47,7 +47,6 @@ export function PublishDatasetModal({
   const { t } = useTranslation('dataset')
   const { user } = useSession()
   const navigate = useNavigate()
-  console.log('requiresMajorVersionUpdate', requiresMajorVersionUpdate)
   const { submissionStatus, submitPublish, publishError } = usePublishDataset(
     repository,
     collectionRepository,
@@ -139,7 +138,10 @@ export function PublishDatasetModal({
         <Button
           variant="primary"
           onClick={() => {
-            submitPublish(selectedVersionUpdateType)
+            const versionUpdateType = requiresMajorVersionUpdate
+              ? VersionUpdateType.MAJOR
+              : selectedVersionUpdateType
+            submitPublish(versionUpdateType)
           }}
           type="submit">
           {t('publish.continueButton')}

diff --git a/tests/component/sections/dataset/dataset-publish/PublishDatasetModal.spec.tsx b/tests/component/sections/dataset/dataset-publish/PublishDatasetModal.spec.tsx
@@ -199,6 +199,12 @@ describe('PublishDatasetModal', () => {
     cy.findByText(
       /Due to the nature of the changes to the current draft this will be a major release \(2\.0\)/
     ).should('exist')
+    cy.findByText('Continue').click()
+    cy.get('@repositoryPublish').should(
+      'have.been.calledWith',
+      'testPersistentId',
+      VersionUpdateType.MAJOR
+    )
   })
 
   it('Displays warning text for unreleased Collection', () => {