-
Notifications
You must be signed in to change notification settings - Fork 309
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
base: main
Are you sure you want to change the base?
The head ref may contain hidden characters: "\u3068\u308A\u3042\u3048\u305AengineMock\u3068\u305D\u306E\u30C6\u30B9\u30C8\u3060\u3051\u5B9F\u88C5\u3059\u308B"
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
const assetsPath = path.resolve(__dirname, "assets"); | ||
const assetsPath = path.resolve( | ||
__dirname, | ||
"../../../src/mock/engineMock/assets", | ||
); |
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.
モック画像の場所が変わったので追従しています。
そもそもモックエンジンを使えばここの書き換えはなくせるので、後々また書き換える予定です。
このプルリクエスト自体は問題ないのですが、Nodeでしか動かない状況になっており、ブラウザでモックを使おうとした方が不利益に時間を使ってしまうようになるので、そっちも解決してからdraftを開けようと思います! |
プルリクエストを開けました! エンジンのモック部分はコードが多すぎてレビューできないと思うので、これはえいやでマージしちゃいたいです。 それ以外の部分のファイル構造やモックの説明readme、あとモック自体のテストとか見ていただけると心強いです!! |
内容
の実装です。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