outrigger 1.2.3 → 2.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a63553d16f5a85d6d18cea29ea8a07f9d5f1f0a9
-  data.tar.gz: b3a9636b7851d488a152fbc9ed03dac3b9345b75
+  metadata.gz: e82a9874959275eb5b4654aeb4374115ce636d83
+  data.tar.gz: 2d769c28f0494b2d1a75b06fc5d6a81d804953ec
 SHA512:
-  metadata.gz: 407cc16bda5c6c9f80294b887c057529f514dd416397a512703fc33dcaba3fba31159b2c7256d6760a1f49a8dc39eb1509e34317c47b3e3cbc1f39e8850dffcc
-  data.tar.gz: 28dc388ab405ed7c03352577f21fa9e995fa35307701eb7fc46a922f69e4d28062934c9aa5be08154a3977c8afc7985e91f60fcfca8c4e703983d5f803715751
+  metadata.gz: 71cf03df7b484f547bcef6b2a6022b6fdfecd2133b0b35ecd235f0fd50404ca0ddd267a0c3c12674be902d2112d54f72248ea80d8edf4823866be0f800ad4d2e
+  data.tar.gz: 59e5e4940e5c7e1836f2d73430b001ba05481461bb25d6b6be4c81370a1b238f6e54a7a76f9187ecda3008f24bbbf3b77ff2cbe76a44a7dd4f7adcf5167bdb89

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

@@ -28,18 +28,18 @@ module RuboCop
           tag = tag_node(node)
 
           if allowed_tags.empty?
-            add_offense(tag,
-              location: :expression,
-              message: 'No allowed tags have been defined in the RuboCop configuration.')
+            add_offense tag,
+                        location: :expression,
+                        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}.")
+            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}.")
+            add_offense klass,
+                        location: :expression,
+                        message: "All migrations require a tag from #{allowed_tags}."
           end
         end
 

data/lib/outrigger/version.rb

@@ -1,3 +1,3 @@
 module Outrigger
-  VERSION = '1.2.3'.freeze
+  VERSION = '2.0.0'.freeze
 end

data/spec/gemfiles/rails-5.0.gemfile

@@ -2,4 +2,4 @@ source 'https://rubygems.org'
 
 gemspec path: '../../'
 
-gem 'rails', '~> 5.0.0'
+gem 'rails', '~> 5.0.7'

data/spec/gemfiles/rails-5.0.gemfile.lock

@@ -1,131 +1,129 @@
 PATH
   remote: ../..
   specs:
-    outrigger (1.2.0)
-      activerecord (>= 4.2, < 5.2)
+    outrigger (1.3.0)
+      activerecord (>= 5.0, < 5.3)
 
 GEM
   remote: https://rubygems.org/
   specs:
-    actioncable (5.0.6)
-      actionpack (= 5.0.6)
+    actioncable (5.0.7)
+      actionpack (= 5.0.7)
       nio4r (>= 1.2, < 3.0)
       websocket-driver (~> 0.6.1)
-    actionmailer (5.0.6)
-      actionpack (= 5.0.6)
-      actionview (= 5.0.6)
-      activejob (= 5.0.6)
+    actionmailer (5.0.7)
+      actionpack (= 5.0.7)
+      actionview (= 5.0.7)
+      activejob (= 5.0.7)
       mail (~> 2.5, >= 2.5.4)
       rails-dom-testing (~> 2.0)
-    actionpack (5.0.6)
-      actionview (= 5.0.6)
-      activesupport (= 5.0.6)
+    actionpack (5.0.7)
+      actionview (= 5.0.7)
+      activesupport (= 5.0.7)
       rack (~> 2.0)
       rack-test (~> 0.6.3)
       rails-dom-testing (~> 2.0)
       rails-html-sanitizer (~> 1.0, >= 1.0.2)
-    actionview (5.0.6)
-      activesupport (= 5.0.6)
+    actionview (5.0.7)
+      activesupport (= 5.0.7)
       builder (~> 3.1)
       erubis (~> 2.7.0)
       rails-dom-testing (~> 2.0)
       rails-html-sanitizer (~> 1.0, >= 1.0.3)
-    activejob (5.0.6)
-      activesupport (= 5.0.6)
+    activejob (5.0.7)
+      activesupport (= 5.0.7)
       globalid (>= 0.3.6)
-    activemodel (5.0.6)
-      activesupport (= 5.0.6)
-    activerecord (5.0.6)
-      activemodel (= 5.0.6)
-      activesupport (= 5.0.6)
+    activemodel (5.0.7)
+      activesupport (= 5.0.7)
+    activerecord (5.0.7)
+      activemodel (= 5.0.7)
+      activesupport (= 5.0.7)
       arel (~> 7.0)
-    activesupport (5.0.6)
+    activesupport (5.0.7)
       concurrent-ruby (~> 1.0, >= 1.0.2)
