add pagination query para

Author: ChengShi-1

docs/useCases.md

@@ -858,7 +858,9 @@ getDatasetVersionsSummaries
 
 _See [use case](../src/datasets/domain/useCases/GetDatasetVersionsSummaries.ts) implementation_.
 
-The `datasetId` parameter can be a string, for persistent identifiers, or a number, for numeric identifiers.
+- The `datasetId` parameter can be a string, for persistent identifiers, or a number, for numeric identifiers.
+- **limit**: (number) Limit for pagination.
+- **offset**: (number) Offset for pagination.
 
 #### Get Dataset Linked Collections
 
@@ -1984,6 +1986,9 @@ getFileVersionSummaries.execute(fileId).then((fileVersionSummaries: fileVersionS
 
 _See [use case](../src/files/domain/useCases/GetFileVersionSummaries.ts) implementation_.
 
+- **limit**: (number) Limit for pagination.
+- **offset**: (number) Offset for pagination.
+
 ## Metadata Blocks
 
 ### Metadata Blocks read use cases

src/datasets/domain/repositories/IDatasetsRepository.ts

@@ -66,7 +66,11 @@ export interface IDatasetsRepository {
     datasetId: number | string,
     includeMDC?: boolean
   ): Promise<DatasetDownloadCount>
-  getDatasetVersionsSummaries(datasetId: number | string): Promise<DatasetVersionSummaryInfo[]>
+  getDatasetVersionsSummaries(
+    datasetId: number | string,
+    limit?: number,
+    offset?: number
+  ): Promise<DatasetVersionSummaryInfo[]>
   deleteDatasetDraft(datasetId: number | string): Promise<void>
   linkDataset(datasetId: number | string, collectionIdOrAlias: number | string): Promise<void>
   unlinkDataset(datasetId: number | string, collectionIdOrAlias: number | string): Promise<void>

src/datasets/domain/useCases/GetDatasetVersionsSummaries.ts

@@ -14,9 +14,15 @@ export class GetDatasetVersionsSummaries implements UseCase<DatasetVersionSummar
    * Draft versions will only be available to users who have permission to view unpublished drafts.
    *
    * @param {number | string} [datasetId] - The dataset identifier, which can be a string (for persistent identifiers), or a number (for numeric identifiers).
+   * @param {number} [limit] - Limit for pagination (optional).
+   * @param {number} [offset] - Offset for pagination (optional).
    * @returns {Promise<DatasetVersionSummaryInfo[]>} - An array of DatasetVersionSummaryInfo.
    */
-  async execute(datasetId: number | string): Promise<DatasetVersionSummaryInfo[]> {
-    return await this.datasetsRepository.getDatasetVersionsSummaries(datasetId)
+  async execute(
+    datasetId: number | string,
+    limit?: number,
+    offset?: number
+  ): Promise<DatasetVersionSummaryInfo[]> {
+    return await this.datasetsRepository.getDatasetVersionsSummaries(datasetId, limit, offset)
   }
 }

src/datasets/infra/repositories/DatasetsRepository.ts

@@ -306,11 +306,19 @@ export class DatasetsRepository extends ApiRepository implements IDatasetsReposi
   }
 
   public async getDatasetVersionsSummaries(
-    datasetId: string | number
+    datasetId: string | number,
+    limit?: number,
+    offset?: number
   ): Promise<DatasetVersionSummaryInfo[]> {
+    const queryParams: { per_page?: string; start?: string } = {}
+
+    limit && (queryParams.per_page = limit.toString())
+    offset && (queryParams.start = offset.toString())
+
     return this.doGet(
       this.buildApiEndpoint(this.datasetsResourceName, 'versions/compareSummary', datasetId),
-      true
+      true,
+      queryParams
     )
       .then((response) => response.data.data)
       .catch((error) => {

src/files/domain/repositories/IFilesRepository.ts

@@ -88,7 +88,11 @@ export interface IFilesRepository {
     replace?: boolean
   ): Promise<void>
 
-  getFileVersionSummaries(fileId: number | string): Promise<FileVersionSummaryInfo[]>
+  getFileVersionSummaries(
+    fileId: number | string,
+    limit?: number,
+    offset?: number
+  ): Promise<FileVersionSummaryInfo[]>
 
   isFileDeleted(fileId: number | string): Promise<boolean>
 }

src/files/domain/useCases/GetFileVersionSummaries.ts

@@ -13,9 +13,15 @@ export class GetFileVersionSummaries implements UseCase<FileVersionSummaryInfo[]
    * Returns a list of versions for a given file including a summary of differences between consecutive versions
    *
    * @param {number | string} [fileId] - The file identifier, which can be a string (for persistent identifiers), or a number (for numeric identifiers).
+   * @param {number} [limit] - Limit for pagination (optional).
+   * @param {number} [offset] - Offset for pagination (optional).
    * @returns {Promise<FileVersionSummaryInfo[]>} - An array of FileVersionSummaryInfo.
    */
-  async execute(fileId: number | string): Promise<FileVersionSummaryInfo[]> {
-    return await this.filesRepository.getFileVersionSummaries(fileId)
+  async execute(
+    fileId: number | string,
+    limit?: number,
+    offset?: number
+  ): Promise<FileVersionSummaryInfo[]> {
+    return await this.filesRepository.getFileVersionSummaries(fileId, limit, offset)
   }
 }

src/files/infra/repositories/FilesRepository.ts

@@ -423,10 +423,20 @@ export class FilesRepository extends ApiRepository implements IFilesRepository {
       })
   }
 
-  public async getFileVersionSummaries(fileId: number | string): Promise<FileVersionSummaryInfo[]> {
+  public async getFileVersionSummaries(
+    fileId: number | string,
+    limit?: number,
+    offset?: number
+  ): Promise<FileVersionSummaryInfo[]> {
+    const queryParams: { per_page?: string; start?: string } = {}
+
+    limit && (queryParams.per_page = limit.toString())
+    offset && (queryParams.start = offset.toString())
+
     return this.doGet(
       this.buildApiEndpoint(this.filesResourceName, 'versionDifferences', fileId),
-      true
+      true,
+      queryParams
     )
       .then((response) => transformFileVersionSummaryInfoResponseToFileVersionSummaryInfo(response))
       .catch((error) => {

test/integration/datasets/DatasetsRepository.test.ts

@@ -1482,6 +1482,65 @@ describe('DatasetsRepository', () => {
         expectedError
       )
     })
+
+    test('should return dataset versions summaries with pagination', async () => {
+      const testDatasetIds = await createDataset.execute(
+        TestConstants.TEST_NEW_DATASET_DTO,
+        testDatasetVersionsCollectionAlias
+      )
+
+      await publishDataset.execute(testDatasetIds.numericId, VersionUpdateType.MAJOR)
+      await waitForNoLocks(testDatasetIds.numericId, 10)
+
+      const metadataBlocksRepository = new MetadataBlocksRepository()
+      const citationMetadataBlock = await metadataBlocksRepository.getMetadataBlockByName(
+        'citation'
+      )
+
+      for (let i = 1; i <= 21; i++) {
+        await sut.updateDataset(
+          testDatasetIds.numericId,
+          {
+            metadataBlockValues: [
+              {
+                name: 'citation',
+                fields: {
+                  title: `Updated Dataset Title - Version ${i}`
+                }
+              }
+            ]
+          },
+          [citationMetadataBlock]
+        )
+
+        await publishDataset.execute(testDatasetIds.numericId, VersionUpdateType.MINOR)
+        await waitForNoLocks(testDatasetIds.numericId, 10)
+      }
+
+      const firstPage = await sut.getDatasetVersionsSummaries(testDatasetIds.numericId, 5, 0)
+
+      expect(firstPage.length).toBe(5)
+      expect(firstPage[0].versionNumber).toBe('1.21')
+      expect(firstPage[4].versionNumber).toBe('1.17')
+
+      // Test pagination with limit=5, offset=5 (second page)
+      const secondPage = await sut.getDatasetVersionsSummaries(testDatasetIds.numericId, 5, 5)
+      expect(secondPage.length).toBe(5)
+      expect(secondPage[0].versionNumber).toBe('1.16')
+      expect(secondPage[4].versionNumber).toBe('1.12')
+
+      // Test pagination with limit=5, offset=10 (third page)
+      const thirdPage = await sut.getDatasetVersionsSummaries(testDatasetIds.numericId, 5, 10)
+      expect(thirdPage.length).toBe(5)
+      expect(thirdPage[0].versionNumber).toBe('1.11')
+      expect(thirdPage[4].versionNumber).toBe('1.7')
+
+      // Test that all versions are returned without pagination
+      const allVersions = await sut.getDatasetVersionsSummaries(testDatasetIds.numericId)
+      expect(allVersions.length).toBe(22) // 1 initial + 21 updates
+
+      await deletePublishedDatasetViaApi(testDatasetIds.persistentId)
+    }, 180000)
   })
 
   describe('getDatasetDownloadCount', () => {

test/integration/files/FilesRepository.test.ts

@@ -1047,6 +1047,58 @@ describe('FilesRepository', () => {
       deletePublishedDatasetViaApi(fileTestDatasetIds.persistentId)
     })
 
+    test('should return file version summaries with pagination', async () => {
+      // Create a new dataset and upload a file
+      const paginationTestDatasetIds = await createDataset.execute(
+        TestConstants.TEST_NEW_DATASET_DTO
+      )
+      await uploadFileViaApi(paginationTestDatasetIds.numericId, testTextFile1Name)
+
+      // Publish initial version (creates version 1.0)
+      await publishDatasetViaApi(paginationTestDatasetIds.numericId)
+      await waitForNoLocks(paginationTestDatasetIds.numericId, 10)
+
+      // Get the file ID
+      const datasetFiles = await sut.getDatasetFiles(
+        paginationTestDatasetIds.numericId,
+        latestDatasetVersionId,
+        false,
+        FileOrderCriteria.NAME_AZ
+      )
+      const paginationTestFile = datasetFiles.files[0]
+
+      for (let i = 1; i <= 21; i++) {
+        await sut.updateFileMetadata(paginationTestFile.id, {
+          description: `File description update ${i}`,
+          label: `updated-file-${i}.txt`
+        })
+
+        await publishDatasetViaApi(paginationTestDatasetIds.numericId)
+        await waitForNoLocks(paginationTestDatasetIds.numericId, 10)
+      }
+
+      const firstPage = await sut.getFileVersionSummaries(paginationTestFile.id, 5, 0)
+
+      expect(firstPage.length).toBe(5)
+      expect(firstPage[0].datasetVersion).toBe('1.21')
+      expect(firstPage[4].datasetVersion).toBe('1.17')
+
+      const secondPage = await sut.getFileVersionSummaries(paginationTestFile.id, 5, 5)
+      expect(secondPage.length).toBe(5)
+      expect(secondPage[0].datasetVersion).toBe('1.16')
+      expect(secondPage[4].datasetVersion).toBe('1.12')
+
+      const thirdPage = await sut.getFileVersionSummaries(paginationTestFile.id, 5, 10)
+      expect(thirdPage.length).toBe(5)
+      expect(thirdPage[0].datasetVersion).toBe('1.11')
+      expect(thirdPage[4].datasetVersion).toBe('1.7')
+
+      const allVersions = await sut.getFileVersionSummaries(paginationTestFile.id)
+      expect(allVersions.length).toBe(22)
+
+      await deletePublishedDatasetViaApi(paginationTestDatasetIds.persistentId)
+    }, 180000)
+
     test('should return error when file does not exist', async () => {
       const expectedError = new ReadError(`[404] File with ID ${nonExistentFiledId} not found.`)