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

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

Open
Hiroshiba opened this issue May 22, 2024 · 3 comments
Labels
初心者歓迎タスク 初心者にも優しい簡単めなタスク 機能向上 状態:実装者募集 実装者を募集している状態 非アクティブ

Comments

@Hiroshiba
Copy link
Member

内容

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

@Hiroshiba Hiroshiba added 機能向上 初心者歓迎タスク 初心者にも優しい簡単めなタスク 状態:必要性議論 必要性を議論している状態 labels May 22, 2024
@tarepan
Copy link
Contributor

tarepan commented May 22, 2024

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

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

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

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

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

@tarepan tarepan added 要議論 実行する前に議論が必要そうなもの 状態:設計 設計をおこなっている状態 and removed 状態:必要性議論 必要性を議論している状態 labels May 22, 2024
@Hiroshiba
Copy link
Member Author

Hiroshiba commented May 27, 2024

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

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

@tarepan tarepan added 状態:実装者募集 実装者を募集している状態 and removed 要議論 実行する前に議論が必要そうなもの 状態:設計 設計をおこなっている状態 labels May 27, 2024
Copy link

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
初心者歓迎タスク 初心者にも優しい簡単めなタスク 機能向上 状態:実装者募集 実装者を募集している状態 非アクティブ
Projects
None yet
Development

No branches or pull requests

2 participants