mirror of
https://github.com/rails/rails.git
synced 2022-11-09 12:12:34 -05:00
%s/find(:\(first\|last\|all\), \([^()]*\))/scoped(\2).\1/gcI amongst other things
This commit is contained in:
parent
a57b7842d0
commit
df6f971e3a
30 changed files with 333 additions and 415 deletions
|
@ -119,7 +119,7 @@ module ActiveRecord
|
|||
class << self; remove_possible_method :find_by_session_id; end
|
||||
|
||||
def self.find_by_session_id(session_id)
|
||||
find :first, :conditions => {:session_id=>session_id}
|
||||
where(session_id: session_id).first
|
||||
end
|
||||
end
|
||||
end
|
||||
|
|
|
@ -130,7 +130,7 @@ class MysqlReservedWordTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_associations_work_with_reserved_words
|
||||
assert_nothing_raised { Select.find(:all, :include => [:groups]) }
|
||||
assert_nothing_raised { Select.scoped(:include => [:groups]).all }
|
||||
end
|
||||
|
||||
#the following functions were added to DRY test cases
|
||||
|
|
|
@ -9,7 +9,7 @@ module ActiveRecord
|
|||
|
||||
def test_update_question_marks
|
||||
str = "foo?bar"
|
||||
x = Topic.find :first
|
||||
x = Topic.first
|
||||
x.title = str
|
||||
x.content = str
|
||||
x.save!
|
||||
|
@ -28,7 +28,7 @@ module ActiveRecord
|
|||
|
||||
def test_update_null_bytes
|
||||
str = "foo\0bar"
|
||||
x = Topic.find :first
|
||||
x = Topic.first
|
||||
x.title = str
|
||||
x.content = str
|
||||
x.save!
|
||||
|
|
|
@ -130,7 +130,7 @@ class MysqlReservedWordTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_associations_work_with_reserved_words
|
||||
assert_nothing_raised { Select.find(:all, :include => [:groups]) }
|
||||
assert_nothing_raised { Select.scoped(:include => [:groups]).all }
|
||||
end
|
||||
|
||||
#the following functions were added to DRY test cases
|
||||
|
|
|
@ -90,7 +90,7 @@ class PostgresqlHstoreTest < ActiveRecord::TestCase
|
|||
|
||||
def test_rewrite
|
||||
@connection.execute "insert into hstores (tags) VALUES ('1=>2')"
|
||||
x = Hstore.find :first
|
||||
x = Hstore.first
|
||||
x.tags = { '"a\'' => 'b' }
|
||||
assert x.save!
|
||||
end
|
||||
|
@ -98,13 +98,13 @@ class PostgresqlHstoreTest < ActiveRecord::TestCase
|
|||
|
||||
def test_select
|
||||
@connection.execute "insert into hstores (tags) VALUES ('1=>2')"
|
||||
x = Hstore.find :first
|
||||
x = Hstore.first
|
||||
assert_equal({'1' => '2'}, x.tags)
|
||||
end
|
||||
|
||||
def test_select_multikey
|
||||
@connection.execute "insert into hstores (tags) VALUES ('1=>2,2=>3')"
|
||||
x = Hstore.find :first
|
||||
x = Hstore.first
|
||||
assert_equal({'1' => '2', '2' => '3'}, x.tags)
|
||||
end
|
||||
|
||||
|
|
|
@ -73,14 +73,14 @@ class BelongsToAssociationsTest < ActiveRecord::TestCase
|
|||
def test_eager_loading_with_primary_key
|
||||
Firm.create("name" => "Apple")
|
||||
Client.create("name" => "Citibank", :firm_name => "Apple")
|
||||
citibank_result = Client.find(:first, :conditions => {:name => "Citibank"}, :include => :firm_with_primary_key)
|
||||
citibank_result = Client.scoped(:conditions => {:name => "Citibank"}, :include => :firm_with_primary_key).first
|
||||
assert citibank_result.association_cache.key?(:firm_with_primary_key)
|
||||
end
|
||||
|
||||
def test_eager_loading_with_primary_key_as_symbol
|
||||
Firm.create("name" => "Apple")
|
||||
Client.create("name" => "Citibank", :firm_name => "Apple")
|
||||
citibank_result = Client.find(:first, :conditions => {:name => "Citibank"}, :include => :firm_with_primary_key_symbols)
|
||||
citibank_result = Client.scoped(:conditions => {:name => "Citibank"}, :include => :firm_with_primary_key_symbols).first
|
||||
assert citibank_result.association_cache.key?(:firm_with_primary_key_symbols)
|
||||
end
|
||||
|
||||
|
@ -334,7 +334,7 @@ class BelongsToAssociationsTest < ActiveRecord::TestCase
|
|||
def test_new_record_with_foreign_key_but_no_object
|
||||
c = Client.new("firm_id" => 1)
|
||||
# sometimes tests on Oracle fail if ORDER BY is not provided therefore add always :order with :first
|
||||
assert_equal Firm.find(:first, :order => "id"), c.firm_with_basic_id
|
||||
assert_equal Firm.scoped(:order => "id").first, c.firm_with_basic_id
|
||||
end
|
||||
|
||||
def test_setting_foreign_key_after_nil_target_loaded
|
||||
|
@ -396,7 +396,7 @@ class BelongsToAssociationsTest < ActiveRecord::TestCase
|
|||
def test_association_assignment_sticks
|
||||
post = Post.first
|
||||
|
||||
author1, author2 = Author.find(:all, :limit => 2)
|
||||
author1, author2 = Author.scoped(:limit => 2).all
|
||||
assert_not_nil author1
|
||||
assert_not_nil author2
|
||||
|
||||
|
|
|
@ -16,7 +16,7 @@ class CascadedEagerLoadingTest < ActiveRecord::TestCase
|
|||
:categorizations, :people, :categories, :edges, :vertices
|
||||
|
||||
def test_eager_association_loading_with_cascaded_two_levels
|
||||
authors = Author.find(:all, :include=>{:posts=>:comments}, :order=>"authors.id")
|
||||
authors = Author.scoped(:include=>{:posts=>:comments}, :order=>"authors.id").all
|
||||
assert_equal 3, authors.size
|
||||
assert_equal 5, authors[0].posts.size
|
||||
assert_equal 3, authors[1].posts.size
|
||||
|
@ -24,7 +24,7 @@ class CascadedEagerLoadingTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_eager_association_loading_with_cascaded_two_levels_and_one_level
|
||||
authors = Author.find(:all, :include=>[{:posts=>:comments}, :categorizations], :order=>"authors.id")
|
||||
authors = Author.scoped(:include=>[{:posts=>:comments}, :categorizations], :order=>"authors.id").all
|
||||
assert_equal 3, authors.size
|
||||
assert_equal 5, authors[0].posts.size
|
||||
assert_equal 3, authors[1].posts.size
|
||||
|
@ -84,7 +84,7 @@ class CascadedEagerLoadingTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_eager_association_loading_with_cascaded_two_levels_with_two_has_many_associations
|
||||
authors = Author.find(:all, :include=>{:posts=>[:comments, :categorizations]}, :order=>"authors.id")
|
||||
authors = Author.scoped(:include=>{:posts=>[:comments, :categorizations]}, :order=>"authors.id").all
|
||||
assert_equal 3, authors.size
|
||||
assert_equal 5, authors[0].posts.size
|
||||
assert_equal 3, authors[1].posts.size
|
||||
|
@ -92,7 +92,7 @@ class CascadedEagerLoadingTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_eager_association_loading_with_cascaded_two_levels_and_self_table_reference
|
||||
authors = Author.find(:all, :include=>{:posts=>[:comments, :author]}, :order=>"authors.id")
|
||||
authors = Author.scoped(:include=>{:posts=>[:comments, :author]}, :order=>"authors.id").all
|
||||
assert_equal 3, authors.size
|
||||
assert_equal 5, authors[0].posts.size
|
||||
assert_equal authors(:david).name, authors[0].name
|
||||
|
@ -100,13 +100,13 @@ class CascadedEagerLoadingTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_eager_association_loading_with_cascaded_two_levels_with_condition
|
||||
authors = Author.find(:all, :include=>{:posts=>:comments}, :conditions=>"authors.id=1", :order=>"authors.id")
|
||||
authors = Author.scoped(:include=>{:posts=>:comments}, :conditions=>"authors.id=1", :order=>"authors.id").all
|
||||
assert_equal 1, authors.size
|
||||
assert_equal 5, authors[0].posts.size
|
||||
end
|
||||
|
||||
def test_eager_association_loading_with_cascaded_three_levels_by_ping_pong
|
||||
firms = Firm.find(:all, :include=>{:account=>{:firm=>:account}}, :order=>"companies.id")
|
||||
firms = Firm.scoped(:include=>{:account=>{:firm=>:account}}, :order=>"companies.id").all
|
||||
assert_equal 2, firms.size
|
||||
assert_equal firms.first.account, firms.first.account.firm.account
|
||||
assert_equal companies(:first_firm).account, assert_no_queries { firms.first.account.firm.account }
|
||||
|
@ -114,7 +114,7 @@ class CascadedEagerLoadingTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_eager_association_loading_with_has_many_sti
|
||||
topics = Topic.find(:all, :include => :replies, :order => 'topics.id')
|
||||
topics = Topic.scoped(:include => :replies, :order => 'topics.id').all
|
||||
first, second, = topics(:first).replies.size, topics(:second).replies.size
|
||||
assert_no_queries do
|
||||
assert_equal first, topics[0].replies.size
|
||||
|
@ -127,7 +127,7 @@ class CascadedEagerLoadingTest < ActiveRecord::TestCase
|
|||
silly.parent_id = 1
|
||||
assert silly.save
|
||||
|
||||
topics = Topic.find(:all, :include => :replies, :order => ['topics.id', 'replies_topics.id'])
|
||||
topics = Topic.scoped(:include => :replies, :order => ['topics.id', 'replies_topics.id']).all
|
||||
assert_no_queries do
|
||||
assert_equal 2, topics[0].replies.size
|
||||
assert_equal 0, topics[1].replies.size
|
||||
|
@ -135,14 +135,14 @@ class CascadedEagerLoadingTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_eager_association_loading_with_belongs_to_sti
|
||||
replies = Reply.find(:all, :include => :topic, :order => 'topics.id')
|
||||
replies = Reply.scoped(:include => :topic, :order => 'topics.id').all
|
||||
assert replies.include?(topics(:second))
|
||||
assert !replies.include?(topics(:first))
|
||||
assert_equal topics(:first), assert_no_queries { replies.first.topic }
|
||||
end
|
||||
|
||||
def test_eager_association_loading_with_multiple_stis_and_order
|
||||
author = Author.find(:first, :include => { :posts => [ :special_comments , :very_special_comment ] }, :order => ['authors.name', 'comments.body', 'very_special_comments_posts.body'], :conditions => 'posts.id = 4')
|
||||
author = Author.scoped(:include => { :posts => [ :special_comments , :very_special_comment ] }, :order => ['authors.name', 'comments.body', 'very_special_comments_posts.body'], :conditions => 'posts.id = 4').first
|
||||
assert_equal authors(:david), author
|
||||
assert_no_queries do
|
||||
author.posts.first.special_comments
|
||||
|
@ -151,7 +151,7 @@ class CascadedEagerLoadingTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_eager_association_loading_of_stis_with_multiple_references
|
||||
authors = Author.find(:all, :include => { :posts => { :special_comments => { :post => [ :special_comments, :very_special_comment ] } } }, :order => 'comments.body, very_special_comments_posts.body', :conditions => 'posts.id = 4')
|
||||
authors = Author.scoped(:include => { :posts => { :special_comments => { :post => [ :special_comments, :very_special_comment ] } } }, :order => 'comments.body, very_special_comments_posts.body', :conditions => 'posts.id = 4').all
|
||||
assert_equal [authors(:david)], authors
|
||||
assert_no_queries do
|
||||
authors.first.posts.first.special_comments.first.post.special_comments
|
||||
|
@ -160,7 +160,7 @@ class CascadedEagerLoadingTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_eager_association_loading_where_first_level_returns_nil
|
||||
authors = Author.find(:all, :include => {:post_about_thinking => :comments}, :order => 'authors.id DESC')
|
||||
authors = Author.scoped(:include => {:post_about_thinking => :comments}, :order => 'authors.id DESC').all
|
||||
assert_equal [authors(:bob), authors(:mary), authors(:david)], authors
|
||||
assert_no_queries do
|
||||
authors[2].post_about_thinking.comments.first
|
||||
|
@ -168,12 +168,12 @@ class CascadedEagerLoadingTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_eager_association_loading_with_recursive_cascading_four_levels_has_many_through
|
||||
source = Vertex.find(:first, :include=>{:sinks=>{:sinks=>{:sinks=>:sinks}}}, :order => 'vertices.id')
|
||||
source = Vertex.scoped(:include=>{:sinks=>{:sinks=>{:sinks=>:sinks}}}, :order => 'vertices.id').first
|
||||
assert_equal vertices(:vertex_4), assert_no_queries { source.sinks.first.sinks.first.sinks.first }
|
||||
end
|
||||
|
||||
def test_eager_association_loading_with_recursive_cascading_four_levels_has_and_belongs_to_many
|
||||
sink = Vertex.find(:first, :include=>{:sources=>{:sources=>{:sources=>:sources}}}, :order => 'vertices.id DESC')
|
||||
sink = Vertex.scoped(:include=>{:sources=>{:sources=>{:sources=>:sources}}}, :order => 'vertices.id DESC').first
|
||||
assert_equal vertices(:vertex_1), assert_no_queries { sink.sources.first.sources.first.sources.first.sources.first }
|
||||
end
|
||||
end
|
||||
|
|
|
@ -93,7 +93,7 @@ class EagerLoadPolyAssocsTest < ActiveRecord::TestCase
|
|||
|
||||
def test_include_query
|
||||
res = 0
|
||||
res = ShapeExpression.find :all, :include => [ :shape, { :paint => :non_poly } ]
|
||||
res = ShapeExpression.scoped(:include => [ :shape, { :paint => :non_poly } ]).all
|
||||
assert_equal NUM_SHAPE_EXPRESSIONS, res.size
|
||||
assert_queries(0) do
|
||||
res.each do |se|
|
||||
|
|
|
@ -103,43 +103,43 @@ class EagerSingularizationTest < ActiveRecord::TestCase
|
|||
def test_eager_no_extra_singularization_belongs_to
|
||||
return unless @have_tables
|
||||
assert_nothing_raised do
|
||||
Virus.find(:all, :include => :octopus)
|
||||
Virus.scoped(:include => :octopus).all
|
||||
end
|
||||
end
|
||||
|
||||
def test_eager_no_extra_singularization_has_one
|
||||
return unless @have_tables
|
||||
assert_nothing_raised do
|
||||
Octopus.find(:all, :include => :virus)
|
||||
Octopus.scoped(:include => :virus).all
|
||||
end
|
||||
end
|
||||
|
||||
def test_eager_no_extra_singularization_has_many
|
||||
return unless @have_tables
|
||||
assert_nothing_raised do
|
||||
Bus.find(:all, :include => :passes)
|
||||
Bus.scoped(:include => :passes).all
|
||||
end
|
||||
end
|
||||
|
||||
def test_eager_no_extra_singularization_has_and_belongs_to_many
|
||||
return unless @have_tables
|
||||
assert_nothing_raised do
|
||||
Crisis.find(:all, :include => :messes)
|
||||
Mess.find(:all, :include => :crises)
|
||||
Crisis.scoped(:include => :messes).all
|
||||
Mess.scoped(:include => :crises).all
|
||||
end
|
||||
end
|
||||
|
||||
def test_eager_no_extra_singularization_has_many_through_belongs_to
|
||||
return unless @have_tables
|
||||
assert_nothing_raised do
|
||||
Crisis.find(:all, :include => :successes)
|
||||
Crisis.scoped(:include => :successes).all
|
||||
end
|
||||
end
|
||||
|
||||
def test_eager_no_extra_singularization_has_many_through_has_many
|
||||
return unless @have_tables
|
||||
assert_nothing_raised do
|
||||
Crisis.find(:all, :include => :compresses)
|
||||
Crisis.scoped(:include => :compresses).all
|
||||
end
|
||||
end
|
||||
end
|
||||
|
|
|
@ -40,22 +40,22 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_loading_with_one_association
|
||||
posts = Post.find(:all, :include => :comments)
|
||||
posts = Post.scoped(:include => :comments).all
|
||||
post = posts.find { |p| p.id == 1 }
|
||||
assert_equal 2, post.comments.size
|
||||
assert post.comments.include?(comments(:greetings))
|
||||
|
||||
post = Post.find(:first, :include => :comments, :conditions => "posts.title = 'Welcome to the weblog'")
|
||||
post = Post.scoped(:include => :comments, :conditions => "posts.title = 'Welcome to the weblog'").first
|
||||
assert_equal 2, post.comments.size
|
||||
assert post.comments.include?(comments(:greetings))
|
||||
|
||||
posts = Post.find(:all, :include => :last_comment)
|
||||
posts = Post.scoped(:include => :last_comment).all
|
||||
post = posts.find { |p| p.id == 1 }
|
||||
assert_equal Post.find(1).last_comment, post.last_comment
|
||||
end
|
||||
|
||||
def test_loading_with_one_association_with_non_preload
|
||||
posts = Post.find(:all, :include => :last_comment, :order => 'comments.id DESC')
|
||||
posts = Post.scoped(:include => :last_comment, :order => 'comments.id DESC').all
|
||||
post = posts.find { |p| p.id == 1 }
|
||||
assert_equal Post.find(1).last_comment, post.last_comment
|
||||
end
|
||||
|
@ -70,7 +70,7 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_with_ordering
|
||||
list = Post.find(:all, :include => :comments, :order => "posts.id DESC")
|
||||
list = Post.scoped(:include => :comments, :order => "posts.id DESC").all
|
||||
[:other_by_mary, :other_by_bob, :misc_by_mary, :misc_by_bob, :eager_other,
|
||||
:sti_habtm, :sti_post_and_comments, :sti_comments, :authorless, :thinking, :welcome
|
||||
].each_with_index do |post, index|
|
||||
|
@ -84,14 +84,14 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_loading_with_multiple_associations
|
||||
posts = Post.find(:all, :include => [ :comments, :author, :categories ], :order => "posts.id")
|
||||
posts = Post.scoped(:include => [ :comments, :author, :categories ], :order => "posts.id").all
|
||||
assert_equal 2, posts.first.comments.size
|
||||
assert_equal 2, posts.first.categories.size
|
||||
assert posts.first.comments.include?(comments(:greetings))
|
||||
end
|
||||
|
||||
def test_duplicate_middle_objects
|
||||
comments = Comment.find :all, :conditions => 'post_id = 1', :include => [:post => :author]
|
||||
comments = Comment.scoped(:conditions => 'post_id = 1', :include => [:post => :author]).all
|
||||
assert_no_queries do
|
||||
comments.each {|comment| comment.post.author.name}
|
||||
end
|
||||
|
@ -99,25 +99,25 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
|
||||
def test_preloading_has_many_in_multiple_queries_with_more_ids_than_database_can_handle
|
||||
Post.connection.expects(:in_clause_length).at_least_once.returns(5)
|
||||
posts = Post.find(:all, :include=>:comments)
|
||||
posts = Post.scoped(:include=>:comments).all
|
||||
assert_equal 11, posts.size
|
||||
end
|
||||
|
||||
def test_preloading_has_many_in_one_queries_when_database_has_no_limit_on_ids_it_can_handle
|
||||
Post.connection.expects(:in_clause_length).at_least_once.returns(nil)
|
||||
posts = Post.find(:all, :include=>:comments)
|
||||
posts = Post.scoped(:include=>:comments).all
|
||||
assert_equal 11, posts.size
|
||||
end
|
||||
|
||||
def test_preloading_habtm_in_multiple_queries_with_more_ids_than_database_can_handle
|
||||
Post.connection.expects(:in_clause_length).at_least_once.returns(5)
|
||||
posts = Post.find(:all, :include=>:categories)
|
||||
posts = Post.scoped(:include=>:categories).all
|
||||
assert_equal 11, posts.size
|
||||
end
|
||||
|
||||
def test_preloading_habtm_in_one_queries_when_database_has_no_limit_on_ids_it_can_handle
|
||||
Post.connection.expects(:in_clause_length).at_least_once.returns(nil)
|
||||
posts = Post.find(:all, :include=>:categories)
|
||||
posts = Post.scoped(:include=>:categories).all
|
||||
assert_equal 11, posts.size
|
||||
end
|
||||
|
||||
|
@ -154,8 +154,8 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
popular_post.readers.create!(:person => people(:michael))
|
||||
popular_post.readers.create!(:person => people(:david))
|
||||
|
||||
readers = Reader.find(:all, :conditions => ["post_id = ?", popular_post.id],
|
||||
:include => {:post => :comments})
|
||||
readers = Reader.scoped(:conditions => ["post_id = ?", popular_post.id],
|
||||
:include => {:post => :comments}).all
|
||||
readers.each do |reader|
|
||||
assert_equal [comment], reader.post.comments
|
||||
end
|
||||
|
@ -167,8 +167,8 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
car_post.categories << categories(:technology)
|
||||
|
||||
comment = car_post.comments.create!(:body => "hmm")
|
||||
categories = Category.find(:all, :conditions => { 'posts.id' => car_post.id },
|
||||
:include => {:posts => :comments})
|
||||
categories = Category.scoped(:conditions => { 'posts.id' => car_post.id },
|
||||
:include => {:posts => :comments}).all
|
||||
categories.each do |category|
|
||||
assert_equal [comment], category.posts[0].comments
|
||||
end
|
||||
|
@ -234,13 +234,13 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_loading_from_an_association
|
||||
posts = authors(:david).posts.find(:all, :include => :comments, :order => "posts.id")
|
||||
posts = authors(:david).posts.scoped(:include => :comments, :order => "posts.id").all
|
||||
assert_equal 2, posts.first.comments.size
|
||||
end
|
||||
|
||||
def test_loading_from_an_association_that_has_a_hash_of_conditions
|
||||
assert_nothing_raised do
|
||||
Author.find(:all, :include => :hello_posts_with_hash_conditions)
|
||||
Author.scoped(:include => :hello_posts_with_hash_conditions).all
|
||||
end
|
||||
assert !Author.find(authors(:david).id, :include => :hello_posts_with_hash_conditions).hello_posts.empty?
|
||||
end
|
||||
|
@ -300,12 +300,12 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_eager_association_loading_with_belongs_to_and_foreign_keys
|
||||
pets = Pet.find(:all, :include => :owner)
|
||||
pets = Pet.scoped(:include => :owner).all
|
||||
assert_equal 3, pets.length
|
||||
end
|
||||
|
||||
def test_eager_association_loading_with_belongs_to
|
||||
comments = Comment.find(:all, :include => :post)
|
||||
comments = Comment.scoped(:include => :post).all
|
||||
assert_equal 11, comments.length
|
||||
titles = comments.map { |c| c.post.title }
|
||||
assert titles.include?(posts(:welcome).title)
|
||||
|
@ -313,31 +313,31 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_eager_association_loading_with_belongs_to_and_limit
|
||||
comments = Comment.find(:all, :include => :post, :limit => 5, :order => 'comments.id')
|
||||
comments = Comment.scoped(:include => :post, :limit => 5, :order => 'comments.id').all
|
||||
assert_equal 5, comments.length
|
||||
assert_equal [1,2,3,5,6], comments.collect { |c| c.id }
|
||||
end
|
||||
|
||||
def test_eager_association_loading_with_belongs_to_and_limit_and_conditions
|
||||
comments = Comment.find(:all, :include => :post, :conditions => 'post_id = 4', :limit => 3, :order => 'comments.id')
|
||||
comments = Comment.scoped(:include => :post, :conditions => 'post_id = 4', :limit => 3, :order => 'comments.id').all
|
||||
assert_equal 3, comments.length
|
||||
assert_equal [5,6,7], comments.collect { |c| c.id }
|
||||
end
|
||||
|
||||
def test_eager_association_loading_with_belongs_to_and_limit_and_offset
|
||||
comments = Comment.find(:all, :include => :post, :limit => 3, :offset => 2, :order => 'comments.id')
|
||||
comments = Comment.scoped(:include => :post, :limit => 3, :offset => 2, :order => 'comments.id').all
|
||||
assert_equal 3, comments.length
|
||||
assert_equal [3,5,6], comments.collect { |c| c.id }
|
||||
end
|
||||
|
||||
def test_eager_association_loading_with_belongs_to_and_limit_and_offset_and_conditions
|
||||
comments = Comment.find(:all, :include => :post, :conditions => 'post_id = 4', :limit => 3, :offset => 1, :order => 'comments.id')
|
||||
comments = Comment.scoped(:include => :post, :conditions => 'post_id = 4', :limit => 3, :offset => 1, :order => 'comments.id').all
|
||||
assert_equal 3, comments.length
|
||||
assert_equal [6,7,8], comments.collect { |c| c.id }
|
||||
end
|
||||
|
||||
def test_eager_association_loading_with_belongs_to_and_limit_and_offset_and_conditions_array
|
||||
comments = Comment.find(:all, :include => :post, :conditions => ['post_id = ?',4], :limit => 3, :offset => 1, :order => 'comments.id')
|
||||
comments = Comment.scoped(:include => :post, :conditions => ['post_id = ?',4], :limit => 3, :offset => 1, :order => 'comments.id').all
|
||||
assert_equal 3, comments.length
|
||||
assert_equal [6,7,8], comments.collect { |c| c.id }
|
||||
end
|
||||
|
@ -345,7 +345,7 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
def test_eager_association_loading_with_belongs_to_and_conditions_string_with_unquoted_table_name
|
||||
assert_nothing_raised do
|
||||
ActiveSupport::Deprecation.silence do
|
||||
Comment.find(:all, :include => :post, :conditions => ['posts.id = ?',4])
|
||||
Comment.scoped(:include => :post, :conditions => ['posts.id = ?',4]).all
|
||||
end
|
||||
end
|
||||
end
|
||||
|
@ -353,7 +353,7 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
def test_eager_association_loading_with_belongs_to_and_conditions_hash
|
||||
comments = []
|
||||
assert_nothing_raised do
|
||||
comments = Comment.find(:all, :include => :post, :conditions => {:posts => {:id => 4}}, :limit => 3, :order => 'comments.id')
|
||||
comments = Comment.scoped(:include => :post, :conditions => {:posts => {:id => 4}}, :limit => 3, :order => 'comments.id').all
|
||||
end
|
||||
assert_equal 3, comments.length
|
||||
assert_equal [5,6,7], comments.collect { |c| c.id }
|
||||
|
@ -366,14 +366,14 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
quoted_posts_id= Comment.connection.quote_table_name('posts') + '.' + Comment.connection.quote_column_name('id')
|
||||
assert_nothing_raised do
|
||||
ActiveSupport::Deprecation.silence do
|
||||
Comment.find(:all, :include => :post, :conditions => ["#{quoted_posts_id} = ?",4])
|
||||
Comment.scoped(:include => :post, :conditions => ["#{quoted_posts_id} = ?",4]).all
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
def test_eager_association_loading_with_belongs_to_and_order_string_with_unquoted_table_name
|
||||
assert_nothing_raised do
|
||||
Comment.find(:all, :include => :post, :order => 'posts.id')
|
||||
Comment.scoped(:include => :post, :order => 'posts.id').all
|
||||
end
|
||||
end
|
||||
|
||||
|
@ -381,25 +381,25 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
quoted_posts_id= Comment.connection.quote_table_name('posts') + '.' + Comment.connection.quote_column_name('id')
|
||||
assert_nothing_raised do
|
||||
ActiveSupport::Deprecation.silence do
|
||||
Comment.find(:all, :include => :post, :order => quoted_posts_id)
|
||||
Comment.scoped(:include => :post, :order => quoted_posts_id).all
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
def test_eager_association_loading_with_belongs_to_and_limit_and_multiple_associations
|
||||
posts = Post.find(:all, :include => [:author, :very_special_comment], :limit => 1, :order => 'posts.id')
|
||||
posts = Post.scoped(:include => [:author, :very_special_comment], :limit => 1, :order => 'posts.id').all
|
||||
assert_equal 1, posts.length
|
||||
assert_equal [1], posts.collect { |p| p.id }
|
||||
end
|
||||
|
||||
def test_eager_association_loading_with_belongs_to_and_limit_and_offset_and_multiple_associations
|
||||
posts = Post.find(:all, :include => [:author, :very_special_comment], :limit => 1, :offset => 1, :order => 'posts.id')
|
||||
posts = Post.scoped(:include => [:author, :very_special_comment], :limit => 1, :offset => 1, :order => 'posts.id').all
|
||||
assert_equal 1, posts.length
|
||||
assert_equal [2], posts.collect { |p| p.id }
|
||||
end
|
||||
|
||||
def test_eager_association_loading_with_belongs_to_inferred_foreign_key_from_association_name
|
||||
author_favorite = AuthorFavorite.find(:first, :include => :favorite_author)
|
||||
author_favorite = AuthorFavorite.scoped(:include => :favorite_author).first
|
||||
assert_equal authors(:mary), assert_no_queries { author_favorite.favorite_author }
|
||||
end
|
||||
|
||||
|
@ -458,14 +458,14 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_eager_association_loading_with_explicit_join
|
||||
posts = Post.find(:all, :include => :comments, :joins => "INNER JOIN authors ON posts.author_id = authors.id AND authors.name = 'Mary'", :limit => 1, :order => 'author_id')
|
||||
posts = Post.scoped(:include => :comments, :joins => "INNER JOIN authors ON posts.author_id = authors.id AND authors.name = 'Mary'", :limit => 1, :order => 'author_id').all
|
||||
assert_equal 1, posts.length
|
||||
end
|
||||
|
||||
def test_eager_with_has_many_through
|
||||
posts_with_comments = people(:michael).posts.find(:all, :include => :comments, :order => 'posts.id')
|
||||
posts_with_author = people(:michael).posts.find(:all, :include => :author, :order => 'posts.id')
|
||||
posts_with_comments_and_author = people(:michael).posts.find(:all, :include => [ :comments, :author ], :order => 'posts.id')
|
||||
posts_with_comments = people(:michael).posts.scoped(:include => :comments, :order => 'posts.id').all
|
||||
posts_with_author = people(:michael).posts.scoped(:include => :author, :order => 'posts.id').all
|
||||
posts_with_comments_and_author = people(:michael).posts.scoped(:include => [ :comments, :author ], :order => 'posts.id').all
|
||||
assert_equal 2, posts_with_comments.inject(0) { |sum, post| sum += post.comments.size }
|
||||
assert_equal authors(:david), assert_no_queries { posts_with_author.first.author }
|
||||
assert_equal authors(:david), assert_no_queries { posts_with_comments_and_author.first.author }
|
||||
|
@ -476,24 +476,24 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
Post.create!(:author => author, :title => "TITLE", :body => "BODY")
|
||||
author.author_favorites.create(:favorite_author_id => 1)
|
||||
author.author_favorites.create(:favorite_author_id => 2)
|
||||
posts_with_author_favorites = author.posts.find(:all, :include => :author_favorites)
|
||||
posts_with_author_favorites = author.posts.scoped(:include => :author_favorites).all
|
||||
assert_no_queries { posts_with_author_favorites.first.author_favorites.first.author_id }
|
||||
end
|
||||
|
||||
def test_eager_with_has_many_through_an_sti_join_model
|
||||
author = Author.find(:first, :include => :special_post_comments, :order => 'authors.id')
|
||||
author = Author.scoped(:include => :special_post_comments, :order => 'authors.id').first
|
||||
assert_equal [comments(:does_it_hurt)], assert_no_queries { author.special_post_comments }
|
||||
end
|
||||
|
||||
def test_eager_with_has_many_through_an_sti_join_model_with_conditions_on_both
|
||||
author = Author.find(:first, :include => :special_nonexistant_post_comments, :order => 'authors.id')
|
||||
author = Author.scoped(:include => :special_nonexistant_post_comments, :order => 'authors.id').first
|
||||
assert_equal [], author.special_nonexistant_post_comments
|
||||
end
|
||||
|
||||
def test_eager_with_has_many_through_join_model_with_conditions
|
||||
assert_equal Author.find(:first, :include => :hello_post_comments,
|
||||
:order => 'authors.id').hello_post_comments.sort_by(&:id),
|
||||
Author.find(:first, :order => 'authors.id').hello_post_comments.sort_by(&:id)
|
||||
assert_equal Author.scoped(:include => :hello_post_comments,
|
||||
:order => 'authors.id').first.hello_post_comments.sort_by(&:id),
|
||||
Author.scoped(:order => 'authors.id').first.hello_post_comments.sort_by(&:id)
|
||||
end
|
||||
|
||||
def test_eager_with_has_many_through_join_model_with_conditions_on_top_level
|
||||
|
@ -520,16 +520,16 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_eager_with_has_many_and_limit
|
||||
posts = Post.find(:all, :order => 'posts.id asc', :include => [ :author, :comments ], :limit => 2)
|
||||
posts = Post.scoped(:order => 'posts.id asc', :include => [ :author, :comments ], :limit => 2).all
|
||||
assert_equal 2, posts.size
|
||||
assert_equal 3, posts.inject(0) { |sum, post| sum += post.comments.size }
|
||||
end
|
||||
|
||||
def test_eager_with_has_many_and_limit_and_conditions
|
||||
if current_adapter?(:OpenBaseAdapter)
|
||||
posts = Post.find(:all, :include => [ :author, :comments ], :limit => 2, :conditions => "FETCHBLOB(posts.body) = 'hello'", :order => "posts.id")
|
||||
posts = Post.scoped(:include => [ :author, :comments ], :limit => 2, :conditions => "FETCHBLOB(posts.body) = 'hello'", :order => "posts.id").all
|
||||
else
|
||||
posts = Post.find(:all, :include => [ :author, :comments ], :limit => 2, :conditions => "posts.body = 'hello'", :order => "posts.id")
|
||||
posts = Post.scoped(:include => [ :author, :comments ], :limit => 2, :conditions => "posts.body = 'hello'", :order => "posts.id").all
|
||||
end
|
||||
assert_equal 2, posts.size
|
||||
assert_equal [4,5], posts.collect { |p| p.id }
|
||||
|
@ -537,9 +537,9 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
|
||||
def test_eager_with_has_many_and_limit_and_conditions_array
|
||||
if current_adapter?(:OpenBaseAdapter)
|
||||
posts = Post.find(:all, :include => [ :author, :comments ], :limit => 2, :conditions => [ "FETCHBLOB(posts.body) = ?", 'hello' ], :order => "posts.id")
|
||||
posts = Post.scoped(:include => [ :author, :comments ], :limit => 2, :conditions => [ "FETCHBLOB(posts.body) = ?", 'hello' ], :order => "posts.id").all
|
||||
else
|
||||
posts = Post.find(:all, :include => [ :author, :comments ], :limit => 2, :conditions => [ "posts.body = ?", 'hello' ], :order => "posts.id")
|
||||
posts = Post.scoped(:include => [ :author, :comments ], :limit => 2, :conditions => [ "posts.body = ?", 'hello' ], :order => "posts.id").all
|
||||
end
|
||||
assert_equal 2, posts.size
|
||||
assert_equal [4,5], posts.collect { |p| p.id }
|
||||
|
@ -547,7 +547,7 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
|
||||
def test_eager_with_has_many_and_limit_and_conditions_array_on_the_eagers
|
||||
posts = ActiveSupport::Deprecation.silence do
|
||||
Post.find(:all, :include => [ :author, :comments ], :limit => 2, :conditions => [ "authors.name = ?", 'David' ])
|
||||
Post.scoped(:include => [ :author, :comments ], :limit => 2, :conditions => [ "authors.name = ?", 'David' ]).all
|
||||
end
|
||||
assert_equal 2, posts.size
|
||||
|
||||
|
@ -558,23 +558,23 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_eager_with_has_many_and_limit_and_high_offset
|
||||
posts = Post.find(:all, :include => [ :author, :comments ], :limit => 2, :offset => 10, :conditions => { 'authors.name' => 'David' })
|
||||
posts = Post.scoped(:include => [ :author, :comments ], :limit => 2, :offset => 10, :conditions => { 'authors.name' => 'David' }).all
|
||||
assert_equal 0, posts.size
|
||||
end
|
||||
|
||||
def test_eager_with_has_many_and_limit_and_high_offset_and_multiple_array_conditions
|
||||
assert_queries(1) do
|
||||
posts = Post.references(:authors, :comments).
|
||||
find(:all, :include => [ :author, :comments ], :limit => 2, :offset => 10,
|
||||
:conditions => [ "authors.name = ? and comments.body = ?", 'David', 'go crazy' ])
|
||||
scoped(:include => [ :author, :comments ], :limit => 2, :offset => 10,
|
||||
:conditions => [ "authors.name = ? and comments.body = ?", 'David', 'go crazy' ]).all
|
||||
assert_equal 0, posts.size
|
||||
end
|
||||
end
|
||||
|
||||
def test_eager_with_has_many_and_limit_and_high_offset_and_multiple_hash_conditions
|
||||
assert_queries(1) do
|
||||
posts = Post.find(:all, :include => [ :author, :comments ], :limit => 2, :offset => 10,
|
||||
:conditions => { 'authors.name' => 'David', 'comments.body' => 'go crazy' })
|
||||
posts = Post.scoped(:include => [ :author, :comments ], :limit => 2, :offset => 10,
|
||||
:conditions => { 'authors.name' => 'David', 'comments.body' => 'go crazy' }).all
|
||||
assert_equal 0, posts.size
|
||||
end
|
||||
end
|
||||
|
@ -602,7 +602,7 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_eager_with_has_and_belongs_to_many_and_limit
|
||||
posts = Post.find(:all, :include => :categories, :order => "posts.id", :limit => 3)
|
||||
posts = Post.scoped(:include => :categories, :order => "posts.id", :limit => 3).all
|
||||
assert_equal 3, posts.size
|
||||
assert_equal 2, posts[0].categories.size
|
||||
assert_equal 1, posts[1].categories.size
|
||||
|
@ -668,7 +668,7 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_eager_association_loading_with_habtm
|
||||
posts = Post.find(:all, :include => :categories, :order => "posts.id")
|
||||
posts = Post.scoped(:include => :categories, :order => "posts.id").all
|
||||
assert_equal 2, posts[0].categories.size
|
||||
assert_equal 1, posts[1].categories.size
|
||||
assert_equal 0, posts[2].categories.size
|
||||
|
@ -677,7 +677,7 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_eager_with_inheritance
|
||||
SpecialPost.find(:all, :include => [ :comments ])
|
||||
SpecialPost.scoped(:include => [ :comments ]).all
|
||||
end
|
||||
|
||||
def test_eager_has_one_with_association_inheritance
|
||||
|
@ -702,8 +702,8 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
|
||||
def test_eager_with_has_one_dependent_does_not_destroy_dependent
|
||||
assert_not_nil companies(:first_firm).account
|
||||
f = Firm.find(:first, :include => :account,
|
||||
:conditions => ["companies.name = ?", "37signals"])
|
||||
f = Firm.scoped(:include => :account,
|
||||
:conditions => ["companies.name = ?", "37signals"]).first
|
||||
assert_not_nil f.account
|
||||
assert_equal companies(:first_firm, :reload).account, f.account
|
||||
end
|
||||
|
@ -771,7 +771,7 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def find_all_ordered(className, include=nil)
|
||||
className.find(:all, :order=>"#{className.table_name}.#{className.primary_key}", :include=>include)
|
||||
className.scoped(:order=>"#{className.table_name}.#{className.primary_key}", :include=>include).all
|
||||
end
|
||||
|
||||
def test_limited_eager_with_order
|
||||
|
@ -881,13 +881,13 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
end
|
||||
end
|
||||
def test_eager_with_valid_association_as_string_not_symbol
|
||||
assert_nothing_raised { Post.find(:all, :include => 'comments') }
|
||||
assert_nothing_raised { Post.scoped(:include => 'comments').all }
|
||||
end
|
||||
|
||||
def test_eager_with_floating_point_numbers
|
||||
assert_queries(2) do
|
||||
# Before changes, the floating point numbers will be interpreted as table names and will cause this to run in one query
|
||||
Comment.find :all, :conditions => "123.456 = 123.456", :include => :post
|
||||
Comment.scoped(:conditions => "123.456 = 123.456", :include => :post).all
|
||||
end
|
||||
end
|
||||
|
||||
|
@ -941,21 +941,21 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
|
||||
def test_load_with_sti_sharing_association
|
||||
assert_queries(2) do #should not do 1 query per subclass
|
||||
Comment.find :all, :include => :post
|
||||
Comment.includes(:post).all
|
||||
end
|
||||
end
|
||||
|
||||
def test_conditions_on_join_table_with_include_and_limit
|
||||
assert_equal 3, Developer.find(:all, :include => 'projects', :conditions => { 'developers_projects.access_level' => 1 }, :limit => 5).size
|
||||
assert_equal 3, Developer.scoped(:include => 'projects', :conditions => { 'developers_projects.access_level' => 1 }, :limit => 5).all.size
|
||||
end
|
||||
|
||||
def test_order_on_join_table_with_include_and_limit
|
||||
assert_equal 5, Developer.find(:all, :include => 'projects', :order => 'developers_projects.joined_on DESC', :limit => 5).size
|
||||
assert_equal 5, Developer.scoped(:include => 'projects', :order => 'developers_projects.joined_on DESC', :limit => 5).all.size
|
||||
end
|
||||
|
||||
def test_eager_loading_with_order_on_joined_table_preloads
|
||||
posts = assert_queries(2) do
|
||||
Post.find(:all, :joins => :comments, :include => :author, :order => 'comments.id DESC')
|
||||
Post.scoped(:joins => :comments, :include => :author, :order => 'comments.id DESC').all
|
||||
end
|
||||
assert_equal posts(:eager_other), posts[1]
|
||||
assert_equal authors(:mary), assert_no_queries { posts[1].author}
|
||||
|
@ -963,24 +963,24 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
|
||||
def test_eager_loading_with_conditions_on_joined_table_preloads
|
||||
posts = assert_queries(2) do
|
||||
Post.find(:all, :select => 'distinct posts.*', :include => :author, :joins => [:comments], :conditions => "comments.body like 'Thank you%'", :order => 'posts.id')
|
||||
Post.scoped(:select => 'distinct posts.*', :include => :author, :joins => [:comments], :conditions => "comments.body like 'Thank you%'", :order => 'posts.id').all
|
||||
end
|
||||
assert_equal [posts(:welcome)], posts
|
||||
assert_equal authors(:david), assert_no_queries { posts[0].author}
|
||||
|
||||
posts = assert_queries(2) do
|
||||
Post.find(:all, :select => 'distinct posts.*', :include => :author, :joins => [:comments], :conditions => "comments.body like 'Thank you%'", :order => 'posts.id')
|
||||
Post.scoped(:select => 'distinct posts.*', :include => :author, :joins => [:comments], :conditions => "comments.body like 'Thank you%'", :order => 'posts.id').all
|
||||
end
|
||||
assert_equal [posts(:welcome)], posts
|
||||
assert_equal authors(:david), assert_no_queries { posts[0].author}
|
||||
|
||||
posts = assert_queries(2) do
|
||||
Post.find(:all, :include => :author, :joins => {:taggings => :tag}, :conditions => "tags.name = 'General'", :order => 'posts.id')
|
||||
Post.scoped(:include => :author, :joins => {:taggings => :tag}, :conditions => "tags.name = 'General'", :order => 'posts.id').all
|
||||
end
|
||||
assert_equal posts(:welcome, :thinking), posts
|
||||
|
||||
posts = assert_queries(2) do
|
||||
Post.find(:all, :include => :author, :joins => {:taggings => {:tag => :taggings}}, :conditions => "taggings_tags.super_tag_id=2", :order => 'posts.id')
|
||||
Post.scoped(:include => :author, :joins => {:taggings => {:tag => :taggings}}, :conditions => "taggings_tags.super_tag_id=2", :order => 'posts.id').all
|
||||
end
|
||||
assert_equal posts(:welcome, :thinking), posts
|
||||
|
||||
|
@ -988,13 +988,13 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
|
||||
def test_eager_loading_with_conditions_on_string_joined_table_preloads
|
||||
posts = assert_queries(2) do
|
||||
Post.find(:all, :select => 'distinct posts.*', :include => :author, :joins => "INNER JOIN comments on comments.post_id = posts.id", :conditions => "comments.body like 'Thank you%'", :order => 'posts.id')
|
||||
Post.scoped(:select => 'distinct posts.*', :include => :author, :joins => "INNER JOIN comments on comments.post_id = posts.id", :conditions => "comments.body like 'Thank you%'", :order => 'posts.id').all
|
||||
end
|
||||
assert_equal [posts(:welcome)], posts
|
||||
assert_equal authors(:david), assert_no_queries { posts[0].author}
|
||||
|
||||
posts = assert_queries(2) do
|
||||
Post.find(:all, :select => 'distinct posts.*', :include => :author, :joins => ["INNER JOIN comments on comments.post_id = posts.id"], :conditions => "comments.body like 'Thank you%'", :order => 'posts.id')
|
||||
Post.scoped(:select => 'distinct posts.*', :include => :author, :joins => ["INNER JOIN comments on comments.post_id = posts.id"], :conditions => "comments.body like 'Thank you%'", :order => 'posts.id').all
|
||||
end
|
||||
assert_equal [posts(:welcome)], posts
|
||||
assert_equal authors(:david), assert_no_queries { posts[0].author}
|
||||
|
@ -1003,7 +1003,7 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
|
||||
def test_eager_loading_with_select_on_joined_table_preloads
|
||||
posts = assert_queries(2) do
|
||||
Post.find(:all, :select => 'posts.*, authors.name as author_name', :include => :comments, :joins => :author, :order => 'posts.id')
|
||||
Post.scoped(:select => 'posts.*, authors.name as author_name', :include => :comments, :joins => :author, :order => 'posts.id').all
|
||||
end
|
||||
assert_equal 'David', posts[0].author_name
|
||||
assert_equal posts(:welcome).comments, assert_no_queries { posts[0].comments}
|
||||
|
@ -1013,14 +1013,14 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
Author.columns
|
||||
|
||||
authors = assert_queries(2) do
|
||||
Author.find(:all, :include => :author_address, :joins => :comments, :conditions => "posts.title like 'Welcome%'")
|
||||
Author.scoped(:include => :author_address, :joins => :comments, :conditions => "posts.title like 'Welcome%'").all
|
||||
end
|
||||
assert_equal authors(:david), authors[0]
|
||||
assert_equal author_addresses(:david_address), authors[0].author_address
|
||||
end
|
||||
|
||||
def test_preload_belongs_to_uses_exclusive_scope
|
||||
people = Person.males.find(:all, :include => :primary_contact)
|
||||
people = Person.males.scoped(:include => :primary_contact).all
|
||||
assert_not_equal people.length, 0
|
||||
people.each do |person|
|
||||
assert_no_queries {assert_not_nil person.primary_contact}
|
||||
|
@ -1029,7 +1029,7 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_preload_has_many_uses_exclusive_scope
|
||||
people = Person.males.find :all, :include => :agents
|
||||
people = Person.males.includes(:agents).all
|
||||
people.each do |person|
|
||||
assert_equal Person.find(person.id).agents, person.agents
|
||||
end
|
||||
|
@ -1037,7 +1037,7 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
|
||||
def test_preload_has_many_using_primary_key
|
||||
expected = Firm.first.clients_using_primary_key.to_a
|
||||
firm = Firm.find :first, :include => :clients_using_primary_key
|
||||
firm = Firm.includes(:clients_using_primary_key).first
|
||||
assert_no_queries do
|
||||
assert_equal expected, firm.clients_using_primary_key
|
||||
end
|
||||
|
@ -1058,7 +1058,7 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
|
||||
def test_preload_has_one_using_primary_key
|
||||
expected = accounts(:signals37)
|
||||
firm = Firm.find :first, :include => :account_using_primary_key, :order => 'companies.id'
|
||||
firm = Firm.scoped(:include => :account_using_primary_key, :order => 'companies.id').first
|
||||
assert_no_queries do
|
||||
assert_equal expected, firm.account_using_primary_key
|
||||
end
|
||||
|
@ -1066,7 +1066,7 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
|
||||
def test_include_has_one_using_primary_key
|
||||
expected = accounts(:signals37)
|
||||
firm = Firm.find(:all, :include => :account_using_primary_key, :order => 'accounts.id').detect {|f| f.id == 1}
|
||||
firm = Firm.scoped(:include => :account_using_primary_key, :order => 'accounts.id').all.detect {|f| f.id == 1}
|
||||
assert_no_queries do
|
||||
assert_equal expected, firm.account_using_primary_key
|
||||
end
|
||||
|
@ -1130,7 +1130,7 @@ class EagerAssociationTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_deep_including_through_habtm
|
||||
posts = Post.find(:all, :include => {:categories => :categorizations}, :order => "posts.id")
|
||||
posts = Post.scoped(:include => {:categories => :categorizations}, :order => "posts.id").all
|
||||
assert_no_queries { assert_equal 2, posts[0].categories[0].categorizations.length }
|
||||
assert_no_queries { assert_equal 1, posts[0].categories[1].categorizations.length }
|
||||
assert_no_queries { assert_equal 2, posts[1].categories[0].categorizations.length }
|
||||
|
|
|
@ -549,14 +549,14 @@ class HasAndBelongsToManyAssociationsTest < ActiveRecord::TestCase
|
|||
def test_find_with_merged_options
|
||||
assert_equal 1, projects(:active_record).limited_developers.size
|
||||
assert_equal 1, projects(:active_record).limited_developers.all.size
|
||||
assert_equal 3, projects(:active_record).limited_developers.find(:all, :limit => nil).size
|
||||
assert_equal 3, projects(:active_record).limited_developers.scoped(:limit => nil).all.size
|
||||
end
|
||||
|
||||
def test_dynamic_find_should_respect_association_order
|
||||
# Developers are ordered 'name DESC, id DESC'
|
||||
high_id_jamis = projects(:active_record).developers.create(:name => 'Jamis')
|
||||
|
||||
assert_equal high_id_jamis, projects(:active_record).developers.find(:first, :conditions => "name = 'Jamis'")
|
||||
assert_equal high_id_jamis, projects(:active_record).developers.scoped(:conditions => "name = 'Jamis'").first
|
||||
assert_equal high_id_jamis, projects(:active_record).developers.find_by_name('Jamis')
|
||||
end
|
||||
|
||||
|
@ -566,7 +566,7 @@ class HasAndBelongsToManyAssociationsTest < ActiveRecord::TestCase
|
|||
middle_id_jamis = developers(:poor_jamis)
|
||||
high_id_jamis = projects(:active_record).developers.create(:name => 'Jamis')
|
||||
|
||||
assert_equal [high_id_jamis, middle_id_jamis, low_id_jamis], projects(:active_record).developers.find(:all, :conditions => "name = 'Jamis'")
|
||||
assert_equal [high_id_jamis, middle_id_jamis, low_id_jamis], projects(:active_record).developers.scoped(:conditions => "name = 'Jamis'").all
|
||||
assert_equal [high_id_jamis, middle_id_jamis, low_id_jamis], projects(:active_record).developers.find_all_by_name('Jamis')
|
||||
end
|
||||
|
||||
|
@ -576,12 +576,12 @@ class HasAndBelongsToManyAssociationsTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_dynamic_find_all_should_respect_association_limit
|
||||
assert_equal 1, projects(:active_record).limited_developers.find(:all, :conditions => "name = 'Jamis'").length
|
||||
assert_equal 1, projects(:active_record).limited_developers.scoped(:conditions => "name = 'Jamis'").all.length
|
||||
assert_equal 1, projects(:active_record).limited_developers.find_all_by_name('Jamis').length
|
||||
end
|
||||
|
||||
def test_dynamic_find_all_order_should_override_association_limit
|
||||
assert_equal 2, projects(:active_record).limited_developers.find(:all, :conditions => "name = 'Jamis'", :limit => 9_000).length
|
||||
assert_equal 2, projects(:active_record).limited_developers.scoped(:conditions => "name = 'Jamis'", :limit => 9_000).all.length
|
||||
assert_equal 2, projects(:active_record).limited_developers.find_all_by_name('Jamis', :limit => 9_000).length
|
||||
end
|
||||
|
||||
|
@ -705,8 +705,8 @@ class HasAndBelongsToManyAssociationsTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_find_grouped
|
||||
all_posts_from_category1 = Post.find(:all, :conditions => "category_id = 1", :joins => :categories)
|
||||
grouped_posts_of_category1 = Post.find(:all, :conditions => "category_id = 1", :group => "author_id", :select => 'count(posts.id) as posts_count', :joins => :categories)
|
||||
all_posts_from_category1 = Post.scoped(:conditions => "category_id = 1", :joins => :categories).all
|
||||
grouped_posts_of_category1 = Post.scoped(:conditions => "category_id = 1", :group => "author_id", :select => 'count(posts.id) as posts_count', :joins => :categories).all
|
||||
assert_equal 5, all_posts_from_category1.size
|
||||
assert_equal 2, grouped_posts_of_category1.size
|
||||
end
|
||||
|
|
|
@ -242,19 +242,19 @@ class HasManyAssociationsTest < ActiveRecord::TestCase
|
|||
|
||||
# sometimes tests on Oracle fail if ORDER BY is not provided therefore add always :order with :first
|
||||
def test_counting_with_counter_sql
|
||||
assert_equal 2, Firm.find(:first, :order => "id").clients.count
|
||||
assert_equal 2, Firm.scoped(:order => "id").first.clients.count
|
||||
end
|
||||
|
||||
def test_counting
|
||||
assert_equal 2, Firm.find(:first, :order => "id").plain_clients.count
|
||||
assert_equal 2, Firm.scoped(:order => "id").first.plain_clients.count
|
||||
end
|
||||
|
||||
def test_counting_with_single_hash
|
||||
assert_equal 1, Firm.find(:first, :order => "id").plain_clients.where(:name => "Microsoft").count
|
||||
assert_equal 1, Firm.scoped(:order => "id").first.plain_clients.where(:name => "Microsoft").count
|
||||
end
|
||||
|
||||
def test_counting_with_column_name_and_hash
|
||||
assert_equal 2, Firm.find(:first, :order => "id").plain_clients.count(:name)
|
||||
assert_equal 2, Firm.scoped(:order => "id").first.plain_clients.count(:name)
|
||||
end
|
||||
|
||||
def test_counting_with_association_limit
|
||||
|
@ -264,7 +264,7 @@ class HasManyAssociationsTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_finding
|
||||
assert_equal 2, Firm.find(:first, :order => "id").clients.length
|
||||
assert_equal 2, Firm.scoped(:order => "id").first.clients.length
|
||||
end
|
||||
|
||||
def test_finding_array_compatibility
|
||||
|
@ -273,14 +273,14 @@ class HasManyAssociationsTest < ActiveRecord::TestCase
|
|||
|
||||
def test_find_with_blank_conditions
|
||||
[[], {}, nil, ""].each do |blank|
|
||||
assert_equal 2, Firm.find(:first, :order => "id").clients.find(:all, :conditions => blank).size
|
||||
assert_equal 2, Firm.scoped(:order => "id").first.clients.scoped(:conditions => blank).all.size
|
||||
end
|
||||
end
|
||||
|
||||
def test_find_many_with_merged_options
|
||||
assert_equal 1, companies(:first_firm).limited_clients.size
|
||||
assert_equal 1, companies(:first_firm).limited_clients.all.size
|
||||
assert_equal 2, companies(:first_firm).limited_clients.find(:all, :limit => nil).size
|
||||
assert_equal 2, companies(:first_firm).limited_clients.scoped(:limit => nil).all.size
|
||||
end
|
||||
|
||||
def test_find_should_append_to_association_order
|
||||
|
@ -293,22 +293,22 @@ class HasManyAssociationsTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_dynamic_find_should_respect_association_order
|
||||
assert_equal companies(:second_client), companies(:first_firm).clients_sorted_desc.find(:first, :conditions => "type = 'Client'")
|
||||
assert_equal companies(:second_client), companies(:first_firm).clients_sorted_desc.scoped(:conditions => "type = 'Client'").first
|
||||
assert_equal companies(:second_client), companies(:first_firm).clients_sorted_desc.find_by_type('Client')
|
||||
end
|
||||
|
||||
def test_dynamic_find_all_should_respect_association_order
|
||||
assert_equal [companies(:second_client), companies(:first_client)], companies(:first_firm).clients_sorted_desc.find(:all, :conditions => "type = 'Client'")
|
||||
assert_equal [companies(:second_client), companies(:first_client)], companies(:first_firm).clients_sorted_desc.scoped(:conditions => "type = 'Client'").all
|
||||
assert_equal [companies(:second_client), companies(:first_client)], companies(:first_firm).clients_sorted_desc.find_all_by_type('Client')
|
||||
end
|
||||
|
||||
def test_dynamic_find_all_should_respect_association_limit
|
||||
assert_equal 1, companies(:first_firm).limited_clients.find(:all, :conditions => "type = 'Client'").length
|
||||
assert_equal 1, companies(:first_firm).limited_clients.scoped(:conditions => "type = 'Client'").all.length
|
||||
assert_equal 1, companies(:first_firm).limited_clients.find_all_by_type('Client').length
|
||||
end
|
||||
|
||||
def test_dynamic_find_all_limit_should_override_association_limit
|
||||
assert_equal 2, companies(:first_firm).limited_clients.find(:all, :conditions => "type = 'Client'", :limit => 9_000).length
|
||||
assert_equal 2, companies(:first_firm).limited_clients.scoped(:conditions => "type = 'Client'", :limit => 9_000).all.length
|
||||
assert_equal 2, companies(:first_firm).limited_clients.find_all_by_type('Client', :limit => 9_000).length
|
||||
end
|
||||
|
||||
|
@ -328,59 +328,59 @@ class HasManyAssociationsTest < ActiveRecord::TestCase
|
|||
|
||||
def test_triple_equality
|
||||
# sometimes tests on Oracle fail if ORDER BY is not provided therefore add always :order with :first
|
||||
assert !(Array === Firm.find(:first, :order => "id").clients)
|
||||
assert Firm.find(:first, :order => "id").clients === Array
|
||||
assert !(Array === Firm.scoped(:order => "id").first.clients)
|
||||
assert Firm.scoped(:order => "id").first.clients === Array
|
||||
end
|
||||
|
||||
def test_finding_default_orders
|
||||
assert_equal "Summit", Firm.find(:first, :order => "id").clients.first.name
|
||||
assert_equal "Summit", Firm.scoped(:order => "id").first.clients.first.name
|
||||
end
|
||||
|
||||
def test_finding_with_different_class_name_and_order
|
||||
assert_equal "Microsoft", Firm.find(:first, :order => "id").clients_sorted_desc.first.name
|
||||
assert_equal "Microsoft", Firm.scoped(:order => "id").first.clients_sorted_desc.first.name
|
||||
end
|
||||
|
||||
def test_finding_with_foreign_key
|
||||
assert_equal "Microsoft", Firm.find(:first, :order => "id").clients_of_firm.first.name
|
||||
assert_equal "Microsoft", Firm.scoped(:order => "id").first.clients_of_firm.first.name
|
||||
end
|
||||
|
||||
def test_finding_with_condition
|
||||
assert_equal "Microsoft", Firm.find(:first, :order => "id").clients_like_ms.first.name
|
||||
assert_equal "Microsoft", Firm.scoped(:order => "id").first.clients_like_ms.first.name
|
||||
end
|
||||
|
||||
def test_finding_with_condition_hash
|
||||
assert_equal "Microsoft", Firm.find(:first, :order => "id").clients_like_ms_with_hash_conditions.first.name
|
||||
assert_equal "Microsoft", Firm.scoped(:order => "id").first.clients_like_ms_with_hash_conditions.first.name
|
||||
end
|
||||
|
||||
def test_finding_using_primary_key
|
||||
assert_equal "Summit", Firm.find(:first, :order => "id").clients_using_primary_key.first.name
|
||||
assert_equal "Summit", Firm.scoped(:order => "id").first.clients_using_primary_key.first.name
|
||||
end
|
||||
|
||||
def test_finding_using_sql
|
||||
firm = Firm.find(:first, :order => "id")
|
||||
firm = Firm.scoped(:order => "id").first
|
||||
first_client = firm.clients_using_sql.first
|
||||
assert_not_nil first_client
|
||||
assert_equal "Microsoft", first_client.name
|
||||
assert_equal 1, firm.clients_using_sql.size
|
||||
assert_equal 1, Firm.find(:first, :order => "id").clients_using_sql.size
|
||||
assert_equal 1, Firm.scoped(:order => "id").first.clients_using_sql.size
|
||||
end
|
||||
|
||||
def test_finding_using_sql_take_into_account_only_uniq_ids
|
||||
firm = Firm.find(:first, :order => "id")
|
||||
firm = Firm.scoped(:order => "id").first
|
||||
client = firm.clients_using_sql.first
|
||||
assert_equal client, firm.clients_using_sql.find(client.id, client.id)
|
||||
assert_equal client, firm.clients_using_sql.find(client.id, client.id.to_s)
|
||||
end
|
||||
|
||||
def test_counting_using_sql
|
||||
assert_equal 1, Firm.find(:first, :order => "id").clients_using_counter_sql.size
|
||||
assert Firm.find(:first, :order => "id").clients_using_counter_sql.any?
|
||||
assert_equal 0, Firm.find(:first, :order => "id").clients_using_zero_counter_sql.size
|
||||
assert !Firm.find(:first, :order => "id").clients_using_zero_counter_sql.any?
|
||||
assert_equal 1, Firm.scoped(:order => "id").first.clients_using_counter_sql.size
|
||||
assert Firm.scoped(:order => "id").first.clients_using_counter_sql.any?
|
||||
assert_equal 0, Firm.scoped(:order => "id").first.clients_using_zero_counter_sql.size
|
||||
assert !Firm.scoped(:order => "id").first.clients_using_zero_counter_sql.any?
|
||||
end
|
||||
|
||||
def test_counting_non_existant_items_using_sql
|
||||
assert_equal 0, Firm.find(:first, :order => "id").no_clients_using_counter_sql.size
|
||||
assert_equal 0, Firm.scoped(:order => "id").first.no_clients_using_counter_sql.size
|
||||
end
|
||||
|
||||
def test_counting_using_finder_sql
|
||||
|
@ -395,7 +395,7 @@ class HasManyAssociationsTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_find_ids
|
||||
firm = Firm.find(:first, :order => "id")
|
||||
firm = Firm.scoped(:order => "id").first
|
||||
|
||||
assert_raise(ActiveRecord::RecordNotFound) { firm.clients.find }
|
||||
|
||||
|
@ -415,7 +415,7 @@ class HasManyAssociationsTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_find_string_ids_when_using_finder_sql
|
||||
firm = Firm.find(:first, :order => "id")
|
||||
firm = Firm.scoped(:order => "id").first
|
||||
|
||||
client = firm.clients_using_finder_sql.find("2")
|
||||
assert_kind_of Client, client
|
||||
|
@ -431,9 +431,9 @@ class HasManyAssociationsTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_find_all
|
||||
firm = Firm.find(:first, :order => "id")
|
||||
assert_equal 2, firm.clients.find(:all, :conditions => "#{QUOTED_TYPE} = 'Client'").length
|
||||
assert_equal 1, firm.clients.find(:all, :conditions => "name = 'Summit'").length
|
||||
firm = Firm.scoped(:order => "id").first
|
||||
assert_equal 2, firm.clients.scoped(:conditions => "#{QUOTED_TYPE} = 'Client'").all.length
|
||||
assert_equal 1, firm.clients.scoped(:conditions => "name = 'Summit'").all.length
|
||||
end
|
||||
|
||||
def test_find_each
|
||||
|
@ -477,29 +477,29 @@ class HasManyAssociationsTest < ActiveRecord::TestCase
|
|||
|
||||
def test_find_all_sanitized
|
||||
# sometimes tests on Oracle fail if ORDER BY is not provided therefore add always :order with :first
|
||||
firm = Firm.find(:first, :order => "id")
|
||||
summit = firm.clients.find(:all, :conditions => "name = 'Summit'")
|
||||
assert_equal summit, firm.clients.find(:all, :conditions => ["name = ?", "Summit"])
|
||||
assert_equal summit, firm.clients.find(:all, :conditions => ["name = :name", { :name => "Summit" }])
|
||||
firm = Firm.scoped(:order => "id").first
|
||||
summit = firm.clients.scoped(:conditions => "name = 'Summit'").all
|
||||
assert_equal summit, firm.clients.scoped(:conditions => ["name = ?", "Summit"]).all
|
||||
assert_equal summit, firm.clients.scoped(:conditions => ["name = :name", { :name => "Summit" }]).all
|
||||
end
|
||||
|
||||
def test_find_first
|
||||
firm = Firm.find(:first, :order => "id")
|
||||
firm = Firm.scoped(:order => "id").first
|
||||
client2 = Client.find(2)
|
||||
assert_equal firm.clients.first, firm.clients.find(:first, :order => "id")
|
||||
assert_equal client2, firm.clients.find(:first, :conditions => "#{QUOTED_TYPE} = 'Client'", :order => "id")
|
||||
assert_equal firm.clients.first, firm.clients.scoped(:order => "id").first
|
||||
assert_equal client2, firm.clients.scoped(:conditions => "#{QUOTED_TYPE} = 'Client'", :order => "id").first
|
||||
end
|
||||
|
||||
def test_find_first_sanitized
|
||||
firm = Firm.find(:first, :order => "id")
|
||||
firm = Firm.scoped(:order => "id").first
|
||||
client2 = Client.find(2)
|
||||
assert_equal client2, firm.clients.find(:first, :conditions => ["#{QUOTED_TYPE} = ?", 'Client'], :order => "id")
|
||||
assert_equal client2, firm.clients.find(:first, :conditions => ["#{QUOTED_TYPE} = :type", { :type => 'Client' }], :order => "id")
|
||||
assert_equal client2, firm.clients.scoped(:conditions => ["#{QUOTED_TYPE} = ?", 'Client'], :order => "id").first
|
||||
assert_equal client2, firm.clients.scoped(:conditions => ["#{QUOTED_TYPE} = :type", { :type => 'Client' }], :order => "id").first
|
||||
end
|
||||
|
||||
def test_find_all_with_include_and_conditions
|
||||
assert_nothing_raised do
|
||||
Developer.find(:all, :joins => :audit_logs, :conditions => {'audit_logs.message' => nil, :name => 'Smith'})
|
||||
Developer.scoped(:joins => :audit_logs, :conditions => {'audit_logs.message' => nil, :name => 'Smith'}).all
|
||||
end
|
||||
end
|
||||
|
||||
|
@ -509,8 +509,8 @@ class HasManyAssociationsTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_find_grouped
|
||||
all_clients_of_firm1 = Client.find(:all, :conditions => "firm_id = 1")
|
||||
grouped_clients_of_firm1 = Client.find(:all, :conditions => "firm_id = 1", :group => "firm_id", :select => 'firm_id, count(id) as clients_count')
|
||||
all_clients_of_firm1 = Client.scoped(:conditions => "firm_id = 1").all
|
||||
grouped_clients_of_firm1 = Client.scoped(:conditions => "firm_id = 1", :group => "firm_id", :select => 'firm_id, count(id) as clients_count').all
|
||||
assert_equal 2, all_clients_of_firm1.size
|
||||
assert_equal 1, grouped_clients_of_firm1.size
|
||||
end
|
||||
|
@ -568,7 +568,7 @@ class HasManyAssociationsTest < ActiveRecord::TestCase
|
|||
|
||||
def test_create_with_bang_on_has_many_raises_when_record_not_saved
|
||||
assert_raise(ActiveRecord::RecordInvalid) do
|
||||
firm = Firm.find(:first, :order => "id")
|
||||
firm = Firm.scoped(:order => "id").first
|
||||
firm.plain_clients.create!
|
||||
end
|
||||
end
|
||||
|
@ -1110,7 +1110,7 @@ class HasManyAssociationsTest < ActiveRecord::TestCase
|
|||
firm = companies(:first_firm)
|
||||
assert_equal 2, firm.clients.size
|
||||
firm.destroy
|
||||
assert Client.find(:all, :conditions => "firm_id=#{firm.id}").empty?
|
||||
assert Client.scoped(:conditions => "firm_id=#{firm.id}").all.empty?
|
||||
end
|
||||
|
||||
def test_dependence_for_associations_with_hash_condition
|
||||
|
@ -1120,7 +1120,7 @@ class HasManyAssociationsTest < ActiveRecord::TestCase
|
|||
|
||||
def test_destroy_dependent_when_deleted_from_association
|
||||
# sometimes tests on Oracle fail if ORDER BY is not provided therefore add always :order with :first
|
||||
firm = Firm.find(:first, :order => "id")
|
||||
firm = Firm.scoped(:order => "id").first
|
||||
assert_equal 2, firm.clients.size
|
||||
|
||||
client = firm.clients.first
|
||||
|
@ -1148,7 +1148,7 @@ class HasManyAssociationsTest < ActiveRecord::TestCase
|
|||
|
||||
firm.destroy rescue "do nothing"
|
||||
|
||||
assert_equal 2, Client.find(:all, :conditions => "firm_id=#{firm.id}").size
|
||||
assert_equal 2, Client.scoped(:conditions => "firm_id=#{firm.id}").all.size
|
||||
end
|
||||
|
||||
def test_dependence_on_account
|
||||
|
@ -1215,7 +1215,7 @@ class HasManyAssociationsTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_replace_with_less
|
||||
firm = Firm.find(:first, :order => "id")
|
||||
firm = Firm.scoped(:order => "id").first
|
||||
firm.clients = [companies(:first_client)]
|
||||
assert firm.save, "Could not save firm"
|
||||
firm.reload
|
||||
|
@ -1229,7 +1229,7 @@ class HasManyAssociationsTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_replace_with_new
|
||||
firm = Firm.find(:first, :order => "id")
|
||||
firm = Firm.scoped(:order => "id").first
|
||||
firm.clients = [companies(:second_client), Client.new("name" => "New Client")]
|
||||
firm.save
|
||||
firm.reload
|
||||
|
@ -1329,27 +1329,27 @@ class HasManyAssociationsTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_dynamic_find_should_respect_association_order_for_through
|
||||
assert_equal Comment.find(10), authors(:david).comments_desc.find(:first, :conditions => "comments.type = 'SpecialComment'")
|
||||
assert_equal Comment.find(10), authors(:david).comments_desc.scoped(:conditions => "comments.type = 'SpecialComment'").first
|
||||
assert_equal Comment.find(10), authors(:david).comments_desc.find_by_type('SpecialComment')
|
||||
end
|
||||
|
||||
def test_dynamic_find_all_should_respect_association_order_for_through
|
||||
assert_equal [Comment.find(10), Comment.find(7), Comment.find(6), Comment.find(3)], authors(:david).comments_desc.find(:all, :conditions => "comments.type = 'SpecialComment'")
|
||||
assert_equal [Comment.find(10), Comment.find(7), Comment.find(6), Comment.find(3)], authors(:david).comments_desc.scoped(:conditions => "comments.type = 'SpecialComment'").all
|
||||
assert_equal [Comment.find(10), Comment.find(7), Comment.find(6), Comment.find(3)], authors(:david).comments_desc.find_all_by_type('SpecialComment')
|
||||
end
|
||||
|
||||
def test_dynamic_find_all_should_respect_association_limit_for_through
|
||||
assert_equal 1, authors(:david).limited_comments.find(:all, :conditions => "comments.type = 'SpecialComment'").length
|
||||
assert_equal 1, authors(:david).limited_comments.scoped(:conditions => "comments.type = 'SpecialComment'").all.length
|
||||
assert_equal 1, authors(:david).limited_comments.find_all_by_type('SpecialComment').length
|
||||
end
|
||||
|
||||
def test_dynamic_find_all_order_should_override_association_limit_for_through
|
||||
assert_equal 4, authors(:david).limited_comments.find(:all, :conditions => "comments.type = 'SpecialComment'", :limit => 9_000).length
|
||||
assert_equal 4, authors(:david).limited_comments.scoped(:conditions => "comments.type = 'SpecialComment'", :limit => 9_000).all.length
|
||||
assert_equal 4, authors(:david).limited_comments.find_all_by_type('SpecialComment', :limit => 9_000).length
|
||||
end
|
||||
|
||||
def test_find_all_include_over_the_same_table_for_through
|
||||
assert_equal 2, people(:michael).posts.find(:all, :include => :people).length
|
||||
assert_equal 2, people(:michael).posts.scoped(:include => :people).all.length
|
||||
end
|
||||
|
||||
def test_has_many_through_respects_hash_conditions
|
||||
|
@ -1568,7 +1568,7 @@ class HasManyAssociationsTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_creating_using_primary_key
|
||||
firm = Firm.find(:first, :order => "id")
|
||||
firm = Firm.scoped(:order => "id").first
|
||||
client = firm.clients_using_primary_key.create!(:name => 'test')
|
||||
assert_equal firm.name, client.firm_name
|
||||
end
|
||||
|
|
|
@ -25,7 +25,7 @@ class HasOneAssociationsTest < ActiveRecord::TestCase
|
|||
assert_queries(1) { assert_nil firm.account }
|
||||
assert_queries(0) { assert_nil firm.account }
|
||||
|
||||
firms = Firm.find(:all, :include => :account)
|
||||
firms = Firm.scoped(:include => :account).all
|
||||
assert_queries(0) { firms.each(&:account) }
|
||||
end
|
||||
|
||||
|
|
|
@ -73,7 +73,7 @@ class HasOneThroughAssociationsTest < ActiveRecord::TestCase
|
|||
|
||||
def test_has_one_through_eager_loading
|
||||
members = assert_queries(3) do #base table, through table, clubs table
|
||||
Member.find(:all, :include => :club, :conditions => ["name = ?", "Groucho Marx"])
|
||||
Member.scoped(:include => :club, :conditions => ["name = ?", "Groucho Marx"]).all
|
||||
end
|
||||
assert_equal 1, members.size
|
||||
assert_not_nil assert_no_queries {members[0].club}
|
||||
|
@ -81,7 +81,7 @@ class HasOneThroughAssociationsTest < ActiveRecord::TestCase
|
|||
|
||||
def test_has_one_through_eager_loading_through_polymorphic
|
||||
members = assert_queries(3) do #base table, through table, clubs table
|
||||
Member.find(:all, :include => :sponsor_club, :conditions => ["name = ?", "Groucho Marx"])
|
||||
Member.scoped(:include => :sponsor_club, :conditions => ["name = ?", "Groucho Marx"]).all
|
||||
end
|
||||
assert_equal 1, members.size
|
||||
assert_not_nil assert_no_queries {members[0].sponsor_club}
|
||||
|
@ -104,14 +104,14 @@ class HasOneThroughAssociationsTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_eager_has_one_through_polymorphic_with_source_type
|
||||
clubs = Club.find(:all, :include => :sponsored_member, :conditions => ["name = ?","Moustache and Eyebrow Fancier Club"])
|
||||
clubs = Club.scoped(:include => :sponsored_member, :conditions => ["name = ?","Moustache and Eyebrow Fancier Club"]).all
|
||||
# Only the eyebrow fanciers club has a sponsored_member
|
||||
assert_not_nil assert_no_queries {clubs[0].sponsored_member}
|
||||
end
|
||||
|
||||
def test_has_one_through_nonpreload_eagerloading
|
||||
members = assert_queries(1) do
|
||||
Member.find(:all, :include => :club, :conditions => ["members.name = ?", "Groucho Marx"], :order => 'clubs.name') #force fallback
|
||||
Member.scoped(:include => :club, :conditions => ["members.name = ?", "Groucho Marx"], :order => 'clubs.name').all #force fallback
|
||||
end
|
||||
assert_equal 1, members.size
|
||||
assert_not_nil assert_no_queries {members[0].club}
|
||||
|
@ -119,7 +119,7 @@ class HasOneThroughAssociationsTest < ActiveRecord::TestCase
|
|||
|
||||
def test_has_one_through_nonpreload_eager_loading_through_polymorphic
|
||||
members = assert_queries(1) do
|
||||
Member.find(:all, :include => :sponsor_club, :conditions => ["members.name = ?", "Groucho Marx"], :order => 'clubs.name') #force fallback
|
||||
Member.scoped(:include => :sponsor_club, :conditions => ["members.name = ?", "Groucho Marx"], :order => 'clubs.name').all #force fallback
|
||||
end
|
||||
assert_equal 1, members.size
|
||||
assert_not_nil assert_no_queries {members[0].sponsor_club}
|
||||
|
@ -128,7 +128,7 @@ class HasOneThroughAssociationsTest < ActiveRecord::TestCase
|
|||
def test_has_one_through_nonpreload_eager_loading_through_polymorphic_with_more_than_one_through_record
|
||||
Sponsor.new(:sponsor_club => clubs(:crazy_club), :sponsorable => members(:groucho)).save!
|
||||
members = assert_queries(1) do
|
||||
Member.find(:all, :include => :sponsor_club, :conditions => ["members.name = ?", "Groucho Marx"], :order => 'clubs.name DESC') #force fallback
|
||||
Member.scoped(:include => :sponsor_club, :conditions => ["members.name = ?", "Groucho Marx"], :order => 'clubs.name DESC').all #force fallback
|
||||
end
|
||||
assert_equal 1, members.size
|
||||
assert_not_nil assert_no_queries { members[0].sponsor_club }
|
||||
|
@ -197,7 +197,7 @@ class HasOneThroughAssociationsTest < ActiveRecord::TestCase
|
|||
@member.member_detail = @member_detail
|
||||
@member.organization = @organization
|
||||
@member_details = assert_queries(3) do
|
||||
MemberDetail.find(:all, :include => :member_type)
|
||||
MemberDetail.scoped(:include => :member_type).all
|
||||
end
|
||||
@new_detail = @member_details[0]
|
||||
assert @new_detail.send(:association, :member_type).loaded?
|
||||
|
|
|
@ -96,7 +96,7 @@ class InverseHasOneTests < ActiveRecord::TestCase
|
|||
|
||||
|
||||
def test_parent_instance_should_be_shared_with_eager_loaded_child_on_find
|
||||
m = Man.find(:first, :conditions => {:name => 'Gordon'}, :include => :face)
|
||||
m = Man.scoped(:conditions => {:name => 'Gordon'}, :include => :face).first
|
||||
f = m.face
|
||||
assert_equal m.name, f.man.name, "Name of man should be the same before changes to parent instance"
|
||||
m.name = 'Bongo'
|
||||
|
@ -104,7 +104,7 @@ class InverseHasOneTests < ActiveRecord::TestCase
|
|||
f.man.name = 'Mungo'
|
||||
assert_equal m.name, f.man.name, "Name of man should be the same after changes to child-owned instance"
|
||||
|
||||
m = Man.find(:first, :conditions => {:name => 'Gordon'}, :include => :face, :order => 'faces.id')
|
||||
m = Man.scoped(:conditions => {:name => 'Gordon'}, :include => :face, :order => 'faces.id').first
|
||||
f = m.face
|
||||
assert_equal m.name, f.man.name, "Name of man should be the same before changes to parent instance"
|
||||
m.name = 'Bongo'
|
||||
|
@ -179,7 +179,7 @@ class InverseHasManyTests < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_parent_instance_should_be_shared_with_eager_loaded_children
|
||||
m = Man.find(:first, :conditions => {:name => 'Gordon'}, :include => :interests)
|
||||
m = Man.scoped(:conditions => {:name => 'Gordon'}, :include => :interests).first
|
||||
is = m.interests
|
||||
is.each do |i|
|
||||
assert_equal m.name, i.man.name, "Name of man should be the same before changes to parent instance"
|
||||
|
@ -189,7 +189,7 @@ class InverseHasManyTests < ActiveRecord::TestCase
|
|||
assert_equal m.name, i.man.name, "Name of man should be the same after changes to child-owned instance"
|
||||
end
|
||||
|
||||
m = Man.find(:first, :conditions => {:name => 'Gordon'}, :include => :interests, :order => 'interests.id')
|
||||
m = Man.scoped(:conditions => {:name => 'Gordon'}, :include => :interests, :order => 'interests.id').first
|
||||
is = m.interests
|
||||
is.each do |i|
|
||||
assert_equal m.name, i.man.name, "Name of man should be the same before changes to parent instance"
|
||||
|
@ -278,7 +278,7 @@ class InverseBelongsToTests < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_eager_loaded_child_instance_should_be_shared_with_parent_on_find
|
||||
f = Face.find(:first, :include => :man, :conditions => {:description => 'trusting'})
|
||||
f = Face.scoped(:include => :man, :conditions => {:description => 'trusting'}).first
|
||||
m = f.man
|
||||
assert_equal f.description, m.face.description, "Description of face should be the same before changes to child instance"
|
||||
f.description = 'gormless'
|
||||
|
@ -286,7 +286,7 @@ class InverseBelongsToTests < ActiveRecord::TestCase
|
|||
m.face.description = 'pleasing'
|
||||
assert_equal f.description, m.face.description, "Description of face should be the same after changes to parent-owned instance"
|
||||
|
||||
f = Face.find(:first, :include => :man, :order => 'men.id', :conditions => {:description => 'trusting'})
|
||||
f = Face.scoped(:include => :man, :order => 'men.id', :conditions => {:description => 'trusting'}).first
|
||||
m = f.man
|
||||
assert_equal f.description, m.face.description, "Description of face should be the same before changes to child instance"
|
||||
f.description = 'gormless'
|
||||
|
@ -351,7 +351,7 @@ class InversePolymorphicBelongsToTests < ActiveRecord::TestCase
|
|||
fixtures :men, :faces, :interests
|
||||
|
||||
def test_child_instance_should_be_shared_with_parent_on_find
|
||||
f = Face.find(:first, :conditions => {:description => 'confused'})
|
||||
f = Face.scoped(:conditions => {:description => 'confused'}).first
|
||||
m = f.polymorphic_man
|
||||
assert_equal f.description, m.polymorphic_face.description, "Description of face should be the same before changes to child instance"
|
||||
f.description = 'gormless'
|
||||
|
@ -361,7 +361,7 @@ class InversePolymorphicBelongsToTests < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_eager_loaded_child_instance_should_be_shared_with_parent_on_find
|
||||
f = Face.find(:first, :conditions => {:description => 'confused'}, :include => :man)
|
||||
f = Face.scoped(:conditions => {:description => 'confused'}, :include => :man).first
|
||||
m = f.polymorphic_man
|
||||
assert_equal f.description, m.polymorphic_face.description, "Description of face should be the same before changes to child instance"
|
||||
f.description = 'gormless'
|
||||
|
@ -369,7 +369,7 @@ class InversePolymorphicBelongsToTests < ActiveRecord::TestCase
|
|||
m.polymorphic_face.description = 'pleasing'
|
||||
assert_equal f.description, m.polymorphic_face.description, "Description of face should be the same after changes to parent-owned instance"
|
||||
|
||||
f = Face.find(:first, :conditions => {:description => 'confused'}, :include => :man, :order => 'men.id')
|
||||
f = Face.scoped(:conditions => {:description => 'confused'}, :include => :man, :order => 'men.id').first
|
||||
m = f.polymorphic_man
|
||||
assert_equal f.description, m.polymorphic_face.description, "Description of face should be the same before changes to child instance"
|
||||
f.description = 'gormless'
|
||||
|
|
|
@ -237,8 +237,8 @@ class AssociationsJoinModelTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_include_has_many_through
|
||||
posts = Post.find(:all, :order => 'posts.id')
|
||||
posts_with_authors = Post.find(:all, :include => :authors, :order => 'posts.id')
|
||||
posts = Post.scoped(:order => 'posts.id').all
|
||||
posts_with_authors = Post.scoped(:include => :authors, :order => 'posts.id').all
|
||||
assert_equal posts.length, posts_with_authors.length
|
||||
posts.length.times do |i|
|
||||
assert_equal posts[i].authors.length, assert_no_queries { posts_with_authors[i].authors.length }
|
||||
|
@ -262,8 +262,8 @@ class AssociationsJoinModelTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_include_polymorphic_has_many_through
|
||||
posts = Post.find(:all, :order => 'posts.id')
|
||||
posts_with_tags = Post.find(:all, :include => :tags, :order => 'posts.id')
|
||||
posts = Post.scoped(:order => 'posts.id').all
|
||||
posts_with_tags = Post.scoped(:include => :tags, :order => 'posts.id').all
|
||||
assert_equal posts.length, posts_with_tags.length
|
||||
posts.length.times do |i|
|
||||
assert_equal posts[i].tags.length, assert_no_queries { posts_with_tags[i].tags.length }
|
||||
|
@ -271,8 +271,8 @@ class AssociationsJoinModelTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_include_polymorphic_has_many
|
||||
posts = Post.find(:all, :order => 'posts.id')
|
||||
posts_with_taggings = Post.find(:all, :include => :taggings, :order => 'posts.id')
|
||||
posts = Post.scoped(:order => 'posts.id').all
|
||||
posts_with_taggings = Post.scoped(:include => :taggings, :order => 'posts.id').all
|
||||
assert_equal posts.length, posts_with_taggings.length
|
||||
posts.length.times do |i|
|
||||
assert_equal posts[i].taggings.length, assert_no_queries { posts_with_taggings[i].taggings.length }
|
||||
|
@ -292,8 +292,8 @@ class AssociationsJoinModelTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_has_many_find_conditions
|
||||
assert_equal categories(:general), authors(:david).categories.find(:first, :conditions => "categories.name = 'General'")
|
||||
assert_nil authors(:david).categories.find(:first, :conditions => "categories.name = 'Technology'")
|
||||
assert_equal categories(:general), authors(:david).categories.scoped(:conditions => "categories.name = 'General'").first
|
||||
assert_nil authors(:david).categories.scoped(:conditions => "categories.name = 'Technology'").first
|
||||
end
|
||||
|
||||
def test_has_many_class_methods_called_by_method_missing
|
||||
|
@ -374,20 +374,20 @@ class AssociationsJoinModelTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_has_many_through_has_many_find_all
|
||||
assert_equal comments(:greetings), authors(:david).comments.find(:all, :order => 'comments.id').first
|
||||
assert_equal comments(:greetings), authors(:david).comments.scoped(:order => 'comments.id').all.first
|
||||
end
|
||||
|
||||
def test_has_many_through_has_many_find_all_with_custom_class
|
||||
assert_equal comments(:greetings), authors(:david).funky_comments.find(:all, :order => 'comments.id').first
|
||||
assert_equal comments(:greetings), authors(:david).funky_comments.scoped(:order => 'comments.id').all.first
|
||||
end
|
||||
|
||||
def test_has_many_through_has_many_find_first
|
||||
assert_equal comments(:greetings), authors(:david).comments.find(:first, :order => 'comments.id')
|
||||
assert_equal comments(:greetings), authors(:david).comments.scoped(:order => 'comments.id').first
|
||||
end
|
||||
|
||||
def test_has_many_through_has_many_find_conditions
|
||||
options = { :conditions => "comments.#{QUOTED_TYPE}='SpecialComment'", :order => 'comments.id' }
|
||||
assert_equal comments(:does_it_hurt), authors(:david).comments.find(:first, options)
|
||||
assert_equal comments(:does_it_hurt), authors(:david).comments.scoped(options).first
|
||||
end
|
||||
|
||||
def test_has_many_through_has_many_find_by_id
|
||||
|
@ -411,7 +411,7 @@ class AssociationsJoinModelTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_eager_load_has_many_through_has_many
|
||||
author = Author.find :first, :conditions => ['name = ?', 'David'], :include => :comments, :order => 'comments.id'
|
||||
author = Author.scoped(:conditions => ['name = ?', 'David'], :include => :comments, :order => 'comments.id').first
|
||||
SpecialComment.new; VerySpecialComment.new
|
||||
assert_no_queries do
|
||||
assert_equal [1,2,3,5,6,7,8,9,10,12], author.comments.collect(&:id)
|
||||
|
@ -419,7 +419,7 @@ class AssociationsJoinModelTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_eager_load_has_many_through_has_many_with_conditions
|
||||
post = Post.find(:first, :include => :invalid_tags)
|
||||
post = Post.scoped(:include => :invalid_tags).first
|
||||
assert_no_queries do
|
||||
post.invalid_tags
|
||||
end
|
||||
|
@ -427,8 +427,8 @@ class AssociationsJoinModelTest < ActiveRecord::TestCase
|
|||
|
||||
def test_eager_belongs_to_and_has_one_not_singularized
|
||||
assert_nothing_raised do
|
||||
Author.find(:first, :include => :author_address)
|
||||
AuthorAddress.find(:first, :include => :author)
|
||||
Author.scoped(:include => :author_address).first
|
||||
AuthorAddress.scoped(:include => :author).first
|
||||
end
|
||||
end
|
||||
|
||||
|
@ -632,8 +632,8 @@ class AssociationsJoinModelTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_preload_polymorphic_has_many_through
|
||||
posts = Post.find(:all, :order => 'posts.id')
|
||||
posts_with_tags = Post.find(:all, :include => :tags, :order => 'posts.id')
|
||||
posts = Post.scoped(:order => 'posts.id').all
|
||||
posts_with_tags = Post.scoped(:include => :tags, :order => 'posts.id').all
|
||||
assert_equal posts.length, posts_with_tags.length
|
||||
posts.length.times do |i|
|
||||
assert_equal posts[i].tags.length, assert_no_queries { posts_with_tags[i].tags.length }
|
||||
|
@ -641,7 +641,7 @@ class AssociationsJoinModelTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_preload_polymorph_many_types
|
||||
taggings = Tagging.find :all, :include => :taggable, :conditions => ['taggable_type != ?', 'FakeModel']
|
||||
taggings = Tagging.scoped(:include => :taggable, :conditions => ['taggable_type != ?', 'FakeModel']).all
|
||||
assert_no_queries do
|
||||
taggings.first.taggable.id
|
||||
taggings[1].taggable.id
|
||||
|
@ -654,13 +654,13 @@ class AssociationsJoinModelTest < ActiveRecord::TestCase
|
|||
|
||||
def test_preload_nil_polymorphic_belongs_to
|
||||
assert_nothing_raised do
|
||||
Tagging.find(:all, :include => :taggable, :conditions => ['taggable_type IS NULL'])
|
||||
Tagging.scoped(:include => :taggable, :conditions => ['taggable_type IS NULL']).all
|
||||
end
|
||||
end
|
||||
|
||||
def test_preload_polymorphic_has_many
|
||||
posts = Post.find(:all, :order => 'posts.id')
|
||||
posts_with_taggings = Post.find(:all, :include => :taggings, :order => 'posts.id')
|
||||
posts = Post.scoped(:order => 'posts.id').all
|
||||
posts_with_taggings = Post.scoped(:include => :taggings, :order => 'posts.id').all
|
||||
assert_equal posts.length, posts_with_taggings.length
|
||||
posts.length.times do |i|
|
||||
assert_equal posts[i].taggings.length, assert_no_queries { posts_with_taggings[i].taggings.length }
|
||||
|
@ -668,7 +668,7 @@ class AssociationsJoinModelTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_belongs_to_shared_parent
|
||||
comments = Comment.find(:all, :include => :post, :conditions => 'post_id = 1')
|
||||
comments = Comment.scoped(:include => :post, :conditions => 'post_id = 1').all
|
||||
assert_no_queries do
|
||||
assert_equal comments.first.post, comments[1].post
|
||||
end
|
||||
|
|
|
@ -74,8 +74,8 @@ class AssociationsTest < ActiveRecord::TestCase
|
|||
|
||||
|
||||
def test_include_with_order_works
|
||||
assert_nothing_raised {Account.find(:first, :order => 'id', :include => :firm)}
|
||||
assert_nothing_raised {Account.find(:first, :order => :id, :include => :firm)}
|
||||
assert_nothing_raised {Account.scoped(:order => 'id', :include => :firm).first}
|
||||
assert_nothing_raised {Account.scoped(:order => :id, :include => :firm).first}
|
||||
end
|
||||
|
||||
def test_bad_collection_keys
|
||||
|
|
|
@ -484,9 +484,9 @@ class AttributeMethodsTest < ActiveRecord::TestCase
|
|||
topic = Topic.create(:title => 'Budget')
|
||||
# Oracle does not support boolean expressions in SELECT
|
||||
if current_adapter?(:OracleAdapter)
|
||||
topic = Topic.find(:first, :select => "topics.*, 0 as is_test")
|
||||
topic = Topic.scoped(:select => "topics.*, 0 as is_test").first
|
||||
else
|
||||
topic = Topic.find(:first, :select => "topics.*, 1=2 as is_test")
|
||||
topic = Topic.scoped(:select => "topics.*, 1=2 as is_test").first
|
||||
end
|
||||
assert !topic.is_test?
|
||||
end
|
||||
|
@ -495,9 +495,9 @@ class AttributeMethodsTest < ActiveRecord::TestCase
|
|||
topic = Topic.create(:title => 'Budget')
|
||||
# Oracle does not support boolean expressions in SELECT
|
||||
if current_adapter?(:OracleAdapter)
|
||||
topic = Topic.find(:first, :select => "topics.*, 1 as is_test")
|
||||
topic = Topic.scoped(:select => "topics.*, 1 as is_test").first
|
||||
else
|
||||
topic = Topic.find(:first, :select => "topics.*, 2=2 as is_test")
|
||||
topic = Topic.scoped(:select => "topics.*, 2=2 as is_test").first
|
||||
end
|
||||
assert topic.is_test?
|
||||
end
|
||||
|
|
|
@ -155,7 +155,7 @@ class TestDefaultAutosaveAssociationOnAHasOneAssociation < ActiveRecord::TestCas
|
|||
end
|
||||
|
||||
def test_not_resaved_when_unchanged
|
||||
firm = Firm.find(:first, :include => :account)
|
||||
firm = Firm.scoped(:include => :account).first
|
||||
firm.name += '-changed'
|
||||
assert_queries(1) { firm.save! }
|
||||
|
||||
|
|
|
@ -350,13 +350,13 @@ class BasicsTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_load
|
||||
topics = Topic.find(:all, :order => 'id')
|
||||
topics = Topic.scoped(:order => 'id').all
|
||||
assert_equal(4, topics.size)
|
||||
assert_equal(topics(:first).title, topics.first.title)
|
||||
end
|
||||
|
||||
def test_load_with_condition
|
||||
topics = Topic.find(:all, :conditions => "author_name = 'Mary'")
|
||||
topics = Topic.scoped(:conditions => "author_name = 'Mary'").all
|
||||
|
||||
assert_equal(1, topics.size)
|
||||
assert_equal(topics(:second).title, topics.first.title)
|
||||
|
@ -1264,10 +1264,10 @@ class BasicsTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_quoting_arrays
|
||||
replies = Reply.find(:all, :conditions => [ "id IN (?)", topics(:first).replies.collect(&:id) ])
|
||||
replies = Reply.scoped(:conditions => [ "id IN (?)", topics(:first).replies.collect(&:id) ]).all
|
||||
assert_equal topics(:first).replies.size, replies.size
|
||||
|
||||
replies = Reply.find(:all, :conditions => [ "id IN (?)", [] ])
|
||||
replies = Reply.scoped(:conditions => [ "id IN (?)", [] ]).all
|
||||
assert_equal 0, replies.size
|
||||
end
|
||||
|
||||
|
@ -1595,17 +1595,17 @@ class BasicsTest < ActiveRecord::TestCase
|
|||
|
||||
def test_no_limit_offset
|
||||
assert_nothing_raised do
|
||||
Developer.find(:all, :offset => 2)
|
||||
Developer.scoped(:offset => 2).all
|
||||
end
|
||||
end
|
||||
|
||||
def test_find_last
|
||||
last = Developer.last
|
||||
assert_equal last, Developer.find(:first, :order => 'id desc')
|
||||
assert_equal last, Developer.scoped(:order => 'id desc').first
|
||||
end
|
||||
|
||||
def test_last
|
||||
assert_equal Developer.find(:first, :order => 'id desc'), Developer.last
|
||||
assert_equal Developer.scoped(:order => 'id desc').first, Developer.last
|
||||
end
|
||||
|
||||
def test_all
|
||||
|
@ -1615,37 +1615,37 @@ class BasicsTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_all_with_conditions
|
||||
assert_equal Developer.find(:all, :order => 'id desc'), Developer.order('id desc').all
|
||||
assert_equal Developer.scoped(:order => 'id desc').all, Developer.order('id desc').all
|
||||
end
|
||||
|
||||
def test_find_ordered_last
|
||||
last = Developer.find :last, :order => 'developers.salary ASC'
|
||||
assert_equal last, Developer.find(:all, :order => 'developers.salary ASC').last
|
||||
last = Developer.scoped(:order => 'developers.salary ASC').last
|
||||
assert_equal last, Developer.scoped(:order => 'developers.salary ASC').all.last
|
||||
end
|
||||
|
||||
def test_find_reverse_ordered_last
|
||||
last = Developer.find :last, :order => 'developers.salary DESC'
|
||||
assert_equal last, Developer.find(:all, :order => 'developers.salary DESC').last
|
||||
last = Developer.scoped(:order => 'developers.salary DESC').last
|
||||
assert_equal last, Developer.scoped(:order => 'developers.salary DESC').all.last
|
||||
end
|
||||
|
||||
def test_find_multiple_ordered_last
|
||||
last = Developer.find :last, :order => 'developers.name, developers.salary DESC'
|
||||
assert_equal last, Developer.find(:all, :order => 'developers.name, developers.salary DESC').last
|
||||
last = Developer.scoped(:order => 'developers.name, developers.salary DESC').last
|
||||
assert_equal last, Developer.scoped(:order => 'developers.name, developers.salary DESC').all.last
|
||||
end
|
||||
|
||||
def test_find_keeps_multiple_order_values
|
||||
combined = Developer.find(:all, :order => 'developers.name, developers.salary')
|
||||
assert_equal combined, Developer.find(:all, :order => ['developers.name', 'developers.salary'])
|
||||
combined = Developer.scoped(:order => 'developers.name, developers.salary').all
|
||||
assert_equal combined, Developer.scoped(:order => ['developers.name', 'developers.salary']).all
|
||||
end
|
||||
|
||||
def test_find_keeps_multiple_group_values
|
||||
combined = Developer.find(:all, :group => 'developers.name, developers.salary, developers.id, developers.created_at, developers.updated_at')
|
||||
assert_equal combined, Developer.find(:all, :group => ['developers.name', 'developers.salary', 'developers.id', 'developers.created_at', 'developers.updated_at'])
|
||||
combined = Developer.scoped(:group => 'developers.name, developers.salary, developers.id, developers.created_at, developers.updated_at').all
|
||||
assert_equal combined, Developer.scoped(:group => ['developers.name', 'developers.salary', 'developers.id', 'developers.created_at', 'developers.updated_at']).all
|
||||
end
|
||||
|
||||
def test_find_symbol_ordered_last
|
||||
last = Developer.find :last, :order => :salary
|
||||
assert_equal last, Developer.find(:all, :order => :salary).last
|
||||
last = Developer.scoped(:order => :salary).last
|
||||
assert_equal last, Developer.scoped(:order => :salary).all.last
|
||||
end
|
||||
|
||||
def test_abstract_class
|
||||
|
@ -1739,8 +1739,8 @@ class BasicsTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_inspect_limited_select_instance
|
||||
assert_equal %(#<Topic id: 1>), Topic.find(:first, :select => 'id', :conditions => 'id = 1').inspect
|
||||
assert_equal %(#<Topic id: 1, title: "The First Topic">), Topic.find(:first, :select => 'id, title', :conditions => 'id = 1').inspect
|
||||
assert_equal %(#<Topic id: 1>), Topic.scoped(:select => 'id', :conditions => 'id = 1').first.inspect
|
||||
assert_equal %(#<Topic id: 1, title: "The First Topic">), Topic.scoped(:select => 'id, title', :conditions => 'id = 1').first.inspect
|
||||
end
|
||||
|
||||
def test_inspect_class_without_table
|
||||
|
|
|
@ -117,56 +117,6 @@ class FinderTest < ActiveRecord::TestCase
|
|||
assert_raise(ActiveRecord::RecordNotFound) { Topic.find(1, 2, 45) }
|
||||
end
|
||||
|
||||
def test_find_all_with_limit
|
||||
assert_equal(2, Entrant.find(:all, :limit => 2).size)
|
||||
assert_equal(0, Entrant.find(:all, :limit => 0).size)
|
||||
end
|
||||
|
||||
def test_find_all_with_prepared_limit_and_offset
|
||||
entrants = Entrant.find(:all, :order => "id ASC", :limit => 2, :offset => 1)
|
||||
|
||||
assert_equal(2, entrants.size)
|
||||
assert_equal(entrants(:second).name, entrants.first.name)
|
||||
|
||||
assert_equal 3, Entrant.count
|
||||
entrants = Entrant.find(:all, :order => "id ASC", :limit => 2, :offset => 2)
|
||||
assert_equal(1, entrants.size)
|
||||
assert_equal(entrants(:third).name, entrants.first.name)
|
||||
end
|
||||
|
||||
def test_find_all_with_limit_and_offset_and_multiple_order_clauses
|
||||
first_three_posts = Post.find :all, :order => 'author_id, id', :limit => 3, :offset => 0
|
||||
second_three_posts = Post.find :all, :order => ' author_id,id ', :limit => 3, :offset => 3
|
||||
third_three_posts = Post.find :all, :order => ' author_id, id ', :limit => 3, :offset => 6
|
||||
last_posts = Post.find :all, :order => ' author_id, id ', :limit => 3, :offset => 9
|
||||
|
||||
assert_equal [[0,3],[1,1],[1,2]], first_three_posts.map { |p| [p.author_id, p.id] }
|
||||
assert_equal [[1,4],[1,5],[1,6]], second_three_posts.map { |p| [p.author_id, p.id] }
|
||||
assert_equal [[2,7],[2,9],[2,11]], third_three_posts.map { |p| [p.author_id, p.id] }
|
||||
assert_equal [[3,8],[3,10]], last_posts.map { |p| [p.author_id, p.id] }
|
||||
end
|
||||
|
||||
|
||||
def test_find_with_group
|
||||
developers = Developer.find(:all, :group => "salary", :select => "salary")
|
||||
assert_equal 4, developers.size
|
||||
assert_equal 4, developers.map(&:salary).uniq.size
|
||||
end
|
||||
|
||||
def test_find_with_group_and_having
|
||||
developers = Developer.find(:all, :group => "salary", :having => "sum(salary) > 10000", :select => "salary")
|
||||
assert_equal 3, developers.size
|
||||
assert_equal 3, developers.map(&:salary).uniq.size
|
||||
assert developers.all? { |developer| developer.salary > 10000 }
|
||||
end
|
||||
|
||||
def test_find_with_group_and_sanitized_having
|
||||
developers = Developer.find(:all, :group => "salary", :having => ["sum(salary) > ?", 10000], :select => "salary")
|
||||
assert_equal 3, developers.size
|
||||
assert_equal 3, developers.map(&:salary).uniq.size
|
||||
assert developers.all? { |developer| developer.salary > 10000 }
|
||||
end
|
||||
|
||||
def test_find_with_group_and_sanitized_having_method
|
||||
developers = Developer.group(:salary).having("sum(salary) > ?", 10000).select('salary').all
|
||||
assert_equal 3, developers.size
|
||||
|
@ -193,16 +143,6 @@ class FinderTest < ActiveRecord::TestCase
|
|||
assert_equal [Account], accounts.collect(&:class).uniq
|
||||
end
|
||||
|
||||
def test_find_first
|
||||
first = Topic.find(:first, :conditions => "title = 'The First Topic'")
|
||||
assert_equal(topics(:first).title, first.title)
|
||||
end
|
||||
|
||||
def test_find_first_failing
|
||||
first = Topic.find(:first, :conditions => "title = 'The First Topic!'")
|
||||
assert_nil(first)
|
||||
end
|
||||
|
||||
def test_first
|
||||
assert_equal topics(:second).title, Topic.where("title = 'The Second Topic of the day'").first.title
|
||||
end
|
||||
|
@ -296,18 +236,6 @@ class FinderTest < ActiveRecord::TestCase
|
|||
assert_respond_to topic, "author_name"
|
||||
end
|
||||
|
||||
def test_find_on_blank_conditions
|
||||
[nil, " ", [], {}].each do |blank|
|
||||
assert_nothing_raised { Topic.find(:first, :conditions => blank) }
|
||||
end
|
||||
end
|
||||
|
||||
def test_find_on_blank_bind_conditions
|
||||
[ [""], ["",{}] ].each do |blank|
|
||||
assert_nothing_raised { Topic.find(:first, :conditions => blank) }
|
||||
end
|
||||
end
|
||||
|
||||
def test_find_on_array_conditions
|
||||
assert Topic.find(1, :conditions => ["approved = ?", false])
|
||||
assert_raise(ActiveRecord::RecordNotFound) { Topic.find(1, :conditions => ["approved = ?", true]) }
|
||||
|
@ -353,23 +281,23 @@ class FinderTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_find_on_hash_conditions_with_range
|
||||
assert_equal [1,2], Topic.find(:all, :conditions => { :id => 1..2 }).map(&:id).sort
|
||||
assert_equal [1,2], Topic.scoped(:conditions => { :id => 1..2 }).all.map(&:id).sort
|
||||
assert_raise(ActiveRecord::RecordNotFound) { Topic.find(1, :conditions => { :id => 2..3 }) }
|
||||
end
|
||||
|
||||
def test_find_on_hash_conditions_with_end_exclusive_range
|
||||
assert_equal [1,2,3], Topic.find(:all, :conditions => { :id => 1..3 }).map(&:id).sort
|
||||
assert_equal [1,2], Topic.find(:all, :conditions => { :id => 1...3 }).map(&:id).sort
|
||||
assert_equal [1,2,3], Topic.scoped(:conditions => { :id => 1..3 }).all.map(&:id).sort
|
||||
assert_equal [1,2], Topic.scoped(:conditions => { :id => 1...3 }).all.map(&:id).sort
|
||||
assert_raise(ActiveRecord::RecordNotFound) { Topic.find(3, :conditions => { :id => 2...3 }) }
|
||||
end
|
||||
|
||||
def test_find_on_hash_conditions_with_multiple_ranges
|
||||
assert_equal [1,2,3], Comment.find(:all, :conditions => { :id => 1..3, :post_id => 1..2 }).map(&:id).sort
|
||||
assert_equal [1], Comment.find(:all, :conditions => { :id => 1..1, :post_id => 1..10 }).map(&:id).sort
|
||||
assert_equal [1,2,3], Comment.scoped(:conditions => { :id => 1..3, :post_id => 1..2 }).all.map(&:id).sort
|
||||
assert_equal [1], Comment.scoped(:conditions => { :id => 1..1, :post_id => 1..10 }).all.map(&:id).sort
|
||||
end
|
||||
|
||||
def test_find_on_hash_conditions_with_array_of_integers_and_ranges
|
||||
assert_equal [1,2,3,5,6,7,8,9], Comment.find(:all, :conditions => {:id => [1..2, 3, 5, 6..8, 9]}).map(&:id).sort
|
||||
assert_equal [1,2,3,5,6,7,8,9], Comment.scoped(:conditions => {:id => [1..2, 3, 5, 6..8, 9]}).all.map(&:id).sort
|
||||
end
|
||||
|
||||
def test_find_on_multiple_hash_conditions
|
||||
|
@ -380,44 +308,44 @@ class FinderTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_condition_interpolation
|
||||
assert_kind_of Firm, Company.find(:first, :conditions => ["name = '%s'", "37signals"])
|
||||
assert_nil Company.find(:first, :conditions => ["name = '%s'", "37signals!"])
|
||||
assert_nil Company.find(:first, :conditions => ["name = '%s'", "37signals!' OR 1=1"])
|
||||
assert_kind_of Time, Topic.find(:first, :conditions => ["id = %d", 1]).written_on
|
||||
assert_kind_of Firm, Company.where("name = '%s'", "37signals").first
|
||||
assert_nil Company.scoped(:conditions => ["name = '%s'", "37signals!"]).first
|
||||
assert_nil Company.scoped(:conditions => ["name = '%s'", "37signals!' OR 1=1"]).first
|
||||
assert_kind_of Time, Topic.scoped(:conditions => ["id = %d", 1]).first.written_on
|
||||
end
|
||||
|
||||
def test_condition_array_interpolation
|
||||
assert_kind_of Firm, Company.find(:first, :conditions => ["name = '%s'", "37signals"])
|
||||
assert_nil Company.find(:first, :conditions => ["name = '%s'", "37signals!"])
|
||||
assert_nil Company.find(:first, :conditions => ["name = '%s'", "37signals!' OR 1=1"])
|
||||
assert_kind_of Time, Topic.find(:first, :conditions => ["id = %d", 1]).written_on
|
||||
assert_kind_of Firm, Company.scoped(:conditions => ["name = '%s'", "37signals"]).first
|
||||
assert_nil Company.scoped(:conditions => ["name = '%s'", "37signals!"]).first
|
||||
assert_nil Company.scoped(:conditions => ["name = '%s'", "37signals!' OR 1=1"]).first
|
||||
assert_kind_of Time, Topic.scoped(:conditions => ["id = %d", 1]).first.written_on
|
||||
end
|
||||
|
||||
def test_condition_hash_interpolation
|
||||
assert_kind_of Firm, Company.find(:first, :conditions => { :name => "37signals"})
|
||||
assert_nil Company.find(:first, :conditions => { :name => "37signals!"})
|
||||
assert_kind_of Time, Topic.find(:first, :conditions => {:id => 1}).written_on
|
||||
assert_kind_of Firm, Company.scoped(:conditions => { :name => "37signals"}).first
|
||||
assert_nil Company.scoped(:conditions => { :name => "37signals!"}).first
|
||||
assert_kind_of Time, Topic.scoped(:conditions => {:id => 1}).first.written_on
|
||||
end
|
||||
|
||||
def test_hash_condition_find_malformed
|
||||
assert_raise(ActiveRecord::StatementInvalid) {
|
||||
Company.find(:first, :conditions => { :id => 2, :dhh => true })
|
||||
Company.scoped(:conditions => { :id => 2, :dhh => true }).first
|
||||
}
|
||||
end
|
||||
|
||||
def test_hash_condition_find_with_escaped_characters
|
||||
Company.create("name" => "Ain't noth'n like' \#stuff")
|
||||
assert Company.find(:first, :conditions => { :name => "Ain't noth'n like' \#stuff" })
|
||||
assert Company.scoped(:conditions => { :name => "Ain't noth'n like' \#stuff" }).first
|
||||
end
|
||||
|
||||
def test_hash_condition_find_with_array
|
||||
p1, p2 = Post.find(:all, :limit => 2, :order => 'id asc')
|
||||
assert_equal [p1, p2], Post.find(:all, :conditions => { :id => [p1, p2] }, :order => 'id asc')
|
||||
assert_equal [p1, p2], Post.find(:all, :conditions => { :id => [p1, p2.id] }, :order => 'id asc')
|
||||
p1, p2 = Post.scoped(:limit => 2, :order => 'id asc').all
|
||||
assert_equal [p1, p2], Post.scoped(:conditions => { :id => [p1, p2] }, :order => 'id asc').all
|
||||
assert_equal [p1, p2], Post.scoped(:conditions => { :id => [p1, p2.id] }, :order => 'id asc').all
|
||||
end
|
||||
|
||||
def test_hash_condition_find_with_nil
|
||||
topic = Topic.find(:first, :conditions => { :last_read => nil } )
|
||||
topic = Topic.scoped(:conditions => { :last_read => nil } ).first
|
||||
assert_not_nil topic
|
||||
assert_nil topic.last_read
|
||||
end
|
||||
|
@ -425,42 +353,42 @@ class FinderTest < ActiveRecord::TestCase
|
|||
def test_hash_condition_find_with_aggregate_having_one_mapping
|
||||
balance = customers(:david).balance
|
||||
assert_kind_of Money, balance
|
||||
found_customer = Customer.find(:first, :conditions => {:balance => balance})
|
||||
found_customer = Customer.scoped(:conditions => {:balance => balance}).first
|
||||
assert_equal customers(:david), found_customer
|
||||
end
|
||||
|
||||
def test_hash_condition_find_with_aggregate_attribute_having_same_name_as_field_and_key_value_being_aggregate
|
||||
gps_location = customers(:david).gps_location
|
||||
assert_kind_of GpsLocation, gps_location
|
||||
found_customer = Customer.find(:first, :conditions => {:gps_location => gps_location})
|
||||
found_customer = Customer.scoped(:conditions => {:gps_location => gps_location}).first
|
||||
assert_equal customers(:david), found_customer
|
||||
end
|
||||
|
||||
def test_hash_condition_find_with_aggregate_having_one_mapping_and_key_value_being_attribute_value
|
||||
balance = customers(:david).balance
|
||||
assert_kind_of Money, balance
|
||||
found_customer = Customer.find(:first, :conditions => {:balance => balance.amount})
|
||||
found_customer = Customer.scoped(:conditions => {:balance => balance.amount}).first
|
||||
assert_equal customers(:david), found_customer
|
||||
end
|
||||
|
||||
def test_hash_condition_find_with_aggregate_attribute_having_same_name_as_field_and_key_value_being_attribute_value
|
||||
gps_location = customers(:david).gps_location
|
||||
assert_kind_of GpsLocation, gps_location
|
||||
found_customer = Customer.find(:first, :conditions => {:gps_location => gps_location.gps_location})
|
||||
found_customer = Customer.scoped(:conditions => {:gps_location => gps_location.gps_location}).first
|
||||
assert_equal customers(:david), found_customer
|
||||
end
|
||||
|
||||
def test_hash_condition_find_with_aggregate_having_three_mappings
|
||||
address = customers(:david).address
|
||||
assert_kind_of Address, address
|
||||
found_customer = Customer.find(:first, :conditions => {:address => address})
|
||||
found_customer = Customer.scoped(:conditions => {:address => address}).first
|
||||
assert_equal customers(:david), found_customer
|
||||
end
|
||||
|
||||
def test_hash_condition_find_with_one_condition_being_aggregate_and_another_not
|
||||
address = customers(:david).address
|
||||
assert_kind_of Address, address
|
||||
found_customer = Customer.find(:first, :conditions => {:address => address, :name => customers(:david).name})
|
||||
found_customer = Customer.scoped(:conditions => {:address => address, :name => customers(:david).name}).first
|
||||
assert_equal customers(:david), found_customer
|
||||
end
|
||||
|
||||
|
@ -468,7 +396,7 @@ class FinderTest < ActiveRecord::TestCase
|
|||
with_env_tz 'America/New_York' do
|
||||
with_active_record_default_timezone :local do
|
||||
topic = Topic.first
|
||||
assert_equal topic, Topic.find(:first, :conditions => ['written_on = ?', topic.written_on.getutc])
|
||||
assert_equal topic, Topic.scoped(:conditions => ['written_on = ?', topic.written_on.getutc]).first
|
||||
end
|
||||
end
|
||||
end
|
||||
|
@ -477,7 +405,7 @@ class FinderTest < ActiveRecord::TestCase
|
|||
with_env_tz 'America/New_York' do
|
||||
with_active_record_default_timezone :local do
|
||||
topic = Topic.first
|
||||
assert_equal topic, Topic.find(:first, :conditions => {:written_on => topic.written_on.getutc})
|
||||
assert_equal topic, Topic.scoped(:conditions => {:written_on => topic.written_on.getutc}).first
|
||||
end
|
||||
end
|
||||
end
|
||||
|
@ -486,7 +414,7 @@ class FinderTest < ActiveRecord::TestCase
|
|||
with_env_tz 'America/New_York' do
|
||||
with_active_record_default_timezone :utc do
|
||||
topic = Topic.first
|
||||
assert_equal topic, Topic.find(:first, :conditions => ['written_on = ?', topic.written_on.getlocal])
|
||||
assert_equal topic, Topic.scoped(:conditions => ['written_on = ?', topic.written_on.getlocal]).first
|
||||
end
|
||||
end
|
||||
end
|
||||
|
@ -495,32 +423,32 @@ class FinderTest < ActiveRecord::TestCase
|
|||
with_env_tz 'America/New_York' do
|
||||
with_active_record_default_timezone :utc do
|
||||
topic = Topic.first
|
||||
assert_equal topic, Topic.find(:first, :conditions => {:written_on => topic.written_on.getlocal})
|
||||
assert_equal topic, Topic.scoped(:conditions => {:written_on => topic.written_on.getlocal}).first
|
||||
end
|
||||
end
|
||||
end
|
||||
|
||||
def test_bind_variables
|
||||
assert_kind_of Firm, Company.find(:first, :conditions => ["name = ?", "37signals"])
|
||||
assert_nil Company.find(:first, :conditions => ["name = ?", "37signals!"])
|
||||
assert_nil Company.find(:first, :conditions => ["name = ?", "37signals!' OR 1=1"])
|
||||
assert_kind_of Time, Topic.find(:first, :conditions => ["id = ?", 1]).written_on
|
||||
assert_kind_of Firm, Company.scoped(:conditions => ["name = ?", "37signals"]).first
|
||||
assert_nil Company.scoped(:conditions => ["name = ?", "37signals!"]).first
|
||||
assert_nil Company.scoped(:conditions => ["name = ?", "37signals!' OR 1=1"]).first
|
||||
assert_kind_of Time, Topic.scoped(:conditions => ["id = ?", 1]).first.written_on
|
||||
assert_raise(ActiveRecord::PreparedStatementInvalid) {
|
||||
Company.find(:first, :conditions => ["id=? AND name = ?", 2])
|
||||
Company.scoped(:conditions => ["id=? AND name = ?", 2]).first
|
||||
}
|
||||
assert_raise(ActiveRecord::PreparedStatementInvalid) {
|
||||
Company.find(:first, :conditions => ["id=?", 2, 3, 4])
|
||||
Company.scoped(:conditions => ["id=?", 2, 3, 4]).first
|
||||
}
|
||||
end
|
||||
|
||||
def test_bind_variables_with_quotes
|
||||
Company.create("name" => "37signals' go'es agains")
|
||||
assert Company.find(:first, :conditions => ["name = ?", "37signals' go'es agains"])
|
||||
assert Company.scoped(:conditions => ["name = ?", "37signals' go'es agains"]).first
|
||||
end
|
||||
|
||||
def test_named_bind_variables_with_quotes
|
||||
Company.create("name" => "37signals' go'es agains")
|
||||
assert Company.find(:first, :conditions => ["name = :name", {:name => "37signals' go'es agains"}])
|
||||
assert Company.scoped(:conditions => ["name = :name", {:name => "37signals' go'es agains"}]).first
|
||||
end
|
||||
|
||||
def test_bind_arity
|
||||
|
@ -538,10 +466,10 @@ class FinderTest < ActiveRecord::TestCase
|
|||
|
||||
assert_nothing_raised { bind("'+00:00'", :foo => "bar") }
|
||||
|
||||
assert_kind_of Firm, Company.find(:first, :conditions => ["name = :name", { :name => "37signals" }])
|
||||
assert_nil Company.find(:first, :conditions => ["name = :name", { :name => "37signals!" }])
|
||||
assert_nil Company.find(:first, :conditions => ["name = :name", { :name => "37signals!' OR 1=1" }])
|
||||
assert_kind_of Time, Topic.find(:first, :conditions => ["id = :id", { :id => 1 }]).written_on
|
||||
assert_kind_of Firm, Company.scoped(:conditions => ["name = :name", { :name => "37signals" }]).first
|
||||
assert_nil Company.scoped(:conditions => ["name = :name", { :name => "37signals!" }]).first
|
||||
assert_nil Company.scoped(:conditions => ["name = :name", { :name => "37signals!' OR 1=1" }]).first
|
||||
assert_kind_of Time, Topic.scoped(:conditions => ["id = :id", { :id => 1 }]).first.written_on
|
||||
end
|
||||
|
||||
class SimpleEnumerable
|
||||
|
@ -1048,11 +976,6 @@ class FinderTest < ActiveRecord::TestCase
|
|||
assert_raise(ActiveRecord::StatementInvalid) { Topic.find_by_sql "select 1 from badtable" }
|
||||
end
|
||||
|
||||
def test_find_with_invalid_params
|
||||
assert_raise(ArgumentError) { Topic.find :first, :join => "It should be `joins'" }
|
||||
assert_raise(ArgumentError) { Topic.find :first, :conditions => '1 = 1', :join => "It should be `joins'" }
|
||||
end
|
||||
|
||||
def test_dynamic_finder_with_invalid_params
|
||||
assert_raise(ArgumentError) { Topic.find_by_title 'No Title', :join => "It should be `joins'" }
|
||||
end
|
||||
|
@ -1107,13 +1030,13 @@ class FinderTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_find_by_empty_in_condition
|
||||
assert_equal [], Post.find(:all, :conditions => ['id in (?)', []])
|
||||
assert_equal [], Post.where('id in (?)', [])
|
||||
end
|
||||
|
||||
def test_find_by_records
|
||||
p1, p2 = Post.find(:all, :limit => 2, :order => 'id asc')
|
||||
assert_equal [p1, p2], Post.find(:all, :conditions => ['id in (?)', [p1, p2]], :order => 'id asc')
|
||||
assert_equal [p1, p2], Post.find(:all, :conditions => ['id in (?)', [p1, p2.id]], :order => 'id asc')
|
||||
p1, p2 = Post.scoped(:limit => 2, :order => 'id asc').all
|
||||
assert_equal [p1, p2], Post.scoped(:conditions => ['id in (?)', [p1, p2]], :order => 'id asc')
|
||||
assert_equal [p1, p2], Post.scoped(:conditions => ['id in (?)', [p1, p2.id]], :order => 'id asc')
|
||||
end
|
||||
|
||||
def test_select_value
|
||||
|
@ -1140,10 +1063,10 @@ class FinderTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_find_with_order_on_included_associations_with_construct_finder_sql_for_association_limiting_and_is_distinct
|
||||
assert_equal 2, Post.find(:all, :include => { :authors => :author_address }, :order => 'author_addresses.id DESC ', :limit => 2).size
|
||||
assert_equal 2, Post.scoped(:include => { :authors => :author_address }, :order => 'author_addresses.id DESC ', :limit => 2).all.size
|
||||
|
||||
assert_equal 3, Post.find(:all, :include => { :author => :author_address, :authors => :author_address},
|
||||
:order => 'author_addresses_authors.id DESC ', :limit => 3).size
|
||||
assert_equal 3, Post.scoped(:include => { :author => :author_address, :authors => :author_address},
|
||||
:order => 'author_addresses_authors.id DESC ', :limit => 3).all.size
|
||||
end
|
||||
|
||||
def test_find_with_nil_inside_set_passed_for_one_attribute
|
||||
|
@ -1188,7 +1111,7 @@ class FinderTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_finder_with_offset_string
|
||||
assert_nothing_raised(ActiveRecord::StatementInvalid) { Topic.find(:all, :offset => "3") }
|
||||
assert_nothing_raised(ActiveRecord::StatementInvalid) { Topic.scoped(:offset => "3").all }
|
||||
end
|
||||
|
||||
protected
|
||||
|
|
|
@ -98,7 +98,7 @@ class InheritanceTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_inheritance_find_all
|
||||
companies = Company.find(:all, :order => 'id')
|
||||
companies = Company.scoped(:order => 'id').all
|
||||
assert_kind_of Firm, companies[0], "37signals should be a firm"
|
||||
assert_kind_of Client, companies[1], "Summit should be a client"
|
||||
end
|
||||
|
@ -151,7 +151,7 @@ class InheritanceTest < ActiveRecord::TestCase
|
|||
Client.update_all "name = 'I am a client'"
|
||||
assert_equal "I am a client", Client.all.first.name
|
||||
# Order by added as otherwise Oracle tests were failing because of different order of results
|
||||
assert_equal "37signals", Firm.find(:all, :order => "id").first.name
|
||||
assert_equal "37signals", Firm.scoped(:order => "id").all.first.name
|
||||
end
|
||||
|
||||
def test_alt_update_all_within_inheritance
|
||||
|
@ -173,9 +173,9 @@ class InheritanceTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_find_first_within_inheritance
|
||||
assert_kind_of Firm, Company.find(:first, :conditions => "name = '37signals'")
|
||||
assert_kind_of Firm, Firm.find(:first, :conditions => "name = '37signals'")
|
||||
assert_nil Client.find(:first, :conditions => "name = '37signals'")
|
||||
assert_kind_of Firm, Company.scoped(:conditions => "name = '37signals'").first
|
||||
assert_kind_of Firm, Firm.scoped(:conditions => "name = '37signals'").first
|
||||
assert_nil Client.scoped(:conditions => "name = '37signals'").first
|
||||
end
|
||||
|
||||
def test_alt_find_first_within_inheritance
|
||||
|
@ -187,10 +187,10 @@ class InheritanceTest < ActiveRecord::TestCase
|
|||
def test_complex_inheritance
|
||||
very_special_client = VerySpecialClient.create("name" => "veryspecial")
|
||||
assert_equal very_special_client, VerySpecialClient.where("name = 'veryspecial'").first
|
||||
assert_equal very_special_client, SpecialClient.find(:first, :conditions => "name = 'veryspecial'")
|
||||
assert_equal very_special_client, Company.find(:first, :conditions => "name = 'veryspecial'")
|
||||
assert_equal very_special_client, Client.find(:first, :conditions => "name = 'veryspecial'")
|
||||
assert_equal 1, Client.find(:all, :conditions => "name = 'Summit'").size
|
||||
assert_equal very_special_client, SpecialClient.scoped(:conditions => "name = 'veryspecial'").first
|
||||
assert_equal very_special_client, Company.scoped(:conditions => "name = 'veryspecial'").first
|
||||
assert_equal very_special_client, Client.scoped(:conditions => "name = 'veryspecial'").first
|
||||
assert_equal 1, Client.scoped(:conditions => "name = 'Summit'").all.size
|
||||
assert_equal very_special_client, Client.find(very_special_client.id)
|
||||
end
|
||||
|
||||
|
@ -230,7 +230,7 @@ class InheritanceTest < ActiveRecord::TestCase
|
|||
private
|
||||
def switch_to_alt_inheritance_column
|
||||
# we don't want misleading test results, so get rid of the values in the type column
|
||||
Company.find(:all, :order => 'id').each do |c|
|
||||
Company.scoped(:order => 'id').all.each do |c|
|
||||
c['type'] = nil
|
||||
c.save
|
||||
end
|
||||
|
|
|
@ -39,7 +39,7 @@ class ModulesTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_associations_spanning_cross_modules
|
||||
account = MyApplication::Billing::Account.find(:first, :order => 'id')
|
||||
account = MyApplication::Billing::Account.scoped(:order => 'id').first
|
||||
assert_kind_of MyApplication::Business::Firm, account.firm
|
||||
assert_kind_of MyApplication::Billing::Firm, account.qualified_billing_firm
|
||||
assert_kind_of MyApplication::Billing::Firm, account.unqualified_billing_firm
|
||||
|
|
|
@ -58,9 +58,9 @@ class NamedScopeTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_scopes_with_options_limit_finds_to_those_matching_the_criteria_specified
|
||||
assert !Topic.find(:all, :conditions => {:approved => true}).empty?
|
||||
assert !Topic.scoped(:conditions => {:approved => true}).all.empty?
|
||||
|
||||
assert_equal Topic.find(:all, :conditions => {:approved => true}), Topic.approved
|
||||
assert_equal Topic.scoped(:conditions => {:approved => true}).all, Topic.approved
|
||||
assert_equal Topic.where(:approved => true).count, Topic.approved.count
|
||||
end
|
||||
|
||||
|
@ -71,8 +71,8 @@ class NamedScopeTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_scopes_are_composable
|
||||
assert_equal((approved = Topic.find(:all, :conditions => {:approved => true})), Topic.approved)
|
||||
assert_equal((replied = Topic.find(:all, :conditions => 'replies_count > 0')), Topic.replied)
|
||||
assert_equal((approved = Topic.scoped(:conditions => {:approved => true}).all), Topic.approved)
|
||||
assert_equal((replied = Topic.scoped(:conditions => 'replies_count > 0').all), Topic.replied)
|
||||
assert !(approved == replied)
|
||||
assert !(approved & replied).empty?
|
||||
|
||||
|
@ -80,8 +80,8 @@ class NamedScopeTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_procedural_scopes
|
||||
topics_written_before_the_third = Topic.find(:all, :conditions => ['written_on < ?', topics(:third).written_on])
|
||||
topics_written_before_the_second = Topic.find(:all, :conditions => ['written_on < ?', topics(:second).written_on])
|
||||
topics_written_before_the_third = Topic.where('written_on < ?', topics(:third).written_on)
|
||||
topics_written_before_the_second = Topic.where('written_on < ?', topics(:second).written_on)
|
||||
assert_not_equal topics_written_before_the_second, topics_written_before_the_third
|
||||
|
||||
assert_equal topics_written_before_the_third, Topic.written_before(topics(:third).written_on)
|
||||
|
@ -464,7 +464,7 @@ class DynamicScopeTest < ActiveRecord::TestCase
|
|||
|
||||
def test_dynamic_scope
|
||||
assert_equal @test_klass.scoped_by_author_id(1).find(1), @test_klass.find(1)
|
||||
assert_equal @test_klass.scoped_by_author_id_and_title(1, "Welcome to the weblog").first, @test_klass.find(:first, :conditions => { :author_id => 1, :title => "Welcome to the weblog"})
|
||||
assert_equal @test_klass.scoped_by_author_id_and_title(1, "Welcome to the weblog").first, @test_klass.scoped(:conditions => { :author_id => 1, :title => "Welcome to the weblog"}).first
|
||||
end
|
||||
|
||||
def test_dynamic_scope_should_create_methods_after_hitting_method_missing
|
||||
|
|
|
@ -55,7 +55,7 @@ class PersistencesTest < ActiveRecord::TestCase
|
|||
author = authors(:david)
|
||||
assert_nothing_raised do
|
||||
assert_equal 1, author.posts_sorted_by_id_limited.size
|
||||
assert_equal 2, author.posts_sorted_by_id_limited.find(:all, :limit => 2).size
|
||||
assert_equal 2, author.posts_sorted_by_id_limited.scoped(:limit => 2).all.size
|
||||
assert_equal 1, author.posts_sorted_by_id_limited.update_all([ "body = ?", "bulk update!" ])
|
||||
assert_equal "bulk update!", posts(:welcome).body
|
||||
assert_not_equal "bulk update!", posts(:thinking).body
|
||||
|
|
|
@ -323,7 +323,7 @@ class DefaultScopingTest < ActiveRecord::TestCase
|
|||
fixtures :developers, :posts
|
||||
|
||||
def test_default_scope
|
||||
expected = Developer.find(:all, :order => 'salary DESC').collect { |dev| dev.salary }
|
||||
expected = Developer.scoped(:order => 'salary DESC').all.collect { |dev| dev.salary }
|
||||
received = DeveloperOrderedBySalary.all.collect { |dev| dev.salary }
|
||||
assert_equal expected, received
|
||||
end
|
||||
|
@ -396,7 +396,7 @@ class DefaultScopingTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_scope_overwrites_default
|
||||
expected = Developer.find(:all, :order => 'salary DESC, name DESC').collect { |dev| dev.name }
|
||||
expected = Developer.scoped(:order => 'salary DESC, name DESC').all.collect { |dev| dev.name }
|
||||
received = DeveloperOrderedBySalary.by_name.all.collect { |dev| dev.name }
|
||||
assert_equal expected, received
|
||||
end
|
||||
|
@ -414,8 +414,8 @@ class DefaultScopingTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_order_in_default_scope_should_prevail
|
||||
expected = Developer.find(:all, :order => 'salary desc').collect { |dev| dev.salary }
|
||||
received = DeveloperOrderedBySalary.find(:all, :order => 'salary').collect { |dev| dev.salary }
|
||||
expected = Developer.scoped(:order => 'salary desc').all.collect { |dev| dev.salary }
|
||||
received = DeveloperOrderedBySalary.scoped(:order => 'salary').all.collect { |dev| dev.salary }
|
||||
assert_equal expected, received
|
||||
end
|
||||
|
||||
|
|
|
@ -282,7 +282,7 @@ class RelationTest < ActiveRecord::TestCase
|
|||
end
|
||||
|
||||
def test_find_on_hash_conditions
|
||||
assert_equal Topic.find(:all, :conditions => {:approved => false}), Topic.where({ :approved => false }).to_a
|
||||
assert_equal Topic.scoped(:conditions => {:approved => false}).all, Topic.where({ :approved => false }).to_a
|
||||
end
|
||||
|
||||
def test_joins_with_string_array
|
||||
|
@ -1091,11 +1091,6 @@ class RelationTest < ActiveRecord::TestCase
|
|||
assert_equal Post.order(Post.arel_table[:title]).all, Post.order("title").all
|
||||
end
|
||||
|
||||
def test_order_with_find_with_order
|
||||
assert_equal 'zyke', CoolCar.order('name desc').find(:first, :order => 'id').name
|
||||
assert_equal 'zyke', FastCar.order('name desc').find(:first, :order => 'id').name
|
||||
end
|
||||
|
||||
def test_default_scope_order_with_scope_order
|
||||
assert_equal 'zyke', CoolCar.order_using_new_style.limit(1).first.name
|
||||
assert_equal 'zyke', FastCar.order_using_new_style.limit(1).first.name
|
||||
|
@ -1103,12 +1098,12 @@ class RelationTest < ActiveRecord::TestCase
|
|||
|
||||
def test_order_using_scoping
|
||||
car1 = CoolCar.order('id DESC').scoping do
|
||||
CoolCar.find(:first, :order => 'id asc')
|
||||
CoolCar.scoped(:order => 'id asc').first
|
||||
end
|
||||
assert_equal 'zyke', car1.name
|
||||
|
||||
car2 = FastCar.order('id DESC').scoping do
|
||||
FastCar.find(:first, :order => 'id asc')
|
||||
FastCar.scoped(:order => 'id asc').first
|
||||
end
|
||||
assert_equal 'zyke', car2.name
|
||||
end
|
||||
|
|
|
@ -19,7 +19,7 @@ class Comment < ActiveRecord::Base
|
|||
end
|
||||
|
||||
def self.search_by_type(q)
|
||||
self.find(:all, :conditions => ["#{QUOTED_TYPE} = ?", q])
|
||||
self.scoped(:conditions => ["#{QUOTED_TYPE} = ?", q]).all
|
||||
end
|
||||
|
||||
def self.all_as_method
|
||||
|
|
|
@ -2,20 +2,20 @@ require 'ostruct'
|
|||
|
||||
module DeveloperProjectsAssociationExtension
|
||||
def find_most_recent
|
||||
find(:first, :order => "id DESC")
|
||||
scoped(:order => "id DESC").first
|
||||
end
|
||||
end
|
||||
|
||||
module DeveloperProjectsAssociationExtension2
|
||||
def find_least_recent
|
||||
find(:first, :order => "id ASC")
|
||||
scoped(:order => "id ASC").first
|
||||
end
|
||||
end
|
||||
|
||||
class Developer < ActiveRecord::Base
|
||||
has_and_belongs_to_many :projects do
|
||||
def find_most_recent
|
||||
find(:first, :order => "id DESC")
|
||||
scoped(:order => "id DESC").first
|
||||
end
|
||||
end
|
||||
|
||||
|
@ -37,7 +37,7 @@ class Developer < ActiveRecord::Base
|
|||
:association_foreign_key => "project_id",
|
||||
:extend => DeveloperProjectsAssociationExtension do
|
||||
def find_least_recent
|
||||
find(:first, :order => "id ASC")
|
||||
scoped(:order => "id ASC").first
|
||||
end
|
||||
end
|
||||
|
||||
|
|
|
@ -31,7 +31,7 @@ class Post < ActiveRecord::Base
|
|||
|
||||
has_many :comments do
|
||||
def find_most_recent
|
||||
find(:first, :order => "id DESC")
|
||||
scoped(:order => "id DESC").first
|
||||
end
|
||||
|
||||
def newest
|
||||
|
@ -64,8 +64,8 @@ class Post < ActiveRecord::Base
|
|||
has_many :taggings, :as => :taggable
|
||||
has_many :tags, :through => :taggings do
|
||||
def add_joins_and_select
|
||||
find :all, :select => 'tags.*, authors.id as author_id',
|
||||
:joins => 'left outer join posts on taggings.taggable_id = posts.id left outer join authors on posts.author_id = authors.id'
|
||||
scoped(:select => 'tags.*, authors.id as author_id',
|
||||
:joins => 'left outer join posts on taggings.taggable_id = posts.id left outer join authors on posts.author_id = authors.id').all
|
||||
end
|
||||
end
|
||||
|
||||
|
|
Loading…
Reference in a new issue