-      i18n (~> 0.7)
+      i18n (>= 0.7, < 2)
       minitest (~> 5.1)
       tzinfo (~> 1.1)
     arel (7.1.4)
-    ast (2.3.0)
+    ast (2.4.0)
     builder (3.2.3)
     concurrent-ruby (1.0.5)
-    crass (1.0.2)
+    crass (1.0.4)
     diff-lcs (1.3)
-    docile (1.1.5)
+    docile (1.3.1)
     erubis (2.7.0)
-    globalid (0.4.0)
+    globalid (0.4.1)
       activesupport (>= 4.2.0)
-    i18n (0.8.6)
+    i18n (1.0.1)
+      concurrent-ruby (~> 1.0)
     json (2.1.0)
-    loofah (2.1.1)
+    loofah (2.2.2)
       crass (~> 1.0.2)
       nokogiri (>= 1.5.9)
-    mail (2.6.6)
-      mime-types (>= 1.16, < 4)
+    mail (2.7.0)
+      mini_mime (>= 0.1.1)
     method_source (0.9.0)
-    mime-types (3.1)
-      mime-types-data (~> 3.2015)
-    mime-types-data (3.2016.0521)
+    mini_mime (1.0.0)
     mini_portile2 (2.3.0)
-    minitest (5.10.3)
-    nio4r (2.1.0)
-    nokogiri (1.8.1)
+    minitest (5.11.3)
+    nio4r (2.3.1)
+    nokogiri (1.8.2)
       mini_portile2 (~> 2.3.0)
-    parallel (1.12.0)
-    parser (2.4.0.0)
-      ast (~> 2.2)
+    parallel (1.12.1)
+    parser (2.5.1.0)
+      ast (~> 2.4.0)
     powerpack (0.1.1)
-    rack (2.0.3)
+    rack (2.0.5)
     rack-test (0.6.3)
       rack (>= 1.0)
-    rails (5.0.6)
-      actioncable (= 5.0.6)
-      actionmailer (= 5.0.6)
-      actionpack (= 5.0.6)
-      actionview (= 5.0.6)
-      activejob (= 5.0.6)
-      activemodel (= 5.0.6)
-      activerecord (= 5.0.6)
-      activesupport (= 5.0.6)
+    rails (5.0.7)
+      actioncable (= 5.0.7)
+      actionmailer (= 5.0.7)
+      actionpack (= 5.0.7)
+      actionview (= 5.0.7)
+      activejob (= 5.0.7)
+      activemodel (= 5.0.7)
+      activerecord (= 5.0.7)
+      activesupport (= 5.0.7)
       bundler (>= 1.3.0)
-      railties (= 5.0.6)
+      railties (= 5.0.7)
       sprockets-rails (>= 2.0.0)
     rails-dom-testing (2.0.3)
       activesupport (>= 4.2.0)
       nokogiri (>= 1.6)
-    rails-html-sanitizer (1.0.3)
-      loofah (~> 2.0)
-    railties (5.0.6)
-      actionpack (= 5.0.6)
-      activesupport (= 5.0.6)
+    rails-html-sanitizer (1.0.4)
+      loofah (~> 2.2, >= 2.2.2)
+    railties (5.0.7)
+      actionpack (= 5.0.7)
+      activesupport (= 5.0.7)
       method_source
       rake (>= 0.8.7)
       thor (>= 0.18.1, < 2.0)
-    rainbow (2.2.2)
-      rake
-    rake (12.1.0)
-    rspec (3.6.0)
-      rspec-core (~> 3.6.0)
-      rspec-expectations (~> 3.6.0)
-      rspec-mocks (~> 3.6.0)
-    rspec-core (3.6.0)
-      rspec-support (~> 3.6.0)
-    rspec-expectations (3.6.0)
+    rainbow (3.0.0)
+    rake (12.3.1)
+    rspec (3.7.0)
+      rspec-core (~> 3.7.0)
+      rspec-expectations (~> 3.7.0)
+      rspec-mocks (~> 3.7.0)
+    rspec-core (3.7.1)
+      rspec-support (~> 3.7.0)
+    rspec-expectations (3.7.0)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.6.0)
-    rspec-mocks (3.6.0)
+      rspec-support (~> 3.7.0)
+    rspec-mocks (3.7.0)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.6.0)
-    rspec-support (3.6.0)
-    rubocop (0.50.0)
+      rspec-support (~> 3.7.0)
+    rspec-support (3.7.1)
+    rubocop (0.52.1)
       parallel (~> 1.10)
-      parser (>= 2.3.3.1, < 3.0)
+      parser (>= 2.4.0.2, < 3.0)
       powerpack (~> 0.1)
-      rainbow (>= 2.2.2, < 3.0)
+      rainbow (>= 2.2.2, < 4.0)
       ruby-progressbar (~> 1.7)
       unicode-display_width (~> 1.0, >= 1.0.1)
     ruby-progressbar (1.9.0)
-    simplecov (0.15.1)
-      docile (~> 1.1.0)
+    simplecov (0.16.1)
+      docile (~> 1.1)
       json (>= 1.8, < 3)
       simplecov-html (~> 0.10.0)
     simplecov-html (0.10.2)
@@ -138,12 +136,12 @@ GEM
       sprockets (>= 3.0.0)
     thor (0.20.0)
     thread_safe (0.3.6)
-    tzinfo (1.2.3)
+    tzinfo (1.2.5)
       thread_safe (~> 0.1)
-    unicode-display_width (1.3.0)
+    unicode-display_width (1.3.3)
     websocket-driver (0.6.5)
       websocket-extensions (>= 0.1.0)
-    websocket-extensions (0.1.2)
+    websocket-extensions (0.1.3)
 
 PLATFORMS
   ruby
@@ -151,10 +149,11 @@ PLATFORMS
 DEPENDENCIES
   bundler (~> 1.15)
   outrigger!
-  rails (~> 5.0.0)
+  rails (~> 5.0.7)
+  railties (>= 5.0, < 5.3)
   rake (~> 12.0)
-  rspec (~> 3.6.0)
-  rubocop (~> 0.50.0)
+  rspec (~> 3.7.0)
+  rubocop (~> 0.52.0)
   simplecov (~> 0)
 
 BUNDLED WITH

data/spec/gemfiles/{rails-4.2.gemfile → rails-5.1.gemfile}

@@ -2,4 +2,4 @@ source 'https://rubygems.org'
 
 gemspec path: '../../'
 
-gem 'rails', '~> 4.2.6'
+gem 'rails', '~> 5.1.6'

data/spec/gemfiles/rails-5.1.gemfile.lock

@@ -0,0 +1,160 @@
+PATH
+  remote: ../..
+  specs:
+    outrigger (1.3.0)
+      activerecord (>= 5.0, < 5.3)
+
+GEM
+  remote: https://rubygems.org/
+  specs:
+    actioncable (5.1.6)
+      actionpack (= 5.1.6)
+      nio4r (~> 2.0)
+      websocket-driver (~> 0.6.1)
+    actionmailer (5.1.6)
+      actionpack (= 5.1.6)
+      actionview (= 5.1.6)
+      activejob (= 5.1.6)
+      mail (~> 2.5, >= 2.5.4)
+      rails-dom-testing (~> 2.0)
+    actionpack (5.1.6)
+      actionview (= 5.1.6)
+      activesupport (= 5.1.6)
+      rack (~> 2.0)
+      rack-test (>= 0.6.3)
+      rails-dom-testing (~> 2.0)
+      rails-html-sanitizer (~> 1.0, >= 1.0.2)
+    actionview (5.1.6)
+      activesupport (= 5.1.6)
+      builder (~> 3.1)
+      erubi (~> 1.4)
+      rails-dom-testing (~> 2.0)
+      rails-html-sanitizer (~> 1.0, >= 1.0.3)
+    activejob (5.1.6)
+      activesupport (= 5.1.6)
+      globalid (>= 0.3.6)
+    activemodel (5.1.6)
+      activesupport (= 5.1.6)
+    activerecord (5.1.6)
+      activemodel (= 5.1.6)
+      activesupport (= 5.1.6)
+      arel (~> 8.0)
+    activesupport (5.1.6)
+      concurrent-ruby (~> 1.0, >= 1.0.2)
+      i18n (>= 0.7, < 2)
+      minitest (~> 5.1)
+      tzinfo (~> 1.1)
+    arel (8.0.0)
+    ast (2.4.0)
+    builder (3.2.3)
+    concurrent-ruby (1.0.5)
+    crass (1.0.4)
+    diff-lcs (1.3)
+    docile (1.3.1)
+    erubi (1.7.1)
+    globalid (0.4.1)
+      activesupport (>= 4.2.0)
+    i18n (1.0.1)
+      concurrent-ruby (~> 1.0)
+    json (2.1.0)
+    loofah (2.2.2)
+      crass (~> 1.0.2)
+      nokogiri (>= 1.5.9)
+    mail (2.7.0)
+      mini_mime (>= 0.1.1)
+    method_source (0.9.0)
+    mini_mime (1.0.0)
+    mini_portile2 (2.3.0)
+    minitest (5.11.3)
+    nio4r (2.3.1)
+    nokogiri (1.8.2)
+      mini_portile2 (~> 2.3.0)
+    parallel (1.12.1)
+    parser (2.5.1.0)
+      ast (~> 2.4.0)
+    powerpack (0.1.1)
+    rack (2.0.5)
+    rack-test (1.0.0)
+      rack (>= 1.0, < 3)
+    rails (5.1.6)
+      actioncable (= 5.1.6)
+      actionmailer (= 5.1.6)
+      actionpack (= 5.1.6)
+      actionview (= 5.1.6)
+      activejob (= 5.1.6)
+      activemodel (= 5.1.6)
+      activerecord (= 5.1.6)
+      activesupport (= 5.1.6)
+      bundler (>= 1.3.0)
+      railties (= 5.1.6)
+      sprockets-rails (>= 2.0.0)
+    rails-dom-testing (2.0.3)
+      activesupport (>= 4.2.0)
+      nokogiri (>= 1.6)
+    rails-html-sanitizer (1.0.4)
+      loofah (~> 2.2, >= 2.2.2)
+    railties (5.1.6)
+      actionpack (= 5.1.6)
+      activesupport (= 5.1.6)
+      method_source
+      rake (>= 0.8.7)
+      thor (>= 0.18.1, < 2.0)
+    rainbow (3.0.0)
+    rake (12.3.1)
+    rspec (3.7.0)
+      rspec-core (~> 3.7.0)
+      rspec-expectations (~> 3.7.0)
+      rspec-mocks (~> 3.7.0)
+    rspec-core (3.7.1)
+      rspec-support (~> 3.7.0)
+    rspec-expectations (3.7.0)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.7.0)
+    rspec-mocks (3.7.0)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.7.0)
+    rspec-support (3.7.1)
+    rubocop (0.52.1)
+      parallel (~> 1.10)
+      parser (>= 2.4.0.2, < 3.0)
+      powerpack (~> 0.1)
+      rainbow (>= 2.2.2, < 4.0)
+      ruby-progressbar (~> 1.7)
+      unicode-display_width (~> 1.0, >= 1.0.1)
+    ruby-progressbar (1.9.0)
+    simplecov (0.16.1)
+      docile (~> 1.1)
+      json (>= 1.8, < 3)
+      simplecov-html (~> 0.10.0)
+    simplecov-html (0.10.2)
+    sprockets (3.7.1)
+      concurrent-ruby (~> 1.0)
+      rack (> 1, < 3)
+    sprockets-rails (3.2.1)
+      actionpack (>= 4.0)
+      activesupport (>= 4.0)
+      sprockets (>= 3.0.0)
+    thor (0.20.0)
+    thread_safe (0.3.6)
+    tzinfo (1.2.5)
+      thread_safe (~> 0.1)
+    unicode-display_width (1.3.3)
+    websocket-driver (0.6.5)
+      websocket-extensions (>= 0.1.0)
+    websocket-extensions (0.1.3)
+
+PLATFORMS
+  ruby
+
+DEPENDENCIES
+  bundler (~> 1.15)
+  outrigger!
+  rails (~> 5.1.6)
+  railties (>= 5.0, < 5.3)
+  rake (~> 12.0)
+  rspec (~> 3.7.0)
+  rubocop (~> 0.52.0)
+  simplecov (~> 0)
+
+BUNDLED WITH
+   1.15.3

