cmassimo-friendly_id 3.0.4.2

data/test/active_record_adapter/basic_slugged_model_test.rb

@@ -0,0 +1,14 @@
+require File.expand_path('../ar_test_helper', __FILE__)
+
+module FriendlyId
+  module Test
+    module ActiveRecordAdapter
+      class BasicSluggedModelTest < ::Test::Unit::TestCase
+        include FriendlyId::Test::Generic
+        include FriendlyId::Test::Slugged
+        include FriendlyId::Test::ActiveRecordAdapter::Slugged
+        include FriendlyId::Test::ActiveRecordAdapter::Core
+      end
+    end
+  end
+end

data/test/active_record_adapter/cached_slug_test.rb

@@ -0,0 +1,61 @@
+require File.expand_path("../ar_test_helper", __FILE__)
+
+module FriendlyId
+  module Test
+    module ActiveRecordAdapter
+
+      class CachedSlugTest < ::Test::Unit::TestCase
+
+        include FriendlyId::Test::Generic
+        include FriendlyId::Test::Slugged
+        include FriendlyId::Test::ActiveRecordAdapter::Slugged
+        include FriendlyId::Test::ActiveRecordAdapter::Core
+
+        def klass
+          District
+        end
+
+        def other_class
+          Post
+        end
+
+        def cached_slug
+          instance.send(cache_column)
+        end
+
+        def cache_column
+          klass.friendly_id_config.cache_column
+        end
+
+        test "should have a cached_slug" do
+          assert_equal cached_slug, instance.slug.to_friendly_id
+        end
+
+        test "should protect the cached slug value" do
+          old_value = cached_slug
+          instance.update_attributes(cache_column => "Madrid")
+          instance.reload
+          assert_equal old_value, cached_slug
+        end
+
+        test "should update the cached slug when updating the slug" do
+          instance.update_attributes(:name => "new name")
+          assert_equal instance.slug.to_friendly_id, cached_slug
+        end
+
+        test "should not update the cached slug column if it has not changed" do
+          instance.note = "a note"
+          instance.expects("#{cache_column}=".to_sym).never
+          instance.save!
+        end
+
+        test "should cache the incremented sequence for duplicate slug names" do
+          instance_2 = klass.create!(:name => instance.name)
+          assert_match(/2\z/, instance_2.send(cache_column))
+        end
+
+      end
+    end
+  end
+end
+

data/test/active_record_adapter/core.rb

@@ -0,0 +1,98 @@
+require File.expand_path("../ar_test_helper", __FILE__)
+
+module FriendlyId
+
+  module Test
+
+    module ActiveRecordAdapter
+
+      module Core
+
+        def teardown
+          klass.delete_all
+          other_class.delete_all
+          Slug.delete_all
+        end
+
+        def find_method
+          :find
+        end
+
+        def create_method
+          :create!
+        end
+
+        def delete_all_method
+          :delete_all
+        end
+
+        def save_method
+          :save!
+        end
+
+        def validation_exceptions
+          [ActiveRecord::RecordInvalid, FriendlyId::ReservedError, FriendlyId::BlankError]
+        end
+
+        test "should return their friendly_id for #to_param" do
+          assert_match(instance.friendly_id, instance.to_param)
+        end
+
+        test "instances should be findable by their own instance" do
+          assert_equal instance, klass.find(instance)
+        end
+
+        test "instances should be findable by an array of friendly_ids" do
+          second = klass.create!(:name => "second_instance")
+          third = klass.create!(:name => "third_instance")
+          assert_equal 2, klass.find([instance.friendly_id, second.friendly_id]).size
+        end
+
+        test "instances should be findable by an array of numeric ids" do
+          second = klass.create!(:name => "second_instance")
+          third = klass.create!(:name => "third_instance")
+          assert_equal 2, klass.find([instance.id.to_i, second.id.to_i]).size
+        end
+
+        test "instances should be findable by an array of numeric ids as strings" do
+          second = klass.create!(:name => "second_instance")
+          third = klass.create!(:name => "third_instance")
+          assert_equal 2, klass.find([instance.id.to_s, second.id.to_s]).size
+        end
+
+        test "instances should be findable by an array of instances" do
+          second = klass.create!(:name => "second_instance")
+          third = klass.create!(:name => "third_instance")
+          assert_equal 2, klass.find([instance, second]).size
+        end
+
+        test "instances should be findable by an array of mixed types" do
+          second = klass.create!(:name => "second_instance")
+          third = klass.create!(:name => "third_instance")
+          assert_equal 2, klass.find([instance.friendly_id, second]).size
+        end
+
+        test "models should raise an error when not all records are found" do
+          assert_raises(ActiveRecord::RecordNotFound) do
+            klass.find([instance.friendly_id, 'bad-friendly-id'])
+          end
+        end
+
+        test "models should respect finder conditions" do
+          assert_raise ActiveRecord::RecordNotFound do
+            klass.find(instance.friendly_id, :conditions => "1 = 2")
+          end
+        end
+
+        # This emulates a fairly common issue where id's generated by fixtures are very high.
+        test "should continue to admit very large ids" do
+          klass.connection.execute("INSERT INTO #{klass.table_name} (id, name) VALUES (2047483647, 'An instance')")
+          assert_nothing_raised do
+            klass.base_class.find(2047483647)
+          end
+        end
+
+      end
+    end
+  end
+end

