fix: minor changes

Author: ChengShi-1

docs/useCases.md

@@ -779,8 +779,7 @@ deaccessionDataset.execute(datasetId, version, deaccessionDatasetDTO)
 
 _See [use case](../src/datasets/domain/useCases/DeaccesionDataset.ts) implementation_.
 The `datasetId` parameter can be a string for persistent identifiers, or a number for numeric identifiers.
-The `version` parameter should be a string or a [DatasetNotNumberedVersion](../src/datasets/domain/models/DatasetNotNumberedVersion.ts) enum value. If not set, the default value is `DatasetNotNumberedVersion.LATEST`.
-In `deaccessionDatasetDTO`, deaccessionForwardURL is optional.
+The `version` parameter should be a string or a [DatasetNotNumberedVersion](../src/datasets/domain/models/DatasetNotNumberedVersion.ts) enum value.
 
 You cannot deaccession a dataset more than once. If you call this endpoint twice for the same dataset version, you will get a not found error on the second call, since the dataset you are looking for will no longer be published since it is already deaccessioned.
 

src/datasets/domain/useCases/DeaccessionDataset.ts

@@ -13,14 +13,13 @@ export class DeaccessionDataset implements UseCase<void> {
   /**
    * Deaccession a dataset, given a dataset id, a dataset version id, and a DatasetDeaccessionDTO object.
    * @param {number | string} [datasetId] - The dataset identifier, which can be a string (for persistent identifiers), or a number (for numeric identifiers).
-   * @param {string | DatasetNotNumberedVersion} [datasetVersionId=DatasetNotNumberedVersion.LATEST] - The dataset version identifier, which can be a version-specific numeric string (for example, 1.0) or a DatasetNotNumberedVersion enum value. If this parameter is not set, the default value is: DatasetNotNumberedVersion.LATEST
-   * @param {DatasetDeaccessionDTO} [DatasetDeaccessionDTO] - The DatasetDeaccessionDTO object that contains the deaccession reason and an optional deaccession forward URL.
+   * @param {string | DatasetNotNumberedVersion} [datasetVersionId] - The dataset version identifier, which can be a version-specific numeric string (for example, 1.0) or a DatasetNotNumberedVersion enum value.
    * @returns A promise that resolves when the dataset is deaccessioned
    * @throws An error if the dataset could not be deaccessioned
    */
   async execute(
     datasetId: number | string,
-    datasetVersionId: string | DatasetNotNumberedVersion = DatasetNotNumberedVersion.LATEST,
+    datasetVersionId: string | DatasetNotNumberedVersion,
     DatasetDeaccessionDTO: DatasetDeaccessionDTO
   ): Promise<void> {
     return await this.datasetsRepository.deaccessionDataset(

test/functional/datasets/DeaccessionDataset.test.ts

@@ -120,6 +120,8 @@ describe('execute', () => {
         testDeaccessionDataset
       )
     ).rejects.toBeInstanceOf(WriteError)
+
+    await deletePublishedDatasetViaApi(createdDatasetIdentifiers.persistentId)
   })
 
   test('should deaccession a dataset when it is deaccessioned once', async () => {