data/spec/gemfiles/rails-5.2.gemfile

@@ -0,0 +1,5 @@
+source 'https://rubygems.org'
+
+gemspec path: '../../'
+
+gem 'rails', '~> 5.2.0'

data/spec/gemfiles/rails-5.2.gemfile.lock

@@ -0,0 +1,168 @@
+PATH
+  remote: ../..
+  specs:
+    outrigger (1.3.0)
+      activerecord (>= 5.0, < 5.3)
+
+GEM
+  remote: https://rubygems.org/
+  specs:
+    actioncable (5.2.0)
+      actionpack (= 5.2.0)
+      nio4r (~> 2.0)
+      websocket-driver (>= 0.6.1)
+    actionmailer (5.2.0)
+      actionpack (= 5.2.0)
+      actionview (= 5.2.0)
+      activejob (= 5.2.0)
+      mail (~> 2.5, >= 2.5.4)
+      rails-dom-testing (~> 2.0)
+    actionpack (5.2.0)
+      actionview (= 5.2.0)
+      activesupport (= 5.2.0)
+      rack (~> 2.0)
+      rack-test (>= 0.6.3)
+      rails-dom-testing (~> 2.0)
+      rails-html-sanitizer (~> 1.0, >= 1.0.2)
+    actionview (5.2.0)
+      activesupport (= 5.2.0)
+      builder (~> 3.1)
+      erubi (~> 1.4)
+      rails-dom-testing (~> 2.0)
+      rails-html-sanitizer (~> 1.0, >= 1.0.3)
+    activejob (5.2.0)
+      activesupport (= 5.2.0)
+      globalid (>= 0.3.6)
+    activemodel (5.2.0)
+      activesupport (= 5.2.0)
+    activerecord (5.2.0)
+      activemodel (= 5.2.0)
+      activesupport (= 5.2.0)
+      arel (>= 9.0)
+    activestorage (5.2.0)
+      actionpack (= 5.2.0)
+      activerecord (= 5.2.0)
+      marcel (~> 0.3.1)
+    activesupport (5.2.0)
+      concurrent-ruby (~> 1.0, >= 1.0.2)
+      i18n (>= 0.7, < 2)
+      minitest (~> 5.1)
+      tzinfo (~> 1.1)
+    arel (9.0.0)
+    ast (2.4.0)
+    builder (3.2.3)
+    concurrent-ruby (1.0.5)
+    crass (1.0.4)
+    diff-lcs (1.3)
+    docile (1.3.1)
+    erubi (1.7.1)
+    globalid (0.4.1)
+      activesupport (>= 4.2.0)
+    i18n (1.0.1)
+      concurrent-ruby (~> 1.0)
+    json (2.1.0)
+    loofah (2.2.2)
+      crass (~> 1.0.2)
+      nokogiri (>= 1.5.9)
+    mail (2.7.0)
+      mini_mime (>= 0.1.1)
+    marcel (0.3.2)
+      mimemagic (~> 0.3.2)
+    method_source (0.9.0)
+    mimemagic (0.3.2)
+    mini_mime (1.0.0)
+    mini_portile2 (2.3.0)
+    minitest (5.11.3)
+    nio4r (2.3.1)
+    nokogiri (1.8.2)
+      mini_portile2 (~> 2.3.0)
+    parallel (1.12.1)
+    parser (2.5.1.0)
+      ast (~> 2.4.0)
+    powerpack (0.1.1)
+    rack (2.0.5)
+    rack-test (1.0.0)
+      rack (>= 1.0, < 3)
+    rails (5.2.0)
+      actioncable (= 5.2.0)
+      actionmailer (= 5.2.0)
+      actionpack (= 5.2.0)
+      actionview (= 5.2.0)
+      activejob (= 5.2.0)
+      activemodel (= 5.2.0)
+      activerecord (= 5.2.0)
+      activestorage (= 5.2.0)
+      activesupport (= 5.2.0)
+      bundler (>= 1.3.0)
+      railties (= 5.2.0)
+      sprockets-rails (>= 2.0.0)
+    rails-dom-testing (2.0.3)
+      activesupport (>= 4.2.0)
+      nokogiri (>= 1.6)
+    rails-html-sanitizer (1.0.4)
+      loofah (~> 2.2, >= 2.2.2)
+    railties (5.2.0)
+      actionpack (= 5.2.0)
+      activesupport (= 5.2.0)
+      method_source
+      rake (>= 0.8.7)
+      thor (>= 0.18.1, < 2.0)
+    rainbow (3.0.0)
+    rake (12.3.1)
+    rspec (3.7.0)
+      rspec-core (~> 3.7.0)
+      rspec-expectations (~> 3.7.0)
+      rspec-mocks (~> 3.7.0)
+    rspec-core (3.7.1)
+      rspec-support (~> 3.7.0)
+    rspec-expectations (3.7.0)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.7.0)
+    rspec-mocks (3.7.0)
+      diff-lcs (>= 1.2.0, < 2.0)
+      rspec-support (~> 3.7.0)
+    rspec-support (3.7.1)
+    rubocop (0.52.1)
+      parallel (~> 1.10)
+      parser (>= 2.4.0.2, < 3.0)
+      powerpack (~> 0.1)
+      rainbow (>= 2.2.2, < 4.0)
+      ruby-progressbar (~> 1.7)
+      unicode-display_width (~> 1.0, >= 1.0.1)
+    ruby-progressbar (1.9.0)
+    simplecov (0.16.1)
+      docile (~> 1.1)
+      json (>= 1.8, < 3)
+      simplecov-html (~> 0.10.0)
+    simplecov-html (0.10.2)
+    sprockets (3.7.1)
+      concurrent-ruby (~> 1.0)
+      rack (> 1, < 3)
+    sprockets-rails (3.2.1)
+      actionpack (>= 4.0)
+      activesupport (>= 4.0)
+      sprockets (>= 3.0.0)
+    thor (0.20.0)
+    thread_safe (0.3.6)
+    tzinfo (1.2.5)
+      thread_safe (~> 0.1)
+    unicode-display_width (1.3.3)
+    websocket-driver (0.7.0)
+      websocket-extensions (>= 0.1.0)
+    websocket-extensions (0.1.3)
+
+PLATFORMS
+  ruby
+
+DEPENDENCIES
+  bundler (~> 1.15)
+  outrigger!
+  rails (~> 5.2.0)
+  railties (>= 5.0, < 5.3)
+  rake (~> 12.0)
+  rspec (~> 3.7.0)
+  rubocop (~> 0.52.0)
+  simplecov (~> 0)
+
+BUNDLED WITH
+   1.15.3

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

