Skip to content

Commit

Permalink
Merge pull request #29415 from kamipo/remove_unused_defined_association
Browse files Browse the repository at this point in the history
Remove unused defined association
  • Loading branch information
guilleiguaran committed Jun 17, 2017
2 parents a3f758a + c073240 commit 03fa6b2
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 2 deletions.
2 changes: 1 addition & 1 deletion activerecord/test/cases/relation/merging_test.rb
Expand Up @@ -143,7 +143,7 @@ class MergingDifferentRelationsTest < ActiveRecord::TestCase
assert_equal ["Mary", "Mary", "Mary", "David"], posts_by_author_name assert_equal ["Mary", "Mary", "Mary", "David"], posts_by_author_name
end end


test "relation merging (using a proc argument)" do test "relation merging (using a proc argument)" do
dev = Developer.where(name: "Jamis").first dev = Developer.where(name: "Jamis").first


comment_1 = dev.comments.create!(body: "I'm Jamis", post: Post.first) comment_1 = dev.comments.create!(body: "I'm Jamis", post: Post.first)
Expand Down
1 change: 0 additions & 1 deletion activerecord/test/models/comment.rb
Expand Up @@ -9,7 +9,6 @@ class Comment < ActiveRecord::Base
belongs_to :post, counter_cache: true belongs_to :post, counter_cache: true
belongs_to :author, polymorphic: true belongs_to :author, polymorphic: true
belongs_to :resource, polymorphic: true belongs_to :resource, polymorphic: true
belongs_to :developer


has_many :ratings has_many :ratings


Expand Down

0 comments on commit 03fa6b2

Please sign in to comment.