checkability 0.4.0 → 0.6.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 3bb2e7105c2121726c166ed2656e3c0194b7a45f
-  data.tar.gz: a06d6cdd6ff33b35b91bf99022363898e9fb0019
+SHA256:
+  metadata.gz: b81bedd5fd904185fc57a5627f6848dbb608ba3fa5155653c6d096aed6f4c95c
+  data.tar.gz: 29948f5309901c9769b899e4660c8d7a0677e29dbf2b1f36af663525dfc1d957
 SHA512:
-  metadata.gz: 29345eba850827066c0f0a8f2c40c751cbc95d9e808373b13bb822c538316604ef61d813caafe7d7b7b93f0e84edb2a961b25d564957875b263ea1dadf0bd994
-  data.tar.gz: c66eb4665963b1e94f40d510f13b3ea2d21eb0244e79dcd1e80ddc781fa8a135fbc6a469553404995bdda4c76d614d684128d03149af529cd6b395704b1e7ad4
+  metadata.gz: 693df60213127f59a954ef39977c86ad78ac32198b478f7f55e45a8b4df900c3b61cd1d58ac3540db9557979545908266bb2a9c3de60f5051227e079ae6d6444
+  data.tar.gz: 45e80fe8b9b1bf424e529c93b48d66329772bc776dfbccf8055539ca832f219ac3f67685a11621ff4dfb32c23f6d7cebe819f92530d864fda84f60aa74c04aee

data/Rakefile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'bundler/setup'
 
 require 'bundler/gem_tasks'

data/bin/checkability

@@ -1,4 +1,5 @@
 #!/usr/bin/env ruby
+# frozen_string_literal: true
 
 require_relative '../lib/checkability'
 

data/lib/checkability.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'active_record'
 require 'active_support'
 require_relative 'checkability/version'
@@ -7,5 +9,7 @@ require_relative 'checkability/external_api_checker'
 require_relative 'checkability/external_api_connector'
 require_relative 'checkability/validator'
 require_relative 'checkability/acts_as_checkable'
+require_relative 'checkability/chain_of_resp/handler'
+require_relative 'checkability/chain_of_resp/abstract_handler'
 
 ActiveRecord::Base.include Checkability::ActsAsCheckable

data/lib/checkability/acts_as_checkable.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Checkability
   # Adding class and instance methods
   #
@@ -10,9 +12,7 @@ module Checkability
 
     class_methods do
       def acts_as_checkable(options = {})
-        if !options.is_a?(Hash) && !options.empty?
-          raise ArgumentError, "Hash expected, got #{options.class.name}"
-        end
+        raise ArgumentError, "Hash expected, got #{options.class.name}" if !options.is_a?(Hash) && !options.empty?
 
         class_attribute :checkable_conf
 
@@ -24,12 +24,20 @@ module Checkability
 
     def perform_check
       _setup
-      self.allowed = _check
-      messages << "'#{value}' is #{_allowness}. "
+      self.allowed = Checkability::Checkable.new(self).check(checkable_conf)
+      messages << "#{allowed}::'#{_value}' is #{_allowness}. "
     end
 
     private
 
+    def _value
+      send(_attr_name)
+    end
+
+    def _attr_name
+      checkable_conf[:attr_name] || :value
+    end
+
     def _setup
       self.allowed = nil
       self.messages = []
@@ -38,9 +46,5 @@ module Checkability
     def _allowness
       allowed ? 'ALLOWED' : 'NOT allowed'
     end
-
-    def _check
-      Checkability::Checkable.new(self).check(checkable_conf)
-    end
   end
 end

data/lib/checkability/chain_of_resp/abstract_handler.rb

@@ -0,0 +1,42 @@
+# frozen_string_literal: true
+
+require_relative 'handler'
+
+module ChainOfResp
+  # @abstract
+  class AbstractHandler < ChainOfResp::Handler
+    # @return [Handler]
+    attr_reader :stop_process_on_failure, :stop_process_on_success
+    attr_accessor :handler, :result
+
+    def initialize(opts = {})
+      @stop_process_on_failure = opts[:stop_process_on_failure] || false
+      @stop_process_on_success = opts[:stop_process_on_success] || false
+      @next_handler = nil
+      post_initialize(opts) # implemented in subclass
+    end
+
+    # @param [Handler] handler
+    #
+    # @return [Handler]
+    def next_handler(handler)
+      @handler = handler if handler
+
+      handler
+    end
+
+    # @abstract
+    #
+    # @param [String] request
+    #
+    # @return [Boolean, nil]
+    def handle(request)
+      check = check_value(request) # imlemented in subclass
+      return true  if check && stop_process_on_success
+
+      return false if !check && stop_process_on_failure
+
+      handler&.handle(request)
+    end
+  end
+end

data/lib/checkability/chain_of_resp/handler.rb

@@ -0,0 +1,31 @@
+# frozen_string_literal: true
+
+module ChainOfResp
+  # @abstract
+  class Handler
+    # @abstract
+    #
+    # @param [Handler] handler
+    def next_handler=(_handler)
+      raise NotImplementedError, "#{self.class} has not implemented method '#{__method__}'"
+    end
+
+    # @abstract
+    #
+    # @param [String] request
+    #
+    # @return [String, nil]
+    def handle(_request)
+      raise NotImplementedError, "#{self.class} has not implemented method '#{__method__}'"
+    end
+
+    # @abstract
+    #
+    # @param [String] request
+    #
+    # @return [Boolean, true|false]
+    def check_value(_request)
+      raise NotImplementedError, "#{self.class} has not implemented method '#{__method__}'"
+    end
+  end
+end

data/lib/checkability/checkable.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Checkability
   # Implements check method to Iterate on chechers
   # Possible to implemet as Iterator in future
@@ -7,32 +9,42 @@ module Checkability
 
     def initialize(checkable)
       @checkable = checkable
+      @checkable.messages = []
     end
 
-    # sentence is a proc
-    #   like { |a,b,c| a && ( b || c ) }
-    #   where a,b,c are checkers
-    #   and each should return true|false
-    # checkers is an array of checker objects
-    #   e.g. [storage_checker, external_api_checker]
+    # As in result handlers should behave like this:
+    # validator    .set_next(storage)
+    # storage      .set_next(api_validator)
+    # api_validator.set_next(api_finder)
+    # api_validator.set_next(nil)
+    #
+    # validator.handle(request)
+    #
     def check(opts = {})
-      results = []
-      opts[:checkers].each do |checker|
-        results << (res = _checker_to_check_value(checker))
-        break if res && checker[:stop_process_if_success]
-        break if res == false && checker[:stop_process_if_failure]
-      end
-      opts[:strategy].call(results)
+      first_handler_name = opts[:first_handler]
+      _handlers(opts)[first_handler_name].handle(checkable)
+    rescue StandardError => e
+      checkable.messages << "false::#{e}: #{opts}."
+      false
     end
 
     private
 
-    def _checker_to_check_value(checker)
-      k = "Checkability::#{checker[:name].to_s.camelize}".constantize
-      k.new(checker).check_value(checkable)
-    rescue NameError => e
-      checkable.messages << "#{e}: #{checker[:name]}."
-      false
+    def _handlers(opts)
+      handler_confs = opts[:handler_confs]
+      handlers = {}
+      handler_confs.each do |handler_name, handler_conf|
+        handlers[handler_name] = _make_handler(handler_name, handler_conf)
+      end
+      handlers.each do |handler_name, handler|
+        next_handler_name = handler_confs[handler_name][:next_handler]
+        handler.next_handler(handlers[next_handler_name])
+      end
+    end
+
+    def _make_handler(_name, conf)
+      k = Checkability.const_get conf[:name].to_s.camelize
+      k.new(conf)
     end
   end
 end

data/lib/checkability/external_api_checker.rb

@@ -1,16 +1,21 @@
+# frozen_string_literal: true
+
 require 'faraday'
 require 'net/http'
 require 'net/https'
 require 'json'
 
+# frozen_string_literal: true
+
+require_relative 'chain_of_resp/abstract_handler'
 module Checkability
   # Checks if postcode exists in external API
   #
-  class ExternalApiChecker
+  class ExternalApiChecker < ChainOfResp::AbstractHandler
     attr_reader :path, :path_suffix, :check_method, :connection, :http_verb,
                 :failure_message, :success_message
 
-    def initialize(conf = {})
+    def post_initialize(conf = {})
       @path = conf[:path]
       @http_verb = conf[:http_verb] || :get
       @path_suffix = conf[:path_suffix] || ''
@@ -21,8 +26,9 @@ module Checkability
     end
 
     def check_value(checkable)
-      @resp = connection.connect.send(http_verb,
-                                      checkable.value.delete(' ') + path_suffix)
+      @resp = connection
+              .connect
+              .send(http_verb, "#{checkable.value.delete(' ')}#{path_suffix}")
       result, message = _result_and_message
       checkable.messages << message
       result
@@ -30,8 +36,8 @@ module Checkability
 
     private
 
-    def _message(str)
-      "#{path}: #{str}"
+    def _message(str, res)
+      "#{res}::#{path}: #{str}"
     end
 
     def _parsed(resp)
@@ -39,14 +45,14 @@ module Checkability
     end
 
     def _result_and_message
-      return [false, _message(@resp.status)] unless @resp.status == 200
+      return [false, _message(@resp.status, false)] unless @resp.status == 200
 
-      return [true, _message(success_message)] if check_method
-                                                  .call(_parsed(@resp))
+      return [true, _message(success_message, true)] if check_method
+                                                        .call(_parsed(@resp))
 
-      [false, _message(failure_message)]
+      [false, _message(failure_message, false)]
     rescue StandardError => e
-      [false, _message(e)]
+      [false, _message(e, false)]
     end
   end
 end

data/lib/checkability/external_api_connector.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Checkability
   # Create connection
   #

data/lib/checkability/railtie.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Checkability
   class Railtie < ::Rails::Railtie
   end

data/lib/checkability/storage_checker.rb

@@ -1,17 +1,21 @@
+# frozen_string_literal: true
+
+require_relative 'chain_of_resp/abstract_handler'
 module Checkability
   # Checks if postcode exists in Storage
   #
-  class StorageChecker
+  class StorageChecker < ChainOfResp::AbstractHandler
     attr_reader :storage_class
 
-    def initialize(conf = {})
+    def post_initialize(conf = {})
       @storage_class = conf[:storage_class]
     end
 
     def check_value(checkable)
       value = checkable.value.upcase
       result = _present_in_storage(value)
-      checkable.messages << (result ? _message('Found') : _message('Not found'))
+      checkable.messages << (
+        result ? _message('Found', result) : _message('Not found', result))
       result
     end
 
@@ -22,8 +26,8 @@ module Checkability
                    .present?
     end
 
-    def _message(str)
-      "Allowed #{storage_class}s list: #{str}."
+    def _message(str, res)
+      "#{res}::Allowed #{storage_class}s list: #{str}."
     end
   end
 end

data/lib/checkability/validator.rb

@@ -1,10 +1,12 @@
+# frozen_string_literal: true
+
 module Checkability
   # Checks if postcode comply with regex
   #
-  class Validator
+  class Validator < ChainOfResp::AbstractHandler
     attr_reader :format
 
-    def initialize(conf = {})
+    def post_initialize(conf = {})
       @format = conf[:format]
     end
 
@@ -18,9 +20,9 @@ module Checkability
 
     def _result_and_message(checkable)
       if (checkable.value.delete(' ') =~ format[:regex]).nil?
-        [false, "Value is NOT COMPLY with format of #{format[:name]}."]
+        [false, "false::Value is NOT COMPLY with format of #{format[:name]}."]
       else
-        [true, "Value is COMPLY with format of #{format[:name]}."]
+        [true, "true::Value is COMPLY with format of #{format[:name]}."]
       end
     end
   end

data/lib/checkability/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Checkability
-  VERSION = '0.4.0'.freeze
+  VERSION = '0.6.2'
 end

data/lib/tasks/checkability_tasks.rake

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 # desc "Explaining what the task does"
 # task :checkability do
 #   # Task goes here

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: checkability
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 0.6.2
 platform: ruby
 authors:
 - Andrey Eremeev
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-04-01 00:00:00.000000000 Z
+date: 2021-04-03 00:00:00.000000000 Z
 dependencies: []
 description: Provide Checkers functionality.
 email:
@@ -24,6 +24,8 @@ files:
 - bin/checkability
 - lib/checkability.rb
 - lib/checkability/acts_as_checkable.rb
+- lib/checkability/chain_of_resp/abstract_handler.rb
+- lib/checkability/chain_of_resp/handler.rb
 - lib/checkability/checkable.rb
 - lib/checkability/external_api_checker.rb
 - lib/checkability/external_api_connector.rb
@@ -40,7 +42,7 @@ metadata:
   homepage_uri: https://github.com/azazelo/checkability
   source_code_uri: https://rubygems.org
   changelog_uri: https://rubygems.org
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -48,16 +50,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '0'
+      version: 2.6.2
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.4.6
-signing_key: 
+rubygems_version: 3.0.3
+signing_key:
 specification_version: 4
 summary: Provide Checkers functionality.
 test_files: []