Skip to content
This repository
Browse code

Merge pull request #12929 from yahonda/test_merging_where_relations_a…

…dd_order

Address test_merging_where_relations failure by adding order
  • Loading branch information...
commit f60d00c56eb93d8a9ffb85a3af7e405f33ddf7f5 2 parents ea76386 + 8e3474b
Carlos Antonio da Silva carlosantoniodasilva authored

Showing 1 changed file with 1 addition and 1 deletion. Show diff stats Hide diff stats

  1. +1 1  activerecord/test/cases/relation/merging_test.rb
2  activerecord/test/cases/relation/merging_test.rb
@@ -135,7 +135,7 @@ class MergingDifferentRelationsTest < ActiveRecord::TestCase
135 135
136 136 test "merging where relations" do
137 137 hello_by_bob = Post.where(body: "hello").joins(:author).
138   - merge(Author.where(name: "Bob")).pluck("posts.id")
  138 + merge(Author.where(name: "Bob")).order("posts.id").pluck("posts.id")
139 139
140 140 assert_equal [posts(:misc_by_bob).id,
141 141 posts(:other_by_bob).id], hello_by_bob

0 comments on commit f60d00c

Please sign in to comment.
Something went wrong with that request. Please try again.