flipper-active_record 0.16.2 → 0.17.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 4039f5b7764d3f487c80d06f91e0b272abbf4e25
-  data.tar.gz: d40858b49ce9a4697a97b7f5b059dd2a20b30970
+SHA256:
+  metadata.gz: 580a17c75e03a7c98a7361a34f24ae82124491bf02e1351972bd1f6d5b627f79
+  data.tar.gz: f9a57fae9ea74c31101e3e461e0afbd622977351fae841428b35f746badfc803
 SHA512:
-  metadata.gz: cbeb82ee800ebe3709e0f23e28c83b43cbe172256487bfd1d36d4c7204361e6c57aaa4756faec7c64ef0678179d80d8e5bf7973b9a935386e809bc6d42d4e673
-  data.tar.gz: 8df611d967c990cefbec545f5cd3b351de115f24f72e944d4a62573c886f035a2361cbc74ff1b76c176cd592360bdb25e56862d7f540a7f5de128b9e5bc5fce7
+  metadata.gz: 71e7004c6d628e0fe95f3e2d1adaa85d572a557d139634562bb79ccb76a2fd986778773c0aa7895d87db398a16483e6ecf3b3dc26cd067ff7919e810232ab78a
+  data.tar.gz: 05b9b9d298593eddcacb447ac3270a9685bdc25881b67ca37a89c944626e631178ece3910b2931a0b6cb5a7ca553579830f486cd5f2093dfd618e8d978d4eec9

data/docs/active_record/README.md

@@ -4,9 +4,9 @@ An ActiveRecord adapter for [Flipper](https://github.com/jnunemaker/flipper).
 
 Supported Active Record versions:
 
-* 3.2.x
 * 4.2.x
 * 5.0.x
+* 6.0.x
 
 ## Installation
 

data/flipper-active_record.gemspec

@@ -19,12 +19,12 @@ Gem::Specification.new do |gem|
     'lib/flipper/version.rb',
   ]
   gem.files         = `git ls-files`.split("\n").select(&flipper_active_record_files) + extra_files
-  gem.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n").select(&flipper_active_record_files)
+  gem.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n").select(&flipper_active_record_files) # rubocop:disable Metrics/LineLength
   gem.name          = 'flipper-active_record'
   gem.require_paths = ['lib']
   gem.version       = Flipper::VERSION
   gem.metadata      = Flipper::METADATA
 
   gem.add_dependency 'flipper', "~> #{Flipper::VERSION}"
-  gem.add_dependency 'activerecord', '>= 3.2', '< 6'
+  gem.add_dependency 'activerecord', '>= 4.2', '< 7'
 end

data/lib/flipper-active_record.rb

@@ -1,3 +1,5 @@
+require 'active_support/lazy_load_hooks'
+
 ActiveSupport.on_load(:active_record) do
   require 'flipper/adapters/active_record'
 end

data/lib/flipper/adapters/active_record.rb

@@ -54,12 +54,16 @@ module Flipper
       # Public: Adds a feature to the set of known features.
       def add(feature)
         # race condition, but add is only used by enable/disable which happen
-        # super rarely, so it shouldn't matter in practice; additionally
-        # to_a.first is used instead of first because of a Ruby 2.4/Rails 3.2.21
-        # CI failure (https://travis-ci.org/jnunemaker/flipper/jobs/297274000).
-        unless @feature_class.where(key: feature.key).to_a.first
-          @feature_class.create! { |f| f.key = feature.key }
+        # super rarely, so it shouldn't matter in practice
+        @feature_class.transaction do
+          unless @feature_class.where(key: feature.key).first
+            begin
+              @feature_class.create! { |f| f.key = feature.key }
+            rescue ActiveRecord::RecordNotUnique
+            end
+          end
         end
+
         true
       end
 
@@ -97,7 +101,7 @@ module Flipper
       end
 
       def get_all
-        rows = ::ActiveRecord::Base.connection.select_all <<-SQL
+        rows = ::ActiveRecord::Base.connection.select_all <<-SQL.tr("\n", ' ')
           SELECT ff.key AS feature_key, fg.key, fg.value
           FROM #{@feature_class.table_name} ff
           LEFT JOIN #{@gate_class.table_name} fg ON ff.key = fg.feature_key
@@ -122,7 +126,7 @@ module Flipper
       def enable(feature, gate, thing)
         case gate.data_type
         when :boolean, :integer
-          enable_single(feature, gate, thing)
+          set(feature, gate, thing)
         when :set
           enable_multi(feature, gate, thing)
         else
@@ -144,18 +148,7 @@ module Flipper
         when :boolean
           clear(feature)
         when :integer
-          @gate_class.transaction do
-            @gate_class.where(
-              feature_key: feature.key,
-              key: gate.key
-            ).destroy_all
-
-            @gate_class.create! do |g|
-              g.feature_key = feature.key
-              g.key = gate.key
-              g.value = thing.value.to_s
-            end
-          end
+          set(feature, gate, thing)
         when :set
           @gate_class.where(feature_key: feature.key, key: gate.key, value: thing.value).destroy_all
         else
@@ -172,7 +165,7 @@ module Flipper
 
       private
 
-      def enable_single(feature, gate, thing)
+      def set(feature, gate, thing)
         @gate_class.transaction do
           @gate_class.where(feature_key: feature.key, key: gate.key).destroy_all
           @gate_class.create! do |g|
@@ -181,6 +174,8 @@ module Flipper
             g.value = thing.value.to_s
           end
         end
+
+        nil
       end
 
       def enable_multi(feature, gate, thing)
@@ -189,9 +184,10 @@ module Flipper
           g.key = gate.key
           g.value = thing.value.to_s
         end
+
+        nil
       rescue ::ActiveRecord::RecordNotUnique
-      rescue ::ActiveRecord::StatementInvalid => error
-        raise unless error.message =~ /unique/i
+        # already added so no need move on with life
       end
 
       def result_for_feature(feature, db_gates)
@@ -201,12 +197,12 @@ module Flipper
           result[gate.key] =
             case gate.data_type
             when :boolean
-              if db_gate = db_gates.detect { |db_gate| db_gate.key == gate.key.to_s }
-                db_gate.value
+              if detected_db_gate = db_gates.detect { |db_gate| db_gate.key == gate.key.to_s }
+                detected_db_gate.value
               end
             when :integer
-              if db_gate = db_gates.detect { |db_gate| db_gate.key == gate.key.to_s }
-                db_gate.value
+              if detected_db_gate = db_gates.detect { |db_gate| db_gate.key == gate.key.to_s }
+                detected_db_gate.value
               end
             when :set
               db_gates.select { |db_gate| db_gate.key == gate.key.to_s }.map(&:value).to_set

data/lib/flipper/version.rb

@@ -1,3 +1,3 @@
 module Flipper
-  VERSION = '0.16.2'.freeze
+  VERSION = '0.17.1'.freeze
 end

data/lib/generators/flipper/active_record_generator.rb

@@ -13,11 +13,11 @@ module Flipper
       end
 
       def self.migration_version
-        "[#{Rails::VERSION::MAJOR}.#{Rails::VERSION::MINOR}]" if rails5?
+        "[#{Rails::VERSION::MAJOR}.#{Rails::VERSION::MINOR}]" if requires_migration_number?
       end
 
-      def self.rails5?
-        Rails.version.start_with?('5')
+      def self.requires_migration_number?
+        Rails::VERSION::MAJOR.to_i >= 5
       end
 
       def create_migration_file

data/test_rails/generators/flipper/active_record_generator_test.rb

@@ -0,0 +1,43 @@
+require 'helper'
+require 'active_record'
+require 'rails/generators/test_case'
+require 'generators/flipper/active_record_generator'
+
+class FlipperActiveRecordGeneratorTest < Rails::Generators::TestCase
+  tests Flipper::Generators::ActiveRecordGenerator
+  destination File.expand_path('../../../../tmp', __FILE__)
+  setup :prepare_destination
+
+  def test_generates_migration
+    run_generator
+    migration_version = if Rails::VERSION::MAJOR.to_i < 5
+                          ""
+                        else
+                          "[#{Rails::VERSION::MAJOR}.#{Rails::VERSION::MINOR}]"
+                        end
+    assert_migration 'db/migrate/create_flipper_tables.rb', <<~MIGRATION
+      class CreateFlipperTables < ActiveRecord::Migration#{migration_version}
+        def self.up
+          create_table :flipper_features do |t|
+            t.string :key, null: false
+            t.timestamps null: false
+          end
+          add_index :flipper_features, :key, unique: true
+
+          create_table :flipper_gates do |t|
+            t.string :feature_key, null: false
+            t.string :key, null: false
+            t.string :value
+            t.timestamps null: false
+          end
+          add_index :flipper_gates, [:feature_key, :key, :value], unique: true
+        end
+
+        def self.down
+          drop_table :flipper_gates
+          drop_table :flipper_features
+        end
+      end
+    MIGRATION
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: flipper-active_record
 version: !ruby/object:Gem::Version
-  version: 0.16.2
+  version: 0.17.1
 platform: ruby
 authors:
 - John Nunemaker
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-04-20 00:00:00.000000000 Z
+date: 2019-09-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: flipper
@@ -16,34 +16,34 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.16.2
+        version: 0.17.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.16.2
+        version: 0.17.1
 - !ruby/object:Gem::Dependency
   name: activerecord
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '3.2'
+        version: '4.2'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '6'
+        version: '7'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '3.2'
+        version: '4.2'
     - - "<"
       - !ruby/object:Gem::Version
-        version: '6'
+        version: '7'
 description: ActiveRecord adapter for Flipper
 email:
 - nunemaker@gmail.com
@@ -63,7 +63,7 @@ files:
 - lib/generators/flipper/templates/migration.erb
 - spec/flipper/adapters/active_record_spec.rb
 - test/adapters/active_record_test.rb
-- test/generators/flipper/active_record_generator_test.rb
+- test_rails/generators/flipper/active_record_generator_test.rb
 homepage: https://github.com/jnunemaker/flipper
 licenses:
 - MIT
@@ -84,12 +84,10 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.4.5.4
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: ActiveRecord adapter for Flipper
 test_files:
 - spec/flipper/adapters/active_record_spec.rb
 - test/adapters/active_record_test.rb
-- test/generators/flipper/active_record_generator_test.rb

data/test/generators/flipper/active_record_generator_test.rb

@@ -1,43 +0,0 @@
-require 'helper'
-require 'active_record'
-require 'rails/generators/test_case'
-require 'generators/flipper/active_record_generator'
-
-class FlipperActiveRecordGeneratorTest < Rails::Generators::TestCase
-  tests Flipper::Generators::ActiveRecordGenerator
-  destination File.expand_path('../../../../tmp', __FILE__)
-  setup :prepare_destination
-
-  def test_generates_migration
-    run_generator
-    migration_version = if Rails::VERSION::MAJOR.to_i < 5
-                          ""
-                        else
-                          "[#{Rails::VERSION::MAJOR}.#{Rails::VERSION::MINOR}]"
-                        end
-    assert_migration 'db/migrate/create_flipper_tables.rb', <<-EOM
-class CreateFlipperTables < ActiveRecord::Migration#{migration_version}
-  def self.up
-    create_table :flipper_features do |t|
-      t.string :key, null: false
-      t.timestamps null: false
-    end
-    add_index :flipper_features, :key, unique: true
-
-    create_table :flipper_gates do |t|
-      t.string :feature_key, null: false
-      t.string :key, null: false
-      t.string :value
-      t.timestamps null: false
-    end
-    add_index :flipper_gates, [:feature_key, :key, :value], unique: true
-  end
-
-  def self.down
-    drop_table :flipper_gates
-    drop_table :flipper_features
-  end
-end
-EOM
-  end
-end