sequent 4.0.0 → 4.1.0

data/lib/sequent/core/helpers/boolean_validator.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'active_model'
 require_relative 'value_validators'
 
@@ -18,7 +20,10 @@ module Sequent
       # They will be converted to `true`, `false` or `nil`
       class BooleanValidator < ActiveModel::EachValidator
         def validate_each(subject, attribute, value)
-          subject.errors.add attribute, :invalid_boolean unless Sequent::Core::Helpers::ValueValidators.for(Boolean).valid_value?(value)
+          unless Sequent::Core::Helpers::ValueValidators.for(Boolean).valid_value?(value)
+            subject.errors.add attribute,
+                               :invalid_boolean
+          end
         end
       end
     end

data/lib/sequent/core/helpers/copyable.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Sequent
   module Core
     module Helpers
@@ -17,9 +19,7 @@ module Sequent
           end
           the_copy
         end
-
       end
     end
   end
 end
-

data/lib/sequent/core/helpers/date_time_validator.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'active_model'
 
 module Sequent
@@ -10,8 +12,9 @@ module Sequent
       class DateTimeValidator < ActiveModel::EachValidator
         def validate_each(subject, attribute, value)
           return if value.is_a?(DateTime)
+
           DateTime.deserialize_from_json(value)
-        rescue
+        rescue StandardError
           subject.errors.add attribute, :invalid_date_time
         end
       end

data/lib/sequent/core/helpers/date_validator.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'active_model'
 require_relative 'value_validators'
 
@@ -10,7 +12,10 @@ module Sequent
       #   attrs value: Date
       class DateValidator < ActiveModel::EachValidator
         def validate_each(subject, attribute, value)
-          subject.errors.add attribute, :invalid_date unless Sequent::Core::Helpers::ValueValidators.for(Date).valid_value?(value)
+          unless Sequent::Core::Helpers::ValueValidators.for(Date).valid_value?(value)
+            subject.errors.add attribute,
+                               :invalid_date
+          end
         end
       end
     end

data/lib/sequent/core/helpers/default_validators.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative 'string_validator'
 require_relative 'boolean_validator'
 require_relative 'date_time_validator'
@@ -13,18 +15,18 @@ module Sequent
             klass.validates_numericality_of field, only_integer: true, allow_nil: true, allow_blank: true
           end,
           Date => ->(klass, field) do
-            klass.validates field, "sequent::Core::Helpers::Date" => true
+            klass.validates field, 'sequent::Core::Helpers::Date' => true
           end,
           DateTime => ->(klass, field) do
-            klass.validates field, "sequent::Core::Helpers::DateTime" => true
+            klass.validates field, 'sequent::Core::Helpers::DateTime' => true
           end,
-          Boolean => -> (klass, field) do
-            klass.validates field, "sequent::Core::Helpers::Boolean" => true
+          Boolean => ->(klass, field) do
+            klass.validates field, 'sequent::Core::Helpers::Boolean' => true
           end,
-          String => -> (klass, field) do
-            klass.validates field, "sequent::Core::Helpers::String" => true
+          String => ->(klass, field) do
+            klass.validates field, 'sequent::Core::Helpers::String' => true
           end,
-          Sequent::Core::Helpers::Secret => -> (klass, field) do
+          Sequent::Core::Helpers::Secret => ->(klass, field) do
             klass.after_validation do |object|
               if object.errors&.any?
                 object.send("#{field}=", nil)
@@ -33,8 +35,8 @@ module Sequent
                 object.send("#{field}=", Sequent::Secret.new(raw_value)) if raw_value
               end
             end
-          end
-        }
+          end,
+        }.freeze
 
         def self.for(type)
           new(type)
@@ -46,7 +48,7 @@ module Sequent
 
         def add_validations_for(klass, field)
           validator = VALIDATORS[@type]
-          validator.call(klass, field) if validator
+          validator&.call(klass, field)
         end
       end
     end

