rdavila_friendly_id 2.2.6

data/test/custom_slug_normalizer_test.rb

@@ -0,0 +1,36 @@
+require File.dirname(__FILE__) + '/test_helper'
+
+class CustomSlugNormalizerTest < Test::Unit::TestCase
+
+  context "A slugged model using a custom slug generator" do
+
+    setup do
+      Person.friendly_id_options = FriendlyId::DEFAULT_OPTIONS.merge(:method => :name, :use_slug => true)
+    end
+
+    teardown do
+      Person.delete_all
+      Slug.delete_all
+    end
+
+    should "invoke the block code" do
+      @person = Person.create!(:name => "Joe Schmoe")
+      assert_equal "JOE SCHMOE", @person.friendly_id
+    end
+
+    should "respect the max_length option" do
+      Person.friendly_id_options = Person.friendly_id_options.merge(:max_length => 3)
+      @person = Person.create!(:name => "Joe Schmoe")
+      assert_equal "JOE", @person.friendly_id
+    end
+
+    should "respect the reserved option" do
+      Person.friendly_id_options = Person.friendly_id_options.merge(:reserved => ["JOE"])
+      assert_raises FriendlyId::SlugGenerationError do
+        Person.create!(:name => "Joe")
+      end
+    end
+
+  end
+
+end

data/test/non_slugged_test.rb

@@ -0,0 +1,99 @@
+require File.dirname(__FILE__) + '/test_helper'
+
+class NonSluggedTest < Test::Unit::TestCase
+
+  context "A non-slugged model with default FriendlyId options" do
+
+    setup do
+      @user = User.create!(:name => "joe")
+    end
+
+    teardown do
+      User.delete_all
+    end
+
+    should "have friendly_id options" do
+      assert_not_nil User.friendly_id_options
+    end
+
+    should "not have a slug" do
+      assert !@user.respond_to?(:slug)
+    end
+
+    should "be findable by its friendly_id" do
+      assert User.find(@user.friendly_id)
+    end
+
+    should "be findable by its regular id" do
+      assert User.find(@user.id)
+    end
+
+    should "respect finder conditions" do
+      assert_raises ActiveRecord::RecordNotFound do
+        User.find(@user.friendly_id, :conditions => "1 = 2")
+      end
+    end
+
+    should "indicate if it was found by its friendly id" do
+      user = User.find(@user.friendly_id)
+      assert user.found_using_friendly_id?
+    end
+
+    should "indicate if it was found by its numeric id" do
+      user = User.find(@user.id)
+      assert user.found_using_numeric_id?
+    end
+
+    should "indicate if it has a better id" do
+      user = User.find(@user.id)
+      assert user.has_better_id?
+    end
+
+    should "not validate if the friendly_id text is reserved" do
+      user = User.new(:name => "new")
+      assert !user.valid?
+    end
+
+    should "have always string for a friendly_id" do
+      assert_equal String, @user.to_param.class
+    end
+
+    should "return its id if the friendly_id is null" do
+      @user.name = nil
+      assert_equal @user.id.to_s, @user.to_param
+    end
+
+
+    context "when using an array as the find argument" do
+
+      setup do
+        @user2 = User.create(:name => "jane")
+      end
+
+      should "return results" do
+        assert_equal 2, User.find([@user.friendly_id, @user2.friendly_id]).size
+      end
+
+      should "not allow mixed friendly and non-friendly ids for the same record" do
+        assert_raises ActiveRecord::RecordNotFound do
+          User.find([@user.id, @user.friendly_id]).size
+        end
+      end
+
+      should "raise an error when all records are not found" do
+        assert_raises ActiveRecord::RecordNotFound do
+          User.find(['bad', 'bad2'])
+        end
+      end
+
+      should "indicate if the results were found using a friendly_id" do
+        users = User.find([@user.id, @user2.friendly_id], :order => "name ASC")
+        assert users[0].found_using_friendly_id?
+        assert users[1].found_using_numeric_id?
+      end
+
+    end
+
+  end
+
+end

data/test/scoped_model_test.rb

@@ -0,0 +1,64 @@
+require File.dirname(__FILE__) + '/test_helper'
+
+
+class ScopedModelTest < Test::Unit::TestCase
+
+  context "A slugged model that uses a scope" do
+
+    setup do
+      @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)
+    end
+
+    teardown do
+      Resident.delete_all
+      Country.delete_all
+      Slug.delete_all
+    end
+
+    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
+
+    should "find all scoped records without scope" do
+      assert_equal 2, Resident.find(:all, @resident.friendly_id).size
+    end
+
+    should "find a single scoped records with a scope as a string" do
+      assert Resident.find(@resident.friendly_id, :scope => @resident.country.to_param)
+    end
+
+    should "find a single scoped records with a scope" do
+      assert Resident.find(@resident.friendly_id, :scope => @resident.country)
+    end
+
+    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
+
+    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 /expected scope/, e.message
+      end
+    end
+
+    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
+
+  end
+
+end

data/test/slug_test.rb

@@ -0,0 +1,105 @@
+# encoding: utf-8
+require File.dirname(__FILE__) + '/test_helper'
+
+class SlugTest < Test::Unit::TestCase
+
+  context "a slug" do
+
+    teardown do
+      Slug.delete_all
+      Post.delete_all
+    end
+
+    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.last.is_most_recent?
+      assert !post.slugs.first.is_most_recent?
+    end
+
+  end
+
+  context "the Slug class" do
+
+    should "parse the slug name and sequence" do
+      assert_equal ["test", "2"], Slug::parse("test--2")
+    end
+
+    should "parse with a default sequence of 1" do
+      assert_equal ["test", "1"], Slug::parse("test")
+    end
+
+    should "should strip diacritics" do
+      assert_equal "acai", Slug::strip_diacritics("açaí")
+    end
+
+    should "strip diacritics correctly " do
+      input  = "ÀÁÂÃÄÅÆÇÈÊËÌÍÎÏÐÑÒÓÔÕÖØÙÚÛÜÝÞßàáâãäåæçèéêëìíîïðñòóôõöøùúûüýþÿ"
+      output = Slug::strip_diacritics(input).split(//)
+      expected = "AAAAAAAECEEEIIIIDNOOOOOOUUUUYThssaaaaaaaeceeeeiiiidnoooooouuuuythy".split(//)
+      output.each_index do |i|
+        assert_equal expected[i], output[i]
+      end
+    end
+
+  end
+
+  context "the Slug class's to_friendly_id method" do
+
+    should "include the sequence if the sequence is greater than 1" do
+      slug = Slug.new(:name => "test", :sequence => 2)
+      assert_equal "test--2", slug.to_friendly_id
+    end
+
+    should "not include the sequence if the sequence is 1" do
+      slug = Slug.new(:name => "test", :sequence => 1)
+      assert_equal "test", slug.to_friendly_id
+    end
+
+  end
+
+  context "the Slug class's normalize method" do
+
+    should "should lowercase  strings" do
+      assert_match /abc/, Slug::normalize("ABC")
+    end
+
+    should "should replace whitespace with dashes" do
+      assert_match /a-b/, Slug::normalize("a b")
+    end
+
+    should "should replace 2spaces with 1dash" do
+      assert_match /a-b/, Slug::normalize("a  b")
+    end
+
+    should "should remove punctuation" do
+      assert_match /abc/, Slug::normalize('abc!@#$%^&*•¶§∞¢££¡¿()><?"":;][]\.,/')
+    end
+
+    should "should strip trailing space" do
+      assert_match /ab/, Slug::normalize("ab ")
+    end
+
+    should "should strip leading space" do
+      assert_match /ab/, Slug::normalize(" ab")
+    end
+
+    should "should strip trailing slashes" do
+      assert_match /ab/, Slug::normalize("ab-")
+    end
+
+    should "should strip leading slashes" do
+      assert_match /ab/, Slug::normalize("-ab")
+    end
+
+    should "should not modify valid name strings" do
+      assert_match /a-b-c-d/, Slug::normalize("a-b-c-d")
+    end
+
+    should "work with non roman chars" do
+      assert_equal "検-索", Slug::normalize("検 索")
+    end
+
+  end
+end

data/test/slugged_model_test.rb

@@ -0,0 +1,348 @@
+# encoding: utf-8
+require File.dirname(__FILE__) + '/test_helper'
+
+class SluggedModelTest < Test::Unit::TestCase
+
+  context "A slugged model with default FriendlyId options" do
+
+    setup do
+      Post.friendly_id_options = FriendlyId::DEFAULT_OPTIONS.merge(:method => :name, :use_slug => true)
+      @post = Post.new :name => "Test post", :published => true
+      @post.save!
+    end
+
+    teardown do
+      Post.delete_all
+      Person.delete_all
+      Place.delete_all
+      Slug.delete_all
+    end
+
+    should "have friendly_id options" do
+      assert_not_nil Post.friendly_id_options
+    end
+
+    should "have a slug" do
+      assert_not_nil @post.slug
+    end
+
+    should "be findable by its friendly_id" do
+      assert Post.find(@post.friendly_id)
+    end
+
+    should "be findable by its regular id" do
+      assert Post.find(@post.id)
+    end
+
+    should "be findable by its regular id as a string" do
+      assert Post.find(@post.id.to_s)
+    end
+
+    should "be findable by its instance" do
+      assert Post.find(@post)
+    end
+
+    should "not be findable by its id if looking for something else" do
+      assert_raises ActiveRecord::RecordNotFound do
+        Post.find("#{@post.id}-i-dont-exists")
+      end
+    end
+
+    should "generate slug text" do
+      post = Post.new :name => "Test post"
+      assert_not_nil post.slug_text
+    end
+
+    should "respect finder conditions" do
+      assert_raises ActiveRecord::RecordNotFound do
+        Post.find(@post.friendly_id, :conditions => "1 = 2")
+      end
+    end
+
+    should "raise an error if the friendly_id text is reserved" do
+      assert_raises(FriendlyId::SlugGenerationError) do
+        Post.create!(:name => "new")
+      end
+    end
+
+    should "raise an error if the friendly_id text is an empty string" do
+      assert_raises(FriendlyId::SlugGenerationError) do
+        Post.create(:name => "")
+      end
+    end
+
+    should "raise an error if the friendly_id text is nil" do
+      assert_raises(FriendlyId::SlugGenerationError) do
+        Post.create(:name => nil)
+      end
+    end
+
+    should "raise an error if the normalized friendly id becomes blank" do
+      assert_raises(FriendlyId::SlugGenerationError) do
+        post = Post.create!(:name => "-.-")
+      end
+    end
+
+    should "not make a new slug unless the friendly_id method value has changed" do
+      @post.published = !@post.published
+      @post.save!
+      assert_equal 1, @post.slugs.size
+    end
+
+    should "make a new slug if the friendly_id method value has changed" do
+      @post.name = "Changed title"
+      @post.save!
+      assert_equal 2, @post.slugs.size
+    end
+
+    should "have a slug sequence of 1 by default" do
+      assert_equal 1, @post.slug.sequence
+    end
+
+    should "increment sequence for duplicate slug names" do
+      @post2 = Post.create! :name => @post.name
+      assert_equal 2, @post2.slug.sequence
+    end
+
+    should "have a friendly_id that terminates with -- and the slug sequence if the sequence is greater than 1" do
+      @post2 = Post.create! :name => @post.name
+      assert_match(/--2\z/, @post2.friendly_id)
+    end
+
+    should "allow datetime columns to be used as slugs" do
+      assert Event.create(:name => "Test", :event_date => DateTime.now)
+    end
+
+    should "not strip diacritics" do
+      post = Post.new(:name => "¡Feliz año!")
+      assert_match(/#{'ñ'}/, post.slug_text)
+    end
+
+    should "not convert to ASCII" do
+      post = Post.new(:name => "katakana: ゲコゴサザシジ")
+      assert_equal "katakana-ゲコゴサザシジ", post.slug_text
+    end
+
+    should "allow the same friendly_id across models" do
+      district = District.create!(:name => @post.name)
+      assert_equal district.friendly_id, @post.friendly_id
+    end
+
+    should "truncate slug text longer than the max length" do
+      post = Post.new(:name => "a" * (Post.friendly_id_options[:max_length] + 1))
+      assert_equal post.slug_text.length, Post.friendly_id_options[:max_length]
+    end
+
+    should "truncate slug in 'right way' when slug is unicode" do
+      post = Post.new(:name => "ё" * 100 + 'ю' *(Post.friendly_id_options[:max_length] - 100 + 1))
+      assert_equal post.slug_text.mb_chars[-1], 'ю'
+    end
+
+    should "be able to reuse an old friendly_id without incrementing the sequence" do
+      old_title = @post.name
+      old_friendly_id = @post.friendly_id
+      @post.name = "A changed title"
+      @post.save!
+      @post.name = old_title
+      @post.save!
+      assert_equal old_friendly_id, @post.friendly_id
+    end
+
+    should "allow eager loading of slugs" do
+      assert_nothing_raised do
+        Post.find(@post.friendly_id, :include => :slugs)
+      end
+    end
+
+    # This emulates a fairly common issue where id's generated by fixtures are very high.
+    should "continue to admit very large ids" do
+      Person.connection.execute("INSERT INTO people (id, name) VALUES (2147483647, 'Joe Schmoe')")
+      assert Person.find(2147483647)
+    end
+
+    context "and configured to strip diacritics" do
+      setup do
+        Post.friendly_id_options = Post.friendly_id_options.merge(:strip_diacritics => true)
+      end
+
+      should "strip diacritics from Roman alphabet based characters" do
+        post = Post.new(:name => "¡Feliz año!")
+        assert_no_match(/#{'ñ'}/, post.slug_text)
+      end
+
+      should "raise an error if the friendly_id text is an empty string" do
+        assert_raises(FriendlyId::SlugGenerationError) do
+          Post.create(:name => "")
+        end
+      end
+
+      should "raise an error if the friendly_id text is nil" do
+        assert_raises(FriendlyId::SlugGenerationError) do
+          Post.create(:name => nil)
+        end
+      end
+
+    end
+
+    context "and configured to convert to ASCII" do
+      setup do
+        Post.friendly_id_options = Post.friendly_id_options.merge(:strip_non_ascii => true)
+      end
+
+      should "strip non-ascii characters" do
+        post = Post.new(:name => "katakana: ゲコゴサザシジ")
+        assert_equal "katakana", post.slug_text
+      end
+    end
+
+    context "that uses a custom table name" do
+      should "support normal CRUD operations" do
+        assert thing = Place.create!(:name => "a name")
+        thing.name = "a new name"
+        assert thing.save!
+        assert thing.destroy
+      end
+    end
+
+    context "that doesn't have a slug" do
+
+      setup do
+        @post.slug.destroy
+        @post = Post.find(@post.id)
+      end
+
+      should "have a to_param method that returns the id cast to a string" do
+        assert_equal @post.id.to_s, @post.to_param
+      end
+
+    end
+
+    context "when found using its friendly_id" do
+      setup do
+        @post = Post.find(@post.friendly_id)
+      end
+
+      should "indicate that it was found using the friendly_id" do
+        assert @post.found_using_friendly_id?
+      end
+
+      should "not indicate that it has a better id" do
+        assert !@post.has_better_id?
+      end
+
+      should "not indicate that it was found using its numeric id" do
+        assert !@post.found_using_numeric_id?
+      end
+
+      should "have a finder slug" do
+        assert_not_nil @post.finder_slug
+      end
+
+    end
+
+    context "when found using its regular id" do
+      setup do
+        @post = Post.find(@post.id)
+      end
+
+      should "indicate that it was not found using the friendly id" do
+        assert !@post.found_using_friendly_id?
+      end
+
+      should "indicate that it has a better id" do
+        assert @post.has_better_id?
+      end
+
+      should "indicate that it was found using its numeric id" do
+        assert @post.found_using_numeric_id?
+      end
+
+      should "not have a finder slug" do
+        assert_nil @post.finder_slug
+      end
+
+    end
+
+    context "when found using an outdated friendly id" do
+      setup do
+        old_id = @post.friendly_id
+        @post.name = "Title changed"
+        @post.save!
+        @post = Post.find(old_id)
+      end
+
+      should "indicate that it was found using a friendly_id" do
+        assert @post.found_using_friendly_id?
+      end
+
+      should "indicate that it has a better id" do
+        assert @post.has_better_id?
+      end
+
+      should "not indicate that it was found using its numeric id" do
+        assert !@post.found_using_numeric_id?
+      end
+
+      should "should have a finder slug different from its default slug" do
+        assert_not_equal @post.slug, @post.finder_slug
+      end
+
+    end
+
+    context "when table does not exist" do
+      should "not raise an error when doing friendly_id setup" do
+        assert_nothing_raised do
+          Question.has_friendly_id :title, :use_slug => true
+        end
+      end
+    end
+
+    context "when using an array as the find argument" do
+
+      setup do
+        @post2 = Post.create!(:name => "another post", :published => true)
+      end
+
+      should "return results when passed an array of non-friendly ids" do
+        assert_equal 2, Post.find([@post.id, @post2.id]).size
+      end
+
+      should "return results when passed an array of friendly ids" do
+        assert_equal 2, Post.find([@post.friendly_id, @post2.friendly_id]).size
+      end
+
+      should "return results when searching using a named scope" do
+        assert_equal 2, Post.published.find([@post.id, @post2.id]).size
+      end
+
+      should "return results when passed a mixed array of friendly and non-friendly ids" do
+        assert_equal 2, Post.find([@post.friendly_id, @post2.id]).size
+      end
+
+      should "return results when passed an array of non-friendly ids, of which one represents a record with multiple slugs" do
+        @post2.update_attributes(:name => 'another post [updated]')
+        assert_equal 2, Post.find([@post.id, @post2.id]).size
+      end
+
+      should "indicate that the results were found using a friendly_id" do
+        @posts = Post.find [@post.friendly_id, @post2.friendly_id]
+        @posts.each { |p| assert p.found_using_friendly_id? }
+      end
+
+      should "raise an error when all records are not found" do
+        assert_raises(ActiveRecord::RecordNotFound) do
+          Post.find([@post.friendly_id, 'non-existant-slug-record'])
+        end
+      end
+
+      should "allow eager loading of slugs" do
+        assert_nothing_raised do
+          Post.find([@post.friendly_id, @post2.friendly_id], :include => :slugs)
+        end
+      end
+
+    end
+
+  end
+
+end