-
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
アクセントの読み変更時のE2Eテストを追加 #2420
base: main
Are you sure you want to change the base?
アクセントの読み変更時のE2Eテストを追加 #2420
Conversation
tests/e2e/browser/音声パラメータ.spec.ts
Outdated
@@ -34,6 +34,7 @@ test("音声パラメータ引き継ぎの設定", async ({ page }) => { | |||
// パラメータを引き継がないことの確認 | |||
await page.locator(".audio-cell input").first().click(); | |||
await page.getByRole("button").filter({ hasText: "add" }).click(); | |||
await page.waitForTimeout(100); |
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.
テストを追加したことにより、この処理がないと既存の通っていたテストが通らなくなる。
理由は不明。
1) [browser] › browser/音声パラメータ.spec.ts:12:5 › 音声パラメータ引き継ぎの設定 ─────────────────────────────────────
Error: expect(received).toBe(expected) // Object.is equality
Expected: "1.00"
Received: "0.50"
7 | async function validateValue(locator: Locator, expectedValue: string) {
8 | const value = await locator.evaluate((e: HTMLInputElement) => e.value);
> 9 | expect(value).toBe(expectedValue);
| ^
10 | }
11 |
12 | test("音声パラメータ引き継ぎの設定", async ({ page }) => {
at validateValue (/Users/jdkfx/Desktop/workspace/voicevox/tests/e2e/browser/音声パラメータ.spec.ts:9:17)
at /Users/jdkfx/Desktop/workspace/voicevox/tests/e2e/browser/音声パラメータ.spec.ts:37:3
attachment #1: video (video/webm) ──────────────────────────────────────────────────────────────
test-results/音声パラメータ-音声パラメータ引き継ぎの設定-browser/video.webm
────────────────────────────────────────────────────────────────────────────────────────────────
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.
flakyになりやすいから、waitForTimeoutではない方法を使った方がいいかも。
https://zenn.dev/cureapp/articles/73cd66a37cc9e8
🚀 プレビュー用ページを作成しました 🚀 更新時点でのコミットハッシュ: |
内容
関連 Issue
ref #1462
スクリーンショット・動画など
その他