data/lib/sequent/core/helpers/equal_support.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Sequent
   module Core
     module Helpers
@@ -8,16 +10,17 @@ module Sequent
       #
       module EqualSupport
         def ==(other)
-          return false if other == nil
+          return false if other.nil?
           return false if self.class != other.class
+
           self.class.types.each do |name, _|
-            self_value = self.send(name)
+            self_value = send(name)
             other_value = other.send(name)
             if self_value.class == DateTime && other_value.class == DateTime
               # we don't care about milliseconds. If you know a better way of checking for equality please improve.
-              return false unless (self_value.iso8601 == other_value.iso8601)
+              return false unless self_value.iso8601 == other_value.iso8601
             else
-              return false unless (self_value == other_value)
+              return false unless self_value == other_value
             end
           end
           true
@@ -26,7 +29,7 @@ module Sequent
         def hash
           hash = 17
           self.class.types.each do |name, _|
-            hash = hash * 31 + self.send(name).hash
+            hash = hash * 31 + send(name).hash
           end
           hash
         end
@@ -35,7 +38,6 @@ module Sequent
           self == other
         end
       end
-
     end
   end
 end

data/lib/sequent/core/helpers/helpers.rb

@@ -1 +1,3 @@
+# frozen_string_literal: true
+
 Dir["#{File.dirname(__FILE__)}/*rb"].each { |f| require_relative f }

data/lib/sequent/core/helpers/mergable.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Sequent
   module Core
     module Helpers
@@ -6,17 +8,16 @@ module Sequent
       #   ben = Person.new(name: 'Ben').merge!(name: 'Ben Vonk')
       #
       module Mergable
-
         def merge!(attrs = {})
-          warn "[DEPRECATION] `merge!` is deprecated. Please use `copy` instead. This method will no longer be included in the next version of Sequent. You can still use it but you will have to include the module `Sequent::Core::Helpers::Mergable` yourself."
+          warn <<~EOS
+            [DEPRECATION] `merge!` is deprecated. Please use `copy` instead. This method will no longer be included in the next version of Sequent. You can still use it but you will have to include the module `Sequent::Core::Helpers::Mergable` yourself.
+          EOS
           attrs.each do |name, value|
-            self.send("#{name}=", value)
+            send("#{name}=", value)
           end
           self
         end
-
       end
     end
   end
 end
-

data/lib/sequent/core/helpers/message_handler.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Sequent
   module Core
     module Helpers
@@ -56,7 +58,7 @@ module Sequent
 
         def handle_message(message)
           handlers = self.class.message_mapping[message.class]
-          handlers.each { |handler| self.instance_exec(message, &handler) } if handlers
+          handlers&.each { |handler| instance_exec(message, &handler) }
         end
       end
     end

data/lib/sequent/core/helpers/param_support.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'active_support'
 
 module Sequent
@@ -21,7 +23,6 @@ module Sequent
           def from_form_data(params = {})
             from_params(params, false)
           end
-
         end
 
         # extend host class with class methods when we're included
@@ -35,7 +36,8 @@ module Sequent
             value = params[attribute]
 
             next if strict_nil_check && value.nil?
-            next if (!strict_nil_check && value.blank?)
+            next if !strict_nil_check && value.blank?
+
             if type.respond_to? :from_params
               value = type.from_params(value)
             elsif value.is_a?(Array)
@@ -58,10 +60,12 @@ module Sequent
         def as_params
           hash = HashWithIndifferentAccess.new
           self.class.types.each do |field|
-            value = self.instance_variable_get("@#{field[0]}")
-            next if field[0] == "errors"
-            hash[field[0]] = if value.kind_of?(Array)
+            value = instance_variable_get("@#{field[0]}")
+            next if field[0] == 'errors'
+
+            hash[field[0]] = if value.is_a?(Array)
                                next if value.blank?
+
                                value.map { |v| value_to_string(v) }
                              else
                                value_to_string(value)
