flipper-active_record 1.1.0 → 1.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 4c4aa6610a01147dbc81d8e256344ecb508ee59fa87b74d6ad746de90d4306c5
-  data.tar.gz: a5c419782f0d82df4a417a5d5c89200fcc8322a8251f1cbba8c5628c40322aa2
+  metadata.gz: a87a99a5d26e8ffd53a4e4620612f5babe97090fc9a25a13999cbeef90eb38c3
+  data.tar.gz: 4e2523ba1ddd2b72c383893ba161655526292334fb8a3f863891c7a2ae7dbd2b
 SHA512:
-  metadata.gz: 9fb31e98eb30ac6323ed1f9662ef49df1ba44c71e6006aade9b7c55c2d236be1e2b608e2e2c3a12c8ab5ba9e748fc5d56fd63d0f663a009820fee37184a26ff5
-  data.tar.gz: bc3fa0ec74c8af61b1e5b537d1fc16c973d555de8f3d75238c6e620a33527af65e143d8ceee9999666950d6bcc5b2110c481c4ad4036c7dc9d01d3b097fab420
+  metadata.gz: 80587a110384c5ac2e4d11e143ad33797d092b391f6e553b87cd2499dc841dfb75079eb414368470400709ede4fa3c96e12416c48712cd731a4606da50f63b9f
+  data.tar.gz: ec0a0d89da664ae8aedc701a7406a2794391d27987bcc0ce85be3a33e1b2c02b76869206301f49d1c9489c703cd9de47c0b6714b166cd758805996569e25e355

data/flipper-active_record.gemspec

@@ -1,15 +1,22 @@
 # -*- encoding: utf-8 -*-
 require File.expand_path('../lib/flipper/version', __FILE__)
 require File.expand_path('../lib/flipper/metadata', __FILE__)
+require "set"
+
+# Files that should exist in main flipper gem.
+main_flipper_active_record_files = Set[
+  "lib/flipper/model/active_record.rb",
+  "spec/flipper/model/active_record_spec.rb",
+]
 
 flipper_active_record_files = lambda do |file|
-  file =~ /active_record/
+  file =~ /active_record/ && !main_flipper_active_record_files.include?(file)
 end
 
 Gem::Specification.new do |gem|
   gem.authors       = ['John Nunemaker']
   gem.email         = 'support@flippercloud.io'
-  gem.summary       = 'ActiveRecord adapter for Flipper'
+  gem.summary       = 'ActiveRecord feature flag adapter for Flipper'
   gem.license       = 'MIT'
   gem.homepage      = 'https://www.flippercloud.io/docs/adapters/active-record'
 

data/lib/flipper/adapters/active_record.rb

@@ -34,7 +34,7 @@ module Flipper
 
       VALUE_TO_TEXT_WARNING = <<-EOS
         Your database needs migrated to use the latest Flipper features.
-        See https://github.com/flippercloud/flipper/issues/557
+        Run `rails generate flipper:update` and `rails db:migrate`.
       EOS
 
       # Public: Initialize a new ActiveRecord adapter instance.
@@ -278,6 +278,9 @@ module Flipper
       # See https://github.com/flippercloud/flipper/pull/692
       def value_not_text?
         @gate_class.column_for_attribute(:value).type != :text
+      rescue ::ActiveRecord::ActiveRecordError => error
+        # If the table doesn't exist, the column doesn't exist either
+        warn "#{error.message}. You likely need to run `rails g flipper:active_record` and/or `rails db:migrate`."
       end
 
       def with_connection(model = @feature_class, &block)

data/lib/flipper/version.rb

@@ -1,3 +1,3 @@
 module Flipper
-  VERSION = '1.1.0'.freeze
+  VERSION = '1.1.2'.freeze
 end

data/lib/generators/flipper/templates/migration.erb

@@ -1,5 +1,5 @@
 class CreateFlipperTables < ActiveRecord::Migration<%= migration_version %>
-  def self.up
+  def up
     create_table :flipper_features do |t|
       t.string :key, null: false
       t.timestamps null: false
@@ -12,10 +12,10 @@ class CreateFlipperTables < ActiveRecord::Migration<%= migration_version %>
       t.text :value
       t.timestamps null: false
     end
-    add_index :flipper_gates, [:feature_key, :key, :value], unique: true
+    add_index :flipper_gates, [:feature_key, :key, :value], unique: true, length: { value: 255 }
   end
 
-  def self.down
+  def down
     drop_table :flipper_gates
     drop_table :flipper_features
   end

data/spec/flipper/adapters/active_record_spec.rb

@@ -3,191 +3,223 @@ require 'active_support/core_ext/kernel'
 
 # Turn off migration logging for specs
 ActiveRecord::Migration.verbose = false
+ActiveRecord::Tasks::DatabaseTasks.root = File.dirname(__FILE__)
 
 RSpec.describe Flipper::Adapters::ActiveRecord do
   subject { described_class.new }
 
   before(:all) do
