acts_as_paranoid 0.5.0 → 0.7.0

data/test/test_default_scopes.rb

@@ -1,52 +1,53 @@
-require 'test_helper'
+# frozen_string_literal: true
+
+require "test_helper"
 
 class MultipleDefaultScopesTest < ParanoidBaseTest
   def setup
     setup_db
 
-    # Naturally, the default scope for humans is male. Sexism++
-    ParanoidHuman.create! :gender => 'male'
-    ParanoidHuman.create! :gender => 'male'
-    ParanoidHuman.create! :gender => 'male'
-    ParanoidHuman.create! :gender => 'female'
+    ParanoidPolygon.create! sides: 3
+    ParanoidPolygon.create! sides: 3
+    ParanoidPolygon.create! sides: 3
+    ParanoidPolygon.create! sides: 8
 
-    assert_equal 3, ParanoidHuman.count
-    assert_equal 4, ParanoidHuman.unscoped.count
+    assert_equal 3, ParanoidPolygon.count
+    assert_equal 4, ParanoidPolygon.unscoped.count
   end
 
   def test_fake_removal_with_multiple_default_scope
-    ParanoidHuman.first.destroy
-    assert_equal 2, ParanoidHuman.count
-    assert_equal 3, ParanoidHuman.with_deleted.count
-    assert_equal 1, ParanoidHuman.only_deleted.count
-    assert_equal 4, ParanoidHuman.unscoped.count
-
-    ParanoidHuman.destroy_all
-    assert_equal 0, ParanoidHuman.count
-    assert_equal 3, ParanoidHuman.with_deleted.count
-    assert_equal 3, ParanoidHuman.with_deleted.count
-    assert_equal 4, ParanoidHuman.unscoped.count
+    ParanoidPolygon.first.destroy
+    assert_equal 2, ParanoidPolygon.count
+    assert_equal 3, ParanoidPolygon.with_deleted.count
+    assert_equal 1, ParanoidPolygon.only_deleted.count
+    assert_equal 4, ParanoidPolygon.unscoped.count
+
+    ParanoidPolygon.destroy_all
+    assert_equal 0, ParanoidPolygon.count
+    assert_equal 3, ParanoidPolygon.with_deleted.count
+    assert_equal 3, ParanoidPolygon.with_deleted.count
+    assert_equal 4, ParanoidPolygon.unscoped.count
   end
 
   def test_real_removal_with_multiple_default_scope
     # two-step
-    ParanoidHuman.first.destroy
-    ParanoidHuman.only_deleted.first.destroy
-    assert_equal 2, ParanoidHuman.count
-    assert_equal 2, ParanoidHuman.with_deleted.count
-    assert_equal 0, ParanoidHuman.only_deleted.count
-    assert_equal 3, ParanoidHuman.unscoped.count
-
-    ParanoidHuman.first.destroy_fully!
-    assert_equal 1, ParanoidHuman.count
-    assert_equal 1, ParanoidHuman.with_deleted.count
-    assert_equal 0, ParanoidHuman.only_deleted.count
-    assert_equal 2, ParanoidHuman.unscoped.count
-
-    ParanoidHuman.delete_all!
-    assert_equal 0, ParanoidHuman.count
-    assert_equal 0, ParanoidHuman.with_deleted.count
-    assert_equal 0, ParanoidHuman.only_deleted.count
-    assert_equal 1, ParanoidHuman.unscoped.count
+    ParanoidPolygon.first.destroy
+    ParanoidPolygon.only_deleted.first.destroy
+    assert_equal 2, ParanoidPolygon.count
+    assert_equal 2, ParanoidPolygon.with_deleted.count
+    assert_equal 0, ParanoidPolygon.only_deleted.count
+    assert_equal 3, ParanoidPolygon.unscoped.count
+
+    ParanoidPolygon.first.destroy_fully!
+    assert_equal 1, ParanoidPolygon.count
+    assert_equal 1, ParanoidPolygon.with_deleted.count
+    assert_equal 0, ParanoidPolygon.only_deleted.count
+    assert_equal 2, ParanoidPolygon.unscoped.count
+
+    ParanoidPolygon.delete_all!
+    assert_equal 0, ParanoidPolygon.count
+    assert_equal 0, ParanoidPolygon.with_deleted.count
+    assert_equal 0, ParanoidPolygon.only_deleted.count
+    assert_equal 1, ParanoidPolygon.unscoped.count
   end
 end

