database_consistency 1.4.1 → 1.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5c6d7fe7d9a50ab2233cca7b018d096f9d47b850e2fb92b2724fecba15eb78e6
-  data.tar.gz: 5c98abb3adb05dfddb3c586e3eae4a87d35069d07332c516154791514cc9c70f
+  metadata.gz: 24c245f8df70e4e98c102b52a56ead68364ba0613a67d1d60d499c36ddfae5b2
+  data.tar.gz: c1d8ce538657703f75a56f3fa462922ddbc5176d328e94665c83f71fd92bb3ed
 SHA512:
-  metadata.gz: 32ee0e823c4c893f5e062f847f33dca5fe6094bfc98d12786f879cbe6a2d2182a44f5d9756271550b524093c56030d38f72ce4ff58de6175b2936852e3e7d3d0
-  data.tar.gz: 210a2a55056fee4992ad8e7b719aa42604b03944891e2061976b27972ad8fd5c7b8ac920fb5b888f193a0dd2fbb27e2a205ffba1252a83c736a5926a471641db
+  metadata.gz: 00614cf53933e47d9599698d6b412294446b50c6757ea042374247b2b400b4fec12743d00a02401040630e9912aee54fa65e472a4d683e5a2116e93e5f7f48a6
+  data.tar.gz: 5b87a53b2566b6892920ff883909f0a3ae871f137c8ed482b85fdc1a4ece3bd83eb1f68218a391a75dbdfa977a66cd2a34e9eb33810083a024039ccd636ff833

data/lib/database_consistency/checkers/association_checkers/foreign_key_cascade_checker.rb

@@ -0,0 +1,89 @@
+# frozen_string_literal: true
+
+module DatabaseConsistency
+  module Checkers
+    # This class checks if non polymorphic +belongs_to+ association has foreign key constraint
+    class ForeignKeyCascadeChecker < AssociationChecker
+      Report = ReportBuilder.define(
+        DatabaseConsistency::Report,
+        :cascade_option,
+        :primary_table,
+        :foreign_table,
+        :primary_key,
+        :foreign_key
+      )
+
+      OPTION_TO_CASCADE = {
+        delete: :cascade,
+        delete_all: :cascade,
+        nullify: :nullify,
+        restrict_with_exception: :restrict,
+        restrict_with_error: :restrict
+      }.freeze
+
+      DEPENDENT_OPTIONS = OPTION_TO_CASCADE.keys.freeze
+
+      private
+
+      def preconditions
+        !association.polymorphic? &&
+          !association.belongs_to? &&
+          foreign_key &&
+          DEPENDENT_OPTIONS.include?(dependent_option)
+      rescue StandardError
+        false
+      end
+
+      # Table of possible statuses
+      # | foreign key | status |
+      # | ----------- | ------ |
+      # | persisted   | ok     |
+      # | missing     | fail   |
+      def check
+        if correlated_cascade_constraint?
+          report_template(:ok)
+        else
+          report_template(:fail, error_slug: :missing_foreign_key_cascade)
+        end
+      end
+
+      def required_foreign_key_cascade
+        OPTION_TO_CASCADE[dependent_option]
+      end
+
+      def correlated_cascade_constraint?
+        required_foreign_key_cascade == foreign_key_on_delete_option
+      end
+
+      def dependent_option
+        association.options[:dependent]
+      end
+
+      def foreign_key_on_delete_option
+        foreign_key.options[:on_delete]
+      end
+
+      def foreign_key
+        @foreign_key ||=
+          association.klass
+                     .connection
+                     .foreign_keys(association.klass.table_name)
+                     .find { |fk| fk.column == association.foreign_key.to_s }
+      end
+
+      def report_template(status, error_slug: nil)
+        Report.new(
+          status: status,
+          error_message: nil,
+          error_slug: error_slug,
+          primary_table: association.table_name.to_s,
+          primary_key: association.association_primary_key.to_s,
+          foreign_table: association.active_record.table_name.to_s,
+          foreign_key: association.foreign_key.to_s,
+          cascade_option: required_foreign_key_cascade,
+          **report_attributes
+        )
+      end
+    end
+  end
+end

data/lib/database_consistency/processors/associations_processor.rb

