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

test: e2eテストがCIで失敗していた問題をいくつか修正 #8642

Merged
merged 3 commits into from
May 14, 2022

Conversation

ishowta
Copy link
Contributor

@ishowta ishowta commented May 10, 2022

What

  • cypressがindexeddbをテスト毎に初期化してくれないので初期化するコードを追加した
  • ログイン後テストでログイン直後のクライアントが初期設定を行っているタイミングでリロードしてしまっていたため、リロードせずに待つようにした
  • fetch-metaが同時に呼ばれて死ぬことがあったのを修正した

Why

#7986 が直ったかもしれない

Additional info (optional)

@syuilo syuilo self-requested a review May 11, 2022 12:02
@syuilo
Copy link
Member

syuilo commented May 11, 2022

ありがとうございます!
e2eテストがランダムにコケる原因は、Whatで挙げられたものがすべてで、このPRを適用した後は原則ランダムにコケることがなくなる、という期待をしているという認識で合っていますか?

@ishowta
Copy link
Contributor Author

ishowta commented May 11, 2022

ありがとうございます! e2eテストがランダムにコケる原因は、Whatで挙げられたものがすべてで、このPRを適用した後は原則ランダムにコケることがなくなる、という期待をしているという認識で合っていますか?

あーすみませんfixって書いちゃってますね…一応私が見たことのあるエラーは潰したつもりで、10回くらいワークフローを動かして通るか確認しましたが、すべて直っている自信は無いです

@ishowta
Copy link
Contributor Author

ishowta commented May 11, 2022

見たことのあるエラーと対応する修正コミット

  • AssertionError: Timed out retrying after 4000ms: Expected to find element: `[data-cy-open-post-form]`, but never found it.
  • Authentication failed. Please ensure your token is correct.
  • Internal error occurred. Please contact us if the error persists.(DBのログを吐かせたらInternal error occurred in meta: duplicate key value violates unique constraint "PK_c4c17a6c2bd7651338b60fc590b"と出てきた)

@ishowta ishowta changed the title test: e2eテストがCIでたまに失敗するのを修正 test: e2eテストがCIで失敗していた問題をいくつか修正 May 11, 2022
@tamaina

This comment was marked as off-topic.

@ishowta
Copy link
Contributor Author

ishowta commented May 12, 2022

その2つはmochaのテストの修正で、このプルリクはcypressのe2eテストの修正です

@tamaina

This comment was marked as off-topic.

@syuilo syuilo merged commit 22bb1a1 into misskey-dev:develop May 14, 2022
@syuilo
Copy link
Member

syuilo commented May 14, 2022

ありがとうございます🙏🏻🙏🏻🙏🏻

atsu1125 pushed a commit to sakura-tel/metaskey that referenced this pull request Dec 7, 2022
* test: e2eテストがCIで失敗していた問題をいくつか修正 (misskey-dev#8642)

* test: indexeddbをテスト毎に初期化するように

* fix: metaが無いときにfetch-metaを同時に呼ぶと死ぬことがある問題を修正

* test: ログイン後のクライアント側処理を待たずにリロードされてログイン出来ないことがあったのを修正

* redisClient.FLUSHDB

* TypeORM 0.2.x

Co-authored-by: iwata <[email protected]>
Co-authored-by: syuilo <[email protected]>
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