gutentag 2.5.2 → 2.6.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e934d0f1e941db5003a28ee2dc0ca30592db0b594f5633eeac2374f6a0ee8f9f
-  data.tar.gz: 8972fd4264e0dad5499ddbd7c9642caf8b15052fa78b63bebec6de4c57f1b9b6
+  metadata.gz: 4a57320061f9e3181385744a38c797564a7ce45c2fceb6e51ba3bc7a5e7a4d68
+  data.tar.gz: 86d6f5d761a18434e20919b1996234896f33f5764ed2f6032cca5dbe82164e2a
 SHA512:
-  metadata.gz: 55961186203575a716c658bd966102d344badbe8940f88523e6e4e76d795b8a9089be43320fd40735b749ed7f4507d3fe7d509bdd6f942dd9d2fabfb0d32e89b
-  data.tar.gz: 90f27936f1f5a96b522d26ad53ce8c74309fdc62c4409cb662112ee8603e610158ae379785646217b0cc61ea5a5e4150d31eaea77ea83033828c0d0af2db8771
+  metadata.gz: 45e211fdef9849824571e960dfe21d68eaa77ca2244375952e823e031bc112d797f55513aa7adf25c6469d399ba9f7e045c7714cf875c13a754823e91cea3f17
+  data.tar.gz: 86f48f5be03702882c074ff36d055ca735ca97ef0ae9befa1fd95e750cbcea7c44b0f00e52a4df19f8007fb361816661e7415035a083ed288fd01e4f003884b8

data/.github/workflows/ci.yml

@@ -0,0 +1,90 @@
+name: CI
+
+on: [push, pull_request]
+
+jobs:
+  test:
+    runs-on: ubuntu-18.04
+
+    strategy:
+      fail-fast: false
+      matrix:
+        ruby: [ '2.4', '2.5', '2.6', '2.7', '3.0' ]
+        bundler: [ '1.17.3', '2.1.4' ]
+        database: [ 'mysql', 'postgresql', 'sqlite' ]
+        exclude:
+          - ruby: '2.4'
+            bundler: '2.1.4'
+          - ruby: '2.5'
+            bundler: '2.1.4'
+          - ruby: '2.6'
+            bundler: '2.1.4'
+          - ruby: '2.7'
+            bundler: '1.17.3'
+          - ruby: '3.0'
+            bundler: '1.17.3'
+
+    services:
+      postgres:
+        image: postgres:10
+        env:
+          POSTGRES_USER: root
+          POSTGRES_PASSWORD: gutentag
+          POSTGRES_DB: test
+        ports: ['5432:5432']
+        # needed because the postgres container does not provide a healthcheck
+        options: --health-cmd pg_isready --health-interval 10s --health-timeout 5s --health-retries 5
+
+      mysql:
+        image: mysql:5.7
+        env:
+          MYSQL_ROOT_PASSWORD: gutentag
+          MYSQL_DATABASE: test
+        ports: ['3306:3306']
+        options: --health-cmd="mysqladmin ping" --health-interval=10s --health-timeout=5s --health-retries=3
+
+    steps:
+      - name: Check out code
+        uses: actions/checkout@v2
+      - name: Set up ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby }}
+          bundler: ${{ matrix.bundler }}
+          bundler-cache: true
+
+      - name: Set up Bundler
+        run: |
+          export BUNDLER_VERSION=${{ matrix.bundler }}
+          export BUNDLE_PATH=$PWD/vendor/bundle
+
+          gem update --system
+
+          bundle _${{ matrix.bundler }}_ config set path $PWD/$BUNDLE_PATH
+          bundle _${{ matrix.bundler }}_ install --jobs=4 --retry=3
+          bundle _${{ matrix.bundler }}_ update
+      - name: Set up Appraisal
+        run: bundle exec appraisal update
+      - name: Test
+        env:
+          CI: "true"
+          DATABASE: ${{ matrix.database }}
+          DB_USERNAME: root
+          DB_PASSWORD: gutentag
+          DB_HOST: 127.0.0.1
+        run: bundle exec appraisal rspec
+
+  rubocop:
+    runs-on: ubuntu-18.04
+
+    steps:
+      - name: Check out code
+        uses: actions/checkout@v2
+      - name: Set up ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: 2.7
+          bundler: 2.1.4
+          bundler-cache: true
+      - name: Rubocop
+        run: bundle exec rubocop

