From 1d2089b0c4910a0af53b75cc81a74ee9a16a3993 Mon Sep 17 00:00:00 2001 From: "makabe.t" Date: Tue, 16 Apr 2024 05:24:18 +0000 Subject: [PATCH] =?UTF-8?q?Merged=20PR=20868:=20=E9=9F=B3=E5=A3=B0?= =?UTF-8?q?=E3=83=95=E3=82=A1=E3=82=A4=E3=83=AB=E5=90=8D=E5=A4=89=E6=9B=B4?= =?UTF-8?q?=E3=81=AE=E3=81=AB=E4=BC=B4=E3=81=86API=E4=BF=AE=E6=AD=A3?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ## 概要 [Task4053: 音声ファイル名変更のに伴うAPI修正](https://paruru.nds-tyo.co.jp:8443/tfs/ReciproCollection/fa4924a4-d079-4fab-9fb5-a9a11eb205f0/_workitems/edit/4053) - 音声ファイル名について生ファイル名を利用するように修正しました。 - ファイルアップロード完了(タスク登録) - パラメータのファイル名で生ファイル名も登録 - 音声ファイルダウンロード先取得 - タスク削除 - blobストレージのファイル名に生ファイル名を利用 ## レビューポイント - 対象APIは認識通りか ## UIの変更 - なし ## クエリの変更 - なし ## 動作確認状況 - ローカルで確認 - 行った修正がデグレを発生させていないことを確認できるか - 具体的にどのような確認をしたか - UTが通ることを確認 - 各APIで生ファイル名を使って実行できることを確認 --- dictation_server/src/features/files/files.service.ts | 4 ++-- dictation_server/src/features/tasks/tasks.service.spec.ts | 4 ++-- dictation_server/src/features/tasks/tasks.service.ts | 2 +- dictation_server/src/features/tasks/test/utility.ts | 1 + .../src/repositories/tasks/tasks.repository.service.ts | 1 + 5 files changed, 7 insertions(+), 5 deletions(-) diff --git a/dictation_server/src/features/files/files.service.ts b/dictation_server/src/features/files/files.service.ts index f6a023c..3d0084a 100644 --- a/dictation_server/src/features/files/files.service.ts +++ b/dictation_server/src/features/files/files.service.ts @@ -611,7 +611,7 @@ export class FilesService { } } - const filePath = `${file.file_name}`; + const filePath = `${file.raw_file_name}`; const isFileExist = await this.blobStorageService.fileExists( context, @@ -623,7 +623,7 @@ export class FilesService { if (!isFileExist) { this.logger.log(`[${context.getTrackingId()}] filePath:${filePath}`); throw new AudioFileNotFoundError( - `Audio file is not exists in blob storage. audio_file_id:${audioFileId}, url:${file.url}, fileName:${file.file_name}`, + `Audio file is not exists in blob storage. audio_file_id:${audioFileId}, url:${file.url}, fileName:${filePath}`, ); } diff --git a/dictation_server/src/features/tasks/tasks.service.spec.ts b/dictation_server/src/features/tasks/tasks.service.spec.ts index 0799356..5fe8662 100644 --- a/dictation_server/src/features/tasks/tasks.service.spec.ts +++ b/dictation_server/src/features/tasks/tasks.service.spec.ts @@ -4415,7 +4415,7 @@ describe('deleteTask', () => { context, account.id, account.country, - 'x.zip', + 'y.zip', ); } }); @@ -4500,7 +4500,7 @@ describe('deleteTask', () => { context, account.id, account.country, - 'x.zip', + 'y.zip', ); } }); diff --git a/dictation_server/src/features/tasks/tasks.service.ts b/dictation_server/src/features/tasks/tasks.service.ts index 11b42ea..871d83f 100644 --- a/dictation_server/src/features/tasks/tasks.service.ts +++ b/dictation_server/src/features/tasks/tasks.service.ts @@ -932,7 +932,7 @@ export class TasksService { Object.values(TASK_STATUS), ); - const targetFileName = task.file?.file_name; + const targetFileName = task.file?.raw_file_name; if (!targetFileName) { throw new Error(`target file not found. audioFileId: ${audioFileId}`); } diff --git a/dictation_server/src/features/tasks/test/utility.ts b/dictation_server/src/features/tasks/test/utility.ts index 6a1e024..31928e0 100644 --- a/dictation_server/src/features/tasks/test/utility.ts +++ b/dictation_server/src/features/tasks/test/utility.ts @@ -120,6 +120,7 @@ export const createTask = async ( owner_user_id: owner_user_id, url: '', file_name: 'x.zip', + raw_file_name: 'y.zip', author_id: author_id, work_type_id: work_type_id, started_at: new Date(), diff --git a/dictation_server/src/repositories/tasks/tasks.repository.service.ts b/dictation_server/src/repositories/tasks/tasks.repository.service.ts index d712597..4a1e484 100644 --- a/dictation_server/src/repositories/tasks/tasks.repository.service.ts +++ b/dictation_server/src/repositories/tasks/tasks.repository.service.ts @@ -854,6 +854,7 @@ export class TasksRepositoryService { audioFile.owner_user_id = owner_user_id; audioFile.url = url; audioFile.file_name = file_name; + audioFile.raw_file_name = file_name; audioFile.author_id = author_id; audioFile.work_type_id = work_type_id; audioFile.started_at = started_at;