@@ -14,35 +14,20 @@ RSpec.describe RuboCop::Cop::Migration::Tagged do # rubocop:disable Metrics/Bloc
   let(:config) { RuboCop::Config.new(config_hash) }
 
   let(:migration_class) do
-    <<-RUBY
-    class Test < ActiveRecord::Migration
-      tag :predeploy
-      def change
+    <<~RUBY
+      class Test < ActiveRecord::Migration[4.2]
+        tag :predeploy
+        def change
+        end
       end
-    end
-    RUBY
-  end
-
-  let(:migration_class_42) do
-    <<-RUBY
-    class Test < ActiveRecord::Migration[4.2]
-      tag :predeploy
-      def change
-      end
-    end
     RUBY
   end
 
   subject(:cop) { described_class.new(config) }
 
   shared_examples_for 'valid migrations' do
-    it 'passes valid unversioned migration' do
-      inspect_source(migration_class)
-      expect(cop.offenses.empty?).to be(true)
-    end
-
     it 'passes valid versioned migration' do
-      inspect_source(migration_class_42)
+      inspect_source(migration_class)
       expect(cop.offenses.empty?).to be(true)
     end
   end
@@ -50,67 +35,36 @@ RSpec.describe RuboCop::Cop::Migration::Tagged do # rubocop:disable Metrics/Bloc
   context 'valid config' do # rubocop:disable Metrics/BlockLength
     include_examples 'valid migrations'
 
-    context 'missing tags' do # rubocop:disable Metrics/BlockLength
+    context 'missing tags' do
       let(:migration_class) do
-        <<-RUBY
-        class Test < ActiveRecord::Migration
-          def change
-          end
-        end
-        RUBY
-      end
-
-      let(:migration_class_42) do
-        <<-RUBY
-        class Test < ActiveRecord::Migration[4.2]
-          def change
+        <<~RUBY
+          class Test < ActiveRecord::Migration[4.2]
+            def change
+            end
           end
-        end
         RUBY
       end
 
-      it 'finds missing tag in unversioned 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
-
       it 'finds missing tag in versioned migration' do
-        inspect_source(migration_class_42)
+        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 'invalid tag' do # rubocop:disable Metrics/BlockLength
+    context 'invalid tag' do
       let(:migration_class) do
-        <<-RUBY
-        class Test < ActiveRecord::Migration
-          tag :foobar
-          def change
+        <<~RUBY
+          class Test < ActiveRecord::Migration[4.2]
+            tag :foobar
+            def change
+            end
           end
-        end
         RUBY
       end
 
-      let(:migration_class_42) do
-        <<-RUBY
-        class Test < ActiveRecord::Migration[4.2]
-          tag :foobar
-          def change
-          end
-        end
-        RUBY
-      end
-
-      it 'fails on invalid tag in unversioned 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
-
       it 'fails on invalid tag in versioned migration' do
-        inspect_source(migration_class_42)
+        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
@@ -127,14 +81,8 @@ RSpec.describe RuboCop::Cop::Migration::Tagged do # rubocop:disable Metrics/Bloc
       }
     end
 
-    it 'fails on missing tags in configuration on unversioned 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
-
     it 'fails on missing tags in configuration on versioned migration' do
-      inspect_source(migration_class_42)
+      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

data/spec/outrigger/outrigger_spec.rb

@@ -2,19 +2,6 @@ require 'spec_helper'
 
 describe Outrigger do
   describe 'filter' do
-    before do
-      class PreDeployMigration < ActiveRecord::Migration
-        tag :predeploy
-      end
-
-      class UntaggedMigration < ActiveRecord::Migration
-      end
-
-      class MultiMigration < ActiveRecord::Migration
-        tag :predeploy, :dynamo
-      end
-    end
-
     it 'should return a proc that tests migrations' do
       filter = Outrigger.filter(:predeploy)
 
@@ -22,7 +9,7 @@ describe Outrigger do
     end
 
     it 'should accept multiple tags' do
-      filter = Outrigger.filter(:predeploy, :dynamo)
+      filter = Outrigger.filter(:predeploy, :postdeploy)
 
       expect(filter.call(MultiMigration)).to eq(true)
       expect(filter.call(PreDeployMigration)).to eq(false)

data/spec/outrigger/taggable_proxy_spec.rb

@@ -6,12 +6,6 @@ class TestProxy
 end
 
 describe Outrigger::TaggableProxy do
-  before do
-    class PreDeployMigration < ActiveRecord::Migration
-      tag :predeploy
-    end
-  end
-
   it 'it should delegate tags to the migration' do
     proxy = TestProxy.new
     proxy.migration = PreDeployMigration.new

data/spec/outrigger/taggable_spec.rb

@@ -1,25 +1,6 @@
 require 'spec_helper'
 
-module ActiveRecord
-  class Migration
-    include Outrigger::Taggable
-  end
-end
-
 describe Outrigger::Taggable do
-  before do
-    class PreDeployMigration < ActiveRecord::Migration
-      tag :predeploy
-    end
-
-    class UntaggedMigration < ActiveRecord::Migration
-    end
-
-    class PostDeployMigration < ActiveRecord::Migration
-      tag :postdeploy
-    end
-  end
-
   it 'PreDeployMigration should be predeploy' do
     expect(PreDeployMigration.tags).to eq([:predeploy])
   end

data/spec/spec_helper.rb

@@ -4,7 +4,7 @@ SimpleCov.start do
   add_filter 'spec'
   track_files 'lib/**/*.rb'
 end
-SimpleCov.minimum_coverage(75)
+SimpleCov.minimum_coverage(85)
 
 require 'bundler/setup'
 require 'rails/railtie'
@@ -13,6 +13,24 @@ require 'rubocop/rspec/support'
 
 require 'outrigger'
 
+ActiveRecord::Migration.send :include, Outrigger::Taggable
+ActiveRecord::MigrationProxy.send :include, Outrigger::TaggableProxy
+
+class PreDeployMigration < ActiveRecord::Migration[5.0]
+  tag :predeploy
+end
+
+class UntaggedMigration < ActiveRecord::Migration[5.0]
+end
+
+class PostDeployMigration < ActiveRecord::Migration[5.0]
+  tag :postdeploy
+end
+
+class MultiMigration < ActiveRecord::Migration[5.0]
+  tag :predeploy, :postdeploy
+end
+
 RSpec.configure do |config|
   config.order = 'random'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: outrigger
 version: !ruby/object:Gem::Version
-  version: 1.2.3
+  version: 2.0.0
 platform: ruby
 authors:
 - Drew Bowman
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-04-10 00:00:00.000000000 Z
+date: 2018-05-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -16,20 +16,20 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '4.2'
+        version: '5.0'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '5.2'
+        version: '5.3'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '4.2'
+        version: '5.0'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '5.2'
+        version: '5.3'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -50,20 +50,20 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '4.2'
+        version: '5.0'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '5.2'
+        version: '5.3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '4.2'
+        version: '5.0'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '5.2'
+        version: '5.3'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -84,14 +84,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.6.0
+        version: 3.7.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 3.6.0
+        version: 3.7.0
 - !ruby/object:Gem::Dependency
   name: rubocop
   requirement: !ruby/object:Gem::Requirement
@@ -134,10 +134,12 @@ files:
 - lib/outrigger/taggable_proxy.rb
 - lib/outrigger/version.rb
 - lib/tasks/outrigger.rake
-- spec/gemfiles/rails-4.2.gemfile
-- spec/gemfiles/rails-4.2.gemfile.lock
 - spec/gemfiles/rails-5.0.gemfile
 - spec/gemfiles/rails-5.0.gemfile.lock
+- spec/gemfiles/rails-5.1.gemfile
+- spec/gemfiles/rails-5.1.gemfile.lock
+- spec/gemfiles/rails-5.2.gemfile
+- spec/gemfiles/rails-5.2.gemfile.lock
 - spec/outrigger/cops/migration/tagged_spec.rb
 - spec/outrigger/outrigger_spec.rb
 - spec/outrigger/railtie_spec.rb
@@ -169,10 +171,12 @@ signing_key:
 specification_version: 4
 summary: Tag migrations and run them separately
 test_files:
-- spec/gemfiles/rails-4.2.gemfile
-- spec/gemfiles/rails-4.2.gemfile.lock
 - spec/gemfiles/rails-5.0.gemfile
 - spec/gemfiles/rails-5.0.gemfile.lock
+- spec/gemfiles/rails-5.1.gemfile
+- spec/gemfiles/rails-5.1.gemfile.lock
+- spec/gemfiles/rails-5.2.gemfile
+- spec/gemfiles/rails-5.2.gemfile.lock
 - spec/outrigger/cops/migration/tagged_spec.rb
 - spec/outrigger/outrigger_spec.rb
 - spec/outrigger/railtie_spec.rb

data/spec/gemfiles/rails-4.2.gemfile.lock

@@ -1,162 +0,0 @@
-PATH
-  remote: ../..
-  specs:
-    outrigger (1.2.0)
-      activerecord (>= 4.2, < 5.2)
-
-GEM
-  remote: https://rubygems.org/
-  specs:
-    actionmailer (4.2.10)
-      actionpack (= 4.2.10)
-      actionview (= 4.2.10)
-      activejob (= 4.2.10)
-      mail (~> 2.5, >= 2.5.4)
-      rails-dom-testing (~> 1.0, >= 1.0.5)
-    actionpack (4.2.10)
-      actionview (= 4.2.10)
-      activesupport (= 4.2.10)
-      rack (~> 1.6)
-      rack-test (~> 0.6.2)
-      rails-dom-testing (~> 1.0, >= 1.0.5)
-      rails-html-sanitizer (~> 1.0, >= 1.0.2)
-    actionview (4.2.10)
-      activesupport (= 4.2.10)
-      builder (~> 3.1)
-      erubis (~> 2.7.0)
-      rails-dom-testing (~> 1.0, >= 1.0.5)
-      rails-html-sanitizer (~> 1.0, >= 1.0.3)
-    activejob (4.2.10)
-      activesupport (= 4.2.10)
-      globalid (>= 0.3.0)
-    activemodel (4.2.10)
-      activesupport (= 4.2.10)
-      builder (~> 3.1)
-    activerecord (4.2.10)
-      activemodel (= 4.2.10)
-      activesupport (= 4.2.10)
-      arel (~> 6.0)
-    activesupport (4.2.10)
-      i18n (~> 0.7)
-      minitest (~> 5.1)
-      thread_safe (~> 0.3, >= 0.3.4)
-      tzinfo (~> 1.1)
-    arel (6.0.4)
-    ast (2.3.0)
-    awesome_print (1.8.0)
-    builder (3.2.3)
-    coderay (1.1.2)
-    concurrent-ruby (1.0.5)
-    crass (1.0.2)
-    diff-lcs (1.3)
-    docile (1.1.5)
-    erubis (2.7.0)
-    globalid (0.4.0)
-      activesupport (>= 4.2.0)
-    i18n (0.8.6)
-    json (2.1.0)
-    loofah (2.1.1)
-      crass (~> 1.0.2)
-      nokogiri (>= 1.5.9)
-    mail (2.6.6)
-      mime-types (>= 1.16, < 4)
-    method_source (0.9.0)
-    mime-types (3.1)
-      mime-types-data (~> 3.2015)
-    mime-types-data (3.2016.0521)
-    mini_portile2 (2.3.0)
-    minitest (5.10.3)
-    nokogiri (1.8.1)
-      mini_portile2 (~> 2.3.0)
-    parallel (1.12.0)
-    parser (2.4.0.0)
-      ast (~> 2.2)
-    powerpack (0.1.1)
-    pry (0.11.1)
-      coderay (~> 1.1.0)
-      method_source (~> 0.9.0)
-    rack (1.6.8)
-    rack-test (0.6.3)
-      rack (>= 1.0)
-    rails (4.2.10)
-      actionmailer (= 4.2.10)
-      actionpack (= 4.2.10)
-      actionview (= 4.2.10)
-      activejob (= 4.2.10)
-      activemodel (= 4.2.10)
-      activerecord (= 4.2.10)
-      activesupport (= 4.2.10)
-      bundler (>= 1.3.0, < 2.0)
-      railties (= 4.2.10)
-      sprockets-rails
-    rails-deprecated_sanitizer (1.0.3)
-      activesupport (>= 4.2.0.alpha)
-    rails-dom-testing (1.0.8)
-      activesupport (>= 4.2.0.beta, < 5.0)
-      nokogiri (~> 1.6)
-      rails-deprecated_sanitizer (>= 1.0.1)
-    rails-html-sanitizer (1.0.3)
-      loofah (~> 2.0)
-    railties (4.2.10)
-      actionpack (= 4.2.10)
-      activesupport (= 4.2.10)
-      rake (>= 0.8.7)
-      thor (>= 0.18.1, < 2.0)
-    rainbow (2.2.2)
-      rake
-    rake (12.1.0)
-    rspec (3.6.0)
-      rspec-core (~> 3.6.0)
-      rspec-expectations (~> 3.6.0)
-      rspec-mocks (~> 3.6.0)
-    rspec-core (3.6.0)
-      rspec-support (~> 3.6.0)
-    rspec-expectations (3.6.0)
-      diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.6.0)
-    rspec-mocks (3.6.0)
-      diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.6.0)
-    rspec-support (3.6.0)
-    rubocop (0.50.0)
-      parallel (~> 1.10)
-      parser (>= 2.3.3.1, < 3.0)
-      powerpack (~> 0.1)
-      rainbow (>= 2.2.2, < 3.0)
-      ruby-progressbar (~> 1.7)
-      unicode-display_width (~> 1.0, >= 1.0.1)
-    ruby-progressbar (1.9.0)
-    simplecov (0.15.1)
-      docile (~> 1.1.0)
-      json (>= 1.8, < 3)
-      simplecov-html (~> 0.10.0)
-    simplecov-html (0.10.2)
-    sprockets (3.7.1)
-      concurrent-ruby (~> 1.0)
-      rack (> 1, < 3)
-    sprockets-rails (3.2.1)
-      actionpack (>= 4.0)
-      activesupport (>= 4.0)
-      sprockets (>= 3.0.0)
-    thor (0.20.0)
-    thread_safe (0.3.6)
-    tzinfo (1.2.3)
-      thread_safe (~> 0.1)
-    unicode-display_width (1.3.0)
-
-PLATFORMS
-  ruby
-
-DEPENDENCIES
-  awesome_print (~> 1.8)
-  bundler (~> 1.15)
-  outrigger!
-  pry (~> 0)
-  rails (~> 4.2.6)
-  rake (~> 12.0)
-  rspec (~> 3.6.0)
-  rubocop (~> 0.50.0)
-  simplecov (~> 0)
-
-BUNDLED WITH
-   1.15.3