fix: copilot reviews

Author: ChengShi-1

CHANGELOG.md

@@ -16,7 +16,7 @@ This changelog follows the principles of [Keep a Changelog](https://keepachangel
 - New Use Case: [Get a Template](./docs/useCases.md#get-a-template) under Templates.
 - New Use Case: [Delete a Template](./docs/useCases.md#delete-a-template) under Templates.
 - New Use Case: [Update Terms of Access](./docs/useCases.md#update-terms-of-access).
-- Guestbooks: Added use cases and repository support for Guestbook CRUD.
+- Guestbooks: Added use cases and repository support for guestbook creation, listing, and enabling/disabling.
 - Access: Added a dedicated `access` module for guestbook-at-request and download terms/guestbook submission endpoints.
 
 ### Changed

docs/useCases.md

@@ -2917,7 +2917,13 @@ import { submitGuestbookForDatafilesDownload } from '@iqss/dataverse-client-java
 
 submitGuestbookForDatafilesDownload
   .execute([10, 11], {
-    guestbookResponse: { answers: [{ id: 123, value: 'Good' }] }
+    guestbookResponse: {
+      answers: [
+        { id: 123, value: 'Good' },
+        { id: 124, value: ['Multi', 'Line'] },
+        { id: 125, value: 'Yellow' }
+      ]
+    }
   })
   .then((signedUrl: string) => {
     /* ... */
@@ -2937,7 +2943,13 @@ import { submitGuestbookForDatasetDownload } from '@iqss/dataverse-client-javasc
 
 submitGuestbookForDatasetDownload
   .execute('doi:10.5072/FK2/XXXXXX', {
-    guestbookResponse: { answers: [{ id: 123, value: 'Good' }] }
+    guestbookResponse: {
+      answers: [
+        { id: 123, value: 'Good' },
+        { id: 124, value: ['Multi', 'Line'] },
+        { id: 125, value: 'Yellow' }
+      ]
+    }
   })
   .then((signedUrl: string) => {
     /* ... */
@@ -2957,7 +2969,13 @@ import { submitGuestbookForDatasetVersionDownload } from '@iqss/dataverse-client
 
 submitGuestbookForDatasetVersionDownload
   .execute(10, ':latest', {
-    guestbookResponse: { answers: [{ id: 123, value: 'Good' }] }
+    guestbookResponse: {
+      answers: [
+        { id: 123, value: 'Good' },
+        { id: 124, value: ['Multi', 'Line'] },
+        { id: 125, value: 'Yellow' }
+      ]
+    }
   })
   .then((signedUrl: string) => {
     /* ... */

src/guestbooks/domain/useCases/GetGuestbooksByCollectionId.ts

@@ -2,7 +2,7 @@ import { UseCase } from '../../../core/domain/useCases/UseCase'
 import { IGuestbooksRepository } from '../repositories/IGuestbooksRepository'
 import { Guestbook } from '../models/Guestbook'
 
-export class GetGuestbooksBycollectionId implements UseCase<Guestbook[]> {
+export class GetGuestbooksByCollectionId implements UseCase<Guestbook[]> {
   constructor(private readonly guestbooksRepository: IGuestbooksRepository) {}
 
   /**

src/guestbooks/index.ts

@@ -1,14 +1,14 @@
 import { GuestbooksRepository } from './infra/repositories/GuestbooksRepository'
 import { CreateGuestbook } from './domain/useCases/CreateGuestbook'
 import { GetGuestbook } from './domain/useCases/GetGuestbook'
-import { GetGuestbooksBycollectionId } from './domain/useCases/GetGuestbooksByCollectionId'
+import { GetGuestbooksByCollectionId } from './domain/useCases/GetGuestbooksByCollectionId'
 import { SetGuestbookEnabled } from './domain/useCases/SetGuestbookEnabled'
 
 const guestbooksRepository = new GuestbooksRepository()
 
 const createGuestbook = new CreateGuestbook(guestbooksRepository)
 const getGuestbook = new GetGuestbook(guestbooksRepository)
-const getGuestbooksBycollectionId = new GetGuestbooksBycollectionId(guestbooksRepository)
+const getGuestbooksBycollectionId = new GetGuestbooksByCollectionId(guestbooksRepository)
 const setGuestbookEnabled = new SetGuestbookEnabled(guestbooksRepository)
 
 export { createGuestbook, getGuestbook, getGuestbooksBycollectionId, setGuestbookEnabled }

…tGuestbooksByDataverseIdentifier.test.ts …ooks/GetGuestbooksByCollectionId.test.tstest/unit/guestbooks/GetGuestbooksByDataverseIdentifier.test.ts renamed to test/unit/guestbooks/GetGuestbooksByCollectionId.test.ts test/unit/guestbooks/GetGuestbooksByDataverseIdentifier.test.ts renamed to test/unit/guestbooks/GetGuestbooksByCollectionId.test.ts

@@ -1,9 +1,9 @@
 import { ReadError } from '../../../src'
 import { Guestbook } from '../../../src/guestbooks/domain/models/Guestbook'
 import { IGuestbooksRepository } from '../../../src/guestbooks/domain/repositories/IGuestbooksRepository'
-import { GetGuestbooksBycollectionId } from '../../../src/guestbooks/domain/useCases/GetGuestbooksByCollectionId'
+import { GetGuestbooksByCollectionId } from '../../../src/guestbooks/domain/useCases/GetGuestbooksByCollectionId'
 
-describe('GetGuestbooksBycollectionId', () => {
+describe('GetGuestbooksByCollectionId', () => {
   const guestbooks: Guestbook[] = [
     {
       id: 12,
@@ -24,7 +24,7 @@ describe('GetGuestbooksBycollectionId', () => {
     const repository: IGuestbooksRepository = {} as IGuestbooksRepository
     repository.getGuestbooksBycollectionId = jest.fn().mockResolvedValue(guestbooks)
 
-    const sut = new GetGuestbooksBycollectionId(repository)
+    const sut = new GetGuestbooksByCollectionId(repository)
     const actual = await sut.execute(collectionId)
 
     expect(repository.getGuestbooksBycollectionId).toHaveBeenCalledWith(collectionId)
@@ -34,7 +34,7 @@ describe('GetGuestbooksBycollectionId', () => {
   test('should throw ReadError when repository fails', async () => {
     const repository: IGuestbooksRepository = {} as IGuestbooksRepository
     repository.getGuestbooksBycollectionId = jest.fn().mockRejectedValue(new ReadError())
-    const sut = new GetGuestbooksBycollectionId(repository)
+    const sut = new GetGuestbooksByCollectionId(repository)
 
     await expect(sut.execute(collectionId)).rejects.toThrow(ReadError)
   })