flipper-active_record 1.2.1 → 1.3.0.pre

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d93623f7da8df34acc676d8a7f20fdc983eca967539edcb6c2aae35247828eb1
-  data.tar.gz: 2060a98a8c7aba7dd54d62d5d1fad78fd6ca0f8a2136c9c264070383fd23c851
+  metadata.gz: 7c5c569d1b6a5074ed23b730cb49012e31ea81ec72d0d1ee3bc7fd220e9d8de7
+  data.tar.gz: 66c02755000391fc1ffd56944dad2e9ee5b8e563437cfe4fd4a6ccd437eb6aed
 SHA512:
-  metadata.gz: ad53eda1a2aa9e11f719e9a5916f060a9cc628f450b60c3a991777b3ae9bc1a5968009c396fbda779406c979b8129d427cd5e217a1ecc5718497823f528241b1
-  data.tar.gz: a99ace899778db6f3120c8909e78afc595a57524a56759c0bba67a8b65e4721fc82c0fd4603f0a672dc4459216742533cc881ea8f7344bdadcb00aae5fb8ce96
+  metadata.gz: db8c6b8e49eb66d7e680236c000c7b5333ed43f55ede52d868aa1cdf54e6729d7c42c289fa6e9eefa456afd8dbf9227c85385fa05bdab303f5d2520ca00634ad
+  data.tar.gz: 3607f280b38cc442d142d17a4d56e1f1584bb4e1ca298fa52a03ee5b53d2c4adf86f1a25e8fd784df9d835357d34fe5cd76366b80c7cf92c29c3847e093ec665

data/benchmark/active_record_adapter_ips.rb

@@ -6,10 +6,13 @@ require 'benchmark/ips'
 
 flipper = Flipper.new(Flipper::Adapters::ActiveRecord.new)
 
-2000.times do |i|
-  flipper.enable_actor :foo, Flipper::Actor.new("User;#{i}")
+10.times do |n|
+  2000.times do |i|
+    flipper.enable_actor 'feature' + n.to_s, Flipper::Actor.new("User;#{i}")
+  end
 end
 
 Benchmark.ips do |x|
   x.report("get_all") { flipper.preload_all }
+  x.report("features") { flipper.features }
 end

data/lib/flipper/adapters/active_record.rb

@@ -59,7 +59,7 @@ module Flipper
 
       # Public: The set of known features.
       def features
-        with_connection(@feature_class) { @feature_class.all.map(&:key).to_set }
+        with_connection(@feature_class) { @feature_class.distinct.pluck(:key).to_set }
       end
 
       # Public: Adds a feature to the set of known features.
@@ -68,9 +68,9 @@ module Flipper
           # race condition, but add is only used by enable/disable which happen
           # super rarely, so it shouldn't matter in practice
           @feature_class.transaction do
-            unless @feature_class.where(key: feature.key).first
+            unless @feature_class.where(key: feature.key).exists?
               begin
-                @feature_class.create! { |f| f.key = feature.key }
+                @feature_class.create!(key: feature.key)
               rescue ::ActiveRecord::RecordNotUnique
               end
             end

data/lib/flipper/version.rb

@@ -1,5 +1,5 @@
 module Flipper
-  VERSION = '1.2.1'.freeze
+  VERSION = '1.3.0.pre'.freeze
 
   REQUIRED_RUBY_VERSION = '2.6'.freeze
   NEXT_REQUIRED_RUBY_VERSION = '3.0'.freeze

data/spec/flipper/adapters/active_record_spec.rb

@@ -1,5 +1,4 @@
 SpecHelpers.silence { require 'flipper/adapters/active_record' }
-require 'active_support/core_ext/kernel'
 
 # Turn off migration logging for specs
 ActiveRecord::Migration.verbose = false
