fix: Turbo DriveのDOM差し替え中に起きるsystem specの不安定な失敗を解消する

[Tsuchiya2]

背景

mainのCIでsystem specがまれに落ちていた。

1) [SystemTest] AlarmContents 新規アラームコンテンツ作成 ...
   Failure/Error: expect(page).to have_content('アラームコンテンツ一覧')

   Selenium::WebDriver::Error::UnknownError:
     unknown error: unhandled inspector error:
     {"code":-32000,"message":"Node with given id does not belong to the document"}

原因

フォーム送信後のリダイレクトをTurbo Driveが処理する際、bodyの差し替えが終わる前にCapybaraがhave_contentの問い合わせを投げると、chromedriverが参照していたノードが旧ドキュメント側に取り残されてこのエラーになる。

差し替えが終われば解消する一過性のエラーだが、Capybaraが再試行対象とするinvalid_element_errors（StaleElementReferenceErrorなど）にUnknownErrorは含まれないため、待機時間を使い切ることなく即失敗していた。

対応

Capybara::Node::Base#catch_error?をprependで拡張し、このメッセージを持つUnknownErrorに限り再試行対象として扱うようにした。

• 待機時間（Capybara.default_max_wait_time）内に差し替えが完了すれば成功する
• 超過すれば元の例外がそのまま送出されるため、本物のエラーを握り潰すことはない
• メッセージで絞っているので、他のUnknownErrorは従来どおり即失敗する

specごとにsleepや明示的なwaitを足す案は採らなかった。同じ競合はTurboで遷移するすべてのsystem specで起こりうるため、発生箇所ごとの対処では漏れが出るため。

確認したこと

• spec/system全件（62 examples, 0 failures, 6 pending）
• 対象メッセージのUnknownErrorは再試行され、それ以外のUnknownErrorは再試行されないことを検証
• Rubocop: no offenses

[coderabbitai]

Warning

Review limit reached

Next included review available in 31 minutes.

View limit details

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 8923be4e-1420-4e32-82dc-4f36329ee67c

📥 Commits

Reviewing files that changed from the base of the PR and between 02ec30f and 5d14d7d.

📒 Files selected for processing (1)

• spec/support/capybara.rb

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}