database_consistency 1.7.27 → 2.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5ba4d8b566474e443336b5b7c471ad85bb492581847cb721daa5ba41be272bc0
-  data.tar.gz: c5c80cbfd15eae2e8b948cc44233d1c2c74a67a734ee0867a30defd3ed8a877d
+  metadata.gz: ba84801df64868302240abb6b325652bc21c34086c2b4d32460bb092f55d7681
+  data.tar.gz: 49ba63ec3857648e90121ef8e7692fe217fcb41592b21087876628caa7888427
 SHA512:
-  metadata.gz: 5d49d5e2e5f15577114ae6adaa8453db9ad9573bd0af6f0bc2eafb8319b30a04fae8f7ef6005e7c5990da1686d79a1d5e58ac535a182152767bf68764daf4edb
-  data.tar.gz: f9a07275157c90e2e20b04706a5f6b95974918d7c0716ea16f91cd2737a4b60bc0a82b65f9f2e8a286c1f9d3976d1d0f4f78b0dabc832079c37644010a8563d6
+  metadata.gz: f29ab6930f1317041b38879890a0195ed03d7620647a08baa5061fd82cd5b124587f0acafa9fe15b1d92747b4a8e7f05528fb4882b170cdf6feb23ed326547aa
+  data.tar.gz: a4e7fb560d24c41aa1dc02ad3127658ac9ab6f268be885843ca03cf24bc49e4354cec95f8d78a69ad44579eaef984a555c578e347eb98645de27248c75edfe3e

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

@@ -21,7 +21,8 @@ module DatabaseConsistency
       def preconditions
         supported? &&
           association.belongs_to? && !association.polymorphic? &&
-          same_database?
+          same_database? &&
+          model.connection.table_exists?(model.table_name)
       rescue NameError
         false
       end

data/lib/database_consistency/checkers/base_checker.rb

@@ -11,9 +11,19 @@ module DatabaseConsistency
         configuration.enabled?('DatabaseConsistencyCheckers', checker_name)
       end
 
+      def self.inherited(subclass)
+        super
+
+        return if subclass.superclass.name == 'DatabaseConsistency::Checkers::BaseChecker'
+
+        processor_prefix = subclass.superclass.name.demodulize.delete_suffix('Checker').pluralize
+        processor = "DatabaseConsistency::Processors::#{processor_prefix}Processor".constantize
+        processor.checkers << subclass
+      end
+
       # @return [String]
       def self.checker_name
-        @checker_name ||= name.split('::').last
+        @checker_name ||= name.demodulize
       end
 
       # @param [Boolean] catch_errors

data/lib/database_consistency/checkers/column_checkers/three_state_boolean_checker.rb

@@ -12,8 +12,9 @@ module DatabaseConsistency
 
       private
 
+      # column should be a boolean and table shouldn't be a view
       def preconditions
-        column.type == :boolean
+        column.type == :boolean && model.connection.table_exists?(model.table_name)
       end
 
       def check

data/lib/database_consistency/checkers/validators_fraction_checkers/column_presence_checker.rb

@@ -19,7 +19,7 @@ module DatabaseConsistency
       end
 
       def preconditions
-        (regular_column || association) && validators.any?
+        (regular_column || association) && model.connection.table_exists?(model.table_name) && validators.any?
       end
 
       def report_template(status, column_name:, error_slug: nil)

data/lib/database_consistency/configuration.rb

@@ -16,6 +16,8 @@ module DatabaseConsistency
         end
         extract_configurations(existing_paths)
       end
+
+      load_custom_checkers
     end
 
     def debug?
@@ -123,5 +125,13 @@ module DatabaseConsistency
           settings && settings['log_level']
         end
     end
+
+    def load_custom_checkers
+      return unless configuration.key?('require') && configuration['require'].is_a?(Array)
+
+      configuration['require'].each do |path|
+        require path
+      end
+    end
   end
 end