@@ -7,7 +7,8 @@ module DatabaseConsistency
       CHECKERS = [
         Checkers::MissingIndexChecker,
         Checkers::ForeignKeyChecker,
-        Checkers::ForeignKeyTypeChecker
+        Checkers::ForeignKeyTypeChecker,
+        Checkers::ForeignKeyCascadeChecker
       ].freeze
 
       private

data/lib/database_consistency/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module DatabaseConsistency
-  VERSION = '1.4.1'
+  VERSION = '1.5.0'
 end

data/lib/database_consistency/writers/simple/missing_foreign_key_cascade.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+module DatabaseConsistency
+  module Writers
+    module Simple
+      class MissingForeignKeyCascade < Base # :nodoc:
+        private
+
+        def template
+          'should have foreign key with on_delete: :%<cascade_option>s in the database'
+        end
+
+        def attributes
+          {
+            cascade_option: report.cascade_option
+          }
+        end
+
+        def unique_attributes
+          {
+            foreign_table: report.foreign_table,
+            foreign_key: report.foreign_key,
+            primary_table: report.primary_table,
+            primary_key: report.primary_key,
+            cascade_option: report.cascade_option
+          }
+        end
+      end
+    end
+  end
+end

data/lib/database_consistency/writers/simple_writer.rb

@@ -23,7 +23,8 @@ module DatabaseConsistency
         redundant_index: Simple::RedundantIndex,
         redundant_unique_index: Simple::RedundantUniqueIndex,
         small_primary_key: Simple::SmallPrimaryKey,
-        inconsistent_enum_type: Simple::InconsistentEnumType
+        inconsistent_enum_type: Simple::InconsistentEnumType,
+        missing_foreign_key_cascade: Simple::MissingForeignKeyCascade
       }.freeze
 
       def write

data/lib/database_consistency.rb

@@ -18,6 +18,7 @@ require 'database_consistency/writers/simple/default_message'
 require 'database_consistency/writers/simple/inconsistent_types'
 require 'database_consistency/writers/simple/null_constraint_association_misses_validator'
 require 'database_consistency/writers/simple/redundant_index'
+require 'database_consistency/writers/simple/missing_foreign_key_cascade'
 require 'database_consistency/writers/simple/redundant_unique_index'
 require 'database_consistency/writers/simple/association_missing_index'
 require 'database_consistency/writers/simple/association_missing_null_constraint'
@@ -59,6 +60,7 @@ require 'database_consistency/checkers/association_checkers/association_checker'
 require 'database_consistency/checkers/association_checkers/missing_index_checker'
 require 'database_consistency/checkers/association_checkers/foreign_key_checker'
 require 'database_consistency/checkers/association_checkers/foreign_key_type_checker'
+require 'database_consistency/checkers/association_checkers/foreign_key_cascade_checker'
 
 require 'database_consistency/checkers/column_checkers/column_checker'
 require 'database_consistency/checkers/column_checkers/null_constraint_checker'

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: database_consistency
 version: !ruby/object:Gem::Version
-  version: 1.4.1
+  version: 1.5.0
 platform: ruby
 authors:
 - Evgeniy Demin
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-11-27 00:00:00.000000000 Z
+date: 2022-11-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -147,6 +147,7 @@ files:
 - bin/database_consistency
 - lib/database_consistency.rb
 - lib/database_consistency/checkers/association_checkers/association_checker.rb
+- lib/database_consistency/checkers/association_checkers/foreign_key_cascade_checker.rb
 - lib/database_consistency/checkers/association_checkers/foreign_key_checker.rb
 - lib/database_consistency/checkers/association_checkers/foreign_key_type_checker.rb
 - lib/database_consistency/checkers/association_checkers/missing_index_checker.rb
@@ -211,6 +212,7 @@ files:
 - lib/database_consistency/writers/simple/length_validator_lower_limit.rb
 - lib/database_consistency/writers/simple/length_validator_missing.rb
 - lib/database_consistency/writers/simple/missing_foreign_key.rb
+- lib/database_consistency/writers/simple/missing_foreign_key_cascade.rb
 - lib/database_consistency/writers/simple/missing_unique_index.rb
 - lib/database_consistency/writers/simple/missing_uniqueness_validation.rb
 - lib/database_consistency/writers/simple/null_constraint_association_misses_validator.rb