fix: optional replace

Author: ChengShi-1

src/files/domain/useCases/UpdateFileCategories.ts

@@ -14,7 +14,7 @@ export class UpdateFileCategories implements UseCase<void> {
    *
    * @param {number | string} [fileId] - The file identifier, which can be a string (for persistent identifiers), or a number (for numeric identifiers).
    * @param {string[]} [categories] - The categories to be added to the file.
-   * @param {boolean} [replace] - If true, replaces the existing categories with the new ones. If false, adds the new categories to the existing ones.
+   * @param {boolean} [replace](optional) - If true, replaces the existing categories with the new ones. If false, adds the new categories to the existing ones.
    * @returns {Promise<void>}
    */
   async execute(fileId: number | string, categories: string[], replace?: boolean): Promise<void> {

src/files/domain/useCases/UpdateFileTabularTags.ts

@@ -14,7 +14,7 @@ export class UpdateFileTabularTags implements UseCase<void> {
    *
    * @param {number | string} [fileId] - The file identifier, which can be a string (for persistent identifiers), or a number (for numeric identifiers).
    * @param {string[]} [tabularTags] - The tabular tags to be added to the file.
-   * @param {boolean} [replace] - If true, replaces the existing tabularTags with the new ones. If false, adds the new tabularTags to the existing ones.
+   * @param {boolean} [replace](optional) - If true, replaces the existing tabularTags with the new ones. If false, adds the new tabularTags to the existing ones.
    * @returns {Promise<void>}
    */
   async execute(fileId: number | string, tabularTags: string[], replace?: boolean): Promise<void> {

src/files/infra/repositories/FilesRepository.ts

@@ -387,10 +387,11 @@ export class FilesRepository extends ApiRepository implements IFilesRepository {
     tabularTags: string[],
     replace?: boolean
   ): Promise<void> {
+    const queryParams = replace !== undefined ? { replace } : {}
     return this.doPost(
       this.buildApiEndpoint(this.filesResourceName, 'metadata/tabularTags', fileId),
       { tabularTags },
-      { replace: replace ?? false }
+      queryParams
     )
       .then(() => undefined)
       .catch((error) => {
@@ -403,10 +404,11 @@ export class FilesRepository extends ApiRepository implements IFilesRepository {
     categories: string[],
     replace?: boolean
   ): Promise<void> {
+    const queryParams = replace !== undefined ? { replace } : {}
     return this.doPost(
       this.buildApiEndpoint(this.filesResourceName, 'metadata/categories', fileId),
       { categories },
-      { replace: replace ?? false }
+      queryParams
     )
       .then(() => undefined)
       .catch((error) => {

test/integration/files/FilesRepository.test.ts

@@ -398,7 +398,7 @@ describe('FilesRepository', () => {
     })
 
     test('should return error when file does not exist', async () => {
-      const expectedError = new ReadError(`[404] File with ID ${nonExistentFiledId} not found.`)
+      const expectedError = new ReadError(`[404] File with ID ${nonExistentFiledId} not found`)
 
       await expect(sut.getFileDownloadCount(nonExistentFiledId)).rejects.toThrow(expectedError)
     })