data/lib/database_consistency/processors/associations_processor.rb

@@ -4,14 +4,6 @@ module DatabaseConsistency
   module Processors
     # The class to process associations
     class AssociationsProcessor < BaseProcessor
-      CHECKERS = [
-        Checkers::MissingIndexChecker,
-        Checkers::ForeignKeyChecker,
-        Checkers::ForeignKeyTypeChecker,
-        Checkers::ForeignKeyCascadeChecker,
-        Checkers::MissingAssociationClassChecker
-      ].freeze
-
       private
 
       def check # rubocop:disable Metrics/MethodLength

data/lib/database_consistency/processors/base_processor.rb

@@ -4,21 +4,17 @@ module DatabaseConsistency
   # The module for processors
   module Processors
     def self.reports(configuration)
-      [
-        ColumnsProcessor,
-        ValidatorsProcessor,
-        AssociationsProcessor,
-        ValidatorsFractionsProcessor,
-        IndexesProcessor,
-        EnumsProcessor,
-        ModelsProcessor
-      ].flat_map do |processor|
+      BaseProcessor.descendants.flat_map do |processor|
         processor.new(configuration).reports
       end
     end
 
     # The base class for processors
     class BaseProcessor
+      def self.checkers
+        @checkers ||= Set.new
+      end
+
       attr_reader :configuration
 
       # @param [DatabaseConsistency::Configuration] configuration
@@ -38,7 +34,7 @@ module DatabaseConsistency
 
       # @return [Array<Class>]
       def enabled_checkers
-        self.class::CHECKERS.select { |checker| checker.enabled?(configuration) }
+        self.class.checkers.select { |checker| checker.enabled?(configuration) }
       end
 
       private

data/lib/database_consistency/processors/columns_processor.rb

@@ -4,15 +4,6 @@ module DatabaseConsistency
   module Processors
     # The class to process columns
     class ColumnsProcessor < BaseProcessor
-      CHECKERS = [
-        Checkers::NullConstraintChecker,
-        Checkers::LengthConstraintChecker,
-        Checkers::PrimaryKeyTypeChecker,
-        Checkers::EnumValueChecker,
-        Checkers::ThreeStateBooleanChecker,
-        Checkers::ImplicitOrderingChecker
-      ].freeze
-
       private
 
       def check # rubocop:disable Metrics/MethodLength

data/lib/database_consistency/processors/enums_processor.rb

@@ -4,10 +4,6 @@ module DatabaseConsistency
   module Processors
     # The class to process enums
     class EnumsProcessor < BaseProcessor
-      CHECKERS = [
-        Checkers::EnumTypeChecker
-      ].freeze
-
       private
 
       def check # rubocop:disable Metrics/MethodLength

data/lib/database_consistency/processors/indexes_processor.rb

@@ -4,12 +4,6 @@ module DatabaseConsistency
   module Processors
     # The class to process indexes
     class IndexesProcessor < BaseProcessor
-      CHECKERS = [
-        Checkers::UniqueIndexChecker,
-        Checkers::RedundantIndexChecker,
-        Checkers::RedundantUniqueIndexChecker
-      ].freeze
-
       private
 
       def check # rubocop:disable Metrics/AbcSize, Metrics/MethodLength

data/lib/database_consistency/processors/models_processor.rb

@@ -4,10 +4,6 @@ module DatabaseConsistency
   module Processors
     # The class to process models
     class ModelsProcessor < BaseProcessor
-      CHECKERS = [
-        Checkers::MissingTableChecker
-      ].freeze
-
       private
 
       def check

data/lib/database_consistency/processors/validators_fractions_processor.rb

@@ -4,10 +4,6 @@ module DatabaseConsistency
   module Processors
     # The class to process validators
     class ValidatorsFractionsProcessor < BaseProcessor
