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テストをできるだけ改良してみた #8159

Merged
merged 53 commits into from
Jan 21, 2022

Conversation

tamaina
Copy link
Contributor

@tamaina tamaina commented Jan 18, 2022

@tamaina tamaina changed the title e2eがコケる(#7986)原因をテストしてみるPR enhance: e2eテストをできるだけ改良してみた Jan 18, 2022
@tamaina
Copy link
Contributor Author

tamaina commented Jan 18, 2022

やってみたこと

  • データベースのバージョンを上げてみた(Postgres→13, Redis→6)
  • Cypress公式のGitHub Actionを使うようにしてみた
  • resetDbのDELETE FROM "${table}" CASCADEは、Promise.allでの並列実行からfor { await }での直列実行にして確実に消去されるようにしてみた(「前のテストの結果がデータベースに残存していると思われる」はこれでなくなるはず)
  • Authentication failedエラーを握りつぶしてみたら、なんとなく動くようになった

できないこと

  • 音声周りでエラー(PcmOpen)が出る
    • ALSA Envアクションを追加したらALSAのもう一つのエラーは大丈夫になった
    • video: falseしてみたけど効果なし
  • PcmOpenエラーのせいなのかわからないが、最後のテストは通らない

@tamaina
Copy link
Contributor Author

tamaina commented Jan 18, 2022

(もうちょっとテストしてみる)

@syuilo
Copy link
Member

syuilo commented Jan 18, 2022

「前のテストの結果がデータベースに残存していると思われる」はこれでなくなるはず

だとするとAuthentication failedエラーは出なくなるはずだから握り潰さなくても良くなると思ったけど違うかな

@syuilo
Copy link
Member

syuilo commented Jan 18, 2022

それともAuthentication failedエラーがランダムに発生するのはデータベース関係ない…?

@tamaina
Copy link
Contributor Author

tamaina commented Jan 18, 2022

  • Firefoxのテストはあきらめた
  • artifacts機能でスクリーンショットと動画を取得できるようになった

@tamaina
Copy link
Contributor Author

tamaina commented Jan 18, 2022

音声周りのエラーは(electronではなく)chromeブラウザを用いることで解消したっぽい

@tamaina
Copy link
Contributor Author

tamaina commented Jan 18, 2022

  • 最後のテストが通らないのは普通にMisskeyのバグだったので解消 @ 240f01c
  • Firefoxはどうしてもテストが通らないのであきらめる(なんかリソース不足感がある)

@syuilo syuilo merged commit e1d69e2 into misskey-dev:develop Jan 21, 2022
@syuilo
Copy link
Member

syuilo commented Jan 21, 2022

🙏🏻🙏🏻🙏🏻
ただAuthentication failedは無視すると本当にMisskeyのバグでそうなってもテスト成功するから消した

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