make_taggable 0.6.3

data/spec/dummy/db/migrate/20201119220853_create_taggable_models.rb

@@ -0,0 +1,8 @@
+class CreateTaggableModels < ActiveRecord::Migration[5.2]
+  def change
+    create_table :taggable_models do |t|
+      t.column :name, :string
+      t.column :type, :string
+    end
+  end
+end

data/spec/dummy/db/migrate/20201119221037_create_columns_override_models.rb

@@ -0,0 +1,9 @@
+class CreateColumnsOverrideModels < ActiveRecord::Migration[5.2]
+  def change
+    create_table :columns_override_models do |t|
+      t.column :name, :string
+      t.column :type, :string
+      t.column :ignored_column, :string
+    end
+  end
+end

data/spec/dummy/db/migrate/20201119221121_create_non_standard_id_taggable_models.rb

@@ -0,0 +1,8 @@
+class CreateNonStandardIdTaggableModels < ActiveRecord::Migration[5.2]
+  def change
+    create_table :non_standard_id_taggable_models, primary_key: "an_id" do |t|
+      t.column :name, :string
+      t.column :type, :string
+    end
+  end
+end

data/spec/dummy/db/migrate/20201119221228_create_untaggable_models.rb

@@ -0,0 +1,8 @@
+class CreateUntaggableModels < ActiveRecord::Migration[5.2]
+  def change
+    create_table :untaggable_models do |t|
+      t.column :taggable_model_id, :integer
+      t.column :name, :string
+    end
+  end
+end

data/spec/dummy/db/migrate/20201119221247_create_cached_models.rb

@@ -0,0 +1,9 @@
+class CreateCachedModels < ActiveRecord::Migration[5.2]
+  def change
+    create_table :cached_models do |t|
+      t.column :name, :string
+      t.column :type, :string
+      t.column :cached_tag_list, :string
+    end
+  end
+end

data/spec/dummy/db/migrate/20201119221314_create_other_cached_models.rb

@@ -0,0 +1,11 @@
+class CreateOtherCachedModels < ActiveRecord::Migration[5.2]
+  def change
+    create_table :other_cached_models 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
+  end
+end

data/spec/dummy/db/migrate/20201119221343_create_companies.rb

@@ -0,0 +1,7 @@
+class CreateCompanies < ActiveRecord::Migration[5.2]
+  def change
+    create_table :companies do |t|
+      t.column :name, :string
+    end
+  end
+end

data/spec/dummy/db/migrate/20201119221416_create_users.rb

@@ -0,0 +1,7 @@
+class CreateUsers < ActiveRecord::Migration[5.2]
+  def change
+    create_table :users do |t|
+      t.column :name, :string
+    end
+  end
+end

data/spec/dummy/db/migrate/20201119221434_create_other_taggable_models.rb

@@ -0,0 +1,8 @@
+class CreateOtherTaggableModels < ActiveRecord::Migration[5.2]
+  def change
+    create_table :other_taggable_models do |t|
+      t.column :name, :string
+      t.column :type, :string
+    end
+  end
+end

data/spec/dummy/db/migrate/20201119221507_create_ordered_taggable_models.rb

@@ -0,0 +1,8 @@
+class CreateOrderedTaggableModels < ActiveRecord::Migration[5.2]
+  def change
+    create_table :ordered_taggable_models do |t|
+      t.column :name, :string
+      t.column :type, :string
+    end
+  end
+end

data/spec/dummy/db/migrate/20201119221530_create_cache_methods_injected_models.rb

@@ -0,0 +1,7 @@
+class CreateCacheMethodsInjectedModels < ActiveRecord::Migration[5.2]
+  def change
+    create_table :cache_methods_injected_models do |t|
+      t.column :cached_tag_list, :string
+    end
+  end
+end

data/spec/dummy/db/migrate/20201119221629_create_other_cached_with_array_models.rb

@@ -0,0 +1,11 @@
+class CreateOtherCachedWithArrayModels < ActiveRecord::Migration[5.2]
+  def change
+    create_table :other_cached_with_array_models do |t|
+      t.column :name, :string
+      t.column :type, :string
+      t.column :cached_language_list, :string, array: true
+      t.column :cached_status_list, :string, array: true
+      t.column :cached_glass_list, :string, array: true
+    end
+  end
+end

data/spec/dummy/db/migrate/20201119221746_create_taggable_model_with_jsons.rb

@@ -0,0 +1,9 @@
+class CreateTaggableModelWithJsons < ActiveRecord::Migration[5.2]
+  def change
+    create_table :taggable_model_with_jsons do |t|
+      t.column :name, :string
+      t.column :type, :string
+      t.column :opts, :json
+    end
+  end
+end

data/spec/dummy/db/migrate/20201119222429_create_make_taggable_tags.make_taggable_engine.rb

@@ -0,0 +1,11 @@
+# This migration comes from make_taggable_engine (originally 1)
+class CreateMakeTaggableTags < ActiveRecord::Migration[5.2]
+  def change
+    create_table MakeTaggable.tags_table do |t|
+      t.string :name
+      t.integer :taggings_count, default: 0
+
+      t.timestamps
+    end
+  end
+end

data/spec/dummy/db/migrate/20201119222430_create_make_taggable_taggings.make_taggable_engine.rb

@@ -0,0 +1,13 @@
+# This migration comes from make_taggable_engine (originally 2)
+class CreateMakeTaggableTaggings < ActiveRecord::Migration[5.2]
+  def change
+    create_table MakeTaggable.taggings_table do |t|
+      t.references :tag, foreign_key: {to_table: MakeTaggable.tags_table}
+      t.references :taggable, polymorphic: true
+      t.references :tagger, polymorphic: true
+      t.string :context, limit: 128
+
+      t.timestamps
+    end
+  end
+end

data/spec/dummy/db/migrate/20201119222431_add_index_to_tags.make_taggable_engine.rb

@@ -0,0 +1,6 @@
+# This migration comes from make_taggable_engine (originally 3)
+class AddIndexToTags < ActiveRecord::Migration[5.2]
+  def change
+    add_index MakeTaggable.tags_table, :name, unique: true
+  end
+end

data/spec/dummy/db/migrate/20201119222432_add_index_to_taggings.make_taggable_engine.rb

@@ -0,0 +1,13 @@
+# This migration comes from make_taggable_engine (originally 4)
+class AddIndexToTaggings < ActiveRecord::Migration[5.2]
+  def change
+    add_index MakeTaggable.taggings_table, :taggable_id
+    add_index MakeTaggable.taggings_table, :tagger_id
+    add_index MakeTaggable.taggings_table, :taggable_type
+    add_index MakeTaggable.taggings_table, :context
+    add_index MakeTaggable.taggings_table, [:tagger_id, :tagger_type]
+    add_index MakeTaggable.taggings_table, [:tag_id, :taggable_id, :taggable_type, :context, :tagger_id, :tagger_type], unique: true, name: "taggings_idx"
+    add_index MakeTaggable.taggings_table, [:taggable_id, :taggable_type, :context], name: "taggings_taggable_context_idx"
+    add_index MakeTaggable.taggings_table, [:taggable_id, :taggable_type, :tagger_id, :context], name: "taggings_idy"
+  end
+end

data/spec/dummy/db/schema.rb

@@ -0,0 +1,117 @@
+# This file is auto-generated from the current state of the database. Instead
+# of editing this file, please use the migrations feature of Active Record to
+# incrementally modify your database, and then regenerate this schema definition.
+#
+# This file is the source Rails uses to define your schema when running `rails
+# db:schema:load`. When creating a new database, `rails db:schema:load` tends to
+# be faster and is potentially less error prone than running all of your
+# migrations from scratch. Old migrations may fail to apply correctly if those
+# migrations use external dependencies or application code.
+#
+# It's strongly recommended that you check this file into your version control system.
+
+ActiveRecord::Schema.define(version: 2020_11_19_222432) do
+
+  create_table "cache_methods_injected_models", force: :cascade do |t|
+    t.string "cached_tag_list"
+  end
+
+  create_table "cached_models", force: :cascade do |t|
+    t.string "name"
+    t.string "type"
+    t.string "cached_tag_list"
+  end
+
+  create_table "columns_override_models", force: :cascade do |t|
+    t.string "name"
+    t.string "type"
+    t.string "ignored_column"
+  end
+
+  create_table "companies", force: :cascade do |t|
+    t.string "name"
+  end
+
+  create_table "non_standard_id_taggable_models", primary_key: "an_id", force: :cascade do |t|
+    t.string "name"
+    t.string "type"
+  end
+
+  create_table "ordered_taggable_models", force: :cascade do |t|
+    t.string "name"
+    t.string "type"
+  end
+
+  create_table "other_cached_models", force: :cascade do |t|
+    t.string "name"
+    t.string "type"
+    t.string "cached_language_list"
+    t.string "cached_status_list"
+    t.string "cached_glass_list"
+  end
+
+  create_table "other_cached_with_array_models", force: :cascade do |t|
+    t.string "name"
+    t.string "type"
+    t.string "cached_language_list"
+    t.string "cached_status_list"
+    t.string "cached_glass_list"
+  end
+
+  create_table "other_taggable_models", force: :cascade do |t|
+    t.string "name"
+    t.string "type"
+  end
+
+  create_table "taggable_model_with_jsons", force: :cascade do |t|
+    t.string "name"
+    t.string "type"
+    t.json "opts"
+  end
+
+  create_table "taggable_models", force: :cascade do |t|
+    t.string "name"
+    t.string "type"
+  end
+
+  create_table "taggings", force: :cascade do |t|
+    t.integer "tag_id"
+    t.string "taggable_type"
+    t.integer "taggable_id"
+    t.string "tagger_type"
+    t.integer "tagger_id"
+    t.string "context", limit: 128
+    t.datetime "created_at", null: false
+    t.datetime "updated_at", null: false
+    t.index ["context"], name: "index_taggings_on_context"
+    t.index ["tag_id", "taggable_id", "taggable_type", "context", "tagger_id", "tagger_type"], name: "taggings_idx", unique: true
+    t.index ["tag_id"], name: "index_taggings_on_tag_id"
+    t.index ["taggable_id", "taggable_type", "context"], name: "taggings_taggable_context_idx"
+    t.index ["taggable_id", "taggable_type", "tagger_id", "context"], name: "taggings_idy"
+    t.index ["taggable_id"], name: "index_taggings_on_taggable_id"
+    t.index ["taggable_type", "taggable_id"], name: "index_taggings_on_taggable_type_and_taggable_id"
+    t.index ["taggable_type"], name: "index_taggings_on_taggable_type"
+    t.index ["tagger_id", "tagger_type"], name: "index_taggings_on_tagger_id_and_tagger_type"
+    t.index ["tagger_id"], name: "index_taggings_on_tagger_id"
+    t.index ["tagger_type", "tagger_id"], name: "index_taggings_on_tagger_type_and_tagger_id"
+  end
+
+  create_table "tags", force: :cascade do |t|
+    t.string "name"
+    t.integer "taggings_count", default: 0
+    t.datetime "created_at", null: false
+    t.datetime "updated_at", null: false
+    t.index ["name"], name: "index_tags_on_name", unique: true
+  end
+
+  create_table "untaggable_models", force: :cascade do |t|
+    t.integer "taggable_model_id"
+    t.string "name"
+  end
+
+  create_table "users", force: :cascade do |t|
+    t.string "name"
+  end
+
+  add_foreign_key "taggings", "tags"
+end

data/spec/dummy/db/seeds.rb

@@ -0,0 +1,7 @@
+# This file should contain all the record creation needed to seed the database with its default values.
+# The data can then be loaded with the rails db:seed command (or created alongside the database with db:setup).
+#
+# Examples:
+#
+#   movies = Movie.create([{ name: 'Star Wars' }, { name: 'Lord of the Rings' }])
+#   Character.create(name: 'Luke', movie: movies.first)

data/spec/dummy/public/robots.txt

@@ -0,0 +1 @@
+# See https://www.robotstxt.org/robotstxt.html for documentation on how to use the robots.txt file

data/spec/dummy/test/channels/application_cable/connection_test.rb

@@ -0,0 +1,11 @@
+require "test_helper"
+
+class ApplicationCable::ConnectionTest < ActionCable::Connection::TestCase
+  # test "connects with cookies" do
+  #   cookies.signed[:user_id] = 42
+  #
+  #   connect
+  #
+  #   assert_equal connection.user_id, "42"
+  # end
+end

data/spec/dummy/test/test_helper.rb

@@ -0,0 +1,13 @@
+ENV['RAILS_ENV'] ||= 'test'
+require_relative '../config/environment'
+require 'rails/test_help'
+
+class ActiveSupport::TestCase
+  # Run tests in parallel with specified workers
+  parallelize(workers: :number_of_processors)
+
+  # Setup all fixtures in test/fixtures/*.yml for all tests in alphabetical order.
+  fixtures :all
+
+  # Add more helper methods to be used by all tests here...
+end

data/spec/make_taggable/acts_as_tagger_spec.rb

@@ -0,0 +1,112 @@
+require "spec_helper"
+
+describe "acts_as_tagger" do
+  describe "Tagger Method Generation" do
+    before(:each) do
+      @tagger = User.new
+    end
+
+    it "should add #is_tagger? query method to the class-side" do
+      expect(User).to respond_to(:is_tagger?)
+    end
+
+    it "should return true from the class-side #is_tagger?" do
+      expect(User.is_tagger?).to be_truthy
+    end
+
+    it "should return false from the base #is_tagger?" do
+      expect(ActiveRecord::Base.is_tagger?).to be_falsy
+    end
+
+    it "should add #is_tagger? query method to the singleton" do
+      expect(@tagger).to respond_to(:is_tagger?)
+    end
+
+    it "should add #tag method on the instance-side" do
+      expect(@tagger).to respond_to(:tag)
+    end
+
+    it "should generate an association for #owned_taggings and #owned_tags" do
+      expect(@tagger).to respond_to(:owned_taggings, :owned_tags)
+    end
+  end
+
+  describe "#tag" do
+    context "when called with a non-existent tag context" do
+      before(:each) do
+        @tagger = User.new
+        @taggable = TaggableModel.new(name: "Richard Prior")
+      end
+
+      it "should by default not throw an exception " do
+        expect(@taggable.tag_list_on(:foo)).to be_empty
+        expect(-> {
+          @tagger.tag(@taggable, with: "this, and, that", on: :foo)
+        }).to_not raise_error
+      end
+
+      it "should by default create the tag context on-the-fly" do
+        expect(@taggable.tag_list_on(:here_ond_now)).to be_empty
+        @tagger.tag(@taggable, with: "that", on: :here_ond_now)
+        expect(@taggable.tag_list_on(:here_ond_now)).to_not include("that")
+        expect(@taggable.all_tags_list_on(:here_ond_now)).to include("that")
+      end
+
+      it "should show all the tag list when both public and owned tags exist" do
+        @taggable.tag_list = "ruby, python"
+        @tagger.tag(@taggable, with: "java, lisp", on: :tags)
+        expect(@taggable.all_tags_on(:tags).map(&:name).sort).to eq(%w[ruby python java lisp].sort)
+      end
+
+      it "should not add owned tags to the common list" do
+        @taggable.tag_list = "ruby, python"
+        @tagger.tag(@taggable, with: "java, lisp", on: :tags)
+        expect(@taggable.tag_list).to eq(%w[ruby python])
+        @tagger.tag(@taggable, with: "", on: :tags)
+        expect(@taggable.tag_list).to eq(%w[ruby python])
+      end
+
+      it "should throw an exception when the default is over-ridden" do
+        expect(@taggable.tag_list_on(:foo_boo)).to be_empty
+        expect(-> {
+          @tagger.tag(@taggable, with: "this, and, that", on: :foo_boo, force: false)
+        }).to raise_error(RuntimeError)
+      end
+
+      it "should not create the tag context on-the-fly when the default is over-ridden" do
+        expect(@taggable.tag_list_on(:foo_boo)).to be_empty
+        begin
+          @tagger.tag(@taggable, with: "this, and, that", on: :foo_boo, force: false)
+        rescue
+          expect(@taggable.tag_list_on(:foo_boo)).to be_empty
+        end
+      end
+    end
+
+    describe "when called by multiple tagger's" do
+      before(:each) do
+        @user_x = User.create(name: "User X")
+        @user_y = User.create(name: "User Y")
+        @taggable = TaggableModel.create(name: "make_taggable", tag_list: "plugin")
+
+        @user_x.tag(@taggable, with: "ruby, rails", on: :tags)
+        @user_y.tag(@taggable, with: "ruby, plugin", on: :tags)
+
+        @user_y.tag(@taggable, with: "", on: :tags)
+        @user_y.tag(@taggable, with: "", on: :tags)
+      end
+
+      it "should delete owned tags" do
+        expect(@user_y.owned_tags).to be_empty
+      end
+
+      it "should not delete other taggers tags" do
+        expect(@user_x.owned_tags.count).to eq(2)
+      end
+
+      it "should not delete original tags" do
+        expect(@taggable.all_tags_list_on(:tags)).to include("plugin")
+      end
+    end
+  end
+end