friendly_id 3.3.3.0 → 4.0.0.beta7

data/test/active_record_adapter/core.rb

@@ -1,138 +0,0 @@
-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 unfriendly_class
-          Unfriendly
-        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 "failing finds with array of unfriendly_id should raise errors normally" do
-          assert_raise ActiveRecord::RecordNotFound do
-            klass.find([0, -1])
-          end
-        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 "should not raise error when finding with empty array" do
-          assert_nothing_raised do
-            klass.find []
-          end
-        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
-
-        test "should not change failure behavior for models not using friendly_id" do
-          assert_raise ActiveRecord::RecordNotFound do
-            unfriendly_class.find(-1)
-          end
-        end
-
-        test "failing finds with unfriendly_id should raise errors normally" do
-          assert_raise ActiveRecord::RecordNotFound do
-            klass.send(find_method, 0)
-          end
-        end
-
-        test "instances found by a single id should not be read-only" do
-          i = klass.find(instance.friendly_id)
-          assert !i.readonly?, "expected instance not to be readonly"
-        end
-
-        test "instances found by an array of ids should not be read-only" do
-          second = klass.create!(:name => "second_instance")
-          third = klass.create!(:name => "third_instance")
-          klass.find([instance.friendly_id, second.friendly_id]).each do |record|
-            assert !record.readonly?, "expected instance not to be readonly"
-          end
-        end
-      end
-    end
-  end
-end

data/test/active_record_adapter/custom_normalizer_test.rb

@@ -1,20 +0,0 @@
-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

@@ -1,22 +0,0 @@
-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/default_scope_test.rb

@@ -1,30 +0,0 @@
-require File.expand_path('../ar_test_helper', __FILE__)
-
-ActiveRecord::Migration.create_table :articles do |t|
-  t.string :name
-  t.string :status
-end
-
-class Article < ActiveRecord::Base
-  has_friendly_id :name, :use_slug => true
-  default_scope :conditions => "articles.status = 'published'"
-end
-
-module FriendlyId
-  module Test
-    module ActiveRecordAdapter
-      class DefaultScopeTest < ::Test::Unit::TestCase
-
-        def setup
-          Article.delete_all
-          Slug.delete_all
-        end
-
-        test "slug should load sluggable without default scope" do
-          Article.create!(:name => "hello world", :status => "draft")
-          assert_not_nil Slug.first.sluggable
-        end
-      end
-    end
-  end
-end

data/test/active_record_adapter/optimistic_locking_test.rb

@@ -1,18 +0,0 @@
-require File.expand_path("../ar_test_helper", __FILE__)
-
-module FriendlyId
-  module Test
-    module ActiveRecordAdapter
-      class OptimisticLockingTest < ::Test::Unit::TestCase
-        test "should update the cached slug when updating the slug" do
-          region = Region.create! :name => 'some name'
-          assert_nothing_raised do
-            region.update_attributes(:name => "new name")
-          end
-          assert_equal region.slug.to_friendly_id, region.cached_slug
-        end
-      end
-    end
-  end
-end
-

data/test/active_record_adapter/scoped_model_test.rb

@@ -1,129 +0,0 @@
-require File.expand_path("../ar_test_helper", __FILE__)
-
-module FriendlyId
-  module Test
-
-    class ScopedModelTest < ::Test::Unit::TestCase
-
-      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)
-        @resident3 = Resident.create!(:name => "Jim Beam", :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
-        Tourist.delete_all
-      end
-
-      test "should not use cached slug column with scopes" do
-        @tourist  = Tourist.create!(:name => "John Smith", :country => @usa)
-        @tourist2 = Tourist.create!(:name => "John Smith", :country => @canada)
-        assert_equal @canada, @canada.residents.find(@tourist2.friendly_id).country
-      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 "updating the scope should increment sequence correctly when there's more than one
-            existing slug in scope" do
-        resident4 = Resident.create!(:name => "John Smith", :country => @canada)
-        old_friendly_id = @resident.friendly_id
-        @resident.update_attributes! :country => @canada
-        assert_equal "#{old_friendly_id}--3", @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
-        name, sequence = @resident.friendly_id.parse_friendly_id
-        assert_equal 2, Resident.find(:all, :joins => :slugs, :conditions => {
-            :slugs => {:name => name, :sequence => sequence}}).size
-      end
-
-      test "should find a scoped record by friendly_id" do
-        assert Resident.find(@resident.friendly_id)
-      end
-
-      test "should find a scope record as a relation member" do
-        assert_equal @resident, @usa.residents.find("john-smith")
-        assert_equal @resident2, @canada.residents.find("john-smith")
-      end
-
-      # test "scope records found as a relation member should be 'best'" do
-      #   assert_equal @resident, @usa.residents.find("john-smith")
-      #   assert @resident.friendly_id_status.best?
-      #   assert_equal @resident2, @canada.residents.find("john-smith")
-      #   assert @resident2.friendly_id_status.best?
-      # end
-
-      test "should find a single scoped record using slug conditions" do
-        assert_equal @resident, Resident.find(@resident.friendly_id, :include => :slugs,
-          :conditions => {:slugs => {:scope => @resident.country.to_param}})
-      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
-
-      test "should not assume that AR's reflect_on_all_associations with return AR classes" do
-        reflections = Resident.reflect_on_all_associations
-        reflections << Struct.new("Dummy", :options, :klass).new(:options => [], :klass => Struct)
-        Resident.expects(:reflect_on_all_associations).returns(reflections)
-        assert_nothing_raised do
-          Resident.friendly_id_config.send(:associated_friendly_classes)
-        end
-      end
-    end
-  end
-end

data/test/active_record_adapter/simple_test.rb

@@ -1,76 +0,0 @@
-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

@@ -1,34 +0,0 @@
-# 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