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

Searcher モデルの Flaky なテストを修正 #4126

Merged
merged 1 commit into from
Feb 3, 2022

Conversation

AudioStakes
Copy link
Contributor

@AudioStakes AudioStakes commented Feb 3, 2022

目的

次の Flaky なテストを常に成功させるため、検索結果の並び順が一意に定まるようにする

test 'sort search results in descending order of updated date' do
result = Searcher.search('検索結果確認用', document_type: :reports)
assert_equal [reports(:report12), reports(:report14), reports(:report13)], result
assert_not_includes(result, Answer)
end

調査したこと

このテストで確認していること

検索結果が次のコードで指定されている .sort_by(&:updated_at).reverse の順番になっていること

def result_for_comments(document_type, word)
[document_type, :comments].flat_map { |type| result_for(type, word, commentable_type: model_name(document_type)) }.sort_by(&:updated_at).reverse
end

テストが Flaky な原因

検索結果の reports(:report12), reports(:report14), reports(:report13) のフィクスチャに updated_at が設定されていないため、すべて updated_at が同値となり、 sort_by(&:updated_at) の結果が一意に定まらないため

やったこと

検索結果の reports(:report12), reports(:report14), reports(:report13) それぞれのフィクスチャに updated_at の値を設定した
なお、updated_at の値はそれぞれの created_at の値と同じにした

@AudioStakes AudioStakes marked this pull request as ready for review February 3, 2022 07:51
@AudioStakes
Copy link
Contributor Author

@komagata
Flaky なテストにより CI が落ちていたので、修正してみました。
お時間のあるときに見てもらえますとありがたいですー🙏

@AudioStakes AudioStakes requested a review from komagata February 3, 2022 07:53
Copy link
Member

@komagata komagata left a comment

Choose a reason for hiding this comment

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

修正ありがとうございます〜。

@komagata komagata merged commit c785d97 into main Feb 3, 2022
@komagata komagata deleted the chore/fix-flaxy-test-of-sort-search-result branch February 3, 2022 14:39
@github-actions github-actions bot mentioned this pull request Feb 3, 2022
54 tasks
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