data/.gitignore

@@ -2,4 +2,5 @@
 gemfiles
 pkg/*
 spec/internal/db/*.sqlite
+spec/internal/db/migrate/
 .rubocop-*-yml

data/.rubocop.yml

@@ -7,6 +7,10 @@ AllCops:
   TargetRubyVersion: 2.3
   Exclude:
     - gemfiles/*.gemfile
+    - vendor/**/*
+
+Bundler/DuplicatedGem:
+  Enabled: false
 
 Metrics/MethodLength:
   Exclude:
@@ -20,3 +24,19 @@ Style/MultilineIfModifier:
 Style/MultilineTernaryOperator:
   Exclude:
     - db/migrate/*.rb
+
+# 0.80
+
+Style/HashEachMethods:
+  Enabled: true
+Style/HashTransformKeys:
+  Enabled: true
+Style/HashTransformValues:
+  Enabled: true
+
+# 0.81
+
+Lint/RaiseException:
+  Enabled: true
+Lint/StructNewOverride:
+  Enabled: true

data/Appraisals

@@ -17,27 +17,34 @@ appraise "rails_4_2" do
   gem "activerecord-jdbcmysql-adapter",      "~> 1.3.23", :platform => :jruby
   gem "activerecord-jdbcpostgresql-adapter", "~> 1.3.23", :platform => :jruby
   gem "activerecord-jdbcsqlite3-adapter",    "~> 1.3.23", :platform => :jruby
-end
+end if RUBY_VERSION.to_f < 2.7
 
 appraise "rails_5_0" do
   gem "rails",  "~> 5.0.3"
   gem "mysql2", "~> 0.4.0", :platform => :ruby
-end if RUBY_VERSION.to_f >= 2.2
+end if RUBY_VERSION.to_f >= 2.2 && RUBY_VERSION.to_f <= 2.7
 
 appraise "rails_5_1" do
   gem "rails",  "~> 5.1.1"
   gem "mysql2", "~> 0.4.0", :platform => :ruby
-end if RUBY_VERSION.to_f >= 2.2
+end if RUBY_VERSION.to_f >= 2.2 && RUBY_VERSION.to_f <= 2.7
 
 appraise "rails_5_2" do
   gem "rails",  "~> 5.2.0"
   gem "pg",     "~> 1.0",   :platform => :ruby
   gem "mysql2", "~> 0.5.0", :platform => :ruby
-end if RUBY_VERSION.to_f >= 2.2
+end if RUBY_VERSION.to_f >= 2.2 && RUBY_VERSION.to_f <= 2.7
 
 appraise "rails_6_0" do
-  gem "rails",   "~> 6.0.0.rc1"
+  gem "rails",   "~> 6.0.0"
   gem "pg",      "~> 1.0",   :platform => :ruby
   gem "mysql2",  "~> 0.5.0", :platform => :ruby
-  gem "sqlite3", "~> 1.4", :platform => :ruby
+  gem "sqlite3", "~> 1.4",   :platform => :ruby
+end if RUBY_VERSION.to_f >= 2.5 && RUBY_PLATFORM != "java"
+
+appraise "rails_6_1" do
+  gem "rails",   "~> 6.1.0"
+  gem "pg",      "~> 1.0",   :platform => :ruby
+  gem "mysql2",  "~> 0.5.0", :platform => :ruby
+  gem "sqlite3", "~> 1.4",   :platform => :ruby
 end if RUBY_VERSION.to_f >= 2.5 && RUBY_PLATFORM != "java"

data/CHANGELOG.md

@@ -2,6 +2,36 @@
 
 All notable changes to this project (at least, from v0.5.0 onwards) will be documented in this file.
 
+## 2.6.1 - 2021-09-18
+
+### Fixed
+
+* The initially-generated migrations now work out of the box with Rails 5.0+, using the 4.2 version. The generator will now update that version to the latest, or remove it for Rails 4.2. See later discussion in [#80](https://github.com/pat/gutentag/issues/80).
+
+## 2.6.0 - 2021-07-10
+
+### Added
+
+* Queries can now be made for objects that have _none_ of the specified tags using `:match => :none` ([Rares S](https://github.com/laleshii) in [#79](https://github.com/pat/gutentag/pull/79)).
+* Added a generator `gutentag:migration_versions` to update generated migrations so they use the current version of Rails/ActiveRecord's Migration superclass. See discussion in [#80](https://github.com/pat/gutentag/issues/80).
+
+### Changed
+
+* When adding Gutentag to a new app, the migrations require the `gutentag:migration_versions` generator to be run to ensure the latest ActiveRecord migration superclass is used. This change has no impact to existing apps. See discussion in [#80](https://github.com/pat/gutentag/issues/80).
+
+## 2.5.4 - 2021-02-21
+
+### Fixed
+
+* Don't apply the tag length validation when `ActiveRecord::ConnectionNotEstablished` exceptions are raised. ([John Duff](https://github.com/jduff) in [#77](https://github.com/pat/gutentag/pull/77)).
+
+## 2.5.3 - 2020-06-28
+
+### Fixed
+
+* Use `saved_change_to_tag_names?` instead of `tag_names_previously_changed?` for Rails 5.1+ ([Morten Trolle](https://github.com/mtrolle) in [#70](https://github.com/pat/gutentag/pull/70)).
+* `Gutentag::Tag.names_for_scope` now handles empty scopes ([Mike Gunderloy](https://github.com/ffmike) in [#73](https://github.com/pat/gutentag/pull/73)).
+
 ## 2.5.2 - 2019-07-08
 
 ### Fixed

data/Gemfile

@@ -8,7 +8,12 @@ gem "test-unit", :platform => :ruby_22
 
 gem "mysql2",  "~> 0.3",    :platform => :ruby
 gem "pg",      "~> 0.18",   :platform => :ruby
-gem "sqlite3", "~> 1.3.13", :platform => :ruby
+
+if RUBY_VERSION.to_f < 3.0
+  gem "sqlite3", "~> 1.3.13"
+else
+  gem "sqlite3", "~> 1.4"
+end
 
 gem "activerecord-jdbcmysql-adapter",      ">= 1.3.23", :platform => :jruby
 gem "activerecord-jdbcpostgresql-adapter", ">= 1.3.23", :platform => :jruby

data/README.md

@@ -1,14 +1,14 @@
 # Gutentag
 
-[![Gem Version](https://badge.fury.io/rb/gutentag.png)](http://badge.fury.io/rb/gutentag)
-[![Build Status](https://travis-ci.org/pat/gutentag.png?branch=master)](https://travis-ci.org/pat/gutentag)
-[![Code Climate](https://codeclimate.com/github/pat/gutentag.png)](https://codeclimate.com/github/pat/gutentag)
+[![Gem Version](https://badge.fury.io/rb/gutentag.svg)](http://badge.fury.io/rb/gutentag)
+[![Build Status](https://travis-ci.org/pat/gutentag.svg?branch=master)](https://travis-ci.org/pat/gutentag)
+[![Code Climate](https://codeclimate.com/github/pat/gutentag.svg)](https://codeclimate.com/github/pat/gutentag)
 
 A good, simple, solid tagging extension for ActiveRecord.
 
 This was initially built partly as a proof-of-concept, partly to see how a tagging gem could work when it's not all stuffed within models, and partly just because I wanted a simpler tagging library. It's now a solid little tagging Rails engine.
 
-If you want to know more, read [this blog post](http://freelancing-gods.com/posts/gutentag_simple_rails_tagging).
+If you want to know more, read [this blog post](http://freelancing-gods.com/posts/gutentag_simple_rails_tagging), or have a look at [the Examples page](https://github.com/pat/gutentag/wiki/Examples) in the wiki (which includes a starting point for accepting tag values in a form).
 
 ## Contents
 
@@ -67,6 +67,13 @@ To return records that have _all_ specified tags, use `:match => :all`:
 Article.tagged_with(:ids => [tag_a.id, tag_b.id], :match => :all)
 ```
 
+To return records that have _none_ of the specified tags, use `:match => :none`:
+
+```ruby
+# Returns all articles that have *neither* tag_a nor tag_b.
+Article.tagged_with(:ids => [tag_a.id, tag_b.id], :match => :none)
+```
+
 <h2 id="installation">Installation</h2>
 
 ### Dependencies
@@ -74,7 +81,7 @@ Article.tagged_with(:ids => [tag_a.id, tag_b.id], :match => :all)
 These are the versions the test suite runs against. It's possible it may work on older versions of Ruby, but it definitely won't work on older versions of Rails.
 
 * Ruby: MRI v2.3-v2.6, JRuby v9.2.5
-* Rails/ActiveRecord: v4.0-v6.0
+* Rails/ActiveRecord: v4.0-v6.1
 
 If you're using MRI v2.2 and/or ActiveRecord v3.2, the last version of Gutentag that fully supported those versions is v2.4.1.
 
@@ -83,14 +90,15 @@ If you're using MRI v2.2 and/or ActiveRecord v3.2, the last version of Gutentag
 Get it into your Gemfile - and don't forget the version constraint!
 
 ```Ruby
-gem 'gutentag', '~> 2.5'
+gem 'gutentag', '~> 2.6'
 ```
 
-Next: your tags get persisted to your database, so let's import and run the migrations to get the tables set up:
+Next: your tags get persisted to your database, so let's import the migrations, update them to your current version of Rails, and then migrate:
 
 ```Bash
-rake gutentag:install:migrations
-rake db:migrate
+bundle exec rake gutentag:install:migrations
+bundle exec rails generate gutentag:migration_versions
+bundle exec rake db:migrate
 ```
 
 If you're using UUID primary keys, make sure you alter the migration files before running `db:migrate` to use UUIDs for the `taggable_id` foreign key column (as noted in [issue 57](https://github.com/pat/gutentag/issues/57).)
@@ -171,4 +179,4 @@ Please note that this project now has a [Contributor Code of Conduct](http://con
 
 <h2 id="licence">Licence</h2>
 
-Copyright (c) 2013-2015, Gutentag is developed and maintained by Pat Allan, and is released under the open MIT Licence.
+Copyright (c) 2013-2021, Gutentag is developed and maintained by Pat Allan, and is released under the open MIT Licence.

data/app/models/gutentag/tag.rb

@@ -21,7 +21,9 @@ class Gutentag::Tag < ActiveRecord::Base
 
   def self.names_for_scope(scope)
     join_conditions = {:taggable_type => scope.name}
-    if scope.current_scope.present?
+    if scope.is_a?(ActiveRecord::Relation)
+      return Gutentag::Tag.none unless scope.current_scope.present?
+
       join_conditions[:taggable_id] = scope.select(:id)
     end
 

data/db/migrate/1_gutentag_tables.rb

@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 
-superclass = ActiveRecord::VERSION::MAJOR < 5 ?
-  ActiveRecord::Migration : ActiveRecord::Migration[4.2]
-class GutentagTables < superclass
+class GutentagTables < ActiveRecord::Migration[4.2]
   def up
     create_table :gutentag_taggings do |t|
       t.integer :tag_id,        :null => false

data/db/migrate/2_gutentag_cache_counter.rb

@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 
-superclass = ActiveRecord::VERSION::MAJOR < 5 ?
-  ActiveRecord::Migration : ActiveRecord::Migration[4.2]
-class GutentagCacheCounter < superclass
+class GutentagCacheCounter < ActiveRecord::Migration[4.2]
   def up
     add_column :gutentag_tags, :taggings_count, :integer, :default => 0
     add_index  :gutentag_tags, :taggings_count

data/db/migrate/3_no_null_counters.rb

@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 
-superclass = ActiveRecord::VERSION::MAJOR < 5 ?
-  ActiveRecord::Migration : ActiveRecord::Migration[4.2]
-class NoNullCounters < superclass
+class NoNullCounters < ActiveRecord::Migration[4.2]
   def up
     change_column :gutentag_tags, :taggings_count, :integer,
       :default => 0,

data/gutentag.gemspec

@@ -2,7 +2,7 @@
 
 Gem::Specification.new do |s|
   s.name        = "gutentag"
-  s.version     = "2.5.2"
+  s.version     = "2.6.1"
   s.authors     = ["Pat Allan"]
   s.email       = ["pat@freelancing-gods.com"]
   s.homepage    = "https://github.com/pat/gutentag"
@@ -16,12 +16,12 @@ Gem::Specification.new do |s|
 
   s.add_runtime_dependency     "activerecord", ">= 3.2.0"
 
-  s.add_development_dependency "appraisal",           "~> 2.1.0"
-  s.add_development_dependency "bundler",             "~> 1.17"
+  s.add_development_dependency "appraisal",           "~> 2.4.1"
+  s.add_development_dependency "bundler",             ">= 1.17"
   s.add_development_dependency "combustion",          "~> 1.1"
   s.add_development_dependency "database_cleaner",    "~> 1.6"
   s.add_development_dependency "rails"
   s.add_development_dependency "rspec-rails",         "~> 3.1"
-  s.add_development_dependency "rubocop",             "~> 0.72.0"
+  s.add_development_dependency "rubocop",             "~> 0.81.0"
   s.add_development_dependency "rubocop-performance", "~> 1"
 end

data/lib/gutentag/active_record/instance_methods.rb

@@ -2,6 +2,8 @@
 
 # For Rails 5.0+
 module Gutentag::ActiveRecord::InstanceMethods
+  AR_VERSION = ActiveRecord::VERSION::STRING.to_f
+
   # If the tag_names attribute was one of the modified values, then let's just
   # use the modifications, rather than overwriting the stored value.
   #
@@ -9,7 +11,12 @@ module Gutentag::ActiveRecord::InstanceMethods
   # the instance directly (e.g. article.tags << tag), which invokes the save
   # callbacks, but the old tag_names value is stored but not updated.
   def reset_tag_names
-    return if tag_names_previously_changed?
+    # Rails 5.1 introduces major changes to how ActiveModel::Dirty works:
+    # https://github.com/pat/gutentag/pull/70#issuecomment-524605448
+    # For Rails <5.1 we'll use *_previously_changed?
+    # and for 5.1+ we'll use saved_change_to_*?
+    return if AR_VERSION < 5.1 && tag_names_previously_changed?
+    return if AR_VERSION >= 5.1 && saved_change_to_tag_names?
 
     # Update the underlying value rather than going through the setter, to
     # ensure this update doesn't get marked as a 'change'.

data/lib/gutentag/engine.rb

@@ -2,4 +2,8 @@
 
 class Gutentag::Engine < Rails::Engine
   engine_name :gutentag
+
+  generators do
+    require "gutentag/generators/migration_versions_generator"
+  end
 end

data/lib/gutentag/generators/migration_versions_generator.rb

@@ -0,0 +1,50 @@
+# frozen_string_literal: true
+
+require "rails/generators"
+
+module Gutentag
+  module Generators
+    class MigrationVersionsGenerator < Rails::Generators::Base
+      desc "Update the ActiveRecord version in Gutentag migrations"
+
+      def update_migration_versions
+        superclass = "ActiveRecord::Migration[#{rails_version}]"
+
+        if ::ActiveRecord::VERSION::MAJOR < 5
+          superclass = "ActiveRecord::Migration"
+        end
+
+        migration_files.each do |file|
+          gsub_file file,
+            /< ActiveRecord::Migration\[4\.2\]$/,
+            "< #{superclass}"
+        end
+      end
+
+      private
+
+      def migration_files
+        Dir[Rails.root.join("db/migrate/*.rb")].select do |path|
+          known_migration_names.any? do |known|
+            File.basename(path)[/\A\d+_#{known}\.gutentag.rb\z/]
+          end
+        end
+      end
+
+      def known_migration_names
+        @known_migration_names ||= begin
+          Dir[File.join(__dir__, "../../../db/migrate/*.rb")].collect do |path|
+            File.basename(path).gsub(/\A\d+_/, "").gsub(/\.rb\z/, "")
+          end
+        end
+      end
+
+      def rails_version
+        @rails_version ||= [
+          ::ActiveRecord::VERSION::MAJOR,
+          ::ActiveRecord::VERSION::MINOR
+        ].join(".")
+      end
+    end
+  end
+end

data/lib/gutentag/tag_validations.rb

@@ -13,6 +13,7 @@ class Gutentag::TagValidations
     if ActiveRecord::VERSION::STRING.to_f > 4.0
       classes << ActiveRecord::NoDatabaseError
     end
+    classes << ActiveRecord::ConnectionNotEstablished
     classes << Mysql2::Error     if defined?(::Mysql2)
     classes << PG::ConnectionBad if defined?(::PG)
     classes

data/lib/gutentag/tagged_with/query.rb

@@ -8,7 +8,7 @@ class Gutentag::TaggedWith::Query
   end
 
   def call
-    model.where "#{model_id} IN (#{query.to_sql})"
+    model.where "#{model_id} #{operator} (#{query.to_sql})"
   end
 
   private
@@ -20,8 +20,16 @@ class Gutentag::TaggedWith::Query
   end
 
   def query
-    return taggable_ids_query if match == :any || values.length == 1
+    return taggable_ids_query if match_any_or_none? || values.length == 1
 
     taggable_ids_query.having("COUNT(*) = #{values.length}").group(:taggable_id)
   end
+
+  def operator
+    match == :none ? "NOT IN" : "IN"
+  end
+
+  def match_any_or_none?
+    %i[any none].include?(match)
+  end
 end

data/spec/acceptance/removing_unused_spec.rb

@@ -2,7 +2,7 @@
 
 require "spec_helper"
 
-describe "Removing unused" do
+RSpec.describe "Removing unused" do
   let(:article) { Article.create }
 
   it "deletes only unused tags" do

data/spec/acceptance/tag_names_for_scope_spec.rb

@@ -27,4 +27,12 @@ RSpec.describe "Tag names for scopes" do
     expect(Gutentag::Tag.names_for_scope(Article)).
       to match_array(%w[ koala wombat cassowary ])
   end
+
+  it "returns an empty array for an empty scope" do
+    Article.create :title => "mammals", :tag_names => %w[ koala wombat ]
+    Article.create :title => "birds",   :tag_names => %w[ cassowary ]
+
+    expect(Gutentag::Tag.names_for_scope(Article.where(:title => "reptiles"))).
+      to match_array([])
+  end
 end

data/spec/acceptance/tag_names_spec.rb

@@ -2,7 +2,7 @@
 
 require "spec_helper"
 
-describe "Managing tags via names" do
+RSpec.describe "Managing tags via names" do
   let(:article) { Article.create }
 
   it "returns tag names" do

data/spec/acceptance/tags_spec.rb

@@ -2,7 +2,7 @@
 
 require "spec_helper"
 
-describe "Adding and removing tags" do
+RSpec.describe "Adding and removing tags" do
   let(:article)  { Article.create }
   let(:pancakes) { Gutentag::Tag.create :name => "pancakes" }
 

data/spec/gutentag/active_record_spec.rb

@@ -2,7 +2,7 @@
 
 require "spec_helper"
 
-describe Gutentag::ActiveRecord do
+RSpec.describe Gutentag::ActiveRecord do
   describe ".tagged_with" do
     let!(:melbourne_article) do
       article = Article.create :title => "Overview"
@@ -160,6 +160,26 @@ describe Gutentag::ActiveRecord do
       it { is_expected.not_to include oregon_article }
     end
 
+    context "matching excluding one tag" do
+      subject do
+        Article.tagged_with(:names => %w[ melbourne ], :match => :none)
+      end
+
+      it { expect(subject.count).to eq 1 }
+      it { is_expected.to include oregon_article }
+      it do
+        is_expected.not_to include melbourne_oregon_article, melbourne_article
+      end
+    end
+
+    context "matching excluding all tags" do
+      subject do
+        Article.tagged_with(:names => %w[ melbourne oregon ], :match => :none)
+      end
+
+      it { expect(subject.count).to eq 0 }
+    end
+
     it "should work on STI subclasses" do
       thinkpiece = Thinkpiece.create! :tag_names => ["pancakes"]
 

data/spec/gutentag_spec.rb

@@ -2,7 +2,7 @@
 
 require "spec_helper"
 
-describe Gutentag do
+RSpec.describe Gutentag do
   describe ".normalizer" do
     it "downcases the provided name" do
       expect(Gutentag.normaliser.call("Tasty Pancakes")).to eq("tasty pancakes")

data/spec/internal/config/database.yml

@@ -2,11 +2,26 @@
 test:
   adapter: mysql2
   database: gutentag
-  username: root
+  username: <%= ENV.fetch("DB_USERNAME", "root") %>
+  <% if ENV["DB_HOST"] %>
+  host: <%= ENV["DB_HOST"] %>
+  <% end %>
+  <% if ENV["DB_PASSWORD"] %>
+  password: <%= ENV["DB_PASSWORD"] %>
+  <% end %>
 <% elsif ENV["DATABASE"] == "postgres" %>
 test:
   adapter: postgresql
   database: gutentag
+  <% if ENV["DB_HOST"] %>
+  host: <%= ENV["DB_HOST"] %>
+  <% end %>
+  <% if ENV["DB_USERNAME"] %>
+  username: <%= ENV["DB_USERNAME"] %>
+  <% end %>
+  <% if ENV["DB_PASSWORD"] %>
+  password: <%= ENV["DB_PASSWORD"] %>
+  <% end %>
 <% else %>
 test:
   adapter: sqlite3

data/spec/models/gutentag/tag_spec.rb

@@ -2,7 +2,7 @@
 
 require "spec_helper"
 
-describe Gutentag::Tag, :type => :model do
+RSpec.describe Gutentag::Tag, :type => :model do
   describe ".find_by_name" do
     it "returns a tag with the same name" do
       existing = Gutentag::Tag.create! :name => "pancakes"

data/spec/models/gutentag/tagging_spec.rb

@@ -2,7 +2,7 @@
 
 require "spec_helper"
 
-describe Gutentag::Tagging, :type => :model do
+RSpec.describe Gutentag::Tagging, :type => :model do
   describe "#valid?" do
     let(:tag)      { Gutentag::Tag.create! :name => "pancakes" }
     let(:taggable) { Article.create! }

data/spec/spec_helper.rb

@@ -4,14 +4,17 @@ require "bundler/setup"
 
 Bundler.require :default, :development
 
-Dir["#{__dir__}/support/**/*.rb"].each { |file| require file }
+Dir["#{__dir__}/support/**/*.rb"].sort.each { |file| require file }
 