@@ -17,212 +16,217 @@ RSpec.describe Flipper::Adapters::ActiveRecord do
 
   [
     {
-      adapter: "sqlite3",
-      database: ":memory:"
+      "adapter" => "sqlite3",
+      "database" => ":memory:"
     },
 
     {
-      adapter: "mysql2",
-      encoding: "utf8mb4",
-      username: ENV["MYSQL_USER"] || "root",
-      password: ENV["MYSQL_PASSWORD"] || "",
-      database: ENV["MYSQL_DATABASE"] || "flipper_test",
-      port: ENV["DB_PORT"] || 3306
+      "adapter" => "mysql2",
+      "encoding" => "utf8mb4",
+      "username" => ENV["MYSQL_USER"] || "root",
+      "password" => ENV["MYSQL_PASSWORD"] || "",
+      "database" => ENV["MYSQL_DATABASE"] || "flipper_test",
+      "port" => ENV["DB_PORT"] || 3306
     },
 
     {
-      adapter: "postgresql",
-      encoding: "unicode",
-      host: "127.0.0.1",
-      username: ENV["POSTGRES_USER"] || "",
-      password: ENV["POSTGRES_PASSWORD"] || "",
-      database: ENV["POSTGRES_DATABASE"] || "flipper_test",
+      "adapter" => "postgresql",
+      "encoding" => "unicode",
+      "host" => "127.0.0.1",
+      "username" => ENV["POSTGRES_USER"] || "",
+      "password" => ENV["POSTGRES_PASSWORD"] || "",
+      "database" => ENV["POSTGRES_DATABASE"] || "flipper_test",
     }
   ].each do |config|
-    config = config.with_indifferent_access
-
-    context "with #{config[:adapter]}" do
-      before(:all) do
-        silence { ActiveRecord::Tasks::DatabaseTasks.create(config) }
-      end
-
-      before(:each) do
-        skip_on_error(ActiveRecord::ConnectionNotEstablished, "#{config[:adapter]} not available") do
-          ActiveRecord::Base.establish_connection(config)
-          CreateFlipperTables.migrate(:up)
-        end
-      end
-
-      after(:each) do
-        ActiveRecord::Tasks::DatabaseTasks.purge(config)
-        ActiveRecord::Base.connection.close
-      end
-
-      after(:all) do
-        silence { ActiveRecord::Tasks::DatabaseTasks.drop(config) }
-      end
-
-      it_should_behave_like 'a flipper adapter'
-
-      it "works when table doesn't exist" do
-        CreateFlipperTables.migrate(:down)
-
-        Flipper.configuration = nil
-        Flipper.instance = nil
-
-        expect {
-          silence do
-            load 'flipper/adapters/active_record.rb'
-            Flipper::Adapters::ActiveRecord.new
+    context "with #{config['adapter']}" do
+      context "with tables created" do
+        before(:all) do
+          skip_on_error(ActiveRecord::ConnectionNotEstablished, "#{config['adapter']} not available") do
+            silence { ActiveRecord::Tasks::DatabaseTasks.create(config) }
           end
-        }.not_to raise_error
-      end
-
-      it "should load actor ids fine" do
-        flipper.enable_percentage_of_time(:foo, 1)
-
-        Flipper::Adapters::ActiveRecord::Gate.create!(
-          feature_key: "foo",
-          key: "actors",
-          value: "Organization;4",
-        )
 
-        flipper = Flipper.new(subject)
-        flipper.preload([:foo])
-      end
-
-      context 'requiring "flipper-active_record"' do
-        before do
           Flipper.configuration = nil
-          Flipper.instance = nil
+        end
 
-          silence { load 'flipper/adapters/active_record.rb' }
+        before(:each) do
+          skip_on_error(ActiveRecord::ConnectionNotEstablished, "#{config['adapter']} not available") do
+            ActiveRecord::Base.establish_connection(config)
+            CreateFlipperTables.migrate(:up)
+          end
         end
 
-        it 'configures itself' do
-          expect(Flipper.adapter.adapter).to be_a(Flipper::Adapters::ActiveRecord)
+        after(:each) do
+          ActiveRecord::Tasks::DatabaseTasks.purge(config)
+          ActiveRecord::Base.connection.close
         end
-      end
 
-      context "ActiveRecord connection_pool" do
-        before do
-          ActiveRecord::Base.clear_active_connections!
+        after(:all) do
+          silence { ActiveRecord::Tasks::DatabaseTasks.drop(config) } unless $skip
         end
 
