database_consistency 1.7.12 → 1.7.14

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8560e4cc5588b8182f8d5258703894839190a758abf57e35e2abe81a630848fc
-  data.tar.gz: 26b96205cd47d48f9a557d6986ef2d2f0eef940f1407ed430760f6201c91cca5
+  metadata.gz: 4aae1ecf43d2307f2006fecf4ba08a40e69565bb240ce8d7c59d2d271ec11ee0
+  data.tar.gz: f59e8e3422196f502c6aefe81c8995c01b2167ad7f158d57ceabb787cd2f46e9
 SHA512:
-  metadata.gz: bfb1870b761338b23481926ec64cea7a814b7a174937d5035ca6a71ef63f7cfd332db6ab243f15ff5e9990a6445522b5565d2103b0c45eeb147f22eddbc48d7f
-  data.tar.gz: feec3a2004be04efe8ab1329a79b4b4395ffa3feec5c7798ad04b8218b7365b917a89a80681409d9a1827a5620105a7295ff6b67863828dd85b555c1dc8e7e7f
+  metadata.gz: cac4e844ed502055658862456132ea245cb5208bfd5d1a184c50097086f7ab469020aa00bf55b64ae9ff056eb69ce222ff3175150950829f0ad1ea8c7f99acb7
+  data.tar.gz: 0a5768efcffc548bd35feba6a6b8f22fd11ee1ae795db9aeb7664422a3bfbd88aef34f1216433be874b7be5fc2a2e4eb367a66048b95a56256922b21ac8ae551

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

@@ -22,6 +22,8 @@ module DatabaseConsistency
         supported? &&
           association.belongs_to? && !association.polymorphic? &&
           same_database?
+      rescue NameError
+        false
       end
 
       def same_database?

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

@@ -25,7 +25,8 @@ module DatabaseConsistency
         !association.polymorphic? &&
           association.through_reflection.nil? &&
           association.klass.present? &&
-          association.macro != :has_and_belongs_to_many
+          association.macro != :has_and_belongs_to_many &&
+          association.klass.table_exists?
       rescue NameError
         false
       end

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

@@ -0,0 +1,36 @@
+# frozen_string_literal: true
+
+module DatabaseConsistency
+  module Checkers
+    # This class checks if an association has existing class defined
+    class MissingAssociationClassChecker < AssociationChecker
+      Report = ReportBuilder.define(
+        DatabaseConsistency::Report,
+        :class_name
+      )
+
+      private
+
+      def preconditions
+        !association.polymorphic?
+      end
+
+      def check
+        association.klass
+        report_template(:ok)
+      rescue NameError
+        report_template(:fail, error_slug: :missing_association_class)
+      end
+
+      def report_template(status, error_slug: nil)
+        Report.new(
+          status: status,
+          error_message: nil,
+          error_slug: error_slug,
+          class_name: association.class_name,
+          **report_attributes
+        )
+      end
+    end
+  end
+end

data/lib/database_consistency/processors/associations_processor.rb

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

data/lib/database_consistency/version.rb

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

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

@@ -0,0 +1,27 @@
+# frozen_string_literal: true
+
+module DatabaseConsistency
+  module Writers
+    module Simple
+      class MissingAssociationClass < Base # :nodoc:
+        private
+
+        def template
+          'refers to undefined model "%<class_name>s"'
+        end
+
+        def unique_attributes
+          {
+            class_name: report.class_name
+          }
+        end
+
+        def attributes
+          {
+            class_name: report.class_name
+          }
+        end
+      end
+    end
+  end
+end

data/lib/database_consistency/writers/simple_writer.rb

@@ -6,29 +6,30 @@ module DatabaseConsistency
     # The simplest formatter
     class SimpleWriter < BaseWriter
       SLUG_TO_WRITER = {
+        association_foreign_type_missing_null_constraint: Simple::AssociationForeignTypeMissingNullConstraint,
         association_missing_index: Simple::AssociationMissingIndex,
         association_missing_null_constraint: Simple::AssociationMissingNullConstraint,
-        association_foreign_type_missing_null_constraint: Simple::AssociationForeignTypeMissingNullConstraint,
+        enum_values_inconsistent_with_ar_enum: Simple::EnumValuesInconsistentWithArEnum,
+        enum_values_inconsistent_with_inclusion: Simple::EnumValuesInconsistentWithInclusion,
         has_one_missing_unique_index: Simple::HasOneMissingUniqueIndex,
+        inconsistent_enum_type: Simple::InconsistentEnumType,
         inconsistent_types: Simple::InconsistentTypes,
         length_validator_greater_limit: Simple::LengthValidatorGreaterLimit,
         length_validator_lower_limit: Simple::LengthValidatorLowerLimit,
         length_validator_missing: Simple::LengthValidatorMissing,
+        missing_association_class: Simple::MissingAssociationClass,
         missing_foreign_key: Simple::MissingForeignKey,
+        missing_foreign_key_cascade: Simple::MissingForeignKeyCascade,
         missing_unique_index: Simple::MissingUniqueIndex,
         missing_uniqueness_validation: Simple::MissingUniquenessValidation,
         null_constraint_association_misses_validator: Simple::NullConstraintAssociationMissesValidator,
         null_constraint_misses_validator: Simple::NullConstraintMissesValidator,
         null_constraint_missing: Simple::NullConstraintMissing,
         possible_null: Simple::PossibleNull,
+        redundant_case_insensitive_option: Simple::RedundantCaseInsensitiveOption,
         redundant_index: Simple::RedundantIndex,
         redundant_unique_index: Simple::RedundantUniqueIndex,
         small_primary_key: Simple::SmallPrimaryKey,
-        inconsistent_enum_type: Simple::InconsistentEnumType,
-        missing_foreign_key_cascade: Simple::MissingForeignKeyCascade,
-        enum_values_inconsistent_with_ar_enum: Simple::EnumValuesInconsistentWithArEnum,
-        enum_values_inconsistent_with_inclusion: Simple::EnumValuesInconsistentWithInclusion,
-        redundant_case_insensitive_option: Simple::RedundantCaseInsensitiveOption,
         three_state_boolean: Simple::ThreeStateBoolean
       }.freeze
 

data/lib/database_consistency.rb

@@ -39,6 +39,7 @@ require 'database_consistency/writers/simple/enum_values_inconsistent_with_ar_en
 require 'database_consistency/writers/simple/enum_values_inconsistent_with_inclusion'
 require 'database_consistency/writers/simple/redundant_case_insensitive_option'
 require 'database_consistency/writers/simple/three_state_boolean'
+require 'database_consistency/writers/simple/missing_association_class'
 require 'database_consistency/writers/simple_writer'
 
 require 'database_consistency/writers/autofix/helpers/migration'
@@ -66,6 +67,7 @@ require 'database_consistency/checkers/association_checkers/missing_index_checke
 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/association_checkers/missing_association_class_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.7.12
+  version: 1.7.14
 platform: ruby
 authors:
 - Evgeniy Demin
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2023-07-05 00:00:00.000000000 Z
+date: 2023-07-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -150,6 +150,7 @@ files:
 - 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_association_class_checker.rb
 - lib/database_consistency/checkers/association_checkers/missing_index_checker.rb
 - lib/database_consistency/checkers/base_checker.rb
 - lib/database_consistency/checkers/column_checkers/column_checker.rb
@@ -217,6 +218,7 @@ files:
 - lib/database_consistency/writers/simple/length_validator_greater_limit.rb
 - 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_association_class.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
@@ -264,7 +266,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.6
+rubygems_version: 3.2.33
 signing_key: 
 specification_version: 4
 summary: Provide an easy way to check the consistency of the database constraints