-Combustion.initialize! :active_record
+Combustion.initialize! :active_record, :database_migrate => false
+AdjustMigrations.call(Combustion::Application)
 ActiveSupport.run_load_hooks :gutentag unless defined?(Gutentag::Engine)
 
 require "rspec/rails"
 
 RSpec.configure do |config|
+  config.disable_monkey_patching!
+
   if config.respond_to?(:use_transactional_tests)
     config.use_transactional_tests = false
   else

data/spec/support/adjust_migrations.rb

@@ -0,0 +1,63 @@
+# frozen_string_literal: true
+
+require "fileutils"
+require "gutentag/generators/migration_versions_generator"
+
+class AdjustMigrations
+  def self.call(application)
+    new(application).call
+  end
+
+  def initialize(application)
+    @application = application
+  end
+
+  def call
+    copy_migrations_to_app
+    run_generator
+    run_migrations
+  end
+
+  private
+
+  attr_reader :application
+
+  def copy_migrations_to_app
+    FileUtils.mkdir_p application.root.join("db/migrate")
+
+    Dir["#{__dir__}/../../db/migrate/*.rb"].each do |file|
+      name = File.basename(file.gsub(/rb\z/, "gutentag.rb"))
+      destination = application.root.join("db/migrate/#{name}")
+
+      FileUtils.cp file, destination.to_s
+    end
+  end
+
+  def migration_context
+    if ActiveRecord::MigrationContext.instance_method(:initialize).arity <= 1
+      ActiveRecord::MigrationContext.new migration_paths
+    else
+      ActiveRecord::MigrationContext.new(
+        migration_paths, ActiveRecord::Base.connection.schema_migration
+      )
+    end
+  end
+
+  def migration_paths
+    application.root.join("db/migrate")
+  end
+
+  def run_generator
+    Gutentag::Generators::MigrationVersionsGenerator.start(
+      ["--quiet"], :destination_root => application.root
+    )
+  end
+
+  def run_migrations
+    if ActiveRecord::VERSION::STRING.to_f >= 5.2
+      migration_context.migrate
+    else
+      ActiveRecord::Migrator.migrate migration_paths, nil
+    end
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gutentag
 version: !ruby/object:Gem::Version
-  version: 2.5.2
+  version: 2.6.1
 platform: ruby
 authors:
 - Pat Allan
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-07-08 00:00:00.000000000 Z
+date: 2021-09-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -30,26 +30,26 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.1.0
+        version: 2.4.1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.1.0
+        version: 2.4.1
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '1.17'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '1.17'
 - !ruby/object:Gem::Dependency
@@ -114,14 +114,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.72.0
+        version: 0.81.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.72.0
+        version: 0.81.0
 - !ruby/object:Gem::Dependency
   name: rubocop-performance
   requirement: !ruby/object:Gem::Requirement
@@ -143,9 +143,9 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/ci.yml"
 - ".gitignore"
 - ".rubocop.yml"
-- ".travis.yml"
 - Appraisals
 - CHANGELOG.md
 - Gemfile
@@ -167,6 +167,7 @@ files:
 - lib/gutentag/change_state.rb
 - lib/gutentag/dirty.rb
 - lib/gutentag/engine.rb
+- lib/gutentag/generators/migration_versions_generator.rb
 - lib/gutentag/persistence.rb
 - lib/gutentag/remove_unused.rb
 - lib/gutentag/tag_names.rb
@@ -191,6 +192,7 @@ files:
 - spec/models/gutentag/tag_spec.rb
 - spec/models/gutentag/tagging_spec.rb
 - spec/spec_helper.rb
+- spec/support/adjust_migrations.rb
 - spec/support/mysql.rb
 homepage: https://github.com/pat/gutentag
 licenses:
@@ -211,7 +213,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
+rubygems_version: 3.1.2
 signing_key: 
 specification_version: 4
 summary: Good Tags

data/.travis.yml

@@ -1,22 +0,0 @@
-language: ruby
-dist: xenial
-script: bundle exec appraisal rake
-rvm:
-  - 2.3.8
-  - 2.4.6
-  - 2.5.5
-  - 2.6.3
-  # - jruby-9.2.8.0
-before_install:
-  - gem install bundler --version 1.17.3
-install:
-  - bundle _1.17.3_ install --jobs=3 --retry=3
-before_script:
-  - bundle exec appraisal install
-env:
-  - DATABASE=postgres
-  - DATABASE=mysql
-  - DATABASE=sqlite
-services:
-- mysql
-- postgresql