data/test/active_record_adapter/custom_normalizer_test.rb

@@ -0,0 +1,20 @@
+require File.expand_path("../ar_test_helper", __FILE__)
+
+module FriendlyId
+  module Test
+    module ActiveRecordAdapter
+
+      class CustomNormalizerTest < ::Test::Unit::TestCase
+
+        include FriendlyId::Test::ActiveRecordAdapter::Core
+        include FriendlyId::Test::ActiveRecordAdapter::Slugged
+        include FriendlyId::Test::CustomNormalizer
+
+        def klass
+          Person
+        end
+
+      end
+    end
+  end
+end

data/test/active_record_adapter/custom_table_name_test.rb

@@ -0,0 +1,22 @@
+require File.expand_path("../ar_test_helper", __FILE__)
+
+module FriendlyId
+  module Test
+    module ActiveRecordAdapter
+
+      class CustomTableNameTest < ::Test::Unit::TestCase
+
+        include FriendlyId::Test::Generic
+        include FriendlyId::Test::Slugged
+        include FriendlyId::Test::ActiveRecordAdapter::Slugged
+        include FriendlyId::Test::ActiveRecordAdapter::Core
+
+        def klass
+          Place
+        end
+
+      end
+
+    end
+  end
+end

data/test/active_record_adapter/scoped_model_test.rb

@@ -0,0 +1,118 @@
+require File.expand_path("../ar_test_helper", __FILE__)
+
+module FriendlyId
+  module Test
+
+    class ScopedModelTest < ::Test::Unit::TestCase
+
+      include FriendlyId::Test::Generic
+      include FriendlyId::Test::Slugged
+      include FriendlyId::Test::ActiveRecordAdapter::Slugged
+      include FriendlyId::Test::ActiveRecordAdapter::Core
+
+      def setup
+        @user = User.create!(:name => "john")
+        @house = House.create!(:name => "123 Main", :user => @user)
+        @usa = Country.create!(:name => "USA")
+        @canada = Country.create!(:name => "Canada")
+        @resident = Resident.create!(:name => "John Smith", :country => @usa)
+        @resident2 = Resident.create!(:name => "John Smith", :country => @canada)
+        @owner = Company.create!(:name => "Acme Events")
+        @site = Site.create!(:name => "Downtown Venue", :owner => @owner)
+      end
+
+      def teardown
+        Resident.delete_all
+        Country.delete_all
+        User.delete_all
+        House.delete_all
+        Slug.delete_all
+      end
+
+      test "a slugged model should auto-detect that it is being used as a parent scope" do
+        assert_equal [Resident], Country.friendly_id_config.child_scopes
+      end
+
+      test "a slugged model should update its child model's scopes when its friendly_id changes" do
+        @usa.update_attributes(:name => "United States")
+        assert_equal "united-states", @usa.to_param
+        assert_equal "united-states", @resident.slugs(true).first.scope
+      end
+
+      test "a non-slugged model should auto-detect that it is being used as a parent scope" do
+        assert_equal [House], User.friendly_id_config.child_scopes
+      end
+
+      test "should update the slug when the scope changes" do
+        @resident.update_attributes! :country => Country.create!(:name => "Argentina")
+        assert_equal "argentina", @resident.slugs(true).first.scope
+      end
+
+      test "updating only the scope should not append sequence to friendly_id" do
+        old_friendly_id = @resident.friendly_id
+        @resident.update_attributes! :country => Country.create!(:name => "Argentina")
+        assert_equal old_friendly_id, @resident.friendly_id
+      end
+
+      test "updating the scope should increment sequence to avoid conflicts" do
+        old_friendly_id = @resident.friendly_id
+        @resident.update_attributes! :country => @canada
+        assert_equal "#{old_friendly_id}--2", @resident.friendly_id
+        assert_equal "canada", @resident.slugs(true).first.scope
+      end
+
+      test "a non-slugged model should update its child model's scopes when its friendly_id changes" do
+        @user.update_attributes(:name => "jack")
+        assert_equal "jack", @user.to_param
+        assert_equal "jack", @house.slugs(true).first.scope
+      end
+
+      test "should should not show the scope in the friendly_id" do
+        assert_equal "john-smith", @resident.friendly_id
+        assert_equal "john-smith", @resident2.friendly_id
+      end
+
+      test "should find all scoped records without scope" do
+        assert_equal 2, Resident.find(:all, @resident.friendly_id).size
+      end
+
+      test "should find a single scoped record with a scope as a string" do
+        assert Resident.find(@resident.friendly_id, :scope => @resident.country)
+      end
+
+      test "should find a single scoped record with a scope" do
+        assert Resident.find(@resident.friendly_id, :scope => @resident.country)
+      end
+
+      test "should raise an error when finding a single scoped record with no scope" do
+        assert_raises ActiveRecord::RecordNotFound do
+          Resident.find(@resident.friendly_id)
+        end
+      end
+
+      test "should append scope error info when missing scope causes a find to fail" do
+        begin
+          Resident.find(@resident.friendly_id)
+          fail "The find should not have succeeded"
+        rescue ActiveRecord::RecordNotFound => e
+          assert_match(/scope: expected/, e.message)
+        end
+      end
+
+      test "should append scope error info when the scope value causes a find to fail" do
+        begin
+          Resident.find(@resident.friendly_id, :scope => "badscope")
+          fail "The find should not have succeeded"
+        rescue ActiveRecord::RecordNotFound => e
+          assert_match(/scope: badscope/, e.message)
+        end
+      end
+      
+      test "should update the sluggable field when a polymorphic relationship exists" do
+        @site.update_attributes(:name => "Uptown Venue")
+        assert_equal "Uptown Venue", @site.name
+      end
+
+    end
+  end
+end

