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テスト用にwindowにsignoutをインジェクトしてタスクごとに実行してみる #8236

Closed
wants to merge 37 commits into from

Conversation

tamaina
Copy link
Contributor

@tamaina tamaina commented Jan 31, 2022

Fix #7986 ?

What

各タスク毎にサインアウトを実行してみる

@tamaina tamaina changed the title e2eテスト用に_signoutをインジェクトしてみる e2eテスト用にwindowに_signoutをインジェクトして実行してみる Jan 31, 2022
@tamaina tamaina changed the title e2eテスト用にwindowに_signoutをインジェクトして実行してみる e2eテスト用にwindowにsignoutをインジェクトしてタスクごとに実行してみる Jan 31, 2022
@tamaina
Copy link
Contributor Author

tamaina commented Jan 31, 2022

手元(Ryzen 5600X, RAM 32GB, WSLg)で動かしても https://github.com/misskey-dev/misskey/runs/5009819361?check_suite_focus=true#step:11:74 のようなエラーは出てこないので、なんか多分スペック不足感がある

@tamaina
Copy link
Contributor Author

tamaina commented Jan 31, 2022

…node.jsのバージョンがちょっと古いのが気になる

@syuilo
Copy link
Member

syuilo commented Jan 31, 2022

自分もローカルで当該エラーが出たことはない

@tamaina
Copy link
Contributor Author

tamaina commented Jan 31, 2022

video: falseにしたらかなり打率高くなったかもしれない

@tamaina
Copy link
Contributor Author

tamaina commented Jan 31, 2022

実験結果

  • たぶんGitHub Actionsのパフォーマンスに起因する
  • video: falseにしないとかなりパフォーマンスが低下する。waitで待てばなんとかなりそうだけど、パフォーマンスを気にしてテストを書くのはちょっと…
  • video: falseにしてもAuthentication failedは治らない。各テストごとにsignoutを実行するとだいぶ安定する。

ってまとめようと思ったら https://github.com/misskey-dev/misskey/runs/5011173934?check_suite_focus=true のテスト結果が出てきたのでもうお手上げかも

@tamaina tamaina closed this Feb 4, 2022
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.

テストがランダムにコケる(e2e)
2 participants