From 9f4e98330b3a7f85a4bca91fca062106ffbee2bf Mon Sep 17 00:00:00 2001
From: Pratik Naik <pratiknaik@gmail.com>
Date: Mon, 28 Dec 2009 12:57:35 +0530
Subject: Remove unused construct_finder_sql

---
 .../associations/inner_join_association_test.rb    | 28 +++++++++++-----------
 1 file changed, 14 insertions(+), 14 deletions(-)

(limited to 'activerecord/test/cases/associations/inner_join_association_test.rb')

diff --git a/activerecord/test/cases/associations/inner_join_association_test.rb b/activerecord/test/cases/associations/inner_join_association_test.rb
index 5f08c40005..6220fab383 100644
--- a/activerecord/test/cases/associations/inner_join_association_test.rb
+++ b/activerecord/test/cases/associations/inner_join_association_test.rb
@@ -9,56 +9,56 @@ class InnerJoinAssociationTest < ActiveRecord::TestCase
   fixtures :authors, :posts, :comments, :categories, :categories_posts, :categorizations
 
   def test_construct_finder_sql_creates_inner_joins
-    sql = Author.send(:construct_finder_sql, :joins => :posts)
+    sql = Author.joins(:posts).to_sql
     assert_match /INNER JOIN .?posts.? ON .?posts.?.author_id = authors.id/, sql
   end
 
   def test_construct_finder_sql_cascades_inner_joins
-    sql = Author.send(:construct_finder_sql, :joins => {:posts => :comments})
+    sql = Author.joins(:posts => :comments).to_sql
     assert_match /INNER JOIN .?posts.? ON .?posts.?.author_id = authors.id/, sql
     assert_match /INNER JOIN .?comments.? ON .?comments.?.post_id = posts.id/, sql
   end
 
   def test_construct_finder_sql_inner_joins_through_associations
-    sql = Author.send(:construct_finder_sql, :joins => :categorized_posts)
+    sql = Author.joins(:categorized_posts).to_sql
     assert_match /INNER JOIN .?categorizations.?.*INNER JOIN .?posts.?/, sql
   end
 
   def test_construct_finder_sql_applies_association_conditions
-    sql = Author.send(:construct_finder_sql, :joins => :categories_like_general, :conditions => "TERMINATING_MARKER")
+    sql = Author.joins(:categories_like_general).where("TERMINATING_MARKER").to_sql
     assert_match /INNER JOIN .?categories.? ON.*AND.*.?General.?(.|\n)*TERMINATING_MARKER/, sql
   end
 
   def test_construct_finder_sql_applies_aliases_tables_on_association_conditions
-    result = Author.find(:all, :joins => [:thinking_posts, :welcome_posts])
+    result = Author.joins(:thinking_posts, :welcome_posts).to_a
     assert_equal authors(:david), result.first
   end
 
   def test_construct_finder_sql_unpacks_nested_joins
-    sql = Author.send(:construct_finder_sql, :joins => {:posts => [[:comments]]})
+    sql = Author.joins(:posts => [[:comments]]).to_sql
     assert_no_match /inner join.*inner join.*inner join/i, sql, "only two join clauses should be present"
     assert_match /INNER JOIN .?posts.? ON .?posts.?.author_id = authors.id/, sql
     assert_match /INNER JOIN .?comments.? ON .?comments.?.post_id = .?posts.?.id/, sql
   end
 
   def test_construct_finder_sql_ignores_empty_joins_hash
-    sql = Author.send(:construct_finder_sql, :joins => {})
+    sql = Author.joins({}).to_sql
     assert_no_match /JOIN/i, sql
   end
 
   def test_construct_finder_sql_ignores_empty_joins_array
-    sql = Author.send(:construct_finder_sql, :joins => [])
+    sql = Author.joins([]).to_sql
     assert_no_match /JOIN/i, sql
   end
 
   def test_find_with_implicit_inner_joins_honors_readonly_without_select
-    authors = Author.find(:all, :joins => :posts)
+    authors = Author.joins(:posts).to_a
     assert !authors.empty?, "expected authors to be non-empty"
     assert authors.all? {|a| a.readonly? }, "expected all authors to be readonly"
   end
 
   def test_find_with_implicit_inner_joins_honors_readonly_with_select
-    authors = Author.find(:all, :select => 'authors.*', :joins => :posts)
+    authors = Author.joins(:posts).select('authors.*').to_a
     assert !authors.empty?, "expected authors to be non-empty"
     assert authors.all? {|a| !a.readonly? }, "expected no authors to be readonly"
   end
@@ -70,23 +70,23 @@ class InnerJoinAssociationTest < ActiveRecord::TestCase
   end
 
   def test_find_with_implicit_inner_joins_does_not_set_associations
-    authors = Author.find(:all, :select => 'authors.*', :joins => :posts)
+    authors = Author.joins(:posts).select('authors.*')
     assert !authors.empty?, "expected authors to be non-empty"
     assert authors.all? {|a| !a.send(:instance_variable_names).include?("@posts")}, "expected no authors to have the @posts association loaded"
   end
 
   def test_count_honors_implicit_inner_joins
-    real_count = Author.find(:all).sum{|a| a.posts.count }
+    real_count = Author.scoped.to_a.sum{|a| a.posts.count }
     assert_equal real_count, Author.count(:joins => :posts), "plain inner join count should match the number of referenced posts records"
   end
 
   def test_calculate_honors_implicit_inner_joins
-    real_count = Author.find(:all).sum{|a| a.posts.count }
+    real_count = Author.scoped.to_a.sum{|a| a.posts.count }
     assert_equal real_count, Author.calculate(:count, 'authors.id', :joins => :posts), "plain inner join count should match the number of referenced posts records"
   end
 
   def test_calculate_honors_implicit_inner_joins_and_distinct_and_conditions
-    real_count = Author.find(:all).select {|a| a.posts.any? {|p| p.title =~ /^Welcome/} }.length
+    real_count = Author.scoped.to_a.select {|a| a.posts.any? {|p| p.title =~ /^Welcome/} }.length
     authors_with_welcoming_post_titles = Author.calculate(:count, 'authors.id', :joins => :posts, :distinct => true, :conditions => "posts.title like 'Welcome%'")
     assert_equal real_count, authors_with_welcoming_post_titles, "inner join and conditions should have only returned authors posting titles starting with 'Welcome'"
   end
-- 
cgit v1.2.3