-    ActiveRecord::Base.establish_connection(adapter: 'sqlite3',
-                                            database: ':memory:')
+    # Eval migration template so we can run migration against each database
+    migration = ERB.new(File.read(File.join(File.dirname(__FILE__), '../../../lib/generators/flipper/templates/migration.erb')))
+    migration_version = "[#{ActiveRecord::VERSION::MAJOR}.#{ActiveRecord::VERSION::MINOR}]"
+    eval migration.result(binding) # defines CreateFlipperTables
   end
 
-  before(:each) do
-    ActiveRecord::Base.connection.execute <<-SQL
-      CREATE TABLE flipper_features (
-        id integer PRIMARY KEY,
-        key string NOT NULL UNIQUE,
-        created_at datetime NOT NULL,
-        updated_at datetime NOT NULL
-      )
-    SQL
-
-    ActiveRecord::Base.connection.execute <<-SQL
-      CREATE TABLE flipper_gates (
-        id integer PRIMARY KEY,
-        feature_key text NOT NULL,
-        key string NOT NULL,
-        value text DEFAULT NULL,
-        created_at datetime NOT NULL,
-        updated_at datetime NOT NULL
-      )
-    SQL
-
-    ActiveRecord::Base.connection.execute <<-SQL
-      CREATE UNIQUE INDEX index_gates_on_keys_and_value on flipper_gates (feature_key, key, value)
-    SQL
-  end
-
-  after(:each) do
-    ActiveRecord::Base.connection.execute("DROP table IF EXISTS `flipper_features`")
-    ActiveRecord::Base.connection.execute("DROP table IF EXISTS `flipper_gates`")
-  end
-
-  it_should_behave_like 'a flipper adapter'
-
-  it "should load actor ids fine" do
-    flipper.enable_percentage_of_time(:foo, 1)
-
-    ActiveRecord::Base.connection.execute <<-SQL
-      INSERT INTO flipper_gates (feature_key, key, value, created_at, updated_at)
-      VALUES ("foo", "actors", "Organization;4", time(), time())
-    SQL
-
-    flipper = Flipper.new(subject)
-    flipper.preload([:foo])
-  end
-
-  context 'requiring "flipper-active_record"' do
-    before do
-      Flipper.configuration = nil
-      Flipper.instance = nil
-
-      silence_warnings { load 'flipper/adapters/active_record.rb' }
-    end
-
-    it 'configures itself' do
-      expect(Flipper.adapter.adapter).to be_a(Flipper::Adapters::ActiveRecord)
-    end
-  end
-
-  context "ActiveRecord connection_pool" do
-    before do
-      ActiveRecord::Base.clear_active_connections!
-    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)
+  [
+    {
+      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: "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
+        ActiveRecord::Tasks::DatabaseTasks.create(config)
       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)
+      before(:each) do
+        skip_on_error(ActiveRecord::ConnectionNotEstablished, "#{config[:adapter]} not available") do
+          ActiveRecord::Base.establish_connection(config)
+          CreateFlipperTables.migrate(:up)
+        end
       end
-    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)
+      after(:each) do
+        ActiveRecord::Tasks::DatabaseTasks.purge(config)
+        ActiveRecord::Base.connection.close
       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)
+      after(:all) do
+        ActiveRecord::Tasks::DatabaseTasks.drop(config)
       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)
-      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.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
+      it "works when table doesn't exist" do
+        CreateFlipperTables.migrate(:down)
 
-    context "#get_multi" do
-      let(:feature) { Flipper::Feature.new(:search, subject) }
+        Flipper.configuration = nil
+        Flipper.instance = nil
 
-      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)
+        silence_warnings { load 'flipper/adapters/active_record.rb' }
+        expect { Flipper::Adapters::ActiveRecord.new }.not_to raise_error
       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 "should load actor ids fine" do
+        flipper.enable_percentage_of_time(:foo, 1)
 
-    context "#enable/#disable boolean" do
-      let(:feature) { Flipper::Feature.new(:search, subject) }
-      let(:gate) { feature.gate(:boolean)}
+        Flipper::Adapters::ActiveRecord::Gate.create!(
+          feature_key: "foo",
+          key: "actors",
+          value: "Organization;4",
+        )
 
-      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)
+        flipper = Flipper.new(subject)
+        flipper.preload([:foo])
       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
+      context 'requiring "flipper-active_record"' do
+        before do
+          Flipper.configuration = nil
+          Flipper.instance = nil
 
-    context "#enable/#disable set" do
-      let(:feature) { Flipper::Feature.new(:search, subject) }
-      let(:gate) { feature.gate(:group) }
+          silence_warnings { load 'flipper/adapters/active_record.rb' }
+        end
 
-      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)
+        it 'configures itself' do
+          expect(Flipper.adapter.adapter).to be_a(Flipper::Adapters::ActiveRecord)
+        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.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)
+      context "ActiveRecord connection_pool" do
+        before do
+          ActiveRecord::Base.clear_active_connections!
+        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 "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
+
+        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
+
+        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
+
+        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
+
+        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
+
+        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
     end
   end

data/test_rails/generators/flipper/active_record_generator_test.rb

