searchlogic 1.5.3 → 1.5.4

data/test/condition_tests/not_equal_test.rb

@@ -0,0 +1,19 @@
+require File.dirname(__FILE__) + '/../test_helper.rb'
+
+module ConditionTests
+  class NotEqualTest < ActiveSupport::TestCase
+    def test_sanitize
+      condition = Searchlogic::Condition::NotEqual.new(Account, :column => Account.columns_hash["id"])
+      condition.value = 12
+      assert_equal ["\"accounts\".\"id\" != ?", 12], condition.sanitize
+    
+      condition = Searchlogic::Condition::NotEqual.new(Account, :column => Account.columns_hash["id"])
+      condition.value = [1,2,3,4]
+      assert_equal ["\"accounts\".\"id\" NOT IN (?)", [1, 2, 3, 4]], condition.sanitize
+    
+      condition = Searchlogic::Condition::NotEqual.new(Account, :column => Account.columns_hash["id"])
+      condition.value = (1..10)
+      assert_equal ["\"accounts\".\"id\" NOT BETWEEN ? AND ?", 1, 10], condition.sanitize
+    end
+  end
+end

data/test/condition_tests/not_have_keywords_test.rb

@@ -0,0 +1,8 @@
+require File.dirname(__FILE__) + '/../test_helper.rb'
+
+module ConditionTests
+  class NotHaveKeywordsTest < ActiveSupport::TestCase
+    def test_sanitize
+    end
+  end
+end

data/test/condition_tests/not_like_test.rb

@@ -0,0 +1,8 @@
+require File.dirname(__FILE__) + '/../test_helper.rb'
+
+module ConditionTests
+  class NotLikeTest < ActiveSupport::TestCase
+    def test_sanitize
+    end
+  end
+end

data/test/condition_tests/not_nil_test.rb

@@ -0,0 +1,13 @@
+require File.dirname(__FILE__) + '/../test_helper.rb'
+
+module ConditionTests
+  class NotNilTest < ActiveSupport::TestCase
+    def test_sanitize
+      condition = Searchlogic::Condition::NotNil.new(Account, :column => Account.columns_hash["created_at"])
+      condition.value = "1"
+      assert_equal "\"accounts\".\"created_at\" IS NOT NULL", condition.sanitize
+      condition.value = "false"
+      assert_equal "\"accounts\".\"created_at\" IS NULL", condition.sanitize
+    end
+  end
+end

data/test/condition_tests/sibling_of_test.rb

@@ -0,0 +1,15 @@
+require File.dirname(__FILE__) + '/../test_helper.rb'
+
+module ConditionTests
+  class SiblingOfTest < ActiveSupport::TestCase
+    def test_sanitize
+      ben = users(:ben)
+      drew = users(:drew)
+      jennifer = users(:jennifer)
+      
+      condition = Searchlogic::Condition::SiblingOf.new(User)
+      condition.value = drew
+      assert_equal ["(\"users\".\"id\" != ?) AND (\"users\".\"parent_id\" = ?)", drew.id, ben.id], condition.sanitize
+    end
+  end
+end

data/test/conditions_tests/base_test.rb

@@ -0,0 +1,221 @@
+require File.dirname(__FILE__) + '/../test_helper.rb'
+
+module ConditionsTests
+  class BaseTest < ActiveSupport::TestCase
+    def test_register_conditions
+      Searchlogic::Conditions::Base.register_condition(Searchlogic::Condition::Keywords)
+      assert [Searchlogic::Condition::Keywords], Searchlogic::Conditions::Base.conditions
+    
+      Searchlogic::Conditions::Base.register_condition(Searchlogic::Condition::Like)
+      assert [Searchlogic::Condition::Keywords, Searchlogic::Condition::Like], Searchlogic::Conditions::Base.conditions
+    end
+  
+    def test_association_names
+      ["dogs", "children", "user_groups", "orders", "account", "parent", "cats"].each do |name|
+        assert Searchlogic::Cache::UserConditions.association_names.include? name
+      end
+      ["admin", "orders", "users"].each do |name|
+        Searchlogic::Cache::AccountConditions.association_names.include? name
+      end
+    end
+  
+    def test_needed
+      assert (not Searchlogic::Conditions::Base.needed?(User, {}))
+      assert (not Searchlogic::Conditions::Base.needed?(User, {:first_name => "Ben"}))
+      assert Searchlogic::Conditions::Base.needed?(User, {:first_name_contains => "Awesome"})
+      assert (not Searchlogic::Conditions::Base.needed?(User, {"orders.id" => 2}))
+    end
+  
+    def test_initialize
+      conditions = Searchlogic::Cache::AccountConditions.new(:name_contains => "Binary")
+      assert_equal conditions.klass, Account
+      assert_equal conditions.name_contains, "Binary"
+    end
+  
+    def test_any
+      conditions = Searchlogic::Cache::AccountConditions.new
+      assert (not conditions.any?)
+      conditions = Searchlogic::Cache::AccountConditions.new(:any => true)
+      assert conditions.any?
+      conditions.any = "false"
+      assert (not conditions.any?)
+      conditions = Searchlogic::Cache::AccountConditions.new
+      conditions.name_contains = "Binary"
+      assert_equal_sql ["\"accounts\".\"name\" LIKE ?", "%Binary%"], conditions.sanitize
+      conditions.id = 1
+      assert_equal_sql ["(\"accounts\".\"id\" = ?) AND (\"accounts\".\"name\" LIKE ?)", 1, "%Binary%"], conditions.sanitize
+      conditions.any = true
+      assert_equal_sql ["(\"accounts\".\"id\" = ?) OR (\"accounts\".\"name\" LIKE ?)", 1, "%Binary%"], conditions.sanitize
+      conditions.any = false
+      assert_equal_sql ["(\"accounts\".\"id\" = ?) AND (\"accounts\".\"name\" LIKE ?)", 1, "%Binary%"], conditions.sanitize
+    end
+  
+    def test_auto_joins
+      conditions = Searchlogic::Cache::AccountConditions.new
+      assert_equal conditions.auto_joins, nil
+    
+      conditions.name_like = "Binary"
+      assert_equal conditions.auto_joins, nil
+    
+      conditions.users.first_name_like = "Ben"
+      assert_equal conditions.auto_joins, :users
+    
+      conditions.users.orders.description_like = "apple"
+      assert_equal conditions.auto_joins, {:users => :orders} 
+    end
+  
+    def test_inspect
+      conditions = Searchlogic::Cache::AccountConditions.new
+      assert_nothing_raised { conditions.inspect }
+    end
+  
+    def test_sanitize
+      conditions = Searchlogic::Cache::AccountConditions.new
+      conditions.name_contains = "Binary"
+      conditions.id_gt = 5
+      now = Time.now
+      conditions.created_after = now
+      assert_equal_sql ["(\"accounts\".\"created_at\" > ?) AND (\"accounts\".\"id\" > ?) AND (\"accounts\".\"name\" LIKE ?)", now, 5, "%Binary%"], conditions.sanitize
+    
+      # test out associations
+      conditions.users.first_name_like = "Ben"
+      conditions.users.id_gt = 10
+      conditions.users.orders.total_lt = 500
+      assert_equal_sql ["(\"accounts\".\"created_at\" > ?) AND (\"accounts\".\"id\" > ?) AND (\"accounts\".\"name\" LIKE ?) AND ((\"users\".\"first_name\" LIKE ?) AND (\"users\".\"id\" > ?) AND (\"orders\".\"total\" < ?))", now, 5, "%Binary%", "%Ben%", 10, 500], conditions.sanitize
+    
+      # test that raw sql is returned
+      conditions.conditions = "awesome"
+      assert_equal "awesome", conditions.sanitize
+    end
+  
+    def test_conditions
+      conditions = Searchlogic::Cache::AccountConditions.new
+      now = Time.now
+      v = {:name_like => "Binary", :created_at_greater_than => now}
+      conditions.conditions = v
+      assert_equal v, conditions.conditions
+    
+      sql = "id in (1,2,3,4)"
+      conditions.conditions = sql
+      assert_equal sql, conditions.conditions
+      assert_equal({}, conditions.send(:objects))
+    
+      v2 = {:id_less_than => 5, :name_begins_with => "Beginning of string"}
+      conditions.conditions = v2
+      assert_equal v2, conditions.conditions
+    
+      v = {:name_like => "Binary", :created_at_greater_than => now}
+      conditions.conditions = v
+      assert_equal v2.merge(v), conditions.conditions
+    
+      sql2 = "id > 5 and name = 'Test'"
+      conditions.conditions = sql2
+      assert_equal sql2, conditions.conditions
+      assert_equal({}, conditions.send(:objects))
+    
+      conditions.name_contains = "awesome"
+      assert_equal({:name_like => "awesome"}, conditions.conditions)
+    
+      now = Time.now
+      conditions.conditions = {:id_gt => "", :id => "", :name => ["", "", ""], :created_at => ["", now], :name_starts_with => "Ben"}
+      assert_equal({:name_like => "awesome", :name_begins_with => "Ben", :created_at_equals => now}, conditions.conditions)
+    end
+  
+    # Other general usage tests, need to clean these up
+  
+    def test_setting_conditions
+      [Account, User, Order].each do |klass|
+        conditions = "Searchlogic::Cache::#{klass}Conditions".constantize.new
+        conditions.class.condition_names.each do |condition_name|
+          conditions.send("#{condition_name}=", 1)
+          assert_equal 1, conditions.send(condition_name)
+        end
+      end
+    end
+  
+    def test_accessible_protected_conditions
+      Account.conditions_accessible << :name_contains
+      conditions = Searchlogic::Cache::AccountConditions.new
+      conditions.conditions = {:created_after => Time.now, :name_contains => "Binary"}
+      assert({:name_contains => "Binary"}, conditions.conditions)
+      Account.send(:write_inheritable_attribute, :conditions_accessible, nil)
+    
+      Account.conditions_protected << :name_contains
+      conditions = Searchlogic::Cache::AccountConditions.new
+      now = Time.now
+      conditions.conditions = {:created_after => now, :name_contains => "Binary"}
+      assert({:created_after => now}, conditions.conditions)
+      Account.send(:write_inheritable_attribute, :conditions_protected, nil)
+    end
+  
+    def test_assert_valid_values
+      conditions = Searchlogic::Cache::UserConditions.new
+      assert_raise(NoMethodError) { conditions.conditions = {:unknown => "blah"} }
+      assert_nothing_raised { conditions.conditions = {:first_name => "blah"} }
+      assert_nothing_raised { conditions.conditions = {:first_name_contains => "blah"} }
+    end
+  
+    def test_setting_associations
+      conditions = Searchlogic::Cache::AccountConditions.new(:users => {:first_name_like => "Ben"})
+      assert_equal conditions.users.first_name_like, "Ben"
+    
+      conditions.users.last_name_begins_with = "Ben"
+      assert_equal conditions.users.last_name_begins_with, "Ben"
+    end
+  
+    def test_virtual_columns
+      conditions = Searchlogic::Cache::AccountConditions.new
+      conditions.hour_of_created_gt = 2
+      assert_equal_sql ["strftime('%H', \"accounts\".\"created_at\") > ?", 2], conditions.sanitize
+      conditions.dow_of_created_at_most = 5
+      assert_equal_sql ["(strftime('%w', \"accounts\".\"created_at\") <= ?) AND (strftime('%H', \"accounts\".\"created_at\") > ?)", 5, 2], conditions.sanitize
+      conditions.month_of_created_at_nil = true
+      assert_equal_sql ["(strftime('%w', \"accounts\".\"created_at\") <= ?) AND (strftime('%H', \"accounts\".\"created_at\") > ?) AND (strftime('%m', \"accounts\".\"created_at\") IS NULL)", 5, 2], conditions.sanitize
+      conditions.min_of_hour_of_month_of_created_at_nil = true
+      assert_equal_sql ["(strftime('%w', \"accounts\".\"created_at\") <= ?) AND (strftime('%H', \"accounts\".\"created_at\") > ?) AND (strftime('%m', strftime('%H', strftime('%M', \"accounts\".\"created_at\"))) IS NULL) AND (strftime('%m', \"accounts\".\"created_at\") IS NULL)", 5, 2], conditions.sanitize
+    end
+  
+    def test_objects
+      conditions = Searchlogic::Cache::AccountConditions.new
+      assert_equal({}, conditions.send(:objects))
+    
+      conditions.name_contains = "Binary"
+      assert_equal 1, conditions.send(:objects).size
+    
+      conditions.users.first_name_contains = "Ben"
+      assert_equal 2, conditions.send(:objects).size
+    end
+  
+    def test_reset
+      conditions = Searchlogic::Cache::AccountConditions.new
+    
+      conditions.name_contains = "Binary"
+      assert_equal 1, conditions.send(:objects).size
+    
+      conditions.reset_name_like!
+      conditions.reset_name_contains! # should set up aliases for reset
+      assert_equal({}, conditions.send(:objects))
+    
+      conditions.users.first_name_like = "Ben"
+      assert_equal 1, conditions.send(:objects).size
+    
+      conditions.reset_users!
+      assert_equal({}, conditions.send(:objects))
+    
+      conditions.name_begins_with ="Binary"
+      conditions.users.orders.total_gt = 200
+      assert_equal 2, conditions.send(:objects).size
+    
+      conditions.reset_name_begins_with!
+      assert_equal 1, conditions.send(:objects).size
+    
+      conditions.reset_users!
+      assert_equal({}, conditions.send(:objects))
+    end
+  
+    def test_method_conflicts
+      conditions = Searchlogic::Cache::AccountConditions.new
+      assert_nil conditions.id
+    end
+  end
+end

data/test/conditions_tests/protection_test.rb

@@ -0,0 +1,18 @@
+require File.dirname(__FILE__) + '/../test_helper.rb'
+
+module ConditionsTests
+  class ProtectionTest < ActiveSupport::TestCase
+    def test_protection
+      assert_raise(ArgumentError) { Account.new_search(:conditions => "(DELETE FROM users)") }
+      assert_nothing_raised { Account.new_search!(:conditions => "(DELETE FROM users)") }
+    
+      account = Account.first
+    
+      assert_raise(ArgumentError) { account.users.new_search(:conditions => "(DELETE FROM users)") }
+      assert_nothing_raised { account.users.new_search!(:conditions => "(DELETE FROM users)") }
+    
+      search = Account.new_search
+      assert_raise(ArgumentError) { search.conditions = "(DELETE FROM users)" }
+    end
+  end
+end

data/test/{test_config.rb → config_test.rb}

@@ -1,6 +1,6 @@
 require File.dirname(__FILE__) + '/test_helper.rb'
 
-class TestConfig < Test::Unit::TestCase
+class ConfigTest < ActiveSupport::TestCase
   def test_per_page
     Searchlogic::Config.search.per_page = 1
     

data/test/fixtures/accounts.yml

@@ -1,15 +1,12 @@
 binary_logic:
-  id: 1
   name: "Binary Logic"
   active: true
   
 neco:
-  id: 2
   name: "NECO"
   active: false
   
 binary_fun:
-  id: 3
   name: "Binary Fun"
   active: true
   

data/test/fixtures/animals.yml

@@ -0,0 +1,7 @@
+pepper:
+  type: Cat
+  description: pepper meows
+  
+harry:
+  type: Doc
+  description: harry is hairy

data/test/fixtures/orders.yml

@@ -1,13 +1,11 @@
 bens_order:
-  id: 1
-  user_id: 1
+  user: ben
   total: 500.23
   description: A bunch of apple products, etc.
   receipt: some binary text
   
 drews_order:
-  id: 2
-  user_id: 2
+  user: drew
   total: 2.12
   description: Some more apple projects, ipod, etc
   receipt: some more binary text

data/test/fixtures/user_groups.yml

@@ -1,13 +1,7 @@
 neco:
-  id: 1
   name: NECO
-  user_ids:
-    - 1
-    - 2
-  
+  users: ben, drew
+
 johnsons:
-  id: 2
   name: Johnsons
-  user_ids:
-    - 1
-    - 3
+  users: ben, jennifer

data/test/fixtures/users.yml

@@ -1,34 +1,28 @@
 ben:
-  id: 1
-  account_id: 1
-  parent_id: null
+  account: binary_logic
   first_name: Ben
   last_name: Johnson
   active: true
   bio: Totally awesome!
   
 drew:
-  id: 2
-  account_id: 2
-  parent_id: 1
+  account: neco
+  parent: ben
   first_name: Drew
   last_name: Mills
   active: false
   bio: Totally not awesome!
   
 jennifer:
-  id: 3
-  account_id: 1
-  parent_id: 2
+  account: binary_logic
+  parent: drew
   first_name: Jennifer
   last_name: Hopkins
   active: false
   bio: Totally not awesome at all!
   
 tren:
-  id: 4
-  account_id: 
-  parent_id: 
+  parent: ben
   first_name: Tren
   last_name: Garfield
   active: false

data/test/modifier_tests/day_of_month_test.rb

@@ -0,0 +1,16 @@
+require File.dirname(__FILE__) + '/../test_helper.rb'
+
+module ConditionsTests
+  class DayOfMonthTest < ActiveSupport::TestCase
+    def test_modifier_names
+      
+    end
+    
+    def test_usage
+      search = User.new_search
+      search.conditions.dom_of_created_at_gt = "1"
+      assert_equal({:conditions => ["(strftime('%d', \"users\".\"created_at\") * 1) > ?", 1], :limit => 25}, search.sanitize)
+      search.all
+    end
+  end
+end