Add: Python APIのテストを追加 #484
Conversation
|
やっとテスト通りました、Draft外します |
There was a problem hiding this comment.
すみません、大作を頂いた後に申し訳ないのですが、現状だと同じテストを別言語で２回書かれている形になるので、メンテナンスが二度手間になってしまっていそうに思いました。
入力内容と出力結果を共通化すれば、Python側のメンテナンスをかなり楽にできそう･･･？
C APIのe2eテストはこちらにあります。これを真似ても（というか真似たほうが）良いかも？
https://github.com/VOICEVOX/voicevox_core/tree/ff7b5a88865d684132f5995ff456273f7cf18f52/crates/voicevox_core_c_api/tests/e2e/testcases
（先に言えてなくて申し訳ないです･･･。）
Python側のテストが落ちたらもうコメントアウトしちゃうくらいの気持ちであればメンテナンス負荷は据え置きなのですが、それはテストとして意味があるのかという････。
楽にしておきたいな～、でもテストあると良いよな～、という微妙なところです！
ちょっとどうすべきか他の方の意見もほしいかもです。
|
おっと、見逃してました。
ですね、それで良さそうです。変えてみます。 |
|
@sevenc-nanashi vvm-async-apiでいくつか足そうと考えています。 |
|
とりあえず現在のC APIテスト準拠にしてみました。 |
|
pythonのテストが落ちてそうでした！ |
|
cwdのズレコピーできてなかったっぽいです。 |
Co-authored-by: Ryo Yamashita <qryxip@gmail.com>
Co-Authored-By: Qryxip <qryxip@users.noreply.github.com>
Co-Authored-By: qryxip <qryxip@users.noreply.github.com>
|
レビューを反映しました（報告遅れました） |
|
今気付いてしまったのですが、project-vvm-async-apiでは |
Co-authored-by: Ryo Yamashita <qryxip@gmail.com>
|
レビューを反映しました。 black、isortのチェックをするActionをどこかに足しても良いかも？ |
There was a problem hiding this comment.
LGTM。
#484 (comment)については、test_engine.pyは抹消してtest_ttsとかにすればいい感じでしょうか?
|
そんな感じで大丈夫だと思います。 |
|
compatible_engineは残るから マージします！ |
内容
Python APIのテストを追加しました。
C APIのe2eテストを移植した感じになっています。
関連 Issue
その他
（なし）