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

JS処理が間に合わないフレーキーテストに対処した #746

Merged
merged 1 commit into from
Jul 15, 2024

Conversation

yonta
Copy link
Collaborator

@yonta yonta commented Jul 1, 2024

fix #745

CIがときどき落ちる、フレーキーテストが存在している。
原因はおそらくJSの処理が間に合っていない。

Capybaraアサーションが失敗したときに、Capybaraは自動で再試行を行う。
この再試行は、デフォルトで2秒まで待ってから実行してくれる。

そこでこのパッチにより、待つ時間を5秒まで許可するようにした。
失敗したときにのみ効果があるので、テスト全体が遅くなることもない。

やったこと

  • Capybaraアサーション失敗時の自動再試行時間をデフォルト2秒から5秒に変更した

Capybaraアサーションが失敗したときに、Capybaraは自動で再試行を行う。
この再試行は、デフォルトで2秒まで待ってから実行してくれる。
このパッチにより、待つ時間を5秒まで許可するようにした。

失敗したときにのみ効果があるので、テスト全体が遅くなることもない。
@github-actions github-actions bot requested a review from momocus July 1, 2024 07:29
Copy link

github-actions bot commented Jul 1, 2024

📦 esbuild Bundle Analysis for sakazuki

This analysis was generated by esbuild-bundle-analyzer. 🤖

Meta File Out File Size (raw) Note
meta.json app/assets/builds/application.js 1.01 MB 🆕 Added
Largest paths These visualization shows top 20 largest paths in the bundle.

Meta file: meta.json, Out file: app/assets/builds/application.js

Path Size
node_modules/chart.js/dist ${{\color{Goldenrod}{ ██████▊ }}}$ 27.4%, 273.88 KB
app/javascript/completion ${{\color{Goldenrod}{ ████▎ }}}$ 17.1%, 170.67 KB
node_modules/@hotwired/turbo ${{\color{Goldenrod}{ ███▎ }}}$ 13.2%, 131.96 KB
node_modules/bootstrap/dist ${{\color{Goldenrod}{ ███▏ }}}$ 12.5%, 124.34 KB
node_modules/simplelightbox/dist ${{\color{Goldenrod}{ ██▏ }}}$ 8.8%, 87.73 KB
node_modules/@hotwired/stimulus ${{\color{Goldenrod}{ ██ }}}$ 8.4%, 84.29 KB
node_modules/@popperjs/core ${{\color{Goldenrod}{ █▍ }}}$ 5.6%, 55.66 KB
node_modules/@rails/actioncable ${{\color{Goldenrod}{ ▌ }}}$ 2.0%, 19.51 KB
node_modules/rater-js/index.js ${{\color{Goldenrod}{ ▍ }}}$ 1.6%, 16.42 KB
node_modules/@kurkle/color ${{\color{Goldenrod}{ ▎ }}}$ 1.3%, 13.37 KB
app/javascript/taste_graph ${{\color{Goldenrod}{ ▏ }}}$ 0.8%, 7.56 KB
app/javascript/controllers ${{\color{Goldenrod}{ ▏ }}}$ 0.7%, 6.63 KB
node_modules/@hotwired/turbo-rails ${{\color{Goldenrod}{ }}}$ 0.4%, 3.82 KB
node_modules/ts-deepmerge/esm ${{\color{Goldenrod}{ }}}$ 0.2%, 1.69 KB
node_modules/just-zip-it/index.mjs ${{\color{Goldenrod}{ }}}$ 0.1%, 785 B
node_modules/chart.js/helpers ${{\color{Goldenrod}{ }}}$ 0.0%, 0 B
app/javascript/application.js ${{\color{Goldenrod}{ }}}$ 0.0%, 0 B
Details

Next to the size is how much the size has increased or decreased compared with the base branch of this PR.

  • ‼️: Size increased by 20% or more. Special attention should be given to this.
  • ⚠️: Size increased in acceptable range (lower than 20%).
  • ✅: No change or even downsized.
  • 🗑️: The out file is deleted: not found in base branch.
  • 🆕: The out file is newly found: will be added to base branch.

@yonta yonta changed the title Fix: JS処理が間に合わないフレーキーテストに対処した JS処理が間に合わないフレーキーテストに対処した Jul 1, 2024
Copy link
Owner

@momocus momocus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

フレーキーテスト、学び

@momocus momocus merged commit 58b20f0 into main Jul 15, 2024
12 checks passed
@momocus momocus deleted the pr/issue745/capybara-flaky-test branch July 15, 2024 09:59
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.

ときどきCIのテストが落ち、ReRunすると落ちない
2 participants