jorahood-ar-extensions 0.9.2.3

data/tests/mysql/test_create_and_update.rb

@@ -0,0 +1,290 @@
+require File.expand_path( File.join( File.dirname( __FILE__ ), '../test_helper') )
+
+class CreateAndUpdateTest < TestCaseSuperClass
+  if ActiveRecord::Base.connection.class.name =~ /sqlite/i
+    self.use_transactional_fixtures = false
+  end
+
+  def setup
+    super
+    Animal.delete_all
+  end
+
+  #Input Validation Tests
+  def test_replace_should_replace_existing_record_with_new_data
+    create_animal
+    assert_nil(@animal.size)
+
+    new_animal = Animal.new(:name => 'giraffe', :size => 'big')
+    assert_raise(ActiveRecord::StatementInvalid,
+      "Should not be able to save duplicate.") { new_animal.save }
+
+    new_animal.replace
+
+    validate_animal(:size => 'big', :name => 'giraffe')
+    assert(@animal.updated_at < @new_animal.updated_at)
+    assert(@animal.created_at < @new_animal.created_at)
+    assert(@new_animal.id > @animal.id)
+
+    assert_nil(Animal.find_by_id(@original_animal_id))
+  end
+
+  #Input Validation Tests
+  def test_create_ignore_without_duplicate_columns_should_throw_exception_for_missing_duplicate_columns
+    assert_raise(ArgumentError) {
+      test_create({:method => 'create'}, :ignore => true, :reload => true)
+    }
+  end
+
+  def test_save_with_invalid_option_should_throw_exception
+    assert_raise(ArgumentError) {
+      test_save({:method => 'create'}, :ignore => true, :reload => true, :unknown_argument => true)
+    }
+  end
+
+  #Create tests
+  def test_create_ignore_should_ignore_existing_data
+    test_create({:method => 'create'}, :ignore => true)
+    validate_animal(:name => 'giraffe',
+                    :updated_at => @animal.updated_at,
+                    :created_at => @animal.created_at)
+  end
+
+  def test_create_ignore_with_reload_should_ignore_existing_data_then_reload
+    create_animal
+
+    @new_animal = Animal.create!({:name => 'giraffe', :size => 'little'}, :ignore => true)
+    assert(@new_animal.stale_record?)
+    assert_equal('little', @new_animal.size)
+
+    @new_animal.reload_duplicate :duplicate_columns => [:name]
+    assert(!@new_animal.stale_record?)
+    assert_nil(@new_animal.size)
+  end
+
+  def test_create_bang_ignore_should_ignore_existing_data
+    test_create({:method => 'create!'}, :ignore => true)
+    validate_animal(:name => 'giraffe',
+                    :updated_at => @animal.updated_at,
+                    :created_at => @animal.created_at)
+  end
+
+  def test_create_duplicate_should_update_duplicate_data
+    test_create({:method =>'create', :size => 'big'}, :on_duplicate_key_update => [:updated_at, :size])
+    validate_animal(:name => 'giraffe',
+                    :size => 'big',
+                    :created_at => @animal.created_at)
+    assert(@animal.updated_at < @new_animal.updated_at)
+  end
+
+  def test_create_bang_duplicate_should_update_duplicate_data
+    test_create({:method =>'create!', :size => 'big'}, :on_duplicate_key_update => [:updated_at, :size])
+    validate_animal(:name => 'giraffe',
+                    :size => 'big',
+                    :created_at => @animal.created_at)
+    assert(@animal.updated_at < @new_animal.updated_at)
+  end
+
+  #Save insert tests
+  def test_save_ignore_should_ignore_existing_data
+    test_save({:method => 'save'}, :ignore => true)
+    validate_animal(:name => 'giraffe',
+                    :updated_at => @animal.updated_at,
+                    :created_at => @animal.created_at)
+  end
+
+  def test_save_bang_ignore_should_ignore_existing_data_and_reload
+    test_save({:method => 'save!'},
+               :ignore => true, :reload => true, :duplicate_columns => [:name])
+    validate_animal(:name => 'giraffe',
+                    :updated_at => @animal.updated_at,
+                    :created_at => @animal.created_at)
+  end
+
+
+  def test_save_duplicate_should_update_duplicate_data_and_reload
+    test_save({:method =>'save', :size => 'big'},
+                :on_duplicate_key_update => [:updated_at, :size],
+                :reload => true, :duplicate_columns => [:name])
+
+    validate_animal(:name => 'giraffe',
+                    :size => 'big',
+                    :created_at => @animal.created_at)
+
+    assert(@animal.updated_at < @new_animal.updated_at)
+  end
+
+  def test_save_bang_duplicate_should_update_duplicate_data
+    test_save({:method => 'save!', :size => 'big'},
+               :on_duplicate_key_update => [:updated_at, :size])
+
+    validate_animal(:name => 'giraffe',
+                    :size => 'big',
+                    :created_at => @animal.created_at)
+
+    assert(@animal.updated_at < @new_animal.updated_at)
+  end
+
+  def test_save_existing_record_should_just_save_without_reload
+    test_save_existing_record(:ignore => true)
+    assert_equal(@bear.name, 'giraffe')
+    assert_equal(@bear.size, 'huge')
+  end
+
+  def test_save_existing_record_should_just_save_with_reload
+    test_save_existing_record(:ignore => true, :reload => true, :duplicate_columns => [:name])
+    assert_equal(@bear.name, 'giraffe')
+    assert_nil(@bear.size)
+  end
+
+  def test_save_existing_record_should_update_duplicate
+    test_save_existing_record(:on_duplicate_key_update => [:updated_at, :size], :reload => true, :duplicate_columns => [:name])
+    assert_equal(@bear.name, 'giraffe')
+    assert_equal(@bear.size, 'huge')
+
+    #ensure record was modified in database
+    @bear = Animal.find_by_name('giraffe')
+    assert_equal(@bear.name, 'giraffe')
+    assert_equal(@bear.size, 'huge')
+  end
+
+  #RELOAD tests
+
+
+    def test_create_ignore_with_reload_should_ignore_existing_data_then_reload
+    create_animal
+
+    @new_animal = Animal.create!({:name => 'giraffe', :size => 'little'}, :ignore => true)
+    assert(@new_animal.stale_record?)
+    assert_equal('little', @new_animal.size)
+
+    @new_animal.reload_duplicate :duplicate_columns => [:name]
+    assert(!@new_animal.stale_record?)
+    assert_nil(@new_animal.size)
+  end
+
+  def test_save_should_update_existing_data_then_reload_later
+    create_animal
+
+    @new_animal = Animal.new :name => 'giraffe', :size => 'little'
+    @new_animal.save!(:on_duplicate_key_update => [:updated_at])
+
+
+    assert(@new_animal.stale_record?)
+    assert_equal(0, @new_animal.id)
+    assert_equal('little', @new_animal.size)
+    assert(@animal.updated_at < @new_animal.updated_at)
+    new_updated = @new_animal.updated_at
+
+    @new_animal.reload_duplicate :duplicate_columns => [:name]
+
+    assert(!@new_animal.stale_record?)
+    assert_nil(@new_animal.size)
+    assert_equal(new_updated.to_s, @new_animal.updated_at.to_s)
+    assert_equal(@animal.id, @new_animal.id)
+  end
+
+  def test_update_should_delete_duplicate_record_on_reload
+    create_animal
+
+    @new_animal = Animal.create! :name => 'bear', :size => 'little'
+    @new_animal.name = 'giraffe'
+    assert(@new_animal.id > @animal.id)
+
+    @new_animal.reload_duplicate :force => true, :duplicate_columns => [:name]
+    assert(!@new_animal.stale_record?)
+    assert_nil(@new_animal.size)
+    assert_equal(@animal.updated_at.to_s, @new_animal.updated_at.to_s)
+    assert_equal(@animal.id, @new_animal.id)
+    assert_nil(Animal.find_by_name('bear'))
+
+  end
+
+  protected
+
+  def test_create(options, ex_options)
+    method = options.delete(:method)
+    create_animal
+    assert_raises(ActiveRecord::StatementInvalid,
+      "Should not be able to create duplicate.") { Animal.send(method, {:name => 'giraffe'}) }
+    @new_animal = Animal.send(method, {:name => 'giraffe'}.merge(options), ex_options)
+    assert(@new_animal)
+    validate_state_state(ex_options)
+  end
+
+  def test_save(options, ex_options)
+    method = options.delete(:method)
+    create_animal
+    @new_animal = Animal.new({:name => 'giraffe'}.merge(options))
+    assert_raise(ActiveRecord::StatementInvalid,
+      "Should not be able to save duplicate.") { @new_animal.send(method) }
+    assert(@new_animal.send(method, ex_options))
+    validate_state_state(ex_options)
+  end
+
+  def validate_state_state(ex_options)
+    if ex_options[:reload]
+      assert_equal(@new_animal.created_at.to_s, @animal.created_at.to_s)
+      assert(!@new_animal.stale_record?)
+      assert_equal(@original_animal_id, @new_animal.id)
+    else
+      assert_equal(0, @new_animal.id)
+      assert(@new_animal.created_at > @animal.created_at)
+      assert(@new_animal.stale_record?)
+      assert(Animal.find_by_name('giraffe'))
+    end
+  end
+
+  def test_save_existing_record(save_options)
+    @giraffe = create_animal
+
+    @bear = create_animal(:name => 'bear')
+    @bear.name = 'giraffe'
+    @bear.size = 'huge'
+
+    @original_bear_id = @bear.id
+    @original_giraffe_id = @giraffe.id
+
+    assert_raises(ActiveRecord::StatementInvalid,
+      "Should not be able to create duplicate."){ @bear.save }
+
+    @bear.save(save_options)
+    assert(Animal.find_by_name('giraffe'))
+
+    if save_options[:reload] || (save_options[:on_duplicate_key_update] && save_options[:duplicate_columns])
+      assert_equal(@original_giraffe_id, @bear.id)
+      assert(!@bear.stale_record?)
+      assert_nil(Animal.find_by_name('bear'))
+    else
+      assert_equal(@original_bear_id, @bear.id)
+      assert(@bear.stale_record?)
+      assert(Animal.find_by_name('bear'))
+    end
+  end
+
+  def create_animal(options={})
+    Animal.create!(options.reverse_merge(:name => 'giraffe'))
+
+    #back set the date so we can compare the new timestamp
+    reset_time = Time.now - 1.day
+    Animal.update_all(['updated_at = ?, created_at = ?', reset_time, reset_time])
+
+    @animal = Animal.find_by_name options[:name] || 'giraffe'
+    @original_animal_id = @animal.id
+    @animal
+  end
+
+  def validate_animal(fields)
+    @new_animal = Animal.find_by_name fields[:name]
+    assert(@new_animal)
+
+    assert(@new_animal.updated_at)
+    assert(@new_animal.created_at)
+
+    fields.each do|field, exp_val|
+      assert_equal(exp_val.to_s, @new_animal.send(field).to_s,
+        "Expecting #{exp_val} for #{field} but got #{@new_animal.send(field)}")
+    end
+  end
+
+end

data/tests/mysql/test_delete.rb

@@ -0,0 +1,142 @@
+require File.expand_path( File.join( File.dirname( __FILE__ ), '../test_helper') )
+
+{:execute => [/^SELECT currval/, /^SELECT CAST/, /^SELECT @@IDENTITY/],
+ :insert => [],
+ :delete => []
+}.each do |method, ignore_list|
+
+  ActiveRecord::Base.connection.class.class_eval <<-END_SRC
+    cattr_accessor :#{method}_count
+    # Array of regexes of queries that are not counted against query_count
+    @@ignore_#{method}_list = #{ignore_list.inspect}
+    alias_method :#{method}_without_query_counting, :#{method}
+    def #{method}_with_query_counting(sql, name = nil, &block)
+      self.#{method}_count += 1 unless @@ignore_#{method}_list.any? { |r| sql =~ r }
+      #{method}_without_query_counting(sql, name, &block)
+    end
+  END_SRC
+end
+
+class DeleteTestCaseSuperClass < TestCaseSuperClass
+  protected
+
+  def assert_query(method, num = 1)
+    ActiveRecord::Base.connection.class.class_eval do
+      self.send "#{method}_count=", 0
+      alias_method method, "#{method}_with_query_counting"
+    end
+    yield
+  ensure
+    ActiveRecord::Base.connection.class.class_eval do
+      alias_method method, "#{method}_without_query_counting"
+    end
+    count = ActiveRecord::Base.connection.send "#{method}_count"
+    assert_equal num, count, "#{count} instead of #{num} #{method} queries were executed."
+  end
+
+end
+
+
+class DeleteTest < DeleteTestCaseSuperClass
+
+  def setup
+    super
+    insert_books
+  end
+  
+  def test_delete_limit_should_only_delete_three_records
+    count = Book.count(:all, :conditions => ['author_name = ?', 'giraffe'])
+    assert_equal 50, count
+
+    deleted = Book.delete_all(['author_name = ?', 'giraffe'], :limit => 24)
+
+    assert_equal 24, deleted
+
+    count = Book.count(:all, :conditions => ['author_name = ?', 'giraffe'])
+    assert_equal 26, count
+  end
+
+  def test_delete_limit_should_delete_less_than_limit
+    count = Book.count(:all, :conditions => ['author_name = ?', 'giraffe'])
+    assert_equal 50, count
+
+    deleted = Book.delete_all(['author_name = ?', 'giraffe'], :limit => 55)
+    assert_equal(50, deleted)
+
+    count = Book.count(:all, :conditions => ['author_name = ?', 'giraffe'])
+    assert_equal 0, count
+  end
+
+  def test_delete_batch_should_execute_6_deletes
+    assert_query(:delete, 6){
+      assert_equal(50, Book.delete_all(nil, :batch => 10))
+    }
+    assert_equal 0, Book.count
+  end
+
+  def test_delete_batch_with_limit_should_delete_until_limit_reached
+    assert_query(:delete, 4){
+      assert_equal(32, Book.delete_all('id > 0', :batch => 10, :limit => 32))
+    }
+    assert_equal 18, Book.count
+  end
+
+
+  def test_delete_batch_default_should_delete_all
+    assert_query(:delete, 1){
+      assert_equal(50, Book.delete_all(nil, :batch => true))
+    }
+    assert_equal 0, Book.count
+  end
+
+
+  def test_delete_batch_bigger_than_limit_should_delete_limit
+    assert_query(:delete, 1){
+      assert_equal(12, Book.delete_all(nil, :batch => 20, :limit => 12))
+    }
+    assert_equal 38, Book.count
+  end
+
+
+  def test_delete_duplicates_should_leave_one_by_author_with_low_id
+    min_id = Book.minimum :id
+
+    Book.delete_duplicates(:fields => [:author_name])
+    assert_equal 0, Book.count(:all, :group => :author_name, :having => 'count(*) > 1' ).length
+
+    assert_equal(min_id, Book.find_by_author_name('giraffe').id)
+  end
+
+
+  def test_delete_duplicates_should_delete_all_but_one_with_publisher_one
+
+    min_id = Book.minimum :id
+
+    Book.delete_duplicates(:fields => [:author_name], :conditions => ['c1.publisher = ? and c2.publisher = ?', 'Pub1', 'Pub1'])
+
+    books = Book.find_all_by_author_name('giraffe', :order => :id)
+    assert_equal 41, books.length
+
+    assert_equal min_id, books.first.id
+
+    assert_equal 1, Book.find_all_by_author_name_and_publisher('giraffe', 'Pub1').length
+
+  end
+
+  def test_delete_duplicates_should_delete_all_but_five
+    Book.delete_duplicates(:fields => [:author_name, :publisher])
+    assert_equal(5, Book.count)
+  end
+
+
+
+  protected
+  def insert_books
+    Book.delete_all
+    1.upto(50){|count|
+      Book.create!(:author_name => 'giraffe', :title => "Title#{count}", :publisher => "Pub#{count % 5}")
+    }
+  end
+
+end
+

data/tests/mysql/test_finder_options.rb

@@ -0,0 +1,121 @@
+require File.expand_path( File.join( File.dirname( __FILE__ ), '..', 'test_helper' ))
+
+
+class FinderOptionsTest < TestCaseSuperClass
+  include ActiveRecord::ConnectionAdapters
+  self.fixture_path = File.join( File.dirname( __FILE__ ), 'fixtures/unit/active_record_base_finders' )
+  self.fixtures 'books'
+
+  def setup
+    @connection = ActiveRecord::Base.connection
+  end
+  
+  def teardown
+    Topic.delete_all
+    Book.delete_all
+  end
+  
+  def test_find_with_having_should_add_having_option_to_sql
+    create_books_and_topics
+    
+    #test having without associations
+    books = Book.find(:all, :select => 'count(*) as count_all, topic_id', :group => :topic_id, :having => 'count(*) > 1')
+    assert_equal 2, books.size
+    
+    #test having with associations
+    books = Book.find(:all, 
+      :include => :topic, 
+      :conditions => " topics.id is not null", #the conditions forces eager loading in Rails 2.2
+      :select => 'count(*) as count_all, topic_id',
+      :group => :topic_id,
+      :having => 'count(*) > 1')
+      
+    assert_equal 2, books.size
+  end
+
+  def test_finder_sql_to_string_should_select_from_one_table
+    book_sql = Book.finder_sql_to_string(:select => 'topic_id', :include => :topic)
+
+    # current generated output looks like this now:
+    # SELECT topic_id FROM `books`
+    assert(/^SELECT\s/.match(book_sql))
+    assert_nil(/\sJOIN\s/.match(book_sql))
+    (Book.column_names - ['topic_id']).each do |col|
+      assert_nil(book_sql.match(Regexp.new("^SELECT .*[`\s]#{col}.*FROM")))
+    end
+
+    assert(/^SELECT\s.*topic_id.*FROM/.match(book_sql))
+  end
+
+  def test_finder_sql_to_string_w_forced_eager_load_should_generate_sql_with_joined_table
+    book_sql = Book.finder_sql_to_string(:select => 'topic_id',
+                                         :include => :topic,
+                                         :force_eager_load => true)
+    assert(/^SELECT\s/.match(book_sql))
+    assert(/\sJOIN\s/.match(book_sql))
+
+    #assert that each column was included in the sql
+    Book.column_names.each do |col|
+      assert(book_sql.match(Regexp.new("^SELECT .*[`\s\.]#{col}.*FROM")))
+    end
+    
+    # current generated output looks like this now:
+    # SELECT `books`.`id` AS t0_r0, `books`.`title` AS t0_r1, `books`.`publisher` AS t0_r2, `books`.`author_name` AS t0_r3, `books`.`created_at` AS t0_r4, `books`.`created_on` AS t0_r5, `books`.`updated_at` AS t0_r6, `books`.`updated_on` AS t0_r7, `books`.`topic_id` AS t0_r8, `books`.`for_sale` AS t0_r9, `topics`.`id` AS t1_r0, `topics`.`title` AS t1_r1, `topics`.`author_name` AS t1_r2, `topics`.`author_email_address` AS t1_r3, `topics`.`written_on` AS t1_r4, `topics`.`bonus_time` AS t1_r5, `topics`.`last_read` AS t1_r6, `topics`.`content` AS t1_r7, `topics`.`approved` AS t1_r8, `topics`.`replies_count` AS t1_r9, `topics`.`parent_id` AS t1_r10, `topics`.`type` AS t1_r11, `topics`.`created_at` AS t1_r12, `topics`.`updated_at` AS t1_r13 FROM `books`  LEFT OUTER JOIN `topics` ON `topics`.id = `books`.topic_id
+  end
+
+  def test_finder_sql_to_string_should_generate_sql_with_joined_table
+    conditions = 'topics.id is not null'
+    book_sql = Book.finder_sql_to_string(:select => 'topic_id', :include => :topic, :conditions => conditions)
+    
+    assert(/^SELECT\s/.match(book_sql))
+    assert(/\sJOIN\s/.match(book_sql))
+    assert(book_sql.include?(conditions))
+
+    #assert that each column was included in the sql
+    Book.column_names.each do |col|
+      assert(book_sql.match(Regexp.new("^SELECT .*[`\s\.]#{col}.*FROM")))
+    end
+    
+    # current generated output looks like this now:
+    #"SELECT `books`.`id` AS t0_r0, `books`.`title` AS t0_r1, `books`.`publisher` AS t0_r2, `books`.`author_name` AS t0_r3, `books`.`created_at` AS t0_r4, `books`.`created_on` AS t0_r5, `books`.`updated_at` AS t0_r6, `books`.`updated_on` AS t0_r7, `books`.`topic_id` AS t0_r8, `books`.`for_sale` AS t0_r9, `topics`.`id` AS t1_r0, `topics`.`title` AS t1_r1, `topics`.`author_name` AS t1_r2, `topics`.`author_email_address` AS t1_r3, `topics`.`written_on` AS t1_r4, `topics`.`bonus_time` AS t1_r5, `topics`.`last_read` AS t1_r6, `topics`.`content` AS t1_r7, `topics`.`approved` AS t1_r8, `topics`.`replies_count` AS t1_r9, `topics`.`parent_id` AS t1_r10, `topics`.`type` AS t1_r11, `topics`.`created_at` AS t1_r12, `topics`.`updated_at` AS t1_r13 FROM `books`  LEFT OUTER JOIN `topics` ON `topics`.id = `books`.topic_id WHERE (topics.id is not null)"
+  end
+  
+  def test_pre_sql_should_ensure_pre_sql_option_is_added_to_beginning_of_sql
+    book_sql = Book.finder_sql_to_string(:select => 'topic_id', :pre_sql => "/* BLAH */")
+    assert(/^\/\* BLAH \*\/\sSELECT/.match(book_sql))
+  end
+
+  def test_pre_sql_should_ensure_pre_sql_option_is_added_to_beginning_of_sql_with_eager_loading
+    book_sql = Book.finder_sql_to_string(:select => 'topic_id', :pre_sql => "/* BLAH */", :include => :topic, :conditions => 'topics.id is not null')
+    assert(/^\/\* BLAH \*\/\sSELECT/.match(book_sql))
+  end
+  
+  def test_pre_sql_should_ensure_post_sql_option_is_added_to_end_of_sql
+    book_sql = Book.finder_sql_to_string(:select => 'topic_id', :post_sql => "/* BLAH */")
+    assert(/\s\/\* BLAH \*\/$/.match(book_sql))
+  end
+
+  def test_pre_sql_should_ensure_post_sql_option_is_added_to_end_of_sql_with_eager_loading
+    book_sql = Book.finder_sql_to_string(:select => 'topic_id', :post_sql => "/* BLAH */", :include => :topic, :conditions => 'topics.id is not null')
+    assert(/\s\/\* BLAH \*\/$/.match(book_sql))
+  end
+  
+  protected
+  
+  def create_books_and_topics
+    Book.destroy_all
+    Topic.destroy_all
+    
+    topics = [Topic.create!(:title => 'My Topic', :author_name => 'Giraffe'),
+              Topic.create!(:title => 'Other Topic', :author_name => 'Giraffe'),
+              Topic.create!(:title => 'Last Topic', :author_name => 'Giraffe')]
+   
+    Book.create!(:title => 'Title A', :topic_id => topics[0].to_param, :author_name => 'Giraffe')
+    Book.create!(:title => 'Title B', :topic_id => topics[0].to_param, :author_name => 'Giraffe')
+    Book.create!(:title => 'Title C', :topic_id => topics[0].to_param, :author_name => 'Giraffe')
+    Book.create!(:title => 'Title D', :topic_id => topics[1].to_param, :author_name => 'Giraffe')
+    Book.create!(:title => 'Title E', :topic_id => topics[1].to_param, :author_name => 'Giraffe')
+    Book.create!(:title => 'Title F', :topic_id => topics[2].to_param, :author_name => 'Giraffe')
+  end
+  
+end