Skip to content
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テストを追加 #1488

Merged

Conversation

Hiroshiba
Copy link
Member

内容

書き出しファイル名パターンのテストを追加します。

連番がパターンに含まれていないとボタンが押せないとか、保存したら内容が表示されるとかをテストしています。

関連 Issue

スクリーンショット・動画など

その他

@Hiroshiba Hiroshiba requested a review from a team as a code owner August 6, 2023 06:12
@Hiroshiba Hiroshiba requested review from y-chan and removed request for a team August 6, 2023 06:12
@thiramisu
Copy link
Contributor

thiramisu commented Aug 6, 2023

以下のパターンも欲しいかな、と思いました、一応コメントまで…

  • 「\」などの利用不可な文字がファイル名に含まれていると警告が表示されて確定が押せない
  • 欄が空だと何か入力してくださいという警告が表示されて確定が押せない

@Hiroshiba
Copy link
Member Author

なるほどです。遷移が不要なので本来はコンポーネント単位のテストのが適切そうですが、無いのでここに書いときますか!

Comment on lines -27 to +28
await page.getByText("ツールバーのカスタマイズ").click();
await getQuasarMenu(page, "ツールバーのカスタマイズ").click();
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

メニューの内容は結構同じ案内があって一意に定まりにくいので専用関数を作ってみました。

Comment on lines -40 to +41
await page
.locator("#q-portal--dialog--6")
await getNewestQuasarDialog(page)
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ダイアログは開いた順番?で最後のindexが変わるので頑健に

Copy link
Member Author

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

問題ないと思うのでマージします!

@stmtk1 (ただの共有です)
tests/e2e/locators.tsというファイルを作って、そこにロケーター周りの便利関数を置いてみました。
もしよかったらどうぞ!!

@Hiroshiba Hiroshiba merged commit 47cd67d into main Aug 13, 2023
@Hiroshiba Hiroshiba deleted the 書き出しファイル名パターンのe2eテストを追加 branch August 13, 2023 15:12
}

/**
* quasarのメニューのlocaltorを取得する
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

あっ、Typoしてますね(時すでに遅し)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants