-
Notifications
You must be signed in to change notification settings - Fork 311
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
test: engineMockとそのテストだけ実装する #2377
The head ref may contain hidden characters: "\u3068\u308A\u3042\u3048\u305AengineMock\u3068\u305D\u306E\u30C6\u30B9\u30C8\u3060\u3051\u5B9F\u88C5\u3059\u308B"
test: engineMockとそのテストだけ実装する #2377
Conversation
🚀 プレビュー用ページを作成しました 🚀 更新時点でのコミットハッシュ: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot reviewed 18 out of 25 changed files in this pull request and generated no suggestions.
Files not reviewed (7)
- package.json: Language not supported
- src/mock/engineMock/index.ts: Evaluated as low risk
- src/mock/engineMock/singModelMock.ts: Evaluated as low risk
- src/mock/engineMock/synthesisMock.ts: Evaluated as low risk
- src/store/singing.ts: Evaluated as low risk
- src/helpers/convertToWavFileData.ts: Evaluated as low risk
- src/mock/engineMock/characterResourceMock.ts: Evaluated as low risk
このプルリクエスト自体は問題ないのですが、Nodeでしか動かない状況になっており、ブラウザでモックを使おうとした方が不利益に時間を使ってしまうようになるので、そっちも解決してからdraftを開けようと思います! |
プルリクエストを開けました! エンジンのモック部分はコードが多すぎてレビューできないと思うので、これはえいやでマージしちゃいたいです。 それ以外の部分のファイル構造やモックの説明readme、あとモック自体のテストとか見ていただけると心強いです!! |
@sevenc-nanashi さんすみません、可能な範囲で大丈夫なのでレビューもらってもいいですか 🙇 🙇 エンジンモックの計算式の中身とかはレビューに相当時間かかると思うので、ここは主に僕がメンテし続ける感じでいいと思ってます!
だけ確認もらえると心強いです・・・!! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
特に問題は無さそうでした。
レビューありがとうございます! マージします!! |
内容
の実装です。e2eを目標としていたプルリクエストから派生しています。
とりあえず、Vuex内で使っているOpenAPI準拠のDefaulApiインターフェースを継承し、利用されている音声合成系の大体の機能を模倣した実装のプルリクエストです。
あとengineMockを利用した単体テストでのスナップショットテストだけ作ってます。
以下、engineMock/README.mdの転載です。
関連 Issue
close #2144
その他
とりあえずこのプルリクエストで実装だけえいやで入れちゃいたいです。
(ものすごい巨大になっているのでおそらくレビューできない・・・。)
モックが生み出す音声を聞いてみたい場合はこちらで実際に動かせます(2024/12/15現在)
https://voicevox.github.io/preview-pages/preview/pr-2152/editor/index.html