整理: e2e single API テスト vol 8

[tarepan]

内容

次のAPIに対しスナップショットテストを追加した。

• POST /audio_query-200
• POST /synthesis-200
• POST /frame_synthesis-200
• POST /multi_synthesis
  • 200
    • snapshot
• POST /synthesis_morphing-200

Notes

.wav ファイルのスナップショットテストは #1064 と同様のワークアラウンドを利用した。

関連 Issue

part of #1065

[tarepan]

POST synthesis_morphing で Linux-MacOS 数値精度問題が発生している...？

[github-actions]

Coverage Result

Resultを開く

Name	Stmts	Miss	Cover
run.py	277	122
voicevox_engine/init.py	1	0
voicevox_engine/app/init.py	0	0
voicevox_engine/app/dependencies.py	9	0
voicevox_engine/app/routers/init.py	0	0
voicevox_engine/app/routers/preset.py	37	4
voicevox_engine/app/routers/setting.py	23	3
voicevox_engine/app/routers/speaker.py	59	5
voicevox_engine/app/routers/tts_pipeline.py	121	29
voicevox_engine/app/routers/user_dict.py	60	29
voicevox_engine/cancellable_engine.py	97	75
voicevox_engine/core/init.py	0	0
voicevox_engine/core/core_adapter.py	81	6
voicevox_engine/core/core_initializer.py	60	30
voicevox_engine/core/core_wrapper.py	228	160
voicevox_engine/dev/init.py	0	0
voicevox_engine/dev/core/init.py	0	0
voicevox_engine/dev/core/mock.py	65	2
voicevox_engine/dev/tts_engine/init.py	0	0
voicevox_engine/dev/tts_engine/mock.py	28	0
voicevox_engine/engine_manifest/EngineManifest.py	36	0
voicevox_engine/engine_manifest/EngineManifestLoader.py	11	0
voicevox_engine/engine_manifest/init.py	0	0
voicevox_engine/library_manager.py	92	4
voicevox_engine/metas/Metas.py	36	0
voicevox_engine/metas/MetasStore.py	28	1
voicevox_engine/metas/init.py	0	0
voicevox_engine/model.py	180	3
voicevox_engine/morphing.py	72	4
voicevox_engine/preset/Preset.py	13	0
voicevox_engine/preset/PresetError.py	2	0
voicevox_engine/preset/PresetManager.py	82	2
voicevox_engine/preset/init.py	0	0
voicevox_engine/setting/Setting.py	9	0
voicevox_engine/setting/SettingLoader.py	20	0
voicevox_engine/setting/init.py	0	0
voicevox_engine/tts_pipeline/init.py	0	0
voicevox_engine/tts_pipeline/kana_converter.py	88	1
voicevox_engine/tts_pipeline/mora_mapping.py	7	0
voicevox_engine/tts_pipeline/phoneme.py	34	0
voicevox_engine/tts_pipeline/text_analyzer.py	146	6
voicevox_engine/tts_pipeline/tts_engine.py	268	9
voicevox_engine/user_dict/part_of_speech_data.py	5	0
voicevox_engine/user_dict/user_dict.py	146	12
voicevox_engine/utility/init.py	0	0
voicevox_engine/utility/connect_base64_waves.py	37	0
voicevox_engine/utility/core_utility.py	6	0
voicevox_engine/utility/core_version_utility.py	8	1
voicevox_engine/utility/mutex_utility.py	13	0
voicevox_engine/utility/path_utility.py	26	6
voicevox_engine/utility/run_utility.py	10	7
TOTAL	2521	521

[tarepan]

POST synthesis_morphing のスナップショットテスト追加をリバートしました。
Review 可能です。

[Hiroshiba]

ちょっと細かいところも含めていっぱいコメントさせていただきました🙇

[Hiroshiba]

たしかnumpy辺りはtobufferみたいなのでバイナリにできるので、それ渡すhash_long_bytesみたいなのをいつか作っても良いかもですね。
ちょっと正確でちょっと早そうでちょっと意図が明確、くらいの気持ちですが

[tarepan]

numpy辺りはtobufferみたいなのでバイナリにできるので、それ渡すhash_long_bytesみたいなのをいつか作っても良い

round_floats() の入力が ndarray でなく float であり、現時点でこの実装をするのは難しそうです。
round_floats() 変更は本 PR で触っていない複数のファイル編集を要するため、別のリファクタリング PR で将来的に対処するのが review 上好ましいと考えます。

「本 PR では対処無し」の方針で問題ないでしょうか？

[Hiroshiba]

ちょっと細かいところかもなのですがコメントしました！

[tarepan]

@Hiroshiba
全指摘箇所の反映・テストパスを確認しました。Re-Re-review よろしくお願いします。

[Hiroshiba]

ほぼLGTMです！！

とてもわかりやすくなったと思います、ありがとうございます！！
１点だけコメントしました 🙇

[tarepan]

@Hiroshiba
全指摘箇所の反映・テストパスを確認しました。Re³-review よろしくお願いします。

[Hiroshiba]

LGTM！！

とてもコーディングしやすくなったと思います、ありがとうございます！！