From 07dad778d18b45eb6d6a003d3ba7cb8528e9c720 Mon Sep 17 00:00:00 2001 From: Kelley Reynolds Date: Thu, 5 Jul 2012 11:23:01 -0400 Subject: [PATCH] Change JoinPart test from an integration to a unit test --- activerecord/test/cases/associations/eager_test.rb | 13 ------------- .../test/cases/associations/join_dependency_test.rb | 8 ++++++++ 2 files changed, 8 insertions(+), 13 deletions(-) create mode 100644 activerecord/test/cases/associations/join_dependency_test.rb diff --git a/activerecord/test/cases/associations/eager_test.rb b/activerecord/test/cases/associations/eager_test.rb index 35070ba506..b79c69bbb5 100644 --- a/activerecord/test/cases/associations/eager_test.rb +++ b/activerecord/test/cases/associations/eager_test.rb @@ -39,19 +39,6 @@ class EagerAssociationTest < ActiveRecord::TestCase assert_nil member.favourite_club end - def test_loading_with_one_association_without_primary_key - Comment.primary_key = nil - begin - assert_nothing_raised do - Post.includes(:comments).references(:comments).all - end - rescue - raise $! - ensure - Comment.primary_key = 'id' - end - end - def test_loading_with_one_association posts = Post.find(:all, :include => :comments) post = posts.find { |p| p.id == 1 } diff --git a/activerecord/test/cases/associations/join_dependency_test.rb b/activerecord/test/cases/associations/join_dependency_test.rb new file mode 100644 index 0000000000..08c166dc33 --- /dev/null +++ b/activerecord/test/cases/associations/join_dependency_test.rb @@ -0,0 +1,8 @@ +require "cases/helper" +require 'models/edge' + +class JoinDependencyTest < ActiveRecord::TestCase + def test_column_names_with_alias_handles_nil_primary_key + assert_equal Edge.column_names, ActiveRecord::Associations::JoinDependency::JoinBase.new(Edge).column_names_with_alias.map(&:first) + end +end \ No newline at end of file