Open Hiroshiba opened 6 months ago
tarepan
commented
6 months ago ファイルが存在するかテストしておきたい
👍️
必要性に同意します。
ファイルがあり、jsonを読め、なにか適当なキーが含まれてることの検証を
check_release_build.pyに実装
check_release_build.py が test-engine-container.yml でも走るため、コンテナ内の engine_manifest.json に触れる設計を練る必要がありそうです。
Hiroshiba
commented
6 months ago engine_manifest.jsonはエンジンを起動させるために必要なもので、dockerの場合はもうエンジンが走っているのでチェックは不要かもです！
（実際docker側のテストは--skip_run_processが指定されてます）
check_release_build.pyにskip_run_processと同じようなskip引数を追加するか、
あるいは「エンジン起動テストをする」みたいなフラグを用意して、そのフラグがONのときにエンジン起動とマニフェストファイルのチェックを行うとかが妥当かなと･･･！
github-actions[bot]
commented
1 week ago 本 Issue は直近 180 日間で活動がありません。今後の方針について VOICEVOX チームによる再検討がおこなわれる予定です。
内容
リリースビルドを自動でテストするコードがあります。github workflowでビルドしたあとに自動的に実行されます。 https://github.com/VOICEVOX/voicevox_engine/blob/master/build_util/check_release_build.py
エンジンにはマニフェストファイルがroot直下にあり、これはエンジン起動にも必要な大事なファイルなので、ファイルが存在するかテストしておきたいです。 ファイルがあり、jsonを読め、なにか適当なキーが含まれてることの検証を
check_release_build.pyに実装できたらタスク完了です。Pros 良くなる点
アプデ時、意図しないバグを防ぎやすくなる。
実現方法
マニフェストファイル
engine_manifest.jsonがあるか確認する。