data/test/active_record_adapter/simple_test.rb

@@ -0,0 +1,76 @@
+require File.expand_path('../ar_test_helper', __FILE__)
+
+module FriendlyId
+  module Test
+    module ActiveRecordAdapter
+      module Simple
+
+        module SimpleTest
+          def klass
+            @klass ||= User
+          end
+
+          def instance
+            @instance ||= User.create! :name => "hello world"
+          end
+
+          def other_class
+            Author
+          end
+        end
+
+        class StatusTest < ::Test::Unit::TestCase
+
+          include SimpleTest
+
+          def setup
+            User.delete_all
+          end
+
+          test "should default to not friendly" do
+            assert !status.friendly?
+          end
+
+          test "should default to numeric" do
+            assert status.numeric?
+          end
+
+          test "should be friendly if name is set" do
+            status.name = "name"
+            assert status.friendly?
+          end
+
+          test "should be best if it is numeric, but record has no friendly_id" do
+            instance.send("#{klass.friendly_id_config.column}=", nil)
+            assert status.best?
+          end
+
+          def status
+            @status ||= instance.friendly_id_status
+          end
+
+        end
+
+        class BasicTest < ::Test::Unit::TestCase
+          include FriendlyId::Test::Generic
+          include FriendlyId::Test::Simple
+          include FriendlyId::Test::ActiveRecordAdapter::Core
+          include SimpleTest
+
+          test "status should be friendly when found using friendly id" do
+            record = klass.send(find_method, instance.friendly_id)
+            assert record.friendly_id_status.friendly?
+          end
+
+          test "status should not be friendly when found using numeric id" do
+            record = klass.send(find_method, instance.id)
+            assert !record.friendly_id_status.friendly?
+          end
+
+        end
+
+      end
+    end
+  end
+end
+

data/test/active_record_adapter/slug_test.rb

@@ -0,0 +1,34 @@
+# encoding: utf-8
+require File.expand_path("../ar_test_helper", __FILE__)
+
+module FriendlyId
+  module Test
+
+    class SlugTest < ::Test::Unit::TestCase
+
+      def teardown
+        Slug.delete_all
+        Post.delete_all
+      end
+
+      test "should indicate if it is the most recent slug" do
+        post = Post.create!(:name => "test title")
+        post.name = "a new title"
+        post.save!
+        assert post.slugs.first.current?
+        assert !post.slugs.last.current?
+      end
+
+      test "should include the sequence if the sequence is greater than 1" do
+        slug = Slug.new(:name => "test", :sluggable => Post.new, :sequence => 2)
+        assert_equal "test--2", slug.to_friendly_id
+      end
+
+      test "should not include the sequence if the sequence is 1" do
+        slug = Slug.new(:name => "test",  :sluggable => Post.new, :sequence => 1)
+        assert_equal "test", slug.to_friendly_id
+      end
+
+    end
+  end
+end