Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
🚀 プレビュー用ページを作成しました 🚀 更新時点でのコミットハッシュ： |
Hiroshiba
commented
Mar 1, 2025
Comment on lines
-29
to
+30
| interrogativeUpspeak: false, | ||
| adjustPauseLength: true, | ||
| interrogativeUpspeak: true, |
There was a problem hiding this comment.
ポーズの長さを変えるのもテストしたかったのでモックを修正しました。
| vowelLength: 1 - 1 / (accentPhrases.length + 1), | ||
| vowelLength: 1 - 1 / (accentPhrases.length + 2), |
There was a problem hiding this comment.
今のaccentPhrasesの数がaccentPhrases.lengthに入っていて、最初は0なので1 - 1/1になって0（ポーズの長さが0）になってしまってたので、+2にしてみました。
| import { test, expect, Page } from "@playwright/test"; | ||
| import { gotoHome, navigateToMain } from "../navigators"; | ||
| import { getQuasarMenu } from "../locators"; | ||
| import { mockShowSaveFileDialog, mockWriteFile } from "./utility"; |
There was a problem hiding this comment.
window.backends関数のモックを作るにあたっていくつか妥協したポイントもあったのでコメント。
より良い解決策があれば試したいかも。
- mockを戻せるように作ってない
- 今モック化したのはダイアログ表示の部分と、ファイル書き出しの部分
- ダイアログは表示した瞬間ハングするのと、ファイル書き出しもできないので、まあどちらも戻さなくても問題にならないかなと思い、とりあえずそのままにしています
- 戻せるようにも作れそうだけど、汎用的に書くのは結構難しいかも（全部のモックを戻す関数を作るとかが難しそう）
- 一般化していない
- 他にモックを刺したかった時にさくっとモックをさせるように作れてない
- 今のうちに共通化した方がいいかもだけど･･･良い方法がパッと思いつかなかったので一旦そのままにしています
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
内容
エンジンモックを使った音声書き出しのテストを書いてみました。
実際の音声も手元にpullしたりダウンロードしたりすれば聞けます。
その他
ブラウザ内でのファイル書き込みをどうモックするかで悩みました。
最終的にbackendのwriteFileをDI（というよりもmock？）することで解決しました。
他の方法としてブラウザに標準的についてる
FileHandleあたりを差し替えることもできるんですが、↓の用にいろいろ問題があったのでやめました。showOpenFilePickerなどのファイル選択ダイアログをmockできないshowOpenFilePickerをDIして適当なFileHandlerを返すこともできるけど、FileHandler周りのテストをPlaywrightがサポートしていないあとテストをplaywrightではなくvitestのブラウザモードで実装するのも考えましたがやめました。
試したところvitest-browser-vueがまだ色々と対応していないのか、ファイルが読み込めてなさそうなエラーが出たので引き返しました。もうちょっと様子見になりそう。