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

デフォルトプリセットがバグってるconfig.jsonがマイグレーションできるテストを加える #2062

Closed
Hiroshiba opened this issue May 6, 2024 · 0 comments · Fixed by #2064
Labels
初心者歓迎タスク 初心者にも優しい簡単めなタスク 機能向上

Comments

@Hiroshiba
Copy link
Member

内容

デフォルトプリセットがバグで再登録できない(というより表示に反映されない)ことがありました。
#1996

これは現在すでに修正されています。 ( thx. @madosuki !!!!!!)
#2053

まあまあ複雑で重大なバグだったのと、設定マイグレーションで一部を削除する処理を行っていることも考えると、テストを書いたほうが良さそうに思いました。
実際に過去にデフォルトプリセットを用い、バグったバージョンでデフォルトプリセットが更新されないことを確認したconfig.jsonを用意して、そのマイグレーションができることを確認できたらこのissueは完了だと思います。

Pros 良くなる点

重大なバグが解消されたという触れ込みを自信を持って出せる。

実現方法

バグってないバージョン(0.15.2)で設定ファイルを用意し、デフォルトプリセットの値を変える。
バグってるバージョン(0.19.1)で設定ファイルをマイグレーションし、デフォルトプリセットの値が変えられないことを確認する。
できたバグconfig.jsonをConfigManagerのマイグレーションテストに設定ファイルを足す。

可能であれば設定ファイルのスナップショットテストも取ると更に良さそう。

その他

0.15.2の正常なデフォルトプリセットが含まれるconfig.jsonと、それをマイグレーションして0.19.1でバグったデフォルトプリセットが含まれるconfig.jsonを実際のVOICEVOXで作ってみました!
使っていただけると!

0.15.2-default_preset.json
0.19.1-bug_default_preset.json

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
初心者歓迎タスク 初心者にも優しい簡単めなタスク 機能向上
Projects
None yet
1 participant