fix: minor change in use case

Author: ChengShi-1

test/functional/datasets/DeaccessionDataset.test.ts

@@ -121,7 +121,7 @@ describe('execute', () => {
       )
     ).rejects.toBeInstanceOf(WriteError)
 
-    await deletePublishedDatasetViaApi(createdDatasetIdentifiers.persistentId)
+    await deleteUnpublishedDatasetViaApi(createdDatasetIdentifiers.numericId)
   })
 
   test('should deaccession a dataset when it is deaccessioned once', async () => {
@@ -152,6 +152,6 @@ describe('execute', () => {
       deaccessionDataset.execute(createdDatasetIdentifiers.numericId, '1.0', testDeaccessionDataset)
     ).rejects.toThrow(Error)
 
-    await deleteUnpublishedDatasetViaApi(createdDatasetIdentifiers.numericId)
+    await deletePublishedDatasetViaApi(createdDatasetIdentifiers.persistentId)
   })
 })

test/testHelpers/datasets/datasetHelper.ts

@@ -10,7 +10,8 @@ import { DvObjectType } from '../../../src/core/domain/models/DvObjectOwnerNode'
 import TurndownService from 'turndown'
 import {
   DatasetDTO,
-  DatasetMetadataFieldValueDTO
+  DatasetMetadataFieldValueDTO,
+  DatasetDeaccessionDTO
 } from '../../../src/datasets/domain/dtos/DatasetDTO'
 import { MetadataBlock, MetadataFieldType } from '../../../src'
 import {
@@ -788,3 +789,9 @@ export const createUpdateDatasetRequestPayload = (): UpdateDatasetRequestPayload
     ]
   }
 }
+
+export const createDatasetDeaccessionDTO = (): DatasetDeaccessionDTO => {
+  return {
+    deaccessionReason: 'Test reason.'
+  }
+}

test/unit/datasets/DatasetsRepository.test.ts

@@ -24,6 +24,7 @@ import {
 } from '../../testHelpers/datasets/datasetPreviewHelper'
 import {
   createDatasetDTO,
+  createDatasetDeaccessionDTO,
   createDatasetMetadataBlockModel,
   createNewDatasetRequestPayload
 } from '../../testHelpers/datasets/datasetHelper'
@@ -902,4 +903,57 @@ describe('DatasetsRepository', () => {
       expect(error).toBeInstanceOf(Error)
     })
   })
+
+  describe('deaccsionDataset', () => {
+    const version = '1.0'
+    const expectedApiEndpoint = `${TestConstants.TEST_API_URL}/datasets/${testDatasetModel.id}/versions/${version}/deaccession`
+    const expectedApiKeyRequestConfig = {
+      ...TestConstants.TEST_EXPECTED_AUTHENTICATED_REQUEST_CONFIG_API_KEY
+    }
+    const testDeaccessionDataset = createDatasetDeaccessionDTO()
+    const testDeaccessionDatasetJSON = JSON.stringify(testDeaccessionDataset)
+    test('should return nothing when providing id, version update type and response is successful', async () => {
+      jest.spyOn(axios, 'post').mockResolvedValue(undefined)
+
+      let actual = await sut.deaccessionDataset(
+        testDatasetModel.id,
+        version,
+        testDeaccessionDataset
+      )
+
+      expect(axios.post).toHaveBeenCalledWith(
+        expectedApiEndpoint,
+        testDeaccessionDatasetJSON,
+        expectedApiKeyRequestConfig
+      )
+      expect(actual).toBeUndefined()
+
+      ApiConfig.init(TestConstants.TEST_API_URL, DataverseApiAuthMechanism.SESSION_COOKIE)
+
+      actual = await sut.deaccessionDataset(testDatasetModel.id, version, testDeaccessionDataset)
+
+      expect(axios.post).toHaveBeenCalledWith(
+        expectedApiEndpoint,
+        testDeaccessionDatasetJSON,
+        expectedApiKeyRequestConfig
+      )
+      expect(actual).toBeUndefined()
+    })
+
+    test('should return error result on error response', async () => {
+      jest.spyOn(axios, 'post').mockRejectedValue(TestConstants.TEST_ERROR_RESPONSE)
+
+      let error: WriteError | undefined = undefined
+      await sut
+        .deaccessionDataset(testDatasetModel.id, version, testDeaccessionDataset)
+        .catch((e) => (error = e))
+
+      expect(axios.post).toHaveBeenCalledWith(
+        expectedApiEndpoint,
+        testDeaccessionDatasetJSON,
+        expectedApiKeyRequestConfig
+      )
+      expect(error).toBeInstanceOf(Error)
+    })
+  })
 })