gravis-searchlogic 2.4.20

data/spec/searchlogic/named_scopes/association_ordering_spec.rb

@@ -0,0 +1,27 @@
+require File.expand_path(File.dirname(__FILE__) + "/../../spec_helper")
+
+describe Searchlogic::NamedScopes::Ordering do
+  it "should allow ascending" do
+    Company.ascend_by_users_username.proxy_options.should == User.ascend_by_username.proxy_options.merge(:joins => :users)
+  end
+  
+  it "should allow descending" do
+    Company.descend_by_users_username.proxy_options.should == User.descend_by_username.proxy_options.merge(:joins => :users)
+  end
+  
+  it "should allow deep ascending" do
+    Company.ascend_by_users_orders_total.proxy_options.should == Order.ascend_by_total.proxy_options.merge(:joins => {:users => :orders})
+  end
+  
+  it "should allow deep descending" do
+    Company.descend_by_users_orders_total.proxy_options.should == Order.descend_by_total.proxy_options.merge(:joins => {:users => :orders})
+  end
+  
+  it "should ascend with a belongs to" do
+    User.ascend_by_company_name.proxy_options.should == Company.ascend_by_name.proxy_options.merge(:joins => :company)
+  end
+  
+  it "should work through #order" do
+    Company.order('ascend_by_users_username').proxy_options.should == Company.ascend_by_users_username.proxy_options
+  end
+end

data/spec/searchlogic/named_scopes/conditions_spec.rb

@@ -0,0 +1,319 @@
+require File.expand_path(File.dirname(__FILE__) + "/../../spec_helper")
+
+describe Searchlogic::NamedScopes::Conditions do
+  it "should be dynamically created and then cached" do
+    User.should_not respond_to(:age_less_than)
+    User.age_less_than(5)
+    User.should respond_to(:age_less_than)
+  end
+  
+  it "should not allow conditions on non columns" do
+    lambda { User.whatever_equals(2) }.should raise_error(NoMethodError)
+  end
+  
+  context "comparison conditions" do
+    it "should have equals" do
+      (5..7).each { |age| User.create(:age => age) }
+      User.age_equals(6).all.should == User.find_all_by_age(6)
+      User.age_equals(5..6).all.should == User.find_all_by_age(5..6)
+      User.age_equals([5, 7]).all.should == User.find_all_by_age([5, 7])
+    end
+    
+    it "should have does not equal" do
+      (5..7).each { |age| User.create(:age => age) }
+      User.age_does_not_equal(6).all.should == User.find_all_by_age([5,7])
+    end
+    
+    it "should have less than" do
+      (5..7).each { |age| User.create(:age => age) }
+      User.age_less_than(6).all.should == User.find_all_by_age(5)
+    end
+    
+    it "should have less than or equal to" do
+      (5..7).each { |age| User.create(:age => age) }
+      User.age_less_than_or_equal_to(6).all.should == User.find_all_by_age([5, 6])
+    end
+    
+    it "should have greater than" do
+      (5..7).each { |age| User.create(:age => age) }
+      User.age_greater_than(6).all.should == User.find_all_by_age(7)
+    end
+    
+    it "should have greater than or equal to" do
+      (5..7).each { |age| User.create(:age => age) }
+      User.age_greater_than_or_equal_to(6).all.should == User.find_all_by_age([6, 7])
+    end
+  end
+  
+  context "wildcard conditions" do
+    it "should have like" do
+      %w(bjohnson thunt).each { |username| User.create(:username => username) }
+      User.username_like("john").all.should == User.find_all_by_username("bjohnson")
+    end
+    
+    it "should have not like" do
+      %w(bjohnson thunt).each { |username| User.create(:username => username) }
+      User.username_not_like("john").all.should == User.find_all_by_username("thunt")
+    end
+    
+    it "should have begins with" do
+      %w(bjohnson thunt).each { |username| User.create(:username => username) }
+      User.username_begins_with("bj").all.should == User.find_all_by_username("bjohnson")
+    end
+    
+    it "should have not begin with" do
+      %w(bjohnson thunt).each { |username| User.create(:username => username) }
+      User.username_not_begin_with("bj").all.should == User.find_all_by_username("thunt")
+    end
+    
+    it "should have ends with" do
+      %w(bjohnson thunt).each { |username| User.create(:username => username) }
+      User.username_ends_with("son").all.should == User.find_all_by_username("bjohnson")
+    end
+    
+    it "should have not end with" do
+      %w(bjohnson thunt).each { |username| User.create(:username => username) }
+      User.username_not_end_with("son").all.should == User.find_all_by_username("thunt")
+    end
+  end
+  
+  context "boolean conditions" do
+    it "should have scopes for boolean columns" do
+      female = User.create(:male => false)
+      male = User.create(:male => true)
+      User.male.all.should == [male]
+      User.not_male.all.should == [female]
+    end
+    
+    it "should have null" do
+      ["bjohnson", nil].each { |username| User.create(:username => username) }
+      User.username_null.all.should == User.find_all_by_username(nil)
+    end
+    
+    it "should have not null" do
+      ["bjohnson", nil].each { |username| User.create(:username => username) }
+      User.username_not_null.all.should == User.find_all_by_username("bjohnson")
+    end
+    
+    it "should have empty" do
+      ["bjohnson", ""].each { |username| User.create(:username => username) }
+      User.username_empty.all.should == User.find_all_by_username("")
+    end
+    
+    it "should have blank" do
+      ["bjohnson", "", nil].each { |username| User.create(:username => username) }
+      User.username_blank.all.should == [User.find_by_username(""), User.find_by_username(nil)]
+    end
+    
+    it "should have not blank" do
+      ["bjohnson", "", nil].each { |username| User.create(:username => username) }
+      User.username_not_blank.all.should == User.find_all_by_username("bjohnson")
+    end
+  end
+  
+  context "any and all conditions" do
+    it "should do nothing if no arguments are passed" do
+      User.username_equals_any.proxy_options.should == {}
+    end
+    
+    it "should treat an array and multiple arguments the same" do
+      %w(bjohnson thunt dgainor).each { |username| User.create(:username => username) }
+      User.username_like_any("bjohnson", "thunt").should == User.username_like_any(["bjohnson", "thunt"])
+    end
+    
+    it "should have equals any" do
+      %w(bjohnson thunt dgainor).each { |username| User.create(:username => username) }
+      User.username_equals_any("bjohnson", "thunt").all.should == User.find_all_by_username(["bjohnson", "thunt"])
+    end
+    
+    it "should have equals all" do
+      %w(bjohnson thunt dainor).each { |username| User.create(:username => username) }
+      User.username_equals_all("bjohnson", "thunt").all.should == []
+    end
+    
+    it "should have does not equal any" do
+      %w(bjohnson thunt dgainor).each { |username| User.create(:username => username) }
+      User.username_does_not_equal_any("bjohnson", "thunt").all.should == User.find_all_by_username(["bjohnson", "thunt", "dgainor"])
+    end
+    
+    it "should have does not equal all" do
+      %w(bjohnson thunt dgainor).each { |username| User.create(:username => username) }
+      User.username_does_not_equal_all("bjohnson", "thunt").all.should == User.find_all_by_username("dgainor")
+    end
+    
+    it "should have less than any" do
+      (5..7).each { |age| User.create(:age => age) }
+      User.age_less_than_any(7,6).all.should == User.find_all_by_age([5, 6])
+    end
+    
+    it "should have less than all" do
+      (5..7).each { |age| User.create(:age => age) }
+      User.age_less_than_all(7,6).all.should == User.find_all_by_age(5)
+    end
+    
+    it "should have less than or equal to any" do
+      (5..7).each { |age| User.create(:age => age) }
+      User.age_less_than_or_equal_to_any(7,6).all.should == User.find_all_by_age([5, 6, 7])
+    end
+    
+    it "should have less than or equal to all" do
+      (5..7).each { |age| User.create(:age => age) }
+      User.age_less_than_or_equal_to_all(7,6).all.should == User.find_all_by_age([5, 6])
+    end
+    
+    it "should have less than any" do
+      (5..7).each { |age| User.create(:age => age) }
+      User.age_greater_than_any(5,6).all.should == User.find_all_by_age([6, 7])
+    end
+    
+    it "should have greater than all" do
+      (5..7).each { |age| User.create(:age => age) }
+      User.age_greater_than_all(5,6).all.should == User.find_all_by_age(7)
+    end
+    
+    it "should have greater than or equal to any" do
+      (5..7).each { |age| User.create(:age => age) }
+      User.age_greater_than_or_equal_to_any(5,6).all.should == User.find_all_by_age([5, 6, 7])
+    end
+    
+    it "should have greater than or equal to all" do
+      (5..7).each { |age| User.create(:age => age) }
+      User.age_greater_than_or_equal_to_all(5,6).all.should == User.find_all_by_age([6, 7])
+    end
+    
+    it "should have like all" do
+      %w(bjohnson thunt dgainor).each { |username| User.create(:username => username) }
+      User.username_like_all("bjohnson", "thunt").all.should == []
+      User.username_like_all("n", "o").all.should == User.find_all_by_username(["bjohnson", "dgainor"])
+    end
+    
+    it "should have like any" do
+      %w(bjohnson thunt dgainor).each { |username| User.create(:username => username) }
+      User.username_like_any("bjohnson", "thunt").all.should == User.find_all_by_username(["bjohnson", "thunt"])
+    end
+    
+    it "should have begins with all" do
+      %w(bjohnson thunt dgainor).each { |username| User.create(:username => username) }
+      User.username_begins_with_all("bjohnson", "thunt").all.should == []
+    end
+    
+    it "should have begins with any" do
+      %w(bjohnson thunt dgainor).each { |username| User.create(:username => username) }
+      User.username_begins_with_any("bj", "th").all.should == User.find_all_by_username(["bjohnson", "thunt"])
+    end
+    
+    it "should have ends with all" do
+      %w(bjohnson thunt dgainor).each { |username| User.create(:username => username) }
+      User.username_ends_with_all("n", "r").all.should == []
+    end
+    
+    it "should have ends with any" do
+      %w(bjohnson thunt dgainor).each { |username| User.create(:username => username) }
+      User.username_ends_with_any("n", "r").all.should == User.find_all_by_username(["bjohnson", "dgainor"])
+    end
+  end
+  
+  context "alias conditions" do
+    it "should have is" do
+      User.age_is(5).proxy_options.should == User.age_equals(5).proxy_options
+    end
+    
+    it "should have eq" do
+      User.age_eq(5).proxy_options.should == User.age_equals(5).proxy_options
+    end
+    
+    it "should have not_equal_to" do
+      User.age_not_equal_to(5).proxy_options.should == User.age_does_not_equal(5).proxy_options
+    end
+    
+    it "should have is_not" do
+      User.age_is_not(5).proxy_options.should == User.age_does_not_equal(5).proxy_options
+    end
+    
+    it "should have not" do
+      User.age_not(5).proxy_options.should == User.age_does_not_equal(5).proxy_options
+    end
+    
+    it "should have ne" do
+      User.age_ne(5).proxy_options.should == User.age_does_not_equal(5).proxy_options
+    end
+    
+    it "should have lt" do
+      User.age_lt(5).proxy_options.should == User.age_less_than(5).proxy_options
+    end
+    
+    it "should have lte" do
+      User.age_lte(5).proxy_options.should == User.age_less_than_or_equal_to(5).proxy_options
+    end
+    
+    it "should have gt" do
+      User.age_gt(5).proxy_options.should == User.age_greater_than(5).proxy_options
+    end
+    
+    it "should have gte" do
+      User.age_gte(5).proxy_options.should == User.age_greater_than_or_equal_to(5).proxy_options
+    end
+    
+    it "should have contains" do
+      User.username_contains(5).proxy_options.should == User.username_like(5).proxy_options
+    end
+    
+    it "should have contains" do
+      User.username_includes(5).proxy_options.should == User.username_like(5).proxy_options
+    end
+    
+    it "should have bw" do
+      User.username_bw(5).proxy_options.should == User.username_begins_with(5).proxy_options
+    end
+    
+    it "should have ew" do
+      User.username_ew(5).proxy_options.should == User.username_ends_with(5).proxy_options
+    end
+    
+    it "should have nil" do
+      User.username_nil.proxy_options.should == User.username_nil.proxy_options
+    end
+  end
+  
+  context "group conditions" do
+    it "should have in" do
+      (5..7).each { |age| User.create(:age => age) }
+      User.age_in([5,6]).all.should == User.find(:all, :conditions => ["users.age IN (?)", [5, 6]])
+    end
+    
+    it "should have not_in" do
+      (5..7).each { |age| User.create(:age => age) }
+      User.age_not_in([5,6]).all.should == User.find(:all, :conditions => ["users.age NOT IN (?)", [5, 6]])
+    end
+  end
+  
+  context "searchlogic lambda" do
+    it "should be a string" do
+      User.username_like("test")
+      User.named_scope_options(:username_like).searchlogic_options[:type].should == :string
+    end
+    
+    it "should be an integer" do
+      User.id_gt(10)
+      User.named_scope_options(:id_gt).searchlogic_options[:type].should == :integer
+    end
+    
+    it "should be a float" do
+      Order.total_gt(10)
+      Order.named_scope_options(:total_gt).searchlogic_options[:type].should == :float
+    end
+  end
+  
+  it "should have priorty to columns over conflicting association conditions" do
+    Company.users_count_gt(10)
+    User.create
+    User.company_id_null.count.should == 1
+    User.company_id_not_null.count.should == 0
+  end
+  
+  it "should fix bug for issue 26" do
+    count1 = User.id_ne(10).username_not_like("root").count
+    count2 = User.id_ne(10).username_not_like("root").count
+    count1.should == count2
+  end
+end

data/spec/searchlogic/named_scopes/or_conditions_spec.rb

@@ -0,0 +1,66 @@
+require File.expand_path(File.dirname(__FILE__) + "/../../spec_helper")
+
+describe Searchlogic::NamedScopes::OrConditions do
+  it "should define a scope by the exact same name as requested by the code" do
+    User.name_or_username_like('Test')
+    User.respond_to?(:name_or_username_like).should be_true
+  end
+  
+  it "should match username or name" do
+    User.username_or_name_like("ben").proxy_options.should == {:conditions => "(users.username LIKE '%ben%') OR (users.name LIKE '%ben%')"}
+  end
+  
+  it "should use the specified condition" do
+    User.username_begins_with_or_name_like("ben").proxy_options.should == {:conditions => "(users.username LIKE 'ben%') OR (users.name LIKE '%ben%')"}
+  end
+  
+  it "should use the last specified condition" do
+    User.username_or_name_like_or_id_or_age_lt(10).proxy_options.should == {:conditions => "(users.username LIKE '%10%') OR (users.name LIKE '%10%') OR (users.id < 10) OR (users.age < 10)"}
+  end
+  
+  it "should raise an error on unknown conditions" do
+    lambda { User.usernme_begins_with_or_name_like("ben") }.should raise_error(Searchlogic::NamedScopes::OrConditions::UnknownConditionError)
+  end
+  
+  it "should work well with _or_equal_to" do
+    User.id_less_than_or_equal_to_or_age_gt(10).proxy_options.should == {:conditions => "(users.id <= 10) OR (users.age > 10)"}
+  end
+  
+  it "should work well with _or_equal_to_any" do
+    User.id_less_than_or_equal_to_all_or_age_gt(10).proxy_options.should == {:conditions => "(users.id <= 10) OR (users.age > 10)"}
+  end
+  
+  it "should work well with _or_equal_to_all" do
+    User.id_less_than_or_equal_to_any_or_age_gt(10).proxy_options.should == {:conditions => "(users.id <= 10) OR (users.age > 10)"}
+  end
+  
+  it "should play nice with other scopes" do
+    User.username_begins_with("ben").id_gt(10).age_not_nil.username_or_name_ends_with("ben").scope(:find).should ==
+      {:conditions => "((users.username LIKE '%ben') OR (users.name LIKE '%ben')) AND ((users.age IS NOT NULL) AND ((users.id > 10) AND (users.username LIKE 'ben%')))"}
+  end
+  
+  it "should play nice with scopes on associations" do
+    lambda { User.name_or_company_name_like("ben") }.should_not raise_error(Searchlogic::NamedScopes::OrConditions::NoConditionSpecifiedError)
+    User.name_or_company_name_like("ben").proxy_options.should == {:joins => :company, :conditions => "(users.name LIKE '%ben%') OR (companies.name LIKE '%ben%')"}
+    User.company_name_or_name_like("ben").proxy_options.should == {:joins => :company, :conditions => "(companies.name LIKE '%ben%') OR (users.name LIKE '%ben%')"}
+    User.company_name_or_company_description_like("ben").proxy_options.should == {:joins =>[:company], :conditions => "(companies.name LIKE '%ben%') OR (companies.description LIKE '%ben%')"}
+  end
+  
+  it "should not get confused by the 'or' in find_or_create_by_* methods" do
+    User.create(:name => "Fred")
+    User.find_or_create_by_name("Fred").should be_a_kind_of User
+  end
+
+  it "should not get confused by the 'or' in compound find_or_create_by_* methods" do
+    User.create(:name => "Fred", :username => "fredb")
+    User.find_or_create_by_name_and_username("Fred", "fredb").should be_a_kind_of User
+  end
+  
+  it "should work with User.search(conditions) method" do
+    User.search(:username_or_name_like => 'ben').proxy_options.should == {:conditions => "(users.username LIKE '%ben%') OR (users.name LIKE '%ben%')"}
+  end
+
+  it "should convert types properly when used with User.search(conditions) method" do
+    User.search(:id_or_age_lte => '10').proxy_options.should == {:conditions => "(users.id <= 10) OR (users.age <= 10)"}
+  end
+end

data/spec/searchlogic/named_scopes/ordering_spec.rb

@@ -0,0 +1,34 @@
+require File.expand_path(File.dirname(__FILE__) + "/../../spec_helper")
+
+describe Searchlogic::NamedScopes::Ordering do
+  it "should be dynamically created and then cached" do
+    User.should_not respond_to(:ascend_by_username)
+    User.ascend_by_username
+    User.should respond_to(:ascend_by_username)
+  end
+  
+  it "should have ascending" do
+    %w(bjohnson thunt).each { |username| User.create(:username => username) }
+    User.ascend_by_username.all.should == User.all(:order => "username ASC")
+  end
+  
+  it "should have descending" do
+    %w(bjohnson thunt).each { |username| User.create(:username => username) }
+    User.descend_by_username.all.should == User.all(:order => "username DESC")
+  end
+  
+  it "should have order" do
+    User.order("ascend_by_username").proxy_options.should == User.ascend_by_username.proxy_options
+  end
+
+  it "should have order by custom scope" do
+    User.column_names.should_not include("custom")
+    %w(bjohnson thunt fisons).each { |username| User.create(:username => username) }
+    User.named_scope(:ascend_by_custom, :order => "username ASC, name DESC")
+    User.order("ascend_by_custom").proxy_options.should == User.ascend_by_custom.proxy_options
+  end
+  
+  it "should have priorty to columns over conflicting association columns" do
+    Company.ascend_by_users_count
+  end
+end

data/spec/searchlogic/search_spec.rb

@@ -0,0 +1,459 @@
+require File.expand_path(File.dirname(__FILE__) + "/../spec_helper")
+
+describe Searchlogic::Search do
+  describe "Implementation" do
+    context "#searchlogic" do
+      it "should create a search proxy" do
+        User.search(:username => "joe").should be_kind_of(Searchlogic::Search)
+      end
+    
+      it "should create a search proxy using the same class" do
+        User.search.klass.should == User
+      end
+    
+      it "should pass on the current scope to the proxy" do
+        company = Company.create
+        user = company.users.create
+        search = company.users.search
+        search.current_scope.should == company.users.scope(:find)
+      end
+    end
+  end
+  
+  context "#initialize" do
+    it "should require a class" do
+      lambda { Searchlogic::Search.new }.should raise_error(ArgumentError)
+    end
+  
+    it "should set the conditions" do
+      search = User.search(:username => "bjohnson")
+      search.conditions.should == {:username => "bjohnson"}
+    end
+  end
+  
+  context "#clone" do
+    it "should clone properly" do
+      company = Company.create
+      user1 = company.users.create(:age => 5)
+      user2 = company.users.create(:age => 25)
+      search1 = company.users.search(:age_gt => 10)
+      search2 = search1.clone
+      search2.age_gt = 1
+      search2.all.should == User.all
+      search1.all.should == [user2]
+    end
+  
+    it "should clone properly without scope" do
+      user1 = User.create(:age => 5)
+      user2 = User.create(:age => 25)
+      search1 = User.search(:age_gt => 10)
+      search2 = search1.clone
+      search2.age_gt = 1
+      search2.all.should == User.all
+      search1.all.should == [user2]
+    end
+  end
+  
+  context "#conditions" do
+    it "should set the conditions and be accessible individually" do
+      search = User.search
+      search.conditions = {:username => "bjohnson"}
+      search.username.should == "bjohnson"
+    end
+    
+    it "should set the conditions and allow string keys" do
+      search = User.search
+      search.conditions = {"username" => "bjohnson"}
+      search.username.should == "bjohnson"
+    end
+    
+    it "should use custom scopes before normalizing" do
+      User.create(:username => "bjohnson")
+      User.named_scope :username, lambda { |value| {:conditions => {:username => value.reverse}} }
+      search1 = User.search(:username => "bjohnson")
+      search2 = User.search(:username => "nosnhojb")
+      search1.count.should == 0
+      search2.count.should == 1
+    end
+    
+    # We ignore them upon execution. But we still want to accept the condition so that returning the conditions
+    # preserves the values.
+    it "should ignore blank values but still return on conditions" do
+      search = User.search
+      search.conditions = {"username" => ""}
+      search.username.should be_nil
+      search.conditions.should == {:username => ""}
+    end
+    
+    it "should not ignore blank values and should not cast them" do
+      search = User.search
+      search.conditions = {"id_equals" => ""}
+      search.id_equals.should be_nil
+      search.conditions.should == {:id_equals => ""}
+    end
+    
+    it "should ignore blank values in arrays" do
+      search = User.search
+      search.conditions = {"username_equals_any" => [""]}
+      search.username_equals_any.should be_nil
+      
+      search.conditions = {"id_equals_any" => ["", "1"]}
+      search.id_equals_any.should == [1]
+    end
+  end
+  
+  context "condition accessors" do
+    it "should allow setting exact columns individually" do
+      search = User.search
+      search.username = "bjohnson"
+      search.username.should == "bjohnson"
+    end
+    
+    it "should allow setting local column conditions individually" do
+      search = User.search
+      search.username_gt = "bjohnson"
+      search.username_gt.should == "bjohnson"
+    end
+    
+    it "should allow chaining conditions" do
+      user = User.create(:username => "bjohnson", :age => 20)
+      User.create(:username => "bjohnson", :age => 5)
+      search = User.search
+      search.username_equals("bjohnson").age_gt(10)
+      search.all.should == [user]
+    end
+    
+    it "should allow setting association conditions" do
+      search = User.search
+      search.orders_total_gt = 10
+      search.orders_total_gt.should == 10
+    end
+    
+    it "should allow setting pre-existing association conditions" do
+      User.named_scope :uname, lambda { |value| {:conditions => ["users.username = ?", value]} }
+      search = Company.search
+      search.users_uname = "bjohnson"
+      search.users_uname.should == "bjohnson"
+    end
+    
+    it "should allow setting pre-existing association alias conditions" do
+      User.alias_scope :username_has, lambda { |value| User.username_like(value) }
+      search = Company.search
+      search.users_username_has = "bjohnson"
+      search.users_username_has.should == "bjohnson"
+    end
+    
+    it "should allow using custom conditions" do
+      User.named_scope(:four_year_olds, { :conditions => { :age => 4 } })
+      search = User.search
+      search.four_year_olds = true
+      search.four_year_olds.should == true
+      search.proxy_options.should == User.four_year_olds.proxy_options
+    end
+    
+    it "should not merge conflicting conditions into one value" do
+      # This class should JUST be a proxy. It should not do anything more than that.
+      # A user would be allowed to call both named scopes if they wanted.
+      search = User.search
+      search.username_greater_than = "bjohnson1"
+      search.username_gt = "bjohnson2"
+      search.username_greater_than.should == "bjohnson1"
+      search.username_gt.should == "bjohnson2"
+    end
+    
+    it "should allow setting custom conditions individually with an arity of 0" do
+      User.named_scope(:four_year_olds, :conditions => {:age => 4})
+      search = User.search
+      search.four_year_olds = true
+      search.four_year_olds.should == true
+    end
+    
+    it "should allow setting custom conditions individually with an arity of 1" do
+      User.named_scope(:username_should_be, lambda { |u| {:conditions => {:username => u}} })
+      search = User.search
+      search.username_should_be = "bjohnson"
+      search.username_should_be.should == "bjohnson"
+    end
+    
+    it "should not allow setting conditions that are not scopes" do
+      search = User.search
+      lambda { search.unknown = true }.should raise_error(Searchlogic::Search::UnknownConditionError)
+    end
+    
+    it "should not allow setting conditions on sensitive methods" do
+      search = User.search
+      lambda { search.destroy = true }.should raise_error(Searchlogic::Search::UnknownConditionError)
+    end
+    
+    it "should not use the ruby implementation of the id method" do
+      search = User.search
+      search.id.should be_nil
+    end
+    
+    context "type casting" do
+      it "should be a Boolean given true" do
+        search = User.search
+        search.id_nil = true
+        search.id_nil.should == true
+      end
+      
+      it "should be a Boolean given 'true'" do
+        search = User.search
+        search.id_nil = "true"
+        search.id_nil.should == true
+      end
+      
+      it "should be a Boolean given '1'" do
+        search = User.search
+        search.id_nil = "1"
+        search.id_nil.should == true
+      end
+      
+      it "should be a Boolean given false" do
+        search = User.search
+        search.id_nil = false
+        search.id_nil.should == false
+      end
+      
+      it "should be a Boolean given 'false'" do
+        search = User.search
+        search.id_nil = "false"
+        search.id_nil.should == false
+      end
+      
+      it "should be a Boolean given '0'" do
+        search = User.search
+        search.id_nil = "0"
+        search.id_nil.should == false
+      end
+      
+      it "should be an Integer given ''" do
+        search = User.search
+        search.id_gt = ''
+        search.id_gt.should == 0
+      end
+      
+      it "should be an Integer given 1" do
+        search = User.search
+        search.id_gt = 1
+        search.id_gt.should == 1
+      end
+      
+      it "should be an Integer given '1'" do
+        search = User.search
+        search.id_gt = "1"
+        search.id_gt.should == 1
+      end
+      
+      it "should be a Float given 1.0" do
+        search = Order.search
+        search.total_gt = 1.0
+        search.total_gt.should == 1.0
+      end
+      
+      it "should be a Float given '1'" do
+        search = Order.search
+        search.total_gt = "1"
+        search.total_gt.should == 1.0
+      end
+      
+      it "should be a Float given '1.5'" do
+        search = Order.search
+        search.total_gt = "1.5"
+        search.total_gt.should == 1.5
+      end
+      
+      it "should be a Range given 1..3" do
+        search = Order.search
+        search.total_eq = (1..3)
+        search.total_eq.should == (1..3)
+      end
+      
+      it "should be a Date given 'Jan 1, 2009'" do
+        search = Order.search
+        search.shipped_on_after = "Jan 1, 2009"
+        search.shipped_on_after.should == Date.parse("Jan 1, 2009")
+      end
+      
+      it "should be a Time given 'Jan 1, 2009'" do
+        search = Order.search
+        search.created_at_after = "Jan 1, 2009"
+        search.created_at_after.should == Time.zone.parse("Jan 1, 2009")
+      end
+      
+      it "should be a Time given 'Jan 1, 2009 9:33AM'" do
+        search = Order.search
+        search.created_at_after = "Jan 1, 2009 9:33AM"
+        search.created_at_after.should == Time.zone.parse("Jan 1, 2009 9:33AM")
+      end
+      
+      it "should skip time zone conversion for attributes skipped" do
+        search = User.search
+        search.whatever_at_after = "Jan 1, 2009 9:33AM"
+        search.whatever_at_after.should == Time.parse("Jan 1, 2009 9:33AM").utc
+      end
+      
+      it "should convert the time to the current zone" do
+        search = Order.search
+        now = Time.now
+        search.created_at_after = now
+        search.created_at_after.should == now.in_time_zone
+      end
+      
+      it "should be an Array and cast it's values given ['1', '2', '3']" do
+        search = Order.search
+        search.id_equals_any = ["1", "2", "3"]
+        search.id_equals_any.should == [1, 2, 3]
+      end
+      
+      it "should type cast association conditions" do
+        search = User.search
+        search.orders_total_gt = "10"
+        search.orders_total_gt.should == 10
+      end
+      
+      it "should type cast deep association conditions" do
+        search = Company.search
+        search.users_orders_total_gt = "10"
+        search.users_orders_total_gt.should == 10
+      end
+
+      it "should support Rails' date_select and datetime_select out of the box" do
+        search = Company.search('created_at_after(1i)' => 2000, 'created_at_after(2i)' => 1, 'created_at_after(3i)' => 1)
+        search.created_at_after.should_not be_nil
+        search.created_at_after.should == Time.zone.local(2000, 1, 1)
+      end
+    end
+  end
+  
+  context "#delete" do
+    it "should delete the condition" do
+      search = User.search(:username_like => "bjohnson")
+      search.delete("username_like")
+      search.username_like.should be_nil
+      search.conditions["username_like"].should be_nil
+    end
+  end
+  
+  context "#ordering_by" do
+    it "should return nil if we aren't ordering" do
+      search = User.search
+      search.ordering_by.should be_nil
+    end
+    
+    it "should return the column name for ascending" do
+      search = User.search(:order => "ascend_by_first_name")
+      search.ordering_by.should == "first_name"
+    end
+    
+    it "should return the column name for descending" do
+      search = User.search(:order => "descend_by_first_name")
+      search.ordering_by.should == "first_name"
+    end
+    
+    it "should handle symbols" do
+      search = User.search(:order => :descend_by_first_name)
+      search.ordering_by.should == "first_name"
+    end
+  end
+  
+  context "#method_missing" do
+    context "setting" do
+      it "should call named scopes for conditions" do
+        User.search(:age_less_than => 5).proxy_options.should == User.age_less_than(5).proxy_options
+      end
+    
+      it "should alias exact column names to use equals" do
+        User.search(:username => "joe").proxy_options.should == User.username_equals("joe").proxy_options
+      end
+    
+      it "should recognize conditions with a value of true where the named scope has an arity of 0" do
+        User.search(:username_nil => true).proxy_options.should == User.username_nil.proxy_options
+      end
+    
+      it "should ignore conditions with a value of false where the named scope has an arity of 0" do
+        User.search(:username_nil => false).proxy_options.should == {}
+      end
+    
+      it "should not ignore conditions with a value of false where the named scope does not have an arity of 0" do
+        User.search(:username_is => false).proxy_options.should == User.username_is(false).proxy_options
+      end
+    
+      it "should recognize the order condition" do
+        User.search(:order => "ascend_by_username").proxy_options.should == User.ascend_by_username.proxy_options
+      end
+    
+      it "should pass array values as multiple arguments with arity -1" do
+        User.named_scope(:multiple_args, lambda { |*args|
+          raise "This should not be an array, it should be 1" if args.first.is_a?(Array)
+          {:conditions => ["id IN (?)", args]}
+        })
+        User.search(:multiple_args => [1,2]).proxy_options.should == User.multiple_args(1,2).proxy_options
+      end
+    
+      it "should pass array as a single value with arity >= 0" do
+        User.named_scope(:multiple_args, lambda { |args|
+          raise "This should be an array" if !args.is_a?(Array)
+          {:conditions => ["id IN (?)", args]}
+        })
+        User.search(:multiple_args => [1,2]).proxy_options.should == User.multiple_args([1,2]).proxy_options
+      end
+    
+      it "should not split out dates or times (big fix)" do
+        s = User.search
+        s.created_at_after = Time.now
+        lambda { s.count }.should_not raise_error
+      end
+    
+      it "should not include blank values" do
+        s = User.search
+        s.conditions = {"id_equals" => ""}
+        s.proxy_options.should == {}
+      end
+    end
+  end
+  
+  context "delegation" do
+    it "should return all when not given any conditions" do
+      3.times { User.create }
+      User.search.all.length.should == 3
+    end
+    
+    it "should implement the current scope based on an association" do
+      User.create
+      company = Company.create
+      user = company.users.create
+      company.users.search.all.should == [user]
+    end
+    
+    it "should implement the current scope based on a named scope" do
+      User.named_scope(:four_year_olds, :conditions => {:age => 4})
+      (3..5).each { |age| User.create(:age => age) }
+      User.four_year_olds.search.all.should == User.find_all_by_age(4)
+    end
+    
+    it "should respond to count" do
+      User.create(:username => "bjohnson")
+      search1 = User.search(:username => "bjohnson")
+      search2 = User.search(:username => "nosnhojb")
+      search1.count.should == 1
+      search2.count.should == 0
+    end
+    
+    it "should respond to empty?" do
+      User.create(:username => "bjohnson")
+      search1 = User.search(:username => "bjohnson")
+      search2 = User.search(:username => "nosnhojb")
+      search1.empty?.should == false
+      search2.empty?.should == true
+    end
+
+    it "should delegate to named scopes with arity > 1" do
+      User.named_scope :paged, lambda {|start, limit| { :limit => limit, :offset => start }}
+      User.create(:username => "bjohnson")
+      search = User.search(:username => "bjohnson")
+      search.paged(0, 1).count.should == 1
+      search.paged(0, 0).count.should == 0
+    end
+  end
+end