@@ -86,16 +90,16 @@ module Sequent
 
         def make_params(key, enumerable, memo = {})
           case enumerable
-            when Array
-              enumerable.each_with_index do |object, index|
-                make_params("#{key}[#{index}]", object, memo)
-              end
-            when Hash
-              enumerable.each do |hash_key, object|
-                make_params("#{key}[#{hash_key}]", object, memo)
-              end
-            else
-              memo[key] = enumerable
+          when Array
+            enumerable.each_with_index do |object, index|
+              make_params("#{key}[#{index}]", object, memo)
+            end
+          when Hash
+            enumerable.each do |hash_key, object|
+              make_params("#{key}[#{hash_key}]", object, memo)
+            end
+          else
+            memo[key] = enumerable
           end
           memo
         end

data/lib/sequent/core/helpers/secret.rb

@@ -1,9 +1,10 @@
+# frozen_string_literal: true
+
 require 'bcrypt'
 
 module Sequent
   module Core
     module Helpers
-
       #
       # You can use this in Commands to handle for instance passwords
       # safely. It uses BCrypt to encrypt the Secret.
@@ -55,7 +56,6 @@ module Sequent
       # See +re_encrypt_secret+
       # See +verify_secret+
       class Secret
-
         class << self
           def deserialize_from_json(value)
             new(value)
@@ -65,7 +65,8 @@ module Sequent
           # Creates a hash for the given clear text password.
           #
           def encrypt_secret(clear_text_secret)
-            fail ArgumentError.new('clear_text_secret can not be blank') if clear_text_secret.blank?
+            fail ArgumentError, 'clear_text_secret can not be blank' if clear_text_secret.blank?
+
             BCrypt::Password.create(clear_text_secret)
           end
 
@@ -74,8 +75,8 @@ module Sequent
           # (essentially re-creating the secret hash).
           #
           def re_encrypt_secret(clear_text_secret, hashed_secret)
-            fail ArgumentError.new('clear_text_secret can not be blank') if clear_text_secret.blank?
-            fail ArgumentError.new('hashed_secret can not be blank') if hashed_secret.blank?
+            fail ArgumentError, 'clear_text_secret can not be blank' if clear_text_secret.blank?
+            fail ArgumentError, 'hashed_secret can not be blank' if hashed_secret.blank?
 
             BCrypt::Engine.hash_secret(clear_text_secret, hashed_secret)
           end
@@ -84,7 +85,7 @@ module Sequent
           # Verifies that the hashed and clear text secret are equal.
           #
           def verify_secret(hashed_secret, clear_text_secret)
-            return false if (hashed_secret.blank? || clear_text_secret.blank?)
+            return false if hashed_secret.blank? || clear_text_secret.blank?
 
             BCrypt::Password.new(hashed_secret) == clear_text_secret
           end
@@ -93,12 +94,13 @@ module Sequent
         attr_reader :value
 
         def initialize(value)
-          fail ArgumentError.new('value can not be blank') if value.blank?
-          if value.is_a?(Secret)
-            @value = value.value
-          else
-            @value = value
-          end
+          fail ArgumentError, 'value can not be blank' if value.blank?
+
+          @value = if value.is_a?(Secret)
+                     value.value
+                   else
+                     value
+                   end
         end
 
         def encrypt

data/lib/sequent/core/helpers/string_support.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Sequent
   module Core
     module Helpers
@@ -9,14 +11,13 @@ module Sequent
       module StringSupport
         def to_s
           s = "#{self.class.name}: "
-          self.instance_variables.each do |name|
-            value = self.instance_variable_get("#{name}")
+          instance_variables.each do |name|
+            value = instance_variable_get(name.to_s)
             s += "#{name}=[#{value}], "
           end
-          "{" + s.chomp(", ") + "}"
+          '{' + s.chomp(', ') + '}'
         end
       end
-
     end
   end
 end

data/lib/sequent/core/helpers/string_to_value_parsers.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative '../ext/ext'
 require_relative 'array_with_type'
 
@@ -16,10 +18,11 @@ module Sequent
           ::DateTime => ->(value) { parse_to_date_time(value) },
           ::Sequent::Core::Helpers::ArrayWithType => ->(values, type_in_array) { parse_array(values, type_in_array) },
           ::Sequent::Core::Helpers::Secret => ->(value) { Sequent::Core::Helpers::Secret.new(value).encrypt },
-        }
+        }.freeze
 
         def self.parse_to_integer(value)
           return value if value.is_a?(Integer)
+
           Integer(value, 10) unless value.blank?
         end
 
@@ -35,12 +38,13 @@ module Sequent
           if value.blank? && !(value.is_a?(TrueClass) || value.is_a?(FalseClass))
             nil
           else
-            (value.is_a?(TrueClass) || value == "true")
+            (value.is_a?(TrueClass) || value == 'true')
           end
         end
 
         def self.parse_to_date(value)
           return if value.blank?
+
           value.is_a?(Date) ? value : Date.iso8601(value.dup)
         end
 
@@ -50,6 +54,7 @@ module Sequent
 
         def self.parse_array(values, type_in_array)
           fail "invalid value for array(): \"#{values}\"" unless values.is_a?(Array)
+
           values.map do |item|
             if item.respond_to?(:parse_attrs_to_correct_types)
               item.parse_attrs_to_correct_types

data/lib/sequent/core/helpers/string_validator.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'active_model'
 require_relative 'value_validators'
 
@@ -16,7 +18,10 @@ module Sequent
       #
       class StringValidator < ActiveModel::EachValidator
         def validate_each(subject, attribute, value)
-          subject.errors.add attribute, :invalid_string unless Sequent::Core::Helpers::ValueValidators.for(String).valid_value?(value)
+          unless Sequent::Core::Helpers::ValueValidators.for(String).valid_value?(value)
+            subject.errors.add attribute,
+                               :invalid_string
+          end
         end
       end
     end

data/lib/sequent/core/helpers/type_conversion_support.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'active_model'
 
 module Sequent
@@ -5,7 +7,6 @@ module Sequent
     TypeConversionError = Class.new(RuntimeError)
 
     module Helpers
-
       # Will parse all values to the correct types.
       # The raw values are typically posted from the web and are therefor mostly strings.
       # To parse a raw value your class must have a parse_from_string method that returns the parsed values.
@@ -14,8 +15,9 @@ module Sequent
         def parse_attrs_to_correct_types
           the_copy = dup
           the_copy.class.types.each do |name, type|
-            raw_value = the_copy.send("#{name}")
+            raw_value = the_copy.send(name.to_s)
             next if raw_value.nil?
+
             if raw_value.respond_to?(:parse_attrs_to_correct_types)
               the_copy.send("#{name}=", raw_value.parse_attrs_to_correct_types)
             else
@@ -24,7 +26,7 @@ module Sequent
             end
           end
           the_copy
-        rescue => e
+        rescue StandardError => e
           raise TypeConversionError, e.message
         end
       end

data/lib/sequent/core/helpers/uuid_helper.rb

@@ -1,12 +1,15 @@
+# frozen_string_literal: true
+
 require 'securerandom'
 
 module Sequent
   module Core
     module Helpers
       module UuidHelper
-
         def new_uuid
-          warn "DEPRECATION WARNING: Sequent::Core::Helpers::UuidHelper.new_uuid is deprecated. Use Sequent.new_uuid instead"
+          warn <<~EOS
+            DEPRECATION WARNING: Sequent::Core::Helpers::UuidHelper.new_uuid is deprecated. Use Sequent.new_uuid instead
+          EOS
           Sequent.new_uuid
         end
 

data/lib/sequent/core/helpers/value_validators.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require_relative '../ext/ext'
 
 module Sequent
@@ -6,7 +8,7 @@ module Sequent
       class ValueValidators
         INVALID_STRING_CHARS = [
           "\u0000",
-        ]
+        ].freeze
 
         VALIDATORS = {
           ::Symbol => ->(_) { true },
@@ -14,36 +16,48 @@ module Sequent
           ::Integer => ->(value) { valid_integer?(value) },
           ::Boolean => ->(value) { valid_bool?(value) },
           ::Date => ->(value) { valid_date?(value) },
-          ::DateTime => ->(value) { valid_date_time?(value) }
-        }
+          ::DateTime => ->(value) { valid_date_time?(value) },
+        }.freeze
 
         def self.valid_integer?(value)
           value.blank? || Integer(value)
-        rescue
+        rescue StandardError
           false
         end
 
         def self.valid_bool?(value)
           return true if value.blank?
-          value.is_a?(TrueClass) || value.is_a?(FalseClass) || value == "true" || value == "false"
+
+          value.is_a?(TrueClass) || value.is_a?(FalseClass) || value == 'true' || value == 'false'
         end
 
         def self.valid_date?(value)
           return true if value.blank?
           return true if value.is_a?(Date)
           return false unless value =~ /\d{4}-\d{2}-\d{2}/
-          !!Date.iso8601(value) rescue false
+
+          begin
+            !!Date.iso8601(value)
+          rescue StandardError
+            false
+          end
         end
 
         def self.valid_date_time?(value)
           return true if value.blank?
-          value.is_a?(DateTime) || !!DateTime.iso8601(value.dup) rescue false
+
+          begin
+            value.is_a?(DateTime) || !!DateTime.iso8601(value.dup)
+          rescue StandardError
+            false
+          end
         end
 
         def self.valid_string?(value)
           return true if value.nil?
-          value.to_s && !INVALID_STRING_CHARS.any? { |invalid_char| value.to_s.include?(invalid_char) }
-        rescue
+
+          value.to_s && INVALID_STRING_CHARS.none? { |invalid_char| value.to_s.include?(invalid_char) }
+        rescue StandardError
           false
         end
 

data/lib/sequent/core/persistors/active_record_persistor.rb

@@ -1,10 +1,11 @@
+# frozen_string_literal: true
+
 require 'active_record'
 require_relative './persistor'
 
 module Sequent
   module Core
     module Persistors
-
       #
       # The ActiveRecordPersistor uses ActiveRecord to update the projection
       #
@@ -17,14 +18,21 @@ module Sequent
       class ActiveRecordPersistor
         include Persistor
 
-        def update_record(record_class, event, where_clause = {aggregate_id: event.aggregate_id}, options = {}, &block)
+        def update_record(record_class, event, where_clause = {aggregate_id: event.aggregate_id}, options = {})
           record = record_class.unscoped.where(where_clause).first
-          raise("Record of class #{record_class} with where clause #{where_clause} not found while handling event #{event}") unless record
+          unless record
+            fail(<<~EOS)
+              Record of class #{record_class} with where clause #{where_clause} not found while handling event #{event}
+            EOS
+          end
+
           record.updated_at = event.created_at if record.respond_to?(:updated_at)
           yield record if block_given?
-          update_sequence_number = options.key?(:update_sequence_number) ?
-                                     options[:update_sequence_number] :
+          update_sequence_number = if options.key?(:update_sequence_number)
+                                     options[:update_sequence_number]
+                                   else
                                      record.respond_to?(:sequence_number=)
+                                   end
           record.sequence_number = event.sequence_number if update_sequence_number
           record.save!
         end
@@ -42,11 +50,13 @@ module Sequent
           query = array_of_value_hashes.map do |values|
             insert_manager = new_insert_manager
             insert_manager.into(table)
-            insert_manager.insert(values.map do |key, value|
-              convert_to_values(key, table, value)
-            end)
+            insert_manager.insert(
+              values.map do |key, value|
+                convert_to_values(key, table, value)
+              end,
+            )
             insert_manager.to_sql
-          end.join(";")
+          end.join(';')
 
           execute_sql(query)
         end