落ちやすいregular_events_testを修正 #6971
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ogawa-tomo
commented
Oct 14, 2023
| visit_with_auth new_regular_event_path, 'komagata' | ||
| within 'form[name=regular_event]' do | ||
| fill_in 'regular_event[title]', with: '全員参加イベント' | ||
| first('.choices__inner').click | ||
| find('#choices--js-choices-multiple-select-item-choice-1').click | ||
| first('.regular-event-repeat-rule').first('.regular-event-repeat-rule__frequency select').select('毎週') | ||
| first('.regular-event-repeat-rule').first('.regular-event-repeat-rule__day-of-the-week select').select(%w[日曜日 月曜日 火曜日 水曜日 木曜日 金曜日 | ||
| 土曜日][DateTime.now.wday].to_s) | ||
| 土曜日][now.to_date.wday].to_s) |
There was a problem hiding this comment.
DateTime型は非推奨ということで、Time型のインスタンスから日付を取得するよう変更しました。
https://docs.ruby-lang.org/ja/latest/class/DateTime.html
There was a problem hiding this comment.
DateTime型は非推奨だったんですね👀
修正ありがとうございます🙏
sochi419
reviewed
Oct 15, 2023
| visit_with_auth new_regular_event_path, 'komagata' | ||
| within 'form[name=regular_event]' do | ||
| fill_in 'regular_event[title]', with: '全員参加イベント' | ||
| first('.choices__inner').click | ||
| find('#choices--js-choices-multiple-select-item-choice-1').click | ||
| first('.regular-event-repeat-rule').first('.regular-event-repeat-rule__frequency select').select('毎週') | ||
| first('.regular-event-repeat-rule').first('.regular-event-repeat-rule__day-of-the-week select').select(%w[日曜日 月曜日 火曜日 水曜日 木曜日 金曜日 | ||
| 土曜日][DateTime.now.wday].to_s) | ||
| 土曜日][now.to_date.wday].to_s) |
There was a problem hiding this comment.
DateTime型は非推奨だったんですね👀
修正ありがとうございます🙏
komagata
approved these changes
Oct 16, 2023
3 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
概要
下記の箇所のテストが19時以降に実行すると必ず落ちるようになっていたので修正しました。
bootcamp/test/system/regular_events_test.rb
Line 240 in 84b62f5
原因と修正内容
このテストの意図は、本日開催予定の定期イベントがトップページに表示されているのを確認することです。
しかしながら、トップページに表示される本日開催予定の定期イベントは、現在時刻より後に開始されるものに限ります。
bootcamp/app/models/regular_event.rb
Line 91 in 84b62f5
そして、テスト中で作成するイベントの開始時刻が19時となっているため、19時以降にこのテストを実行すると作成したイベントがトップページに表示されず、テストが必ず落ちます。
bootcamp/test/system/regular_events_test.rb
Line 223 in 84b62f5
travel_toを用いてテスト中でトップページを確認する時刻をイベント開始前にすることにより、テストが常に通るようにしました。
変更確認方法
bug/debug-flaky-regular-events-testに切り替え、19時以降に以下のテストを実行し、成功すること