Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

整理: 音声合成系テストの utils を統廃合 #1428

Merged
merged 3 commits into from
Jun 24, 2024

Conversation

tarepan
Copy link
Contributor

@tarepan tarepan commented Jun 24, 2024

内容

概要: 音声合成系テストの utils を統廃合してリファクタリング

音声合成系テストは繰り返し処理を util 関数として切り出している。
度重なるテストの更新により、一部の util は切り出す必要がなくなっている。逆に一部の util は複数ファイルでそれぞれ定義されており共通化すべき状態にある。

このような背景から、音声合成系テストの utils を廃止・統合するリファクタリングを提案します。

変更された utils は以下になります:

  • _gen_mora(): 複数ファイルで統一関数が定義(コピペ)されていたため、モジュールへ切り出しリネーム
  • _gen_hello_hiho_text(): ベタ書きして削除
  • _gen_hello_hiho_kana(): ベタ書きして削除

関連 Issue

無し

@tarepan tarepan requested a review from a team as a code owner June 24, 2024 14:12
@tarepan tarepan requested review from Hiroshiba and removed request for a team June 24, 2024 14:12
Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@Hiroshiba Hiroshiba merged commit 4a5541e into VOICEVOX:master Jun 24, 2024
4 checks passed
@tarepan tarepan deleted the refactor/core_test branch June 24, 2024 16:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants