We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
デフォルトプリセットがバグで再登録できない(というより表示に反映されない)ことがありました。 #1996
これは現在すでに修正されています。 ( thx. @madosuki !!!!!!) #2053
まあまあ複雑で重大なバグだったのと、設定マイグレーションで一部を削除する処理を行っていることも考えると、テストを書いたほうが良さそうに思いました。 実際に過去にデフォルトプリセットを用い、バグったバージョンでデフォルトプリセットが更新されないことを確認したconfig.jsonを用意して、そのマイグレーションができることを確認できたらこのissueは完了だと思います。
重大なバグが解消されたという触れ込みを自信を持って出せる。
バグってないバージョン(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
The text was updated successfully, but these errors were encountered:
Successfully merging a pull request may close this issue.
内容
デフォルトプリセットがバグで再登録できない(というより表示に反映されない)ことがありました。
#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
The text was updated successfully, but these errors were encountered: