-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
fix: タイムアウトを起こすe2eテストのタイムアウト時間設定を延長 #15283
Conversation
このPRによるapi.jsonの差分 |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #15283 +/- ##
===========================================
+ Coverage 39.43% 40.46% +1.03%
===========================================
Files 1568 1568
Lines 199142 199142
Branches 3656 3955 +299
===========================================
+ Hits 78537 80592 +2055
+ Misses 120032 117977 -2055
Partials 573 573 ☔ View full report in Codecov by Sentry. |
これで治るとは思うけどなんでタイムアウト引っかかるようになったかは考えたいかも。substringしてからハイライトすると不正なSQLでパフォーマンスがガタ落ちするとか |
テストに時間がかかることの根本原因の排除じゃないのでPRとissueのリンクを外しました |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
暫定対処として
What
#15282 に記載の通り、時間がかかっているテストがタイムアウトを起こすので時間を5秒伸ばしました。
また、#15076 を見る限り前々から起こっているようですが…迷宮入りの気配が漂うため、暫定対処という位置づけです。
Why
#15282
#15076
※根本対処ではないのでfixとはしない
Additional info (optional)
Checklist