acts_as_taggable_on 3.0.0.rc1

data/spec/acts_as_taggable_on/tagger_spec.rb

@@ -0,0 +1,137 @@
+require 'spec_helper'
+
+describe "Tagger" do
+  before(:each) do
+    clean_database!
+    @user = User.create
+    @taggable = TaggableModel.create(:name => "Bob Jones")
+  end
+
+  it "should have taggings" do
+    @user.tag(@taggable, :with=>'ruby,scheme', :on=>:tags)
+    @user.owned_taggings.size == 2
+  end
+
+  it "should have tags" do
+    @user.tag(@taggable, :with=>'ruby,scheme', :on=>:tags)
+    @user.owned_tags.size == 2
+  end
+
+  it "should scope objects returned by tagged_with by owners" do
+    @taggable2 = TaggableModel.create(:name => "Jim Jones")
+    @taggable3 = TaggableModel.create(:name => "Jane Doe")
+
+    @user2 = User.new
+    @user.tag(@taggable, :with => 'ruby, scheme', :on => :tags)
+    @user2.tag(@taggable2, :with => 'ruby, scheme', :on => :tags)
+    @user2.tag(@taggable3, :with => 'ruby, scheme', :on => :tags)
+
+    TaggableModel.tagged_with(%w(ruby scheme), :owned_by => @user).count.should == 1
+    TaggableModel.tagged_with(%w(ruby scheme), :owned_by => @user2).count.should == 2
+  end
+
+  it "only returns objects tagged by owned_by when any is true" do
+    @user2 = User.new
+    @taggable2 = TaggableModel.create(:name => "Jim Jones")
+    @taggable3 = TaggableModel.create(:name => "Jane Doe")
+
+    @user.tag(@taggable, :with => 'ruby', :on => :tags)
+    @user.tag(@taggable2, :with => 'java', :on => :tags)
+    @user2.tag(@taggable3, :with => 'ruby', :on => :tags)
+
+    tags = TaggableModel.tagged_with(%w(ruby java), :owned_by => @user, :any => true)
+    tags.should match_array [@taggable, @taggable2]
+  end
+
+  it "only returns objects tagged by owned_by when exclude is true" do
+    @user2 = User.new
+    @taggable2 = TaggableModel.create(:name => "Jim Jones")
+    @taggable3 = TaggableModel.create(:name => "Jane Doe")
+
+    @user.tag(@taggable, :with => 'ruby', :on => :tags)
+    @user.tag(@taggable2, :with => 'java', :on => :tags)
+    @user2.tag(@taggable3, :with => 'java', :on => :tags)
+
+    tags = TaggableModel.tagged_with(%w(ruby), :owned_by => @user, :exclude => true)
+    tags.should match_array [@taggable2]
+  end
+
+  it "should not overlap tags from different taggers" do
+    @user2 = User.new
+    lambda{
+      @user.tag(@taggable, :with => 'ruby, scheme', :on => :tags)
+      @user2.tag(@taggable, :with => 'java, python, lisp, ruby', :on => :tags)
+    }.should change(ActsAsTaggableOn::Tagging, :count).by(6)
+
+    [@user, @user2, @taggable].each(&:reload)
+
+    @user.owned_tags.map(&:name).sort.should == %w(ruby scheme).sort
+    @user2.owned_tags.map(&:name).sort.should == %w(java python lisp ruby).sort
+
+    @taggable.tags_from(@user).sort.should == %w(ruby scheme).sort
+    @taggable.tags_from(@user2).sort.should == %w(java lisp python ruby).sort
+
+    @taggable.all_tags_list.sort.should == %w(ruby scheme java python lisp).sort
+    @taggable.all_tags_on(:tags).size.should == 5
+  end
+
+  it "should not lose tags from different taggers" do
+    @user2 = User.create
+    @user2.tag(@taggable, :with => 'java, python, lisp, ruby', :on => :tags)
+    @user.tag(@taggable, :with => 'ruby, scheme', :on => :tags)
+
+    lambda {
+      @user2.tag(@taggable, :with => 'java, python, lisp', :on => :tags)
+    }.should change(ActsAsTaggableOn::Tagging, :count).by(-1)
+
+    [@user, @user2, @taggable].each(&:reload)
+
+    @taggable.tags_from(@user).sort.should == %w(ruby scheme).sort
+    @taggable.tags_from(@user2).sort.should == %w(java python lisp).sort
+
+    @taggable.all_tags_list.sort.should == %w(ruby scheme java python lisp).sort
+    @taggable.all_tags_on(:tags).length.should == 5
+  end
+
+  it "should not lose tags" do
+    @user2 = User.create
+
+    @user.tag(@taggable, :with => 'awesome', :on => :tags)
+    @user2.tag(@taggable, :with => 'awesome, epic', :on => :tags)
+
+    lambda {
+      @user2.tag(@taggable, :with => 'epic', :on => :tags)
+    }.should change(ActsAsTaggableOn::Tagging, :count).by(-1)
+
+    @taggable.reload
+    @taggable.all_tags_list.should include('awesome')
+    @taggable.all_tags_list.should include('epic')
+  end
+
+  it "should not lose tags" do
+    @taggable.update_attributes(:tag_list => 'ruby')
+    @user.tag(@taggable, :with => 'ruby, scheme', :on => :tags)
+
+    [@taggable, @user].each(&:reload)
+    @taggable.tag_list.should == %w(ruby)
+    @taggable.all_tags_list.sort.should == %w(ruby scheme).sort
+
+    lambda {
+      @taggable.update_attributes(:tag_list => "")
+    }.should change(ActsAsTaggableOn::Tagging, :count).by(-1)
+
+    @taggable.tag_list.should == []
+    @taggable.all_tags_list.sort.should == %w(ruby scheme).sort
+  end
+
+  it "is tagger" do
+    @user.is_tagger?.should(be_true)
+  end
+
+  it "should skip save if skip_save is passed as option" do
+    lambda {
+      @user.tag(@taggable, :with => 'epic', :on => :tags, :skip_save => true)
+    }.should_not change(ActsAsTaggableOn::Tagging, :count)
+  end
+
+end

data/spec/acts_as_taggable_on/tagging_spec.rb

@@ -0,0 +1,28 @@
+require 'spec_helper'
+
+describe ActsAsTaggableOn::Tagging do
+  before(:each) do
+    clean_database!
+    @tagging = ActsAsTaggableOn::Tagging.new
+  end
+
+  it "should not be valid with a invalid tag" do
+    @tagging.taggable = TaggableModel.create(:name => "Bob Jones")
+    @tagging.tag = ActsAsTaggableOn::Tag.new(:name => "")
+    @tagging.context = "tags"
+
+    @tagging.should_not be_valid
+    
+    @tagging.errors[:tag_id].should == ["can't be blank"]
+  end
+
+  it "should not create duplicate taggings" do
+    @taggable = TaggableModel.create(:name => "Bob Jones")
+    @tag = ActsAsTaggableOn::Tag.create(:name => "awesome")
+
+    lambda {
+      2.times { ActsAsTaggableOn::Tagging.create(:taggable => @taggable, :tag => @tag, :context => 'tags') }
+    }.should change(ActsAsTaggableOn::Tagging, :count).by(1)
+  end
+  
+end

data/spec/acts_as_taggable_on/tags_helper_spec.rb

@@ -0,0 +1,44 @@
+require 'spec_helper'
+
+describe ActsAsTaggableOn::TagsHelper do
+  before(:each) do
+    clean_database!
+    
+    @bob = TaggableModel.create(:name => "Bob Jones",  :language_list => "ruby, php")
+    @tom = TaggableModel.create(:name => "Tom Marley", :language_list => "ruby, java")
+    @eve = TaggableModel.create(:name => "Eve Nodd",   :language_list => "ruby, c++")
+    
+    @helper = class Helper
+      include ActsAsTaggableOn::TagsHelper
+    end.new
+  end
+  
+  it "should yield the proper css classes" do 
+    tags = { }
+    
+    @helper.tag_cloud(TaggableModel.tag_counts_on(:languages), ["sucky", "awesome"]) do |tag, css_class|
+      tags[tag.name] = css_class
+    end
+    
+    tags["ruby"].should == "awesome"
+    tags["java"].should == "sucky"
+    tags["c++"].should == "sucky"
+    tags["php"].should == "sucky"
+  end
+  
+  it "should handle tags with zero counts (build for empty)" do
+    bob = ActsAsTaggableOn::Tag.create(:name => "php")
+    tom = ActsAsTaggableOn::Tag.create(:name => "java")
+    eve = ActsAsTaggableOn::Tag.create(:name => "c++")
+    
+    tags = { }
+    
+    @helper.tag_cloud(ActsAsTaggableOn::Tag.all, ["sucky", "awesome"]) do |tag, css_class|
+      tags[tag.name] = css_class
+    end
+    
+    tags["java"].should == "sucky"
+    tags["c++"].should == "sucky"
+    tags["php"].should == "sucky"
+  end
+end

data/spec/acts_as_taggable_on/utils_spec.rb

@@ -0,0 +1,21 @@
+require 'spec_helper'
+
+describe ActsAsTaggableOn::Utils do
+  describe "like_operator" do
+    before(:each) do
+      clean_database!
+      TaggableModel.acts_as_taggable_on(:tags, :languages, :skills, :needs, :offerings)
+      @taggable = TaggableModel.new(:name => "Bob Jones")
+    end
+
+    it "should return 'ILIKE' when the adapter is PostgreSQL" do
+      TaggableModel.connection.stub(:adapter_name).and_return("PostgreSQL")
+      TaggableModel.send(:like_operator).should == "ILIKE"
+    end
+
+    it "should return 'LIKE' when the adapter is not PostgreSQL" do
+      TaggableModel.connection.stub(:adapter_name).and_return("MySQL")
+      TaggableModel.send(:like_operator).should == "LIKE"
+    end
+  end
+end

data/spec/bm.rb

@@ -0,0 +1,52 @@
+require 'active_record'
+require 'action_view'
+require File.expand_path('../../lib/acts_as_taggable_on', __FILE__)
+
+if defined?(ActiveRecord::Acts::TaggableOn)
+  ActiveRecord::Base.send :include, ActiveRecord::Acts::TaggableOn
+  ActiveRecord::Base.send :include, ActiveRecord::Acts::Tagger
+  ActionView::Base.send :include, TagsHelper if defined?(ActionView::Base)
+end
+
+TEST_DATABASE_FILE = File.join(File.dirname(__FILE__), '..', 'test.sqlite3')
+File.unlink(TEST_DATABASE_FILE) if File.exist?(TEST_DATABASE_FILE)
+ActiveRecord::Base.establish_connection :adapter => 'sqlite3', :database => TEST_DATABASE_FILE
+
+ActiveRecord::Base.silence do
+  ActiveRecord::Migration.verbose = false
+  ActiveRecord::Schema.define :version => 0 do
+    create_table "taggings", :force => true do |t|
+      t.integer  "tag_id",        :limit => 11
+      t.integer  "taggable_id",   :limit => 11
+      t.string   "taggable_type"
+      t.string   "context"
+      t.datetime "created_at"
+      t.integer  "tagger_id",     :limit => 11
+      t.string   "tagger_type"
+    end
+
+    add_index "taggings", ["tag_id"], :name => "index_taggings_on_tag_id"
+    add_index "taggings", ["taggable_id", "taggable_type", "context"], :name => "index_taggings_on_taggable_id_and_taggable_type_and_context"
+
+    create_table "tags", :force => true do |t|
+      t.string "name"
+    end
+
+    create_table :taggable_models, :force => true do |t|
+      t.column :name, :string
+      t.column :type, :string
+      t.column :cached_tag_list, :string
+    end
+  end
+
+  class TaggableModel < ActiveRecord::Base
+    acts_as_taggable
+    acts_as_taggable_on :languages
+    acts_as_taggable_on :skills
+    acts_as_taggable_on :needs, :offerings
+  end
+end
+
+puts Benchmark.measure {
+  1000.times { TaggableModel.create :tag_list => "awesome, epic, neat" }
+}

data/spec/database.yml.sample

@@ -0,0 +1,19 @@
+sqlite3:
+  adapter: sqlite3
+  database: acts_as_taggable_on.sqlite3
+
+mysql:
+  adapter: mysql2
+  hostname: localhost
+  username: root
+  password:
+  database: acts_as_taggable_on
+  charset: utf8
+  
+postgresql:
+  adapter: postgresql
+  hostname: localhost
+  username: postgres
+  password:
+  database: acts_as_taggable_on
+  encoding: utf8

data/spec/models.rb

@@ -0,0 +1,58 @@
+class TaggableModel < ActiveRecord::Base
+  acts_as_taggable
+  acts_as_taggable_on :languages
+  acts_as_taggable_on :skills
+  acts_as_taggable_on :needs, :offerings
+  has_many :untaggable_models
+
+  attr_reader :tag_list_submethod_called
+  def tag_list=v
+    @tag_list_submethod_called = true
+    super
+  end
+end
+
+class CachedModel < ActiveRecord::Base
+  acts_as_taggable
+end
+
+class OtherCachedModel < ActiveRecord::Base
+  acts_as_taggable_on :languages, :statuses, :glasses
+end
+
+class OtherTaggableModel < ActiveRecord::Base
+  acts_as_taggable_on :tags, :languages
+  acts_as_taggable_on :needs, :offerings
+end
+
+class InheritingTaggableModel < TaggableModel
+end
+
+class AlteredInheritingTaggableModel < TaggableModel
+  acts_as_taggable_on :parts
+end
+
+class User < ActiveRecord::Base
+  acts_as_tagger
+end
+
+class Student < User
+end
+
+class UntaggableModel < ActiveRecord::Base
+  belongs_to :taggable_model
+end
+
+class NonStandardIdTaggableModel < ActiveRecord::Base
+  primary_key = "an_id"
+  acts_as_taggable
+  acts_as_taggable_on :languages
+  acts_as_taggable_on :skills
+  acts_as_taggable_on :needs, :offerings
+  has_many :untaggable_models
+end
+
+class OrderedTaggableModel < ActiveRecord::Base
+  acts_as_ordered_taggable
+  acts_as_ordered_taggable_on :colours
+end

data/spec/schema.rb

@@ -0,0 +1,65 @@
+ActiveRecord::Schema.define :version => 0 do
+  create_table :tags, :force => true do |t|
+    t.string :name
+  end
+
+  create_table :taggings, :force => true do |t|
+    t.references :tag
+
+    # You should make sure that the column created is
+    # long enough to store the required class names.
+    t.references :taggable, :polymorphic => true
+    t.references :tagger, :polymorphic => true
+
+    # Limit is created to prevent MySQL error on index
+    # length for MyISAM table type: http://bit.ly/vgW2Ql
+    t.string :context, :limit => 128
+
+    t.datetime :created_at
+  end
+  # above copied from
+  # generators/acts_as_taggable_on/migration/migration_generator
+
+  create_table :taggable_models, :force => true do |t|
+    t.column :name, :string
+    t.column :type, :string
+  end
+
+  create_table :non_standard_id_taggable_models, :primary_key => "an_id", :force => true do |t|
+    t.column :name, :string
+    t.column :type, :string
+  end
+
+  create_table :untaggable_models, :force => true do |t|
+    t.column :taggable_model_id, :integer
+    t.column :name, :string
+  end
+
+  create_table :cached_models, :force => true do |t|
+    t.column :name, :string
+    t.column :type, :string
+    t.column :cached_tag_list, :string
+  end
+
+  create_table :other_cached_models, :force => true do |t|
+    t.column :name, :string
+    t.column :type, :string
+    t.column :cached_language_list, :string
+    t.column :cached_status_list, :string
+    t.column :cached_glass_list, :string
+  end
+
+  create_table :users, :force => true do |t|
+    t.column :name, :string
+  end
+
+  create_table :other_taggable_models, :force => true do |t|
+    t.column :name, :string
+    t.column :type, :string
+  end
+
+  create_table :ordered_taggable_models, :force => true do |t|
+    t.column :name, :string
+    t.column :type, :string
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,87 @@
+$LOAD_PATH << "." unless $LOAD_PATH.include?(".")
+$LOAD_PATH.unshift(File.expand_path('../../lib', __FILE__))
+require 'logger'
+
+begin
+  require "rubygems"
+  require "bundler"
+
+  if Gem::Version.new(Bundler::VERSION) <= Gem::Version.new("0.9.5")
+    raise RuntimeError, "Your bundler version is too old." +
+     "Run `gem install bundler` to upgrade."
+  end
+
+  # Set up load paths for all bundled gems
+  Bundler.setup
+rescue Bundler::GemNotFound
+  raise RuntimeError, "Bundler couldn't find some gems." +
+    "Did you run \`bundle install\`?"
+end
+
+Bundler.require
+require File.expand_path('../../lib/acts_as_taggable_on', __FILE__)
+require 'ammeter/init'
+
+unless [].respond_to?(:freq)
+  class Array
+    def freq
+      k=Hash.new(0)
+      each {|e| k[e]+=1}
+      k
+    end
+  end
+end
+
+# set adapter to use, default is sqlite3
+# to use an alternative adapter run => rake spec DB='postgresql'
+db_name = ENV['DB'] || 'sqlite3'
+database_yml = File.expand_path('../database.yml', __FILE__)
+
+if File.exists?(database_yml)
+  active_record_configuration = YAML.load_file(database_yml)
+  
+  ActiveRecord::Base.configurations = active_record_configuration
+  config = ActiveRecord::Base.configurations[db_name]
+  
+  begin
+    ActiveRecord::Base.establish_connection(db_name)
+    ActiveRecord::Base.connection
+  rescue
+    case db_name
+    when /mysql/      
+      ActiveRecord::Base.establish_connection(config.merge('database' => nil))
+      ActiveRecord::Base.connection.create_database(config['database'],  {:charset => 'utf8', :collation => 'utf8_unicode_ci'})
+    when 'postgresql'
+      ActiveRecord::Base.establish_connection(config.merge('database' => 'postgres', 'schema_search_path' => 'public'))
+      ActiveRecord::Base.connection.create_database(config['database'], config.merge('encoding' => 'utf8'))
+    end
+    
+    ActiveRecord::Base.establish_connection(config)
+  end
+    
+  logger = ActiveRecord::Base.logger = Logger.new(File.join(File.dirname(__FILE__), "debug.log"))
+  ActiveRecord::Base.default_timezone = :utc
+  
+  begin
+    old_logger_level, logger.level = logger.level, ::Logger::ERROR
+    ActiveRecord::Migration.verbose = false
+    
+    load(File.dirname(__FILE__) + '/schema.rb')
+    load(File.dirname(__FILE__) + '/models.rb')
+  ensure
+    logger.level = old_logger_level
+  end
+  
+else
+  raise "Please create #{database_yml} first to configure your database. Take a look at: #{database_yml}.sample"
+end
+
+def clean_database!
+  models = [ActsAsTaggableOn::Tag, ActsAsTaggableOn::Tagging, TaggableModel, OtherTaggableModel, InheritingTaggableModel,
+            AlteredInheritingTaggableModel, User, UntaggableModel, OrderedTaggableModel]
+  models.each do |model|
+    ActiveRecord::Base.connection.execute "DELETE FROM #{model.table_name}"
+  end
+end
+
+clean_database!