Revert "feat: use case for dataset download count"

This reverts commit 20f759c.

Author: ChengShi-1

docs/useCases.md

@@ -807,29 +807,6 @@ The `version` parameter should be a string or a [DatasetNotNumberedVersion](../s
 
 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.
 
-#### Get Download Count of a Dataset
-
-Total number of downloads requested for a dataset, given a dataset numeric identifier,
-
-##### Example call:
-
-```typescript
-import { getDatasetDownloadCount } from '@iqss/dataverse-client-javascript'
-
-/* ... */
-
-const datasetId = 1
-const includeMDC = true
-
-getDatasetDownloadCount.execute(datasetId, includeMDC)
-
-/* ... */
-```
-
-_See [use case](../src/datasets/domain/useCases/GetDatasetDownloadCount.ts) implementation_.
-The `datasetId` parameter is a number for numeric identifiers.
-The `includeMDC` parameter is optional. If MDC is enabled the count will be limited to the time before MDC start if the optional `includeMDC` parameter is not included or set to False. Setting `includeMDC` to True will ignore the `:MDCStartDate` setting and return a total count.
-
 ## Files
 
 ### Files read use cases

src/datasets/domain/repositories/IDatasetsRepository.ts

@@ -51,5 +51,4 @@ export interface IDatasetsRepository {
     datasetVersionId: string,
     deaccessionDTO: DatasetDeaccessionDTO
   ): Promise<void>
-  getDatasetDownloadCount(datasetId: number, includeMDC?: boolean): Promise<number>
 }

src/datasets/domain/useCases/GetDatasetDownloadCount.ts

@@ -17,7 +17,6 @@ import { PublishDataset } from './domain/useCases/PublishDataset'
 import { UpdateDataset } from './domain/useCases/UpdateDataset'
 import { GetDatasetVersionDiff } from './domain/useCases/GetDatasetVersionDiff'
 import { DeaccessionDataset } from './domain/useCases/DeaccessionDataset'
-import { GetDatasetDownloadCount } from './domain/useCases/GetDatasetDownloadCount'
 
 const datasetsRepository = new DatasetsRepository()
 
@@ -49,7 +48,6 @@ const updateDataset = new UpdateDataset(
   datasetResourceValidator
 )
 const deaccessionDataset = new DeaccessionDataset(datasetsRepository)
-const getDatasetDownloadCount = new GetDatasetDownloadCount(datasetsRepository)
 
 export {
   getDataset,
@@ -64,8 +62,7 @@ export {
   publishDataset,
   createDataset,
   updateDataset,
-  deaccessionDataset,
-  getDatasetDownloadCount
+  deaccessionDataset
 }
 export { DatasetNotNumberedVersion } from './domain/models/DatasetNotNumberedVersion'
 export { DatasetUserPermissions } from './domain/models/DatasetUserPermissions'

src/datasets/index.ts

@@ -235,18 +235,4 @@ export class DatasetsRepository extends ApiRepository implements IDatasetsReposi
         throw error
       })
   }
-
-  public async getDatasetDownloadCount(datasetId: number, includeMDC?: boolean): Promise<number> {
-    const queryParams = includeMDC !== undefined ? { includeMDC } : {}
-
-    return this.doGet(
-      this.buildApiEndpoint(this.datasetsResourceName, `${datasetId}/download/count`),
-      true,
-      queryParams
-    )
-      .then((response) => response.data.downloadCount)
-      .catch((error) => {
-        throw error
-      })
-  }
 }

src/datasets/infra/repositories/DatasetsRepository.ts

@@ -925,41 +925,4 @@ describe('DatasetsRepository', () => {
       ).rejects.toBeInstanceOf(WriteError)
     })
   })
-
-  describe('getDatasetDownloadCount', () => {
-    test('should return download count for a dataset', async () => {
-      const testDatasetIds = await createDataset.execute(TestConstants.TEST_NEW_DATASET_DTO)
-      const fileMetadata = {
-        description: 'test description',
-        directoryLabel: 'directoryLabel',
-        categories: ['category1', 'category2']
-      }
-      await uploadFileViaApi(testDatasetIds.numericId, testTextFile1Name, fileMetadata)
-      await publishDatasetViaApi(testDatasetIds.numericId)
-      await waitForNoLocks(testDatasetIds.numericId, 10)
-      const actual = await sut.getDatasetDownloadCount(testDatasetIds.numericId)
-
-      expect(actual).toBe(0)
-
-      await deletePublishedDatasetViaApi(testDatasetIds.persistentId)
-    })
-
-    test('should return download count including MDC data', async () => {
-      const testDatasetIds = await createDataset.execute(TestConstants.TEST_NEW_DATASET_DTO)
-      await publishDatasetViaApi(testDatasetIds.numericId)
-      await waitForNoLocks(testDatasetIds.numericId, 10)
-
-      const actual = await sut.getDatasetDownloadCount(testDatasetIds.numericId, true)
-
-      expect(actual).toBe(0)
-
-      await deletePublishedDatasetViaApi(testDatasetIds.persistentId)
-    })
-
-    test('should return error when dataset does not exist', async () => {
-      await expect(sut.getDatasetDownloadCount(nonExistentTestDatasetId)).rejects.toBeInstanceOf(
-        ReadError
-      )
-    })
-  })
 })