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

enhance(e2e): more resilient version of visit('/') #10128

Merged
merged 1 commit into from
Mar 1, 2023

Conversation

saschanaz
Copy link
Member

やっぱりPR template消えました。

なにを

visitHome コマンドを追加、なにかボタンが現れるまで待つようにする

なんで

ローカルでpnpm devを対象にテスト回したらバックエンド遅すぎて失敗しますが、ちゃんとページロード終わったのかの確認がないので、エラーメッセージだけ読んだらなんか本当に壊れているように見えるのを何とかしました。

@saschanaz
Copy link
Member Author

visitAndWaitにした方がもっと色々使えるかもです🤔

@codecov
Copy link

codecov bot commented Feb 26, 2023

Codecov Report

Merging #10128 (dc680b9) into develop (81e6a21) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop   #10128   +/-   ##
========================================
  Coverage    25.06%   25.06%           
========================================
  Files          705      705           
  Lines        65151    65151           
  Branches      2307     2307           
========================================
  Hits         16329    16329           
  Misses       48822    48822           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@syuilo syuilo merged commit 44058f8 into misskey-dev:develop Mar 1, 2023
@syuilo
Copy link
Member

syuilo commented Mar 1, 2023

👍🏻

@saschanaz saschanaz deleted the visit-home branch March 1, 2023 07:03
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.

2 participants