@@ -1,5 +1,4 @@
-require 'active_record'
-require 'rails/generators/test_case'
+require 'helper'
 require 'generators/flipper/active_record_generator'
 
 class FlipperActiveRecordGeneratorTest < Rails::Generators::TestCase
@@ -16,7 +15,7 @@ class FlipperActiveRecordGeneratorTest < Rails::Generators::TestCase
                         end
     assert_migration 'db/migrate/create_flipper_tables.rb', <<~MIGRATION
       class CreateFlipperTables < ActiveRecord::Migration#{migration_version}
-        def self.up
+        def up
           create_table :flipper_features do |t|
             t.string :key, null: false
             t.timestamps null: false
@@ -26,13 +25,13 @@ class FlipperActiveRecordGeneratorTest < Rails::Generators::TestCase
           create_table :flipper_gates do |t|
             t.string :feature_key, null: false
             t.string :key, null: false
-            t.string :value
+            t.text :value
             t.timestamps null: false
           end
-          add_index :flipper_gates, [:feature_key, :key, :value], unique: true
+          add_index :flipper_gates, [:feature_key, :key, :value], unique: true, length: { value: 255 }
         end
 
-        def self.down
+        def down
           drop_table :flipper_gates
           drop_table :flipper_features
         end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: flipper-active_record
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.1.2
 platform: ruby
 authors:
 - John Nunemaker
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-12-08 00:00:00.000000000 Z
+date: 2023-12-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: flipper
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.1.0
+        version: 1.1.2
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 1.1.0
+        version: 1.1.2
 - !ruby/object:Gem::Dependency
   name: activerecord
   requirement: !ruby/object:Gem::Requirement
@@ -60,12 +60,10 @@ files:
 - flipper-active_record.gemspec
 - lib/flipper-active_record.rb
 - lib/flipper/adapters/active_record.rb
-- lib/flipper/model/active_record.rb
 - lib/flipper/version.rb
 - lib/generators/flipper/active_record_generator.rb
 - lib/generators/flipper/templates/migration.erb
 - spec/flipper/adapters/active_record_spec.rb
-- spec/flipper/model/active_record_spec.rb
 - test/adapters/active_record_test.rb
 - test_rails/generators/flipper/active_record_generator_test.rb
 homepage: https://www.flippercloud.io/docs/adapters/active-record
@@ -95,8 +93,7 @@ requirements: []
 rubygems_version: 3.4.10
 signing_key:
 specification_version: 4
-summary: ActiveRecord adapter for Flipper
+summary: ActiveRecord feature flag adapter for Flipper
 test_files:
 - spec/flipper/adapters/active_record_spec.rb
-- spec/flipper/model/active_record_spec.rb
 - test/adapters/active_record_test.rb

data/lib/flipper/model/active_record.rb

@@ -1,23 +0,0 @@
-module Flipper
-  module Model
-    module ActiveRecord
-      # The id of the record when used as an actor.
-      #
-      #   class User < ActiveRecord::Base
-      #   end
-      #
-      #   user = User.first
-      #   Flipper.enable :some_feature, user
-      #   Flipper.enabled? :some_feature, user #=> true
-      #
-      def flipper_id
-        "#{self.class.base_class.name};#{id}"
-      end
-
-      # Properties used to evaluate expressions
-      def flipper_properties
-        {"type" => self.class.name}.merge(attributes)
-      end
-    end
-  end
-end

data/spec/flipper/model/active_record_spec.rb

@@ -1,61 +0,0 @@
-require 'active_record'
-require 'flipper/model/active_record'
-
-# Turn off migration logging for specs
-ActiveRecord::Migration.verbose = false
-
-RSpec.describe Flipper::Model::ActiveRecord do
-  before(:all) do
-    ActiveRecord::Base.establish_connection(adapter: 'sqlite3', database: ':memory:')
-  end
-
-  before(:each) do
-    ActiveRecord::Base.connection.execute <<-SQL
-      CREATE TABLE users (
-        id integer PRIMARY KEY,
-        name string NOT NULL,
-        age integer,
-        is_confirmed boolean,
-        created_at datetime NOT NULL,
-        updated_at datetime NOT NULL
-      )
-    SQL
-  end
-
-  after(:each) do
-    ActiveRecord::Base.connection.execute("DROP table IF EXISTS `users`")
-  end
-
-  class User < ActiveRecord::Base
-    include Flipper::Model::ActiveRecord
-  end
-
-  class Admin < User
-  end
-
-  describe "flipper_id" do
-    it "returns class name and id" do
-      expect(User.new(id: 1).flipper_id).to eq("User;1")
-    end
-
-    it "uses base class name" do
-      expect(Admin.new(id: 2).flipper_id).to eq("User;2")
-    end
-  end
-
-  describe "flipper_properties" do
-    subject { User.create!(name: "Test", age: 22, is_confirmed: true) }
-
-    it "includes all attributes" do
-      expect(subject.flipper_properties).to eq({
-        "type" => "User",
-        "id" => subject.id,
-        "name" => "Test",
-        "age" => 22,
-        "is_confirmed" => true,
-        "created_at" => subject.created_at,
-        "updated_at" => subject.updated_at
-      })
-    end
-  end
-end