-      CHECKERS = [
-        Checkers::ColumnPresenceChecker
-      ].freeze
-
       private
 
       # @return [Array<Hash>]

data/lib/database_consistency/processors/validators_processor.rb

@@ -4,11 +4,6 @@ module DatabaseConsistency
   module Processors
     # The class to process validators
     class ValidatorsProcessor < BaseProcessor
-      CHECKERS = [
-        Checkers::MissingUniqueIndexChecker,
-        Checkers::CaseSensitiveUniqueValidationChecker
-      ].freeze
-
       private
 
       # @return [Array<Hash>]

data/lib/database_consistency/version.rb

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

data/lib/database_consistency/writers/simple_writer.rb

@@ -5,36 +5,6 @@ module DatabaseConsistency
   module Writers
     # 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,
-        enum_values_inconsistent_with_ar_enum: Simple::EnumValuesInconsistentWithArEnum,
-        enum_values_inconsistent_with_inclusion: Simple::EnumValuesInconsistentWithInclusion,
-        has_one_missing_unique_index: Simple::HasOneMissingUniqueIndex,
-        implicit_order_column_missing: Simple::ImplicitOrderColumnMissing,
-        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_table: Simple::MissingTable,
-        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,
-        three_state_boolean: Simple::ThreeStateBoolean
-      }.freeze
-
       def write
         results.select(&method(:write?))
                .map(&method(:writer))
@@ -58,7 +28,12 @@ module DatabaseConsistency
       end
 
       def writer(report)
-        klass = SLUG_TO_WRITER[report.error_slug] || Simple::DefaultMessage
+        klass = begin
+                  "DatabaseConsistency::Writers::Simple::#{report.error_slug.to_s.classify}".constantize
+                rescue NameError
+                  Simple::DefaultMessage
+                end
+
         klass.new(report, config: config)
       end
     end

data/lib/database_consistency.rb

@@ -1,6 +1,11 @@
 # frozen_string_literal: true
 
 require 'active_record'
+require 'active_support/inflector'
+
+ActiveSupport::Inflector.inflections do |inflect|
+  inflect.irregular 'index', 'indexes'
+end
 
 require 'database_consistency/version'
 require 'database_consistency/helper'
@@ -60,6 +65,15 @@ require 'database_consistency/databases/factory'
 require 'database_consistency/databases/types/base'
 require 'database_consistency/databases/types/sqlite'
 
+require 'database_consistency/processors/base_processor'
+require 'database_consistency/processors/enums_processor'
+require 'database_consistency/processors/associations_processor'
+require 'database_consistency/processors/validators_processor'
+require 'database_consistency/processors/columns_processor'
+require 'database_consistency/processors/validators_fractions_processor'
+require 'database_consistency/processors/indexes_processor'
+require 'database_consistency/processors/models_processor'
+
 require 'database_consistency/checkers/base_checker'
 
 require 'database_consistency/checkers/enum_checkers/enum_checker'
@@ -95,15 +109,6 @@ require 'database_consistency/checkers/index_checkers/unique_index_checker'
 require 'database_consistency/checkers/index_checkers/redundant_index_checker'
 require 'database_consistency/checkers/index_checkers/redundant_unique_index_checker'
 
-require 'database_consistency/processors/base_processor'
-require 'database_consistency/processors/enums_processor'
-require 'database_consistency/processors/associations_processor'
-require 'database_consistency/processors/validators_processor'
-require 'database_consistency/processors/columns_processor'
-require 'database_consistency/processors/validators_fractions_processor'
-require 'database_consistency/processors/indexes_processor'
-require 'database_consistency/processors/models_processor'
-
 # The root module
 module DatabaseConsistency
   class << self

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: database_consistency
 version: !ruby/object:Gem::Version
-  version: 1.7.27
+  version: 2.0.1
 platform: ruby
 authors:
 - Evgeniy Demin
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-12-24 00:00:00.000000000 Z
+date: 2024-12-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord