outrigger 3.1.0 → 3.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6779702303231a2966e2df6e7a143f6b6f17426461a7c9fa932454e49a0fbb4b
-  data.tar.gz: 8f750172ee8f965253677fa83fff8dd79c2e30d8bc7bbc9f61f98075c16483cb
+  metadata.gz: 1e9ee6034865548d34c7f9927301ffc5543f057f074fb09f3cab47e429f1241e
+  data.tar.gz: 9df796e4a6ec0df60c6f4afa2ae13f55f02a7289824ae41db1c02a4b6ee977b9
 SHA512:
-  metadata.gz: 0d7314a030ba43659313def727ea02155770d129047fd05ba845fb99694e4c14e5c37d8639f29ec7255633181f06182770ed8dbcb87122efe8a6e631ffd9c437
-  data.tar.gz: ec484adf66017fe3e871c5a380fda4db3380683ad443ad2c4e066541e7bad0a5fa687133efeba3fea3f627abd252106d06031c765144aaf08d0a1d47e6ba6062
+  metadata.gz: 326bc1cbd0665a6df25b146141119511be9a4b0257877913a699ff1bd48a00562527b60153c8b5506885c9d4bd0e0edd077abd7a44d6d068c955242b1dda915b
+  data.tar.gz: 2bee38a6e7c046708ecacd7a64742ef5f1c498afb3e41e1bdb50485abe5567d5a10093f6ee717c67ed823836fb478bd0b873ff419578d75c6ad8d58ce24c2b69

data/Rakefile

@@ -1,9 +1,9 @@
 # frozen_string_literal: true
 
-require 'bundler/gem_tasks'
+require "bundler/gem_tasks"
 
 begin
-  require 'rspec/core/rake_task'
+  require "rspec/core/rake_task"
 
   RSpec::Core::RakeTask.new(:spec)
 

data/lib/outrigger/cops/migration/tagged.rb

@@ -3,7 +3,7 @@
 module RuboCop
   module Cop
     module Migration
-      class Tagged < Cop
+      class Tagged < Base
         def on_class(node)
           _name, _superclass, body = *node
           return unless body && migration_class?(node)
@@ -32,17 +32,14 @@ module RuboCop
 
           if allowed_tags.empty?
             add_offense tag,
-                        location: :expression,
-                        message: 'No allowed tags have been defined in the RuboCop configuration.'
+                        message: "No allowed tags have been defined in the RuboCop configuration."
           elsif tag
             return if allowed_tags.include? tag.children.last.to_a.last
 
             add_offense tag,
-                        location: :expression,
                         message: "Tags may only be one of #{allowed_tags}."
           else
             add_offense klass,
-                        location: :expression,
                         message: "All migrations require a tag from #{allowed_tags}."
           end
         end
@@ -54,7 +51,7 @@ module RuboCop
         end
 
         def allowed_tags
-          cop_config['AllowedTags'].map(&:to_sym)
+          cop_config["AllowedTags"].map(&:to_sym)
         end
       end
     end

data/lib/outrigger/railtie.rb

@@ -5,10 +5,10 @@ module Outrigger
     railtie_name :taggable_migrations
 
     rake_tasks do
-      load 'tasks/outrigger.rake'
+      load "tasks/outrigger.rake"
     end
 
-    initializer 'extend_migrations' do
+    initializer "extend_migrations" do
       ActiveSupport.on_load :active_record do
         ActiveRecord::Migration.include(Outrigger::Taggable)
         ActiveRecord::MigrationProxy.include(Outrigger::TaggableProxy)

data/lib/outrigger/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Outrigger
-  VERSION = '3.1.0'
+  VERSION = "3.2.0"
 end

data/lib/outrigger.rb

@@ -1,12 +1,12 @@
 # frozen_string_literal: true
 
-require 'active_record' unless defined? ActiveRecord
+require "active_record" unless defined? ActiveRecord
 
-require 'outrigger/migrator'
-require 'outrigger/taggable'
-require 'outrigger/taggable_proxy'
+require "outrigger/migrator"
+require "outrigger/taggable"
+require "outrigger/taggable_proxy"
 
-require 'outrigger/railtie' if defined? Rails::Railtie
+require "outrigger/railtie" if defined? Rails::Railtie
 
 module Outrigger
   class << self
@@ -18,7 +18,7 @@ module Outrigger
     end
 
     def migration_context
-      if ActiveRecord.version < Gem::Version.new('7.2')
+      if ActiveRecord.version < Gem::Version.new("7.2")
         ActiveRecord::Base.connection.migration_context
       else
         ActiveRecord::Base.connection_pool.migration_context

data/lib/tasks/outrigger.rake

@@ -2,7 +2,7 @@
 
 namespace :db do
   namespace :migrate do
-    desc 'Run migrations for a Tag'
+    desc "Run migrations for a Tag"
     task tagged: %i[environment load_config] do |_t, args|
       puts("Migrating Tags: #{args.extras}")
 

data/spec/outrigger/migrator_spec.rb

@@ -10,14 +10,16 @@ module Outrigger
     end
 
     let(:migrations) do
-      [PostDeployMigration.new(nil, 1), UntaggedMigration.new(nil, 2), PreDeployMigration.new(nil, 3),
+      [PostDeployMigration.new(nil, 1),
+       UntaggedMigration.new(nil, 2),
+       PreDeployMigration.new(nil, 3),
        MultiMigration.new(nil, 4)]
     end
 
     let(:versions) { [] }
     let(:direction) { :up }
     let(:schema_migration) do
-      if ActiveRecord.version < Gem::Version.new('7.1')
+      if ActiveRecord.version < Gem::Version.new("7.1")
         object_double(ActiveRecord::SchemaMigration, create_table: nil, all_versions: versions)
       else
         instance_double(ActiveRecord::SchemaMigration, create_table: nil, integer_versions: versions)
@@ -25,7 +27,7 @@ module Outrigger
     end
 
     let(:migrator) do
-      if ActiveRecord.version < Gem::Version.new('7.1')
+      if ActiveRecord.version < Gem::Version.new("7.1")
         ActiveRecord::Migrator.new(direction, migrations, schema_migration)
       else
         internal_metadata = instance_double(ActiveRecord::InternalMetadata, create_table: nil)
@@ -37,7 +39,7 @@ module Outrigger
       allow(ActiveRecord::InternalMetadata).to receive(:create_table)
     end
 
-    it 'sorts' do
+    it "sorts" do
       expect(migrator.runnable.map(&:class)).to eq(
         [
           PreDeployMigration, MultiMigration, UntaggedMigration, PostDeployMigration
@@ -45,11 +47,11 @@ module Outrigger
       )
     end
 
-    context 'when going down' do
+    context "when going down" do
       let(:versions) { [1, 2, 3, 4] }
       let(:direction) { :down }
 
-      it 'reverse sorts' do
+      it "reverse sorts" do
         expect(migrator.runnable.map(&:class)).to eq(
           [
             PostDeployMigration, UntaggedMigration, MultiMigration, PreDeployMigration

data/spec/outrigger/outrigger_spec.rb

@@ -1,14 +1,14 @@
 # frozen_string_literal: true
 
 describe Outrigger do
-  describe 'filter' do
-    it 'returns a proc that tests migrations' do
+  describe "filter" do
+    it "returns a proc that tests migrations" do
       filter = described_class.filter(:predeploy)
 
       expect(filter.call(PreDeployMigration)).to be(true)
     end
 
-    it 'accepts multiple tags' do
+    it "accepts multiple tags" do
       filter = described_class.filter(:predeploy, :postdeploy)
 
       expect(filter.call(MultiMigration)).to be(true)

data/spec/outrigger/railtie_spec.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 describe Outrigger::Railtie do
-  it 'provides a railtie_name' do
-    expect(described_class.railtie_name).to eq 'taggable_migrations'
+  it "provides a railtie_name" do
+    expect(described_class.railtie_name).to eq "taggable_migrations"
   end
 end

data/spec/outrigger/taggable_proxy_spec.rb

@@ -7,7 +7,7 @@ class TestProxy
 end
 
 describe Outrigger::TaggableProxy do
-  it 'delegates tags to the migration' do
+  it "delegates tags to the migration" do
     proxy = TestProxy.new
     proxy.migration = PreDeployMigration.new
 

data/spec/outrigger/taggable_spec.rb

@@ -1,19 +1,19 @@
 # frozen_string_literal: true
 
 describe Outrigger::Taggable do
-  it 'PreDeployMigration should be predeploy' do
+  it "PreDeployMigration should be predeploy" do
     expect(PreDeployMigration.tags).to eq([:predeploy])
   end
 
-  it 'UntaggedMigration should be have no tags' do
+  it "UntaggedMigration should be have no tags" do
     expect(UntaggedMigration.tags).to eq([])
   end
 
-  it 'PostDeployMigration should be predeploy' do
+  it "PostDeployMigration should be predeploy" do
     expect(PostDeployMigration.tags).to eq([:postdeploy])
   end
 
-  it 'instance tags should point to class tags' do
+  it "instance tags should point to class tags" do
     expect(PreDeployMigration.new.tags).to eq([:predeploy])
   end
 end

data/spec/rubocop/cop/migration/tagged_spec.rb

@@ -0,0 +1,91 @@
+# frozen_string_literal: true
+
+require "outrigger/cops/migration/tagged"
+
+RSpec.describe RuboCop::Cop::Migration::Tagged do
+  subject(:cop) { described_class.new(config) }
+
+  let(:config_hash) do
+    {
+      "Migration/Tagged" => {
+        "AllowedTags" => %w[predeploy deploy],
+        "Enabled" => true
+      }
+    }
+  end
+
+  let(:config) { RuboCop::Config.new(config_hash) }
+
+  let(:migration_class) do
+    <<~RUBY
+      class Test < ActiveRecord::Migration[4.2]
+        tag :predeploy
+        def change
+        end
+      end
+    RUBY
+  end
+
+  shared_examples_for "valid migrations" do
+    it "passes valid versioned migration" do
+      offenses = inspect_source(migration_class)
+      expect(offenses.empty?).to be(true)
+    end
+  end
+
+  context "with valid config" do
+    it_behaves_like "valid migrations"
+
+    context "with missing tags" do
+      let(:migration_class) do
+        <<~RUBY
+          class Test < ActiveRecord::Migration[4.2]
+            def change
+            end
+          end
+        RUBY
+      end
+
+      it "finds missing tag in versioned migration" do
+        offenses = inspect_source(migration_class)
+        expect(offenses.empty?).to be(false)
+        expect(offenses.first.message).to match(/All migrations require a tag from/)
+      end
+    end
+
+    context "with invalid tag" do
+      let(:migration_class) do
+        <<~RUBY
+          class Test < ActiveRecord::Migration[4.2]
+            tag :foobar
+            def change
+            end
+          end
+        RUBY
+      end
+
+      it "fails on invalid tag in versioned migration" do
+        offenses = inspect_source(migration_class)
+        expect(offenses.empty?).to be(false)
+        expect(offenses.first.message).to match(/Tags may only be one of/)
+      end
+    end
+  end
+
+  context "with invalid config" do
+    let(:config_hash) do
+      {
+        "Migration/Tagged" => {
+          "AllowedTags" => [],
+          "Enabled" => true
+        }
+      }
+    end
+
+    it "fails on missing tags in configuration on versioned migration" do
+      offenses = inspect_source(migration_class)
+      expect(offenses.empty?).to be(false)
+      expect(offenses.first.message).to match(/No allowed tags have been defined/)
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -1,20 +1,21 @@
 # frozen_string_literal: true
 
-require 'simplecov'
+require "simplecov"
 SimpleCov.start do
-  add_filter 'lib/outrigger/version.rb'
-  add_filter 'spec'
-  track_files 'lib/**/*.rb'
+  add_filter "lib/outrigger/version.rb"
+  add_filter "spec"
+  track_files "lib/**/*.rb"
 end
 SimpleCov.minimum_coverage(85)
 
-require 'bundler/setup'
-require 'byebug'
-require 'rails/railtie'
-require 'rubocop'
-require 'rubocop/rspec/support'
+require "bundler/setup"
+require "debug"
+require "rails"
+require "rails/railtie"
+require "rubocop"
+require "rubocop/rspec/support"
 
-require 'outrigger'
+require "outrigger"
 
 ActiveRecord::Migration.include(Outrigger::Taggable)
 ActiveRecord::MigrationProxy.include(Outrigger::TaggableProxy)
@@ -36,5 +37,5 @@ class MultiMigration < ActiveRecord::Migration[5.0]
 end
 
 RSpec.configure do |config|
-  config.order = 'random'
+  config.order = "random"
 end

metadata

@@ -1,14 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: outrigger
 version: !ruby/object:Gem::Version
-  version: 3.1.0
+  version: 3.2.0
 platform: ruby
 authors:
 - Drew Bowman
-autorequire:
 bindir: bin
 cert_chain: []
-date: 2025-08-26 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -19,7 +18,7 @@ dependencies:
         version: '6.0'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '8.1'
+        version: '8.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -29,7 +28,7 @@ dependencies:
         version: '6.0'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '8.1'
+        version: '8.2'
 - !ruby/object:Gem::Dependency
   name: railties
   requirement: !ruby/object:Gem::Requirement
@@ -39,7 +38,7 @@ dependencies:
         version: '6.0'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '8.1'
+        version: '8.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -49,121 +48,8 @@ dependencies:
         version: '6.0'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '8.1'
-- !ruby/object:Gem::Dependency
-  name: bundler
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '2.2'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '2.2'
-- !ruby/object:Gem::Dependency
-  name: byebug
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '11.1'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '11.1'
-- !ruby/object:Gem::Dependency
-  name: rake
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '13.0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '13.0'
-- !ruby/object:Gem::Dependency
-  name: rspec
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.7'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.7'
-- !ruby/object:Gem::Dependency
-  name: rubocop
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.20'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.20'
-- !ruby/object:Gem::Dependency
-  name: rubocop-rake
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0.6'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0.6'
-- !ruby/object:Gem::Dependency
-  name: rubocop-rspec
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.6'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.6'
-- !ruby/object:Gem::Dependency
-  name: simplecov
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0.21'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '0.21'
+        version: '8.2'
 description: Migrations
-email:
 executables: []
 extensions: []
 extra_rdoc_files: []
@@ -177,19 +63,18 @@ files:
 - lib/outrigger/taggable_proxy.rb
 - lib/outrigger/version.rb
 - lib/tasks/outrigger.rake
-- spec/outrigger/cops/migration/tagged_spec.rb
 - spec/outrigger/migrator_spec.rb
 - spec/outrigger/outrigger_spec.rb
 - spec/outrigger/railtie_spec.rb
 - spec/outrigger/taggable_proxy_spec.rb
 - spec/outrigger/taggable_spec.rb
+- spec/rubocop/cop/migration/tagged_spec.rb
 - spec/spec_helper.rb
 homepage: https://github.com/instructure/outrigger
 licenses:
 - MIT
 metadata:
   rubygems_mfa_required: 'true'
-post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -204,8 +89,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.4.19
-signing_key:
+rubygems_version: 3.6.9
 specification_version: 4
 summary: Tag migrations and run them separately
 test_files: []

data/spec/outrigger/cops/migration/tagged_spec.rb

@@ -1,91 +0,0 @@
-# frozen_string_literal: true
-
-require 'outrigger/cops/migration/tagged'
-
-RSpec.describe RuboCop::Cop::Migration::Tagged do
-  let(:config_hash) do
-    {
-      'Migration/Tagged' => {
-        'AllowedTags' => %w[predeploy deploy],
-        'Enabled' => true
-      }
-    }
-  end
-
-  let(:config) { RuboCop::Config.new(config_hash) }
-
-  let(:migration_class) do
-    <<~RUBY
-      class Test < ActiveRecord::Migration[4.2]
-        tag :predeploy
-        def change
-        end
-      end
-    RUBY
-  end
-
-  subject(:cop) { described_class.new(config) } # rubocop:disable RSpec/LeadingSubject
-
-  shared_examples_for 'valid migrations' do
-    it 'passes valid versioned migration' do
-      inspect_source(migration_class)
-      expect(cop.offenses.empty?).to be(true)
-    end
-  end
-
-  context 'with valid config' do
-    it_behaves_like 'valid migrations'
-
-    context 'with missing tags' do
-      let(:migration_class) do
-        <<~RUBY
-          class Test < ActiveRecord::Migration[4.2]
-            def change
-            end
-          end
-        RUBY
-      end
-
-      it 'finds missing tag in versioned migration' do
-        inspect_source(migration_class)
-        expect(cop.offenses.empty?).to be(false)
-        expect(cop.offenses.first.message).to match(/All migrations require a tag from/)
-      end
-    end
-
-    context 'with invalid tag' do
-      let(:migration_class) do
-        <<~RUBY
-          class Test < ActiveRecord::Migration[4.2]
-            tag :foobar
-            def change
-            end
-          end
-        RUBY
-      end
-
-      it 'fails on invalid tag in versioned migration' do
-        inspect_source(migration_class)
-        expect(cop.offenses.empty?).to be(false)
-        expect(cop.offenses.first.message).to match(/Tags may only be one of/)
-      end
-    end
-  end
-
-  context 'with invalid config' do
-    let(:config_hash) do
-      {
-        'Migration/Tagged' => {
-          'AllowedTags' => [],
-          'Enabled' => true
-        }
-      }
-    end
-
-    it 'fails on missing tags in configuration on versioned migration' do
-      inspect_source(migration_class)
-      expect(cop.offenses.empty?).to be(false)
-      expect(cop.offenses.first.message).to match(/No allowed tags have been defined/)
-    end
-  end
-end