data/test/test_helper.rb

@@ -1,23 +1,32 @@
-require 'bundler'
+# frozen_string_literal: true
+
+require "bundler"
 begin
-  Bundler.require(:default, :development)
+  Bundler.load
 rescue Bundler::BundlerError => e
-  $stderr.puts e.message
-  $stderr.puts "Run `bundle install` to install missing gems"
+  warn e.message
+  warn "Run `bundle install` to install missing gems"
   exit e.status_code
 end
 
-require 'acts_as_paranoid'
-require 'minitest/autorun'
+require "simplecov"
+SimpleCov.start do
+  enable_coverage :branch
+end
+
+require "acts_as_paranoid"
+require "minitest/autorun"
 
 # Silence deprecation halfway through the test
 I18n.enforce_available_locales = true
 
-ActiveRecord::Base.establish_connection(:adapter => "sqlite3", :database => ":memory:")
+ActiveRecord::Base.establish_connection(adapter: "sqlite3", database: ":memory:")
 ActiveRecord::Schema.verbose = false
 
+# rubocop:disable Metrics/AbcSize
+# rubocop:disable Metrics/MethodLength
 def setup_db
-  ActiveRecord::Schema.define(:version => 1) do
+  ActiveRecord::Schema.define(version: 1) do # rubocop:disable Metrics/BlockLength
     create_table :paranoid_times do |t|
       t.string    :name
       t.datetime  :deleted_at
@@ -31,7 +40,8 @@ def setup_db
       t.string    :name
       t.boolean   :is_deleted
       t.integer   :paranoid_time_id
-
+      t.integer   :paranoid_with_counter_caches_count
+      t.integer   :custom_counter_cache
       timestamps t
     end
 
@@ -111,14 +121,15 @@ def setup_db
 
     create_table :super_paranoids do |t|
       t.string :type
-      t.references :has_many_inherited_super_paranoidz, index: { name: 'index__sp_id_on_has_many_isp' }
+      t.references :has_many_inherited_super_paranoidz,
+                   index: { name: "index__sp_id_on_has_many_isp" }
       t.datetime :deleted_at
 
       timestamps t
     end
 
     create_table :has_many_inherited_super_paranoidzs do |t|
-      t.references :super_paranoidz, index: { name: 'index_has_many_isp_on_sp_id' }
+      t.references :super_paranoidz, index: { name: "index_has_many_isp_on_sp_id" }
       t.datetime :deleted_at
 
       timestamps t
@@ -148,7 +159,7 @@ def setup_db
       timestamps t
     end
 
-   create_table :paranoid_forests do |t|
+    create_table :paranoid_forests do |t|
       t.string   :name
       t.boolean  :rainforest
       t.datetime :deleted_at
@@ -164,8 +175,8 @@ def setup_db
       timestamps t
     end
 
-    create_table :paranoid_humen do |t|
-      t.string   :gender
+    create_table :paranoid_polygons do |t|
+      t.integer :sides
       t.datetime :deleted_at
 
       timestamps t
@@ -184,7 +195,7 @@ def setup_db
 
     create_table :paranoid_boolean_not_nullables do |t|
       t.string :name
-      t.boolean :deleted, :boolean, :null => false, :default => false
+      t.boolean :deleted, :boolean, null: false, default: false
     end
 
     create_table :paranoid_belongs_to_polymorphics do |t|
@@ -193,36 +204,47 @@ def setup_db
       t.integer :parent_id
       t.datetime :deleted_at
 
-      t.timestamps
+      timestamps t
     end
 
     create_table :not_paranoid_has_many_as_parents do |t|
       t.string :name
 
-      t.timestamps
+      timestamps t
     end
 
     create_table :paranoid_has_many_as_parents do |t|
       t.string :name
       t.datetime :deleted_at
 
-      t.timestamps
+      timestamps t
+    end
+
+    create_table :paranoid_no_double_tap_destroys_fullies do |t|
+      t.datetime :deleted_at
+    end
+
+    create_table :paranoid_with_counter_caches do |t|
+      t.string    :name
+      t.datetime  :deleted_at
+      t.integer   :paranoid_boolean_id
+
+      timestamps t
     end
   end
 end
+# rubocop:enable Metrics/AbcSize
+# rubocop:enable Metrics/MethodLength
 
 def timestamps(table)
-  table.column  :created_at , :timestamp, :null => false
-  table.column  :updated_at , :timestamp, :null => false
+  table.column  :created_at, :timestamp, null: false
+  table.column  :updated_at, :timestamp, null: false
 end
 
 def teardown_db
-  tables = if ActiveRecord::VERSION::MAJOR < 5
-    ActiveRecord::Base.connection.tables
-  else
-    ActiveRecord::Base.connection.data_sources
+  ActiveRecord::Base.connection.data_sources.each do |table|
+    ActiveRecord::Base.connection.drop_table(table)
   end
-  tables.each { |table| ActiveRecord::Base.connection.drop_table(table) }
 end
 
 class ParanoidTime < ActiveRecord::Base
@@ -230,49 +252,92 @@ class ParanoidTime < ActiveRecord::Base
 
   validates_uniqueness_of :name
 
-  has_many :paranoid_has_many_dependants, :dependent => :destroy
-  has_many :paranoid_booleans, :dependent => :destroy
-  has_many :not_paranoids, :dependent => :delete_all
-  has_many :paranoid_sections, :dependent => :destroy
+  has_many :paranoid_has_many_dependants, dependent: :destroy
+  has_many :paranoid_booleans, dependent: :destroy
+  has_many :not_paranoids, dependent: :delete_all
+  has_many :paranoid_sections, dependent: :destroy
 
-  has_one :has_one_not_paranoid, :dependent => :destroy
+  has_one :has_one_not_paranoid, dependent: :destroy
 
-  belongs_to :not_paranoid, :dependent => :destroy
+  belongs_to :not_paranoid, dependent: :destroy
 end
 
 class ParanoidBoolean < ActiveRecord::Base
-  acts_as_paranoid :column_type => "boolean", :column => "is_deleted"
+  acts_as_paranoid column_type: "boolean", column: "is_deleted"
   validates_as_paranoid
   validates_uniqueness_of_without_deleted :name
 
   belongs_to :paranoid_time
-  has_one :paranoid_has_one_dependant, :dependent => :destroy
+  has_one :paranoid_has_one_dependant, dependent: :destroy
+  has_many :paranoid_with_counter_cache, dependent: :destroy
+  has_many :paranoid_with_custom_counter_cache, dependent: :destroy
 end
 
 class ParanoidString < ActiveRecord::Base
-  acts_as_paranoid :column_type => "string", :column => "deleted", :deleted_value => "dead"
+  acts_as_paranoid column_type: "string", column: "deleted", deleted_value: "dead"
 end
 
 class NotParanoid < ActiveRecord::Base
 end
 
+class ParanoidNoDoubleTapDestroysFully < ActiveRecord::Base
+  acts_as_paranoid double_tap_destroys_fully: false
+end
+
 class HasOneNotParanoid < ActiveRecord::Base
-  belongs_to :paranoid_time, :with_deleted => true
+  belongs_to :paranoid_time, with_deleted: true
 end
 
 class DoubleHasOneNotParanoid < HasOneNotParanoid
-  belongs_to :paranoid_time, :with_deleted => true
-  belongs_to :paranoid_time, :with_deleted => true
+  belongs_to :paranoid_time, with_deleted: true
+  begin
+    verbose = $VERBOSE
+    $VERBOSE = false
+    belongs_to :paranoid_time, with_deleted: true
+  ensure
+    $VERBOSE = verbose
+  end
+end
+
+class ParanoidWithCounterCache < ActiveRecord::Base
+  acts_as_paranoid
+  belongs_to :paranoid_boolean, counter_cache: true
+end
+
+class ParanoidWithCustomCounterCache < ActiveRecord::Base
+  self.table_name = "paranoid_with_counter_caches"
+
+  acts_as_paranoid
+  belongs_to :paranoid_boolean, counter_cache: :custom_counter_cache
+end
+
+class ParanoidWithCounterCacheOnOptionalBelognsTo < ActiveRecord::Base
+  self.table_name = "paranoid_with_counter_caches"
+
+  acts_as_paranoid
+  if ActiveRecord::VERSION::MAJOR < 5
+    belongs_to :paranoid_boolean, counter_cache: true, required: false
+  else
+    belongs_to :paranoid_boolean, counter_cache: true, optional: true
+  end
 end
 
 class ParanoidHasManyDependant < ActiveRecord::Base
   acts_as_paranoid
   belongs_to :paranoid_time
-  belongs_to :paranoid_time_with_scope, -> { includes(:not_paranoid) }, :class_name => 'ParanoidTime', :foreign_key => :paranoid_time_id
-  belongs_to :paranoid_time_with_deleted, :class_name => 'ParanoidTime', :foreign_key => :paranoid_time_id, :with_deleted => true
-  belongs_to :paranoid_time_polymorphic_with_deleted, :class_name => 'ParanoidTime', :foreign_key => :paranoid_time_id, :polymorphic => true, :with_deleted => true
+  belongs_to :paranoid_time_with_scope,
+             -> { where(name: "hello").includes(:not_paranoid) },
+             class_name: "ParanoidTime", foreign_key: :paranoid_time_id
+  belongs_to :paranoid_time_with_deleted, class_name: "ParanoidTime",
+                                          foreign_key: :paranoid_time_id, with_deleted: true
+  belongs_to :paranoid_time_with_scope_with_deleted,
+             -> { where(name: "hello").includes(:not_paranoid) },
+             class_name: "ParanoidTime", foreign_key: :paranoid_time_id, with_deleted: true
+  belongs_to :paranoid_time_polymorphic_with_deleted, class_name: "ParanoidTime",
+                                                      foreign_key: :paranoid_time_id,
+                                                      polymorphic: true, with_deleted: true
 
-  belongs_to :paranoid_belongs_dependant, :dependent => :destroy
+  belongs_to :paranoid_belongs_dependant, dependent: :destroy
 end
 
 class ParanoidBelongsDependant < ActiveRecord::Base
@@ -290,13 +355,14 @@ end
 class ParanoidWithCallback < ActiveRecord::Base
   acts_as_paranoid
 
-  attr_accessor :called_before_destroy, :called_after_destroy, :called_after_commit_on_destroy
-  attr_accessor :called_before_recover, :called_after_recover
+  attr_accessor :called_before_destroy, :called_after_destroy,
+                :called_after_commit_on_destroy, :called_before_recover,
+                :called_after_recover
 
   before_destroy :call_me_before_destroy
   after_destroy :call_me_after_destroy
 
-  after_commit :call_me_after_commit_on_destroy, :on => :destroy
+  after_commit :call_me_after_commit_on_destroy, on: :destroy
 
   before_recover :call_me_before_recover
   after_recover :call_me_after_recover
@@ -329,14 +395,14 @@ end
 
 class ParanoidDestroyCompany < ActiveRecord::Base
   acts_as_paranoid
-  validates :name, :presence => true
-  has_many :paranoid_products, :dependent => :destroy
+  validates :name, presence: true
+  has_many :paranoid_products, dependent: :destroy
 end
 
 class ParanoidDeleteCompany < ActiveRecord::Base
   acts_as_paranoid
-  validates :name, :presence => true
-  has_many :paranoid_products, :dependent => :delete_all
+  validates :name, presence: true
+  has_many :paranoid_products, dependent: :delete_all
 end
 
 class ParanoidProduct < ActiveRecord::Base
@@ -352,22 +418,21 @@ class SuperParanoid < ActiveRecord::Base
 end
 
 class HasManyInheritedSuperParanoidz < ActiveRecord::Base
-  has_many :super_paranoidz, :class_name => "InheritedParanoid", :dependent => :destroy
+  has_many :super_paranoidz, class_name: "InheritedParanoid", dependent: :destroy
 end
 
 class InheritedParanoid < SuperParanoid
   acts_as_paranoid
 end
 
-
 class ParanoidManyManyParentLeft < ActiveRecord::Base
   has_many :paranoid_many_many_children
-  has_many :paranoid_many_many_parent_rights, :through => :paranoid_many_many_children
+  has_many :paranoid_many_many_parent_rights, through: :paranoid_many_many_children
 end
 
 class ParanoidManyManyParentRight < ActiveRecord::Base
   has_many :paranoid_many_many_children
-  has_many :paranoid_many_many_parent_lefts, :through => :paranoid_many_many_children
+  has_many :paranoid_many_many_parent_lefts, through: :paranoid_many_many_children
 end
 
 class ParanoidManyManyChild < ActiveRecord::Base
@@ -378,21 +443,21 @@ end
 
 class ParanoidWithScopedValidation < ActiveRecord::Base
   acts_as_paranoid
-  validates_uniqueness_of :name, :scope => :category
+  validates_uniqueness_of :name, scope: :category
 end
 
 class ParanoidBelongsToPolymorphic < ActiveRecord::Base
   acts_as_paranoid
-  belongs_to :parent, :polymorphic => true, :with_deleted => true
+  belongs_to :parent, polymorphic: true, with_deleted: true
 end
 
 class NotParanoidHasManyAsParent < ActiveRecord::Base
-  has_many :paranoid_belongs_to_polymorphics, :as => :parent, :dependent => :destroy
+  has_many :paranoid_belongs_to_polymorphics, as: :parent, dependent: :destroy
 end
 
 class ParanoidHasManyAsParent < ActiveRecord::Base
   acts_as_paranoid
-  has_many :paranoid_belongs_to_polymorphics, :as => :parent, :dependent => :destroy
+  has_many :paranoid_belongs_to_polymorphics, as: :parent, dependent: :destroy
 end
 
 class ParanoidBaseTest < ActiveSupport::TestCase
@@ -400,13 +465,13 @@ class ParanoidBaseTest < ActiveSupport::TestCase
     setup_db
 
     ["paranoid", "really paranoid", "extremely paranoid"].each do |name|
-      ParanoidTime.create! :name => name
-      ParanoidBoolean.create! :name => name
+      ParanoidTime.create! name: name
+      ParanoidBoolean.create! name: name
     end
 
-    ParanoidString.create! :name => "strings can be paranoid"
-    NotParanoid.create! :name => "no paranoid goals"
-    ParanoidWithCallback.create! :name => "paranoid with callbacks"
+    ParanoidString.create! name: "strings can be paranoid"
+    NotParanoid.create! name: "no paranoid goals"
+    ParanoidWithCallback.create! name: "paranoid with callbacks"
   end
 
   def teardown
@@ -440,9 +505,9 @@ class ParanoidForest < ActiveRecord::Base
 
   ActiveRecord::Base.logger = Logger.new(StringIO.new)
 
-  scope :rainforest, lambda{ where(:rainforest => true) }
+  scope :rainforest, -> { where(rainforest: true) }
 
-  has_many :paranoid_trees, :dependent => :destroy
+  has_many :paranoid_trees, dependent: :destroy
 end
 
 class ParanoidTree < ActiveRecord::Base
@@ -451,9 +516,9 @@ class ParanoidTree < ActiveRecord::Base
   validates_presence_of :name
 end
 
-class ParanoidHuman < ActiveRecord::Base
+class ParanoidPolygon < ActiveRecord::Base
   acts_as_paranoid
-  default_scope { where('gender = ?', 'male') }
+  default_scope { where("sides = ?", 3) }
 end
 
 class ParanoidAndroid < ActiveRecord::Base
@@ -463,10 +528,14 @@ end
 class ParanoidSection < ActiveRecord::Base
   acts_as_paranoid
   belongs_to :paranoid_time
-  belongs_to :paranoid_thing, :polymorphic => true, :dependent => :destroy
+  belongs_to :paranoid_thing, polymorphic: true, dependent: :destroy
 end
 
 class ParanoidBooleanNotNullable < ActiveRecord::Base
-  acts_as_paranoid column: 'deleted', column_type: 'boolean', allow_nulls: false
+  acts_as_paranoid column: "deleted", column_type: "boolean", allow_nulls: false
 end
 
+class ParanoidWithExplicitTableNameAfterMacro < ActiveRecord::Base
+  acts_as_paranoid
+  self.table_name = "explicit_table"
+end