acts-as-taggable-on 2.0.6 → 2.3.0

data/spec/acts_as_taggable_on/acts_as_taggable_on_spec.rb

@@ -8,11 +8,25 @@ describe "Acts As Taggable On" do
   it "should provide a class method 'taggable?' that is false for untaggable models" do
     UntaggableModel.should_not be_taggable
   end
+  
+  describe "Taggable Method Generation To Preserve Order" do
+    before(:each) do
+      clean_database!
+      TaggableModel.tag_types = []
+      TaggableModel.preserve_tag_order = false
+      TaggableModel.acts_as_ordered_taggable_on(:ordered_tags)
+      @taggable = TaggableModel.new(:name => "Bob Jones")
+    end
+
+    it "should respond 'true' to preserve_tag_order?" do
+      @taggable.class.preserve_tag_order?.should be_true
+    end
+  end
 
   describe "Taggable Method Generation" do
     before(:each) do
       clean_database!
-      TaggableModel.write_inheritable_attribute(:tag_types, [])
+      TaggableModel.tag_types = []
       TaggableModel.acts_as_taggable_on(:tags, :languages, :skills, :needs, :offerings)
       @taggable = TaggableModel.new(:name => "Bob Jones")
     end
@@ -28,10 +42,22 @@ describe "Acts As Taggable On" do
     it "should create an instance attribute for tag types" do
       @taggable.should respond_to(:tag_types)
     end
-    
+
     it "should have all tag types" do
       @taggable.tag_types.should == [:tags, :languages, :skills, :needs, :offerings]
     end
+    
+    it "should create a class attribute for preserve tag order" do
+      @taggable.class.should respond_to(:preserve_tag_order?)
+    end
+
+    it "should create an instance attribute for preserve tag order" do
+      @taggable.should respond_to(:preserve_tag_order?)
+    end
+    
+    it "should respond 'false' to preserve_tag_order?" do
+      @taggable.class.preserve_tag_order?.should be_false
+    end
 
     it "should generate an association for each tag type" do
       @taggable.should respond_to(:tags, :skills, :languages)
@@ -45,7 +71,7 @@ describe "Acts As Taggable On" do
       @taggable.should respond_to(:tag_list, :skill_list, :language_list)
       @taggable.should respond_to(:tag_list=, :skill_list=, :language_list=)
     end
-    
+
     it "should generate a tag_list accessor, that includes owned tags, for each tag type" do
       @taggable.should respond_to(:all_tags_list, :all_skills_list, :all_languages_list)
     end
@@ -57,17 +83,17 @@ describe "Acts As Taggable On" do
       @inherited_same = InheritingTaggableModel.new(:name => "inherited same")
       @inherited_different = AlteredInheritingTaggableModel.new(:name => "inherited different")
     end
-  
+
     it "should pass on tag contexts to STI-inherited models" do
       @inherited_same.should respond_to(:tag_list, :skill_list, :language_list)
       @inherited_different.should respond_to(:tag_list, :skill_list, :language_list)
     end
-  
+
     it "should have tag contexts added in altered STI models" do
       @inherited_different.should respond_to(:part_list)
     end
   end
-  
+
   describe "Reloading" do
     it "should save a model instantiated by Model.find" do
       taggable = TaggableModel.create!(:name => "Taggable")
@@ -81,51 +107,121 @@ describe "Acts As Taggable On" do
       taggable1 = TaggableModel.create!(:name => "Taggable 1")
       taggable2 = TaggableModel.create!(:name => "Taggable 2")
       taggable3 = TaggableModel.create!(:name => "Taggable 3")
-  
+
       taggable1.tag_list = "one, two"
       taggable1.save
-  
+
       taggable2.tag_list = "three, four"
       taggable2.save
-  
+
       taggable3.tag_list = "one, four"
       taggable3.save
-  
+
       taggable1.find_related_tags.should include(taggable3)
       taggable1.find_related_tags.should_not include(taggable2)
     end
-  
+
+    it "should find related objects based on tag names on context - non standard id" do
+      taggable1 = NonStandardIdTaggableModel.create!(:name => "Taggable 1")
+      taggable2 = NonStandardIdTaggableModel.create!(:name => "Taggable 2")
+      taggable3 = NonStandardIdTaggableModel.create!(:name => "Taggable 3")
+
+      taggable1.tag_list = "one, two"
+      taggable1.save
+
+      taggable2.tag_list = "three, four"
+      taggable2.save
+
+      taggable3.tag_list = "one, four"
+      taggable3.save
+
+      taggable1.find_related_tags.should include(taggable3)
+      taggable1.find_related_tags.should_not include(taggable2)
+    end
+
     it "should find other related objects based on tag names on context" do
       taggable1 = TaggableModel.create!(:name => "Taggable 1")
       taggable2 = OtherTaggableModel.create!(:name => "Taggable 2")
       taggable3 = OtherTaggableModel.create!(:name => "Taggable 3")
-  
+
       taggable1.tag_list = "one, two"
       taggable1.save
-  
+
       taggable2.tag_list = "three, four"
       taggable2.save
-  
+
       taggable3.tag_list = "one, four"
       taggable3.save
-  
+
       taggable1.find_related_tags_for(OtherTaggableModel).should include(taggable3)
       taggable1.find_related_tags_for(OtherTaggableModel).should_not include(taggable2)
     end
-  
+
     it "should not include the object itself in the list of related objects" do
       taggable1 = TaggableModel.create!(:name => "Taggable 1")
       taggable2 = TaggableModel.create!(:name => "Taggable 2")
-  
+
       taggable1.tag_list = "one"
       taggable1.save
-  
+
       taggable2.tag_list = "one, two"
       taggable2.save
-  
+
       taggable1.find_related_tags.should include(taggable2)
       taggable1.find_related_tags.should_not include(taggable1)
     end
+
+    it "should not include the object itself in the list of related objects - non standard id" do
+      taggable1 = NonStandardIdTaggableModel.create!(:name => "Taggable 1")
+      taggable2 = NonStandardIdTaggableModel.create!(:name => "Taggable 2")
+
+      taggable1.tag_list = "one"
+      taggable1.save
+
+      taggable2.tag_list = "one, two"
+      taggable2.save
+
+      taggable1.find_related_tags.should include(taggable2)
+      taggable1.find_related_tags.should_not include(taggable1)
+    end
+
+    context "Inherited Models" do
+      before do
+        @taggable1 = InheritingTaggableModel.create!(:name => "InheritingTaggable 1")
+        @taggable2 = InheritingTaggableModel.create!(:name => "InheritingTaggable 2")
+        @taggable3 = InheritingTaggableModel.create!(:name => "InheritingTaggable 3")
+        @taggable4 = TaggableModel.create!(:name => "Taggable 4")
+
+        @taggable1.tag_list = "one, two"
+        @taggable1.save
+
+        @taggable2.tag_list = "three, four"
+        @taggable2.save
+
+        @taggable3.tag_list = "one, four"
+        @taggable3.save
+
+        @taggable4.tag_list = "one, two, three, four"
+        @taggable4.save
+      end
+
+      it "should find related objects based on tag names on context" do
+        @taggable1.find_related_tags.should include(@taggable3)
+        @taggable1.find_related_tags.should_not include(@taggable2)
+        @taggable1.find_related_tags.should_not include(@taggable4)
+
+        @taggable1.find_related_tags_for(TaggableModel).should include(@taggable3)
+        @taggable1.find_related_tags_for(TaggableModel).should_not include(@taggable2)
+        @taggable1.find_related_tags_for(TaggableModel).should include(@taggable4)
+      end
+
+      it "should not include the object itself in the list of related objects" do
+        @taggable1.find_related_tags.should_not include(@taggable1)
+        @taggable1.find_related_tags_for(InheritingTaggableModel).should_not include(@taggable1)
+        @taggable1.find_related_tags_for(TaggableModel).should_not include(@taggable1)
+      end
+    end
+
   end
 
   describe "Matching Contexts" do
@@ -133,50 +229,95 @@ describe "Acts As Taggable On" do
       taggable1 = TaggableModel.create!(:name => "Taggable 1")
       taggable2 = TaggableModel.create!(:name => "Taggable 2")
       taggable3 = TaggableModel.create!(:name => "Taggable 3")
-  
+
       taggable1.offering_list = "one, two"
       taggable1.save!
-  
+
       taggable2.need_list = "one, two"
       taggable2.save!
-  
+
       taggable3.offering_list = "one, two"
       taggable3.save!
-  
+
       taggable1.find_matching_contexts(:offerings, :needs).should include(taggable2)
       taggable1.find_matching_contexts(:offerings, :needs).should_not include(taggable3)
     end
-  
+
     it "should find other related objects with tags of matching contexts" do
       taggable1 = TaggableModel.create!(:name => "Taggable 1")
       taggable2 = OtherTaggableModel.create!(:name => "Taggable 2")
       taggable3 = OtherTaggableModel.create!(:name => "Taggable 3")
-  
+
       taggable1.offering_list = "one, two"
       taggable1.save
-  
+
       taggable2.need_list = "one, two"
       taggable2.save
-  
+
       taggable3.offering_list = "one, two"
       taggable3.save
-  
+
       taggable1.find_matching_contexts_for(OtherTaggableModel, :offerings, :needs).should include(taggable2)
       taggable1.find_matching_contexts_for(OtherTaggableModel, :offerings, :needs).should_not include(taggable3)
     end
-  
-    it "should not include the object itself in the list of related objects" do
+
+    it "should not include the object itself in the list of related objects with tags of matching contexts" do
       taggable1 = TaggableModel.create!(:name => "Taggable 1")
       taggable2 = TaggableModel.create!(:name => "Taggable 2")
-  
-      taggable1.tag_list = "one"
+
+      taggable1.offering_list = "one, two"
+      taggable1.need_list = "one, two"
       taggable1.save
-  
-      taggable2.tag_list = "one, two"
+
+      taggable2.need_list = "one, two"
       taggable2.save
-  
-      taggable1.find_related_tags.should include(taggable2)
-      taggable1.find_related_tags.should_not include(taggable1)
+
+      taggable1.find_matching_contexts_for(TaggableModel, :offerings, :needs).should include(taggable2)
+      taggable1.find_matching_contexts_for(TaggableModel, :offerings, :needs).should_not include(taggable1)
+    end
+
+    context "Inherited Models" do
+      before do
+        @taggable1 = InheritingTaggableModel.create!(:name => "InheritingTaggable 1")
+        @taggable2 = InheritingTaggableModel.create!(:name => "InheritingTaggable 2")
+        @taggable3 = InheritingTaggableModel.create!(:name => "InheritingTaggable 3")
+        @taggable4 = InheritingTaggableModel.create!(:name => "InheritingTaggable 4")
+        @taggable5 = TaggableModel.create!(:name => "Taggable 5")
+
+        @taggable1.offering_list = "one, two"
+        @taggable1.need_list = "one, two"
+        @taggable1.save!
+
+        @taggable2.need_list = "one, two"
+        @taggable2.save!
+
+        @taggable3.offering_list = "one, two"
+        @taggable3.save!
+
+        @taggable4.tag_list = "one, two, three, four"
+        @taggable4.save!
+
+        @taggable5.need_list = "one, two"
+        @taggable5.save!
+      end
+
+      it "should find objects with tags of matching contexts" do
+        @taggable1.find_matching_contexts(:offerings, :needs).should include(@taggable2)
+        @taggable1.find_matching_contexts(:offerings, :needs).should_not include(@taggable3)
+        @taggable1.find_matching_contexts(:offerings, :needs).should_not include(@taggable4)
+        @taggable1.find_matching_contexts(:offerings, :needs).should_not include(@taggable5)
+
+        @taggable1.find_matching_contexts_for(TaggableModel, :offerings, :needs).should include(@taggable2)
+        @taggable1.find_matching_contexts_for(TaggableModel, :offerings, :needs).should_not include(@taggable3)
+        @taggable1.find_matching_contexts_for(TaggableModel, :offerings, :needs).should_not include(@taggable4)
+        @taggable1.find_matching_contexts_for(TaggableModel, :offerings, :needs).should include(@taggable5)
+      end
+
+      it "should not include the object itself in the list of related objects with tags of matching contexts" do
+        @taggable1.find_matching_contexts(:offerings, :needs).should_not include(@taggable1)
+        @taggable1.find_matching_contexts_for(InheritingTaggableModel, :offerings, :needs).should_not include(@taggable1)
+        @taggable1.find_matching_contexts_for(TaggableModel, :offerings, :needs).should_not include(@taggable1)
+      end
     end
   end
 
@@ -208,55 +349,67 @@ describe "Acts As Taggable On" do
       }.should_not raise_error
     end
   end
-  
+
   describe 'Caching' do
     before(:each) do
-      @taggable = CachedModel.new(:name => "Bob Jones")  
+      @taggable = CachedModel.new(:name => "Bob Jones")
+      @another_taggable = OtherCachedModel.new(:name => "John Smith")
     end
-    
+
     it "should add saving of tag lists and cached tag lists to the instance" do
       @taggable.should respond_to(:save_cached_tag_list)
+      @another_taggable.should respond_to(:save_cached_tag_list)
+
       @taggable.should respond_to(:save_tags)
-    end  
+    end
+
+    it "should add cached tag lists to the instance if cached column is not present" do
+      TaggableModel.new(:name => "Art Kram").should_not respond_to(:save_cached_tag_list)
+    end
 
     it "should generate a cached column checker for each tag type" do
       CachedModel.should respond_to(:caching_tag_list?)
-    end  
-    
+      OtherCachedModel.should respond_to(:caching_language_list?)
+    end
+
     it 'should not have cached tags' do
-      @taggable.cached_tag_list.should be_blank  
+      @taggable.cached_tag_list.should be_blank
+      @another_taggable.cached_language_list.should be_blank
     end
-    
+
     it 'should cache tags' do
       @taggable.update_attributes(:tag_list => 'awesome, epic')
       @taggable.cached_tag_list.should == 'awesome, epic'
+
+      @another_taggable.update_attributes(:language_list => 'ruby, .net')
+      @another_taggable.cached_language_list.should == 'ruby, .net'
     end
-    
+
     it 'should keep the cache' do
       @taggable.update_attributes(:tag_list => 'awesome, epic')
-      @taggable = CachedModel.find(@taggable)  
+      @taggable = CachedModel.find(@taggable)
       @taggable.save!
-      @taggable.cached_tag_list.should == 'awesome, epic'   
+      @taggable.cached_tag_list.should == 'awesome, epic'
     end
-    
+
     it 'should update the cache' do
       @taggable.update_attributes(:tag_list => 'awesome, epic')
       @taggable.update_attributes(:tag_list => 'awesome')
-      @taggable.cached_tag_list.should == 'awesome'      
+      @taggable.cached_tag_list.should == 'awesome'
     end
-    
+
     it 'should remove the cache' do
       @taggable.update_attributes(:tag_list => 'awesome, epic')
       @taggable.update_attributes(:tag_list => '')
-      @taggable.cached_tag_list.should be_blank  
+      @taggable.cached_tag_list.should be_blank
     end
-    
+
     it 'should have a tag list' do
       @taggable.update_attributes(:tag_list => 'awesome, epic')
       @taggable = CachedModel.find(@taggable.id)
       @taggable.tag_list.sort.should == %w(awesome epic).sort
     end
-    
+
     it 'should keep the tag list' do
       @taggable.update_attributes(:tag_list => 'awesome, epic')
       @taggable = CachedModel.find(@taggable.id)
@@ -265,4 +418,97 @@ describe "Acts As Taggable On" do
     end
   end
 
+  context 'when tagging context ends in an "s" when singular (ex. "status", "glass", etc.)' do
+   describe 'caching' do
+     before  { @taggable = OtherCachedModel.new(:name => "John Smith") }
+     subject { @taggable }
+
+     it { should respond_to(:save_cached_tag_list) }
+     its(:cached_language_list) { should be_blank }
+     its(:cached_status_list)   { should be_blank }
+     its(:cached_glass_list)    { should be_blank }
+
+     context 'language taggings cache after update' do
+       before  { @taggable.update_attributes(:language_list => 'ruby, .net') }
+       subject { @taggable }
+
+       its(:language_list)        { should == ['ruby', '.net']}
+       its(:cached_language_list) { should == 'ruby, .net' }           # passes
+       its(:instance_variables)   { should     include((RUBY_VERSION < '1.9' ? '@language_list' : :@language_list)) }
+     end
+
+     context 'status taggings cache after update' do
+       before  { @taggable.update_attributes(:status_list => 'happy, married') }
+       subject { @taggable }
+
+       its(:status_list)        { should     == ['happy', 'married'] }
+       its(:cached_status_list) { should     == 'happy, married'     } # fails
+       its(:cached_status_list) { should_not == ''                   } # fails, is blank
+       its(:instance_variables) { should     include((RUBY_VERSION < '1.9' ? '@status_list' : :@status_list)) }
+       its(:instance_variables) { should_not include((RUBY_VERSION < '1.9' ? '@statu_list' : :@statu_list))  } # fails, note: one "s"
+
+     end
+
+     context 'glass taggings cache after update' do
+       before do
+         @taggable.update_attributes(:glass_list => 'rectangle, aviator')
+       end
+
+       subject { @taggable }
+       its(:glass_list)         { should     == ['rectangle', 'aviator'] }
+       its(:cached_glass_list)  { should     == 'rectangle, aviator'     } # fails
+       its(:cached_glass_list)  { should_not == ''                       } # fails, is blank
+       if RUBY_VERSION < '1.9'
+         its(:instance_variables) { should     include('@glass_list')      }
+         its(:instance_variables) { should_not include('@glas_list')       } # fails, note: one "s"
+       else
+         its(:instance_variables) { should     include(:@glass_list)      }
+         its(:instance_variables) { should_not include(:@glas_list)       } # fails, note: one "s"
+       end
+
+     end
+   end
+  end
+
+  describe "taggings" do
+    before(:each) do
+      @taggable = TaggableModel.new(:name => "Art Kram")
+    end
+
+    it 'should return [] taggings' do
+      @taggable.taggings.should == []
+    end
+  end
+
+  describe "@@remove_unused_tags" do
+    before do
+      @taggable = TaggableModel.create(:name => "Bob Jones")
+      @tag = ActsAsTaggableOn::Tag.create(:name => "awesome")
+
+      @tagging = ActsAsTaggableOn::Tagging.create(:taggable => @taggable, :tag => @tag, :context => 'tags')
+    end
+
+    context "if set to true" do
+      before do
+        ActsAsTaggableOn.remove_unused_tags = true
+      end
+
+      it "should remove unused tags after removing taggings" do
+        @tagging.destroy
+        ActsAsTaggableOn::Tag.find_by_name("awesome").should be_nil
+      end
+    end
+
+    context "if set to false" do
+      before do
+        ActsAsTaggableOn.remove_unused_tags = false
+      end
+
+      it "should not remove unused tags after removing taggings" do
+        @tagging.destroy
+        ActsAsTaggableOn::Tag.find_by_name("awesome").should == @tag
+      end
+    end
+  end
+
 end

data/spec/acts_as_taggable_on/tag_list_spec.rb

@@ -1,70 +1,93 @@
 require File.expand_path('../../spec_helper', __FILE__)
 
 describe ActsAsTaggableOn::TagList do
-  before(:each) do
-    @tag_list = ActsAsTaggableOn::TagList.new("awesome","radical")
-  end
-  
-  it "should be an array" do
-    @tag_list.is_a?(Array).should be_true
-  end
-  
-  it "should be able to be add a new tag word" do
-    @tag_list.add("cool")
-    @tag_list.include?("cool").should be_true
-  end
-  
-  it "should be able to add delimited lists of words" do
-    @tag_list.add("cool, wicked", :parse => true)
-    @tag_list.include?("cool").should be_true
-    @tag_list.include?("wicked").should be_true
-  end
-  
-  it "should be able to add delimited list of words with quoted delimiters" do
-    @tag_list.add("'cool, wicked', \"really cool, really wicked\"", :parse => true)
-    @tag_list.include?("cool, wicked").should be_true
-    @tag_list.include?("really cool, really wicked").should be_true
-  end
-  
-  it "should be able to handle other uses of quotation marks correctly" do
-    @tag_list.add("john's cool car, mary's wicked toy", :parse => true)
-    @tag_list.include?("john's cool car").should be_true
-    @tag_list.include?("mary's wicked toy").should be_true
-  end
-  
-  it "should be able to add an array of words" do
-    @tag_list.add(["cool", "wicked"], :parse => true)
-    @tag_list.include?("cool").should be_true
-    @tag_list.include?("wicked").should be_true
-  end
-  
-  it "should be able to remove words" do
-    @tag_list.remove("awesome")
-    @tag_list.include?("awesome").should be_false
-  end
-  
-  it "should be able to remove delimited lists of words" do
-    @tag_list.remove("awesome, radical", :parse => true)
-    @tag_list.should be_empty
+  let(:tag_list) { ActsAsTaggableOn::TagList.new("awesome","radical") }
+
+  it { should be_kind_of Array }
+
+  it "#from should return empty array if empty array is passed" do
+    ActsAsTaggableOn::TagList.from([]).should be_empty
   end
-  
-  it "should be able to remove an array of words" do
-    @tag_list.remove(["awesome", "radical"], :parse => true)
-    @tag_list.should be_empty
+
+  describe "#add" do
+    it "should be able to be add a new tag word" do
+      tag_list.add("cool")
+      tag_list.include?("cool").should be_true
+    end
+
+    it "should be able to add delimited lists of words" do
+      tag_list.add("cool, wicked", :parse => true)
+      tag_list.should include("cool", "wicked")
+    end
+
+    it "should be able to add delimited list of words with quoted delimiters" do
+      tag_list.add("'cool, wicked', \"really cool, really wicked\"", :parse => true)
+      tag_list.should include("cool, wicked", "really cool, really wicked")
+    end
+
+    it "should be able to handle other uses of quotation marks correctly" do
+      tag_list.add("john's cool car, mary's wicked toy", :parse => true)
+      tag_list.should include("john's cool car", "mary's wicked toy")
+    end
+
+    it "should be able to add an array of words" do
+      tag_list.add(["cool", "wicked"], :parse => true)
+      tag_list.should include("cool", "wicked")
+    end
+
+    it "should quote escape tags with commas in them" do
+      tag_list.add("cool","rad,bodacious")
+      tag_list.to_s.should == "awesome, radical, cool, \"rad,bodacious\""
+    end
+
   end
-  
-  it "should give a delimited list of words when converted to string" do
-    @tag_list.to_s.should == "awesome, radical"
+
+  describe "#remove" do
+    it "should be able to remove words" do
+      tag_list.remove("awesome")
+      tag_list.should_not include("awesome")
+    end
+
+    it "should be able to remove delimited lists of words" do
+      tag_list.remove("awesome, radical", :parse => true)
+      tag_list.should be_empty
+    end
+
+    it "should be able to remove an array of words" do
+      tag_list.remove(["awesome", "radical"], :parse => true)
+      tag_list.should be_empty
+    end
   end
-  
-  it "should quote escape tags with commas in them" do
-    @tag_list.add("cool","rad,bodacious")
-    @tag_list.to_s.should == "awesome, radical, cool, \"rad,bodacious\""
+
+  describe "#to_s" do
+    it "should give a delimited list of words when converted to string" do
+      tag_list.to_s.should == "awesome, radical"
+    end
+
+    it "should be able to call to_s on a frozen tag list" do
+      tag_list.freeze
+      lambda { tag_list.add("cool","rad,bodacious") }.should raise_error
+      lambda { tag_list.to_s }.should_not raise_error
+    end
   end
-  
-  it "should be able to call to_s on a frozen tag list" do
-    @tag_list.freeze
-    lambda { @tag_list.add("cool","rad,bodacious") }.should raise_error
-    lambda { @tag_list.to_s }.should_not raise_error
+
+  describe "cleaning" do
+    it "should parameterize if force_parameterize is set to true" do
+      ActsAsTaggableOn.force_parameterize = true
+      tag_list = ActsAsTaggableOn::TagList.new("awesome()","radical)(cc")
+
+      tag_list.to_s.should == "awesome, radical-cc"
+      ActsAsTaggableOn.force_parameterize = false
+    end
+
+    it "should lowercase if force_lowercase is set to true" do
+      ActsAsTaggableOn.force_lowercase = true
+
+      tag_list = ActsAsTaggableOn::TagList.new("aweSomE","RaDicaL")
+      tag_list.to_s.should == "awesome, radical"
+
+      ActsAsTaggableOn.force_lowercase = false
+    end
+
   end
 end