-        context "#features" do
-          it "does not hold onto connections" do
-            expect(ActiveRecord::Base.connection_handler.active_connections?).to be(false)
-            subject.features
-            expect(ActiveRecord::Base.connection_handler.active_connections?).to be(false)
-          end
+        it_should_behave_like 'a flipper adapter'
 
-          it "does not release previously held connection" do
-            ActiveRecord::Base.connection # establish a new connection
-            expect(ActiveRecord::Base.connection_handler.active_connections?).to be(true)
-            subject.features
-            expect(ActiveRecord::Base.connection_handler.active_connections?).to be(true)
-          end
+        it "should load actor ids fine" do
+          flipper.enable_percentage_of_time(:foo, 1)
+
+          Flipper::Adapters::ActiveRecord::Gate.create!(
+            feature_key: "foo",
+            key: "actors",
+            value: "Organization;4",
+          )
+
+          flipper = Flipper.new(subject)
+          flipper.preload([:foo])
         end
 
-        context "#get_all" do
-          it "does not hold onto connections" do
-            expect(ActiveRecord::Base.connection_handler.active_connections?).to be(false)
-            subject.get_all
-            expect(ActiveRecord::Base.connection_handler.active_connections?).to be(false)
+        context "ActiveRecord connection_pool" do
+          before do
+            ActiveRecord::Base.connection_handler.clear_active_connections!
           end
 
-          it "does not release previously held connection" do
-            ActiveRecord::Base.connection # establish a new connection
-            expect(ActiveRecord::Base.connection_handler.active_connections?).to be(true)
-            subject.get_all
-            expect(ActiveRecord::Base.connection_handler.active_connections?).to be(true)
+          context "#features" do
+            it "does not hold onto connections" do
+              expect(ActiveRecord::Base.connection_handler.active_connections?).to be(false)
+              subject.features
+              expect(ActiveRecord::Base.connection_handler.active_connections?).to be(false)
+            end
+
+            it "does not release previously held connection" do
+              ActiveRecord::Base.connection # establish a new connection
+              expect(ActiveRecord::Base.connection_handler.active_connections?).to be(true)
+              subject.features
+              expect(ActiveRecord::Base.connection_handler.active_connections?).to be(true)
+            end
           end
-        end
 
-        context "#add / #remove / #clear" do
-          let(:feature) { Flipper::Feature.new(:search, subject) }
-
-          it "does not hold onto connections" do
-            expect(ActiveRecord::Base.connection_handler.active_connections?).to be(false)
-            subject.add(feature)
-            expect(ActiveRecord::Base.connection_handler.active_connections?).to be(false)
-            subject.remove(feature)
-            expect(ActiveRecord::Base.connection_handler.active_connections?).to be(false)
-            subject.clear(feature)
-            expect(ActiveRecord::Base.connection_handler.active_connections?).to be(false)
+          context "#get_all" do
+            it "does not hold onto connections" do
+              expect(ActiveRecord::Base.connection_handler.active_connections?).to be(false)
+              subject.get_all
+              expect(ActiveRecord::Base.connection_handler.active_connections?).to be(false)
+            end
+
+            it "does not release previously held connection" do
+              ActiveRecord::Base.connection # establish a new connection
+              expect(ActiveRecord::Base.connection_handler.active_connections?).to be(true)
+              subject.get_all
+              expect(ActiveRecord::Base.connection_handler.active_connections?).to be(true)
+            end
           end
 
-          it "does not release previously held connection" do
-            ActiveRecord::Base.connection # establish a new connection
-            expect(ActiveRecord::Base.connection_handler.active_connections?).to be(true)
-            subject.add(feature)
-            expect(ActiveRecord::Base.connection_handler.active_connections?).to be(true)
-            subject.remove(feature)
-            expect(ActiveRecord::Base.connection_handler.active_connections?).to be(true)
-            subject.clear(feature)
-            expect(ActiveRecord::Base.connection_handler.active_connections?).to be(true)
+          context "#add / #remove / #clear" do
+            let(:feature) { Flipper::Feature.new(:search, subject) }
+
+            it "does not hold onto connections" do
+              expect(ActiveRecord::Base.connection_handler.active_connections?).to be(false)
+              subject.add(feature)
+              expect(ActiveRecord::Base.connection_handler.active_connections?).to be(false)
+              subject.remove(feature)
+              expect(ActiveRecord::Base.connection_handler.active_connections?).to be(false)
+              subject.clear(feature)
+              expect(ActiveRecord::Base.connection_handler.active_connections?).to be(false)
+            end
+
+            it "does not release previously held connection" do
+              ActiveRecord::Base.connection # establish a new connection
+              expect(ActiveRecord::Base.connection_handler.active_connections?).to be(true)
+              subject.add(feature)
+              expect(ActiveRecord::Base.connection_handler.active_connections?).to be(true)
+              subject.remove(feature)
+              expect(ActiveRecord::Base.connection_handler.active_connections?).to be(true)
+              subject.clear(feature)
+              expect(ActiveRecord::Base.connection_handler.active_connections?).to be(true)
+            end
           end
-        end
 
-        context "#get_multi" do
-          let(:feature) { Flipper::Feature.new(:search, subject) }
+          context "#get_multi" do
+            let(:feature) { Flipper::Feature.new(:search, subject) }
+
+            it "does not hold onto connections" do
+              expect(ActiveRecord::Base.connection_handler.active_connections?).to be(false)
+              subject.get_multi([feature])
+              expect(ActiveRecord::Base.connection_handler.active_connections?).to be(false)
+            end
+
+            it "does not release previously held connection" do
+              ActiveRecord::Base.connection # establish a new connection
+              expect(ActiveRecord::Base.connection_handler.active_connections?).to be(true)
+              subject.get_multi([feature])
+              expect(ActiveRecord::Base.connection_handler.active_connections?).to be(true)
+            end
+          end
 
-          it "does not hold onto connections" do
-            expect(ActiveRecord::Base.connection_handler.active_connections?).to be(false)
-            subject.get_multi([feature])
-            expect(ActiveRecord::Base.connection_handler.active_connections?).to be(false)
+          context "#enable/#disable boolean" do
+            let(:feature) { Flipper::Feature.new(:search, subject) }
+            let(:gate) { feature.gate(:boolean)}
+
+            it "does not hold onto connections" do
+              expect(ActiveRecord::Base.connection_handler.active_connections?).to be(false)
+              subject.enable(feature, gate, gate.wrap(true))
+              expect(ActiveRecord::Base.connection_handler.active_connections?).to be(false)
+              subject.disable(feature, gate, gate.wrap(false))
+              expect(ActiveRecord::Base.connection_handler.active_connections?).to be(false)
+            end
+
+            it "does not release previously held connection" do
+              ActiveRecord::Base.connection # establish a new connection
+              expect(ActiveRecord::Base.connection_handler.active_connections?).to be(true)
+              subject.enable(feature, gate, gate.wrap(true))
+              expect(ActiveRecord::Base.connection_handler.active_connections?).to be(true)
+              subject.disable(feature, gate, gate.wrap(false))
+              expect(ActiveRecord::Base.connection_handler.active_connections?).to be(true)
+            end
           end
 
-          it "does not release previously held connection" do
-            ActiveRecord::Base.connection # establish a new connection
-            expect(ActiveRecord::Base.connection_handler.active_connections?).to be(true)
-            subject.get_multi([feature])
-            expect(ActiveRecord::Base.connection_handler.active_connections?).to be(true)
+          context "#enable/#disable set" do
+            let(:feature) { Flipper::Feature.new(:search, subject) }
+            let(:gate) { feature.gate(:group) }
+
+            it "does not hold onto connections" do
+              expect(ActiveRecord::Base.connection_handler.active_connections?).to be(false)
+              subject.enable(feature, gate, gate.wrap(:admin))
+              expect(ActiveRecord::Base.connection_handler.active_connections?).to be(false)
+              subject.disable(feature, gate, gate.wrap(:admin))
+              expect(ActiveRecord::Base.connection_handler.active_connections?).to be(false)
+            end
+
+            it "does not release previously held connection" do
+              ActiveRecord::Base.connection # establish a new connection
+              expect(ActiveRecord::Base.connection_handler.active_connections?).to be(true)
+              subject.enable(feature, gate, gate.wrap(:admin))
+              expect(ActiveRecord::Base.connection_handler.active_connections?).to be(true)
+              subject.disable(feature, gate, gate.wrap(:admin))
+              expect(ActiveRecord::Base.connection_handler.active_connections?).to be(true)
+            end
           end
         end
 
-        context "#enable/#disable boolean" do
-          let(:feature) { Flipper::Feature.new(:search, subject) }
-          let(:gate) { feature.gate(:boolean)}
+        context 'requiring "flipper-active_record"' do
+          before do
+            Flipper.configuration = nil
+            Flipper.instance = nil
 
-          it "does not hold onto connections" do
-            expect(ActiveRecord::Base.connection_handler.active_connections?).to be(false)
-            subject.enable(feature, gate, gate.wrap(true))
-            expect(ActiveRecord::Base.connection_handler.active_connections?).to be(false)
-            subject.disable(feature, gate, gate.wrap(false))
-            expect(ActiveRecord::Base.connection_handler.active_connections?).to be(false)
+            silence { load 'flipper/adapters/active_record.rb' }
           end
 
-          it "does not release previously held connection" do
-            ActiveRecord::Base.connection # establish a new connection
-            expect(ActiveRecord::Base.connection_handler.active_connections?).to be(true)
-            subject.enable(feature, gate, gate.wrap(true))
-            expect(ActiveRecord::Base.connection_handler.active_connections?).to be(true)
-            subject.disable(feature, gate, gate.wrap(false))
-            expect(ActiveRecord::Base.connection_handler.active_connections?).to be(true)
+          it 'configures itself' do
+            expect(Flipper.adapter.adapter).to be_a(Flipper::Adapters::ActiveRecord)
           end
         end
+      end
 
-        context "#enable/#disable set" do
-          let(:feature) { Flipper::Feature.new(:search, subject) }
-          let(:gate) { feature.gate(:group) }
+      it "works when table doesn't exist" do
 
-          it "does not hold onto connections" do
-            expect(ActiveRecord::Base.connection_handler.active_connections?).to be(false)
-            subject.enable(feature, gate, gate.wrap(:admin))
-            expect(ActiveRecord::Base.connection_handler.active_connections?).to be(false)
-            subject.disable(feature, gate, gate.wrap(:admin))
-            expect(ActiveRecord::Base.connection_handler.active_connections?).to be(false)
-          end
+        Flipper.configuration = nil
+        Flipper.instance = nil
 
-          it "does not release previously held connection" do
-            ActiveRecord::Base.connection # establish a new connection
-            expect(ActiveRecord::Base.connection_handler.active_connections?).to be(true)
-            subject.enable(feature, gate, gate.wrap(:admin))
-            expect(ActiveRecord::Base.connection_handler.active_connections?).to be(true)
-            subject.disable(feature, gate, gate.wrap(:admin))
-            expect(ActiveRecord::Base.connection_handler.active_connections?).to be(true)
-          end
+        Flipper::Adapters.send(:remove_const, :ActiveRecord) if Flipper::Adapters.const_defined?(:ActiveRecord)
+
+        silence do
+          expect {
+            load 'flipper/adapters/active_record.rb'
+            Flipper::Adapters::ActiveRecord.new
+          }.not_to raise_error
         end
       end
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: flipper-active_record
 version: !ruby/object:Gem::Version
-  version: 1.2.1
+  version: 1.3.0.pre
 platform: ruby
 authors:
 - John Nunemaker
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-01-15 00:00:00.000000000 Z
+date: 2024-03-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: flipper
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.2.1
+        version: 1.3.0.pre
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.2.1
+        version: 1.3.0.pre
 - !ruby/object:Gem::Dependency
   name: activerecord
   requirement: !ruby/object:Gem::Requirement
@@ -74,7 +74,7 @@ metadata:
   homepage_uri: https://www.flippercloud.io
   source_code_uri: https://github.com/flippercloud/flipper
   bug_tracker_uri: https://github.com/flippercloud/flipper/issues
-  changelog_uri: https://github.com/flippercloud/flipper/releases/tag/v1.2.1
+  changelog_uri: https://github.com/flippercloud/flipper/releases/tag/v1.3.0.pre
 post_install_message:
 rdoc_options: []
 require_paths: