リリースビルドのチェックテスト時に、engine_manifest.jsonファイルがあるか検証する

[Hiroshiba]

内容

リリースビルドを自動でテストするコードがあります。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があるか確認する。

[tarepan]

#1259 で明確化されたパス要件のテスト、と認識しました。

ファイルが存在するかテストしておきたい

👍️
必要性に同意します。

ファイルがあり、jsonを読め、なにか適当なキーが含まれてることの検証をcheck_release_build.pyに実装

check_release_build.py が test-engine-container.yml でも走るため、コンテナ内の engine_manifest.json に触れる設計を練る必要がありそうです。

[Hiroshiba]

engine_manifest.jsonはエンジンを起動させるために必要なもので、dockerの場合はもうエンジンが走っているのでチェックは不要かもです！
（実際docker側のテストは--skip_run_processが指定されてます）

check_release_build.pyにskip_run_processと同じようなskip引数を追加するか、
あるいは「エンジン起動テストをする」みたいなフラグを用意して、そのフラグがONのときにエンジン起動とマニフェストファイルのチェックを行うとかが妥当かなと･･･！

[github-actions]

本 Issue は直近 180 日間で活動がありません。今後の方針について VOICEVOX チームによる再検討がおこなわれる予定です。

[nanae772]

初めまして。VOICEVOXの開発に興味があり、こちらのタスクに取り組んでみてもよいでしょうか？

対応内容としては

voicevox_engine/tools/check_release_build.py

Lines 24 to 27 in ab6f180

	# 起動
	process = None
	if not skip_run_process:
	process = Popen([run_file.absolute()], cwd=dist_dir)

この箇所の前でmanifestの存在確認やjsonとして読めるかなどを行えばよいという感じでしょうか？

[nanae772]

続けて失礼します。一旦上記方針でこちらPR作成してみました。

#1526