rservicebus2 0.2.8 → 0.2.9

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 5f94672750deba44e6495d728b6c45bbf0aba95efe35d18b6877359e27173da4
-  data.tar.gz: 27e94561da7ab338ee3ff5ac99f850f872900f2559bb56ce3dffd604c452173a
+  metadata.gz: 7cf46f6455ed14f25642855bd4839b0c910447bb07007bdb2dba0c9a65b7f4d0
+  data.tar.gz: 5ef71ec901e2d142d731d2dd8be176b297116ef8b6b07ab182c295496f1dfd85
 SHA512:
-  metadata.gz: ac819c545740eb82ecf1ae2b95286f837ae1189ba3eded36ddcc98680ce4b7dcc84b975d9c75ab48e62f99cef32a83d98fdf2ad195bab0012f2804392abb3289
-  data.tar.gz: c10528a4a06fc27f9e2c5541dd4d500a1ae2120cc0f89a4a66c165a09c6fb5eae014c04ab74735c29069c6b6f8ebfaf089c7f2f63d80cdc02fb7d4acf3f7cc7b
+  metadata.gz: 35999f1a63cf911f2dc2d2d9580dc3afd26f70203db43bf7c58ee079606bb318246e292ffd382ceadfe4747d37a826a0784b3407d611c1957d61b7ea266aef05
+  data.tar.gz: c19a66ce0e83b4c2ae617ed0e8b9827f91cf31c9820e7f0e8ba5f405f83792b097e5ed0bd9769faf331bd15c4bdfeaf9748cea40677c29e050f4f3d1d6e58815

data/lib/rservicebus2/agent.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module RServiceBus2
   class QueueNotFoundForMsg < StandardError
   end
@@ -22,9 +24,9 @@ module RServiceBus2
     # @param [Object] messageObj The msg to be sent
     # @param [String] queueName the name of the queue to be send the msg to
     # @param [String] returnAddress the name of a queue to send replies to
-    # rubocop:disable Metrics/AbcSize,Metrics/MethodLength
+    # rubocop:disable Metrics/MethodLength
     def send_msg(message_obj, queue_name, return_address = nil)
-      fail QueueNotFoundForMsg, message_obj.class.name if queue_name.nil?
+      raise QueueNotFoundForMsg, message_obj.class.name if queue_name.nil?
 
       msg = RServiceBus2::Message.new(message_obj, return_address)
       if queue_name.index('@').nil?
@@ -40,6 +42,7 @@ module RServiceBus2
 
       @mq.send(q, serialized_object)
     end
+    # rubocop:enable Metrics/MethodLength
 
     # Gives an agent the means to receive a reply
     #

data/lib/rservicebus2/appresource.rb

@@ -18,7 +18,7 @@ module RServiceBus2
       RServiceBus2.rlog "#{self.class.name}. Connected to, #{@uri}"
     end
 
-    def get_resource
+    def resource
       @connection
     end
 

data/lib/rservicebus2/appresource_configure.rb

@@ -1,12 +1,14 @@
+# frozen_string_literal: true
+
 require 'uri'
 
 module RServiceBus2
   # Configure AppResources for an rservicebus host
   class ConfigureAppResource
-    # rubocop:disable Metrics/AbcSize,Metrics/MethodLength,CyclomaticComplexity
+    # rubocop:disable Metrics/AbcSize,Metrics/MethodLength,Metrics/CyclomaticComplexity,Metrics/PerceivedComplexity
     def get_resources(env, host, state_manager, saga_storage)
-      # rm = resource_manager
       rm = ResourceManager.new(state_manager, saga_storage)
+      # rubocop:disable Metrics/BlockLength
       env.each do |k, v|
         if v.is_a?(String) && k.start_with?('RSBFDB2_')
           uri = URI.parse(v)
@@ -15,8 +17,8 @@ module RServiceBus2
           k = k.sub('RSBFDB2_', '')
           rm.add k, AppResourceFluidDb2.new(host, uri)
         elsif v.is_a?(String) &&
-              (k.start_with?('RSBFDB_') || v.index('fluiddb') == 0)
-          v = v['fluiddb'.length..-1] if v.index('fluiddb') == 0
+              (k.start_with?('RSBFDB_') || v.start_with?('fluiddb'))
+          v = v['fluiddb'.length..-1] if v.start_with?('fluiddb')
           uri = URI.parse(v)
           require 'rservicebus2/appresource/fluiddb'
 
@@ -46,8 +48,10 @@ module RServiceBus2
           end
         end
       end
+      # rubocop:enable Metrics/BlockLength
 
       rm
     end
+    # rubocop:enable Metrics/AbcSize,Metrics/MethodLength,Metrics/CyclomaticComplexity,Metrics/PerceivedComplexity
   end
 end

data/lib/rservicebus2/audit.rb

@@ -1,8 +1,10 @@
+# frozen_string_literal: true
+
 module RServiceBus2
   # Audit Class
   class Audit
-    def initialize(mq)
-      @mq = mq
+    def initialize(message_queue)
+      @mq = message_queue
       audit_queue_name = RServiceBus2.get_value('AUDIT_QUEUE_NAME')
       if audit_queue_name.nil?
         @sent_messages_to = RServiceBus2.get_value('sent_messages_to')

data/lib/rservicebus2/config.rb

@@ -154,11 +154,14 @@ module RServiceBus2
 
   # Class
   class ConfigFromEnv < Config
+    def initialize; end;
   end
 
   # Class
   class ConfigFromSetter < Config
     attr_writer :app_name, :message_endpoint_mappings, :handler_path_list, :error_queue_name, \
                 :max_retries, :forward_received_messages_to, :beanstalk_host
+
+    def initialize; end;
   end
 end

data/lib/rservicebus2/helper_functions.rb

@@ -1,3 +1,6 @@
+# frozen_string_literal: true
+
+# Helper functions
 module RServiceBus2
   def self.convert_dto_to_hash(obj)
     hash = {}
@@ -35,7 +38,7 @@ module RServiceBus2
   end
 
   def self.get_value(name, default = nil)
-    value = (ENV[name].nil? || ENV[name] == '') ? default : ENV[name]
+    value = ENV[name].nil? || ENV[name] == '' ? default : ENV[name]
     log "Env value: #{name}: #{value}"
     value
   end
@@ -50,7 +53,6 @@ module RServiceBus2
     agent = RServiceBus2::Agent.new
     Audit.new(agent).audit_to_queue(msg)
     agent.send_msg(msg, queue_name, response_queue)
-
   rescue QueueNotFoundForMsg => e
     msg = "\n"
     msg = "#{msg}*** Queue not found for, #{e.message}\n"
@@ -76,10 +78,12 @@ module RServiceBus2
 
   def self.check_environment_variable(string)
     return false if ENV[string].nil?
+    return false if ENV[string] == ''
     return true if ENV[string] == true || ENV[string] =~ (/(true|t|yes|y|1)$/i)
     return false if ENV[string] == false ||
                     ENV[string].nil? ||
                     ENV[string] =~ (/(false|f|no|n|0)$/i)
-    fail ArgumentError, "invalid value for Environment Variable: \"#{string}\""
+
+    raise ArgumentError, "invalid value for Environment Variable: \"#{string}\""
   end
 end

data/lib/rservicebus2/host.rb

@@ -1,10 +1,15 @@
+# frozen_string_literal: true
+
 module RServiceBus2
   class NoHandlerFound < StandardError
   end
+
   class ClassNotFoundForMsg < StandardError
   end
+
   class NoMsgToProcess < StandardError
   end
+
   class PropertyNotSet < StandardError
   end
 
@@ -14,9 +19,8 @@ module RServiceBus2
 
     # Provides a thin logging veneer
     # @param [String] string Log entry
-    # @param [Boolean] ver Indicator for a verbose log entry
-    def log(string, ver = false)
-      RServiceBus2.log(string, ver)
+    def log(string)
+      RServiceBus2.log(string)
     end
 
     # Thin veneer for Configuring external resources
@@ -26,19 +30,6 @@ module RServiceBus2
                                                              self,
                                                              @state_manager,
                                                              @saga_storage)
-      self
-    end
-
-    # Thin veneer for Configuring SendAt
-    def configure_send_at_manager
-      @send_at_manager = SendAtManager.new(self)
-      self
-    end
-
-    # Thin veneer for Configuring state
-    def configure_state_manager
-      @state_manager = StateManager.new
-      self
     end
 
     # Thin veneer for Configuring state
@@ -48,25 +39,6 @@ module RServiceBus2
 
       uri = URI.parse(string)
       @saga_storage = SagaStorage.get(uri)
-      self
-    end
-
-    # Thin veneer for Configuring Cron
-    def configure_circuit_breaker
-      @circuit_breaker = CircuitBreaker.new(self)
-      self
-    end
-
-    # Thin veneer for Configuring external resources
-    def configure_monitors
-      @monitors = ConfigureMonitor.new(self, @resource_manager).get_monitors(ENV)
-      self
-    end
-
-    # Thin veneer for Configuring the Message Queue
-    def connect_to_mq
-      @mq = MQ.get
-      self
     end
 
     # Subscriptions are specified by adding events to the
@@ -77,8 +49,6 @@ module RServiceBus2
       @endpoint_mapping.subscription_endpoints.each do |event_name|
         subscribe(event_name)
       end
-
-      self
     end
 
     # Load and configure Message Handlers
@@ -90,8 +60,6 @@ module RServiceBus2
       @config.handler_path_list.each do |path|
         @handler_loader.load_handlers_from_path(path)
       end
-
-      self
     end
 
     # Load and configure Sagas
@@ -103,14 +71,6 @@ module RServiceBus2
       @config.saga_path_list.each do |path|
         @saga_loader.load_sagas_from_path(path)
       end
-
-      self
-    end
-
-    # Thin veneer for Configuring Cron
-    def configure_cron_manager
-      @cron_manager = CronManager.new(self, @handler_manager.msg_names)
-      self
     end
 
     # Load Contracts
@@ -121,62 +81,50 @@ module RServiceBus2
         require path
         RServiceBus2.rlog "Loaded Contract: #{path}"
       end
-
-      self
     end
 
     # For each directory given, find and load all librarys
     def load_libs
       log 'Load Libs'
       @config.lib_list.each do |path|
-        $:.unshift path
+        $LOAD_PATH.unshift path
       end
-
-      self
     end
 
     # Load, configure and initialise Subscriptions
     def configure_subscriptions
       subscription_storage = ConfigureSubscriptionStorage.new.get(@config.app_name, @config.subscription_uri)
       @subscription_manager = SubscriptionManager.new(subscription_storage)
-      self
-    end
-
-    # Initialise statistics monitor
-    def configure_statistics
-      @stats = StatisticManager.new( self )
-      self
     end
 
     def initialize
       RServiceBus2.rlog "Current directory: #{Dir.pwd}"
-      @config = ConfigFromEnv.new.load_host_section
-                                  .load_contracts
-                                  .load_handler_path_list
-                                  .load_saga_path_list
-                                  .load_libs
-                                  .load_working_dir_list
+      @config = ConfigFromEnv.new
+                             .load_host_section
+                             .load_contracts
+                             .load_handler_path_list
+                             .load_saga_path_list
+                             .load_libs
+                             .load_working_dir_list
 
-      connect_to_mq
+      @mq = MQ.get
 
       @endpoint_mapping = EndpointMapping.new.configure(@mq.local_queue_name)
 
-      self.configure_statistics
-          .load_contracts
-          .load_libs
-          .configure_send_at_manager
-          .configure_state_manager
-          .configure_saga_storage
-          .configure_app_resource
-          .configure_circuit_breaker
-          .configure_monitors
-          .load_handlers
-          .load_sagas
-          .configure_cron_manager
-          .configure_subscriptions
-          .send_subscriptions
-
-      self
+      @stats = StatisticManager.new(self)
+      load_contracts
+      load_libs
+      @send_at_manager = SendAtManager.new(self)
+      @state_manager = StateManager.new
+      configure_saga_storage
+      configure_app_resource
+      @circuit_breaker = CircuitBreaker.new(self)
+      @monitors = ConfigureMonitor.new(self, @resource_manager).get_monitors(ENV)
+      load_handlers
+      load_sagas
+      @cron_manager = CronManager.new(self, @handler_manager.msg_names)
+      configure_subscriptions
+      send_subscriptions
     end
 
     # Ignition
@@ -185,10 +133,10 @@ module RServiceBus2
       log "Watching, #{@mq.local_queue_name}"
       $0 = "rservicebus - #{@mq.local_queue_name}"
       unless @config.forward_received_messages_to.nil?
-        log 'Forwarding all received messages to: ' + @config.forward_received_messages_to.to_s
+        log "Forwarding all received messages to: #{@config.forward_received_messages_to}"
       end
       unless @config.forward_sent_messages_to.nil?
-        log 'Forwarding all sent messages to: ' + @config.forward_sent_messages_to.to_s
+        log "Forwarding all sent messages to: #{@config.forward_sent_messages_to}"
       end
 
       start_listening_to_endpoints
@@ -258,7 +206,6 @@ module RServiceBus2
             _send_already_wrapped_and_serialised(serialized_object,
                                                  @config.error_queue_name)
             @mq.ack
-
           rescue PropertyNotSet => e
             # This has been re-rasied from a rescue in the handler
             puts "*** #{e.message}"
@@ -266,7 +213,6 @@ module RServiceBus2
             property_name = e.message[10, e.message.index(',', 10) - 10]
             puts "*** Ensure the environment variable, RSB_#{property_name},
               has been set at startup."
-
           rescue StandardError => e
             sleep 0.5
 
@@ -275,7 +221,7 @@ module RServiceBus2
             puts e.backtrace
             puts '***'
 
-            if retries > 0
+            if retries.positive?
               retries -= 1
               @mq.return_to_queue
             else
@@ -293,7 +239,7 @@ module RServiceBus2
                 abort
               end
 
-              error_string = e.message + '. ' + e.backtrace.join('. ')
+              error_string = "#{e.message}. #{e.backtrace.join('. ')}"
               @msg.add_error_msg(@mq.local_queue_name, error_string)
               serialized_object = YAML.dump(@msg)
               _send_already_wrapped_and_serialised(serialized_object, @config.error_queue_name)
@@ -318,7 +264,6 @@ module RServiceBus2
 
           @send_at_manager.process
           @circuit_breaker.success
-
         rescue StandardError => e
           if e.message == 'SIGTERM' || e.message == 'SIGINT'
             puts 'Exiting on request ...'
@@ -326,7 +271,7 @@ module RServiceBus2
           else
             puts '*** This is really unexpected.'
             message_loop = false
-            puts 'Message: ' + e.message
+            puts "Message: #{e.message}"
             puts e.backtrace
           end
         end
@@ -338,33 +283,29 @@ module RServiceBus2
       @resource_manager.begin
       msg_name = @msg.msg.class.name
       handler_list = @handler_manager.get_handler_list_for_msg(msg_name)
-      RServiceBus2.rlog 'Handler found for: ' + msg_name
+      RServiceBus2.rlog "Handler found for: #{msg_name}"
       begin
         @queue_for_msgs_to_be_sent_on_complete = []
 
         log "Started processing msg, #{msg_name}"
         handler_list.each do |handler|
-          begin
-            log "Handler, #{handler.class.name}, Started"
-            handler.handle(@msg.msg)
-            log "Handler, #{handler.class.name}, Finished"
-          rescue PropertyNotSet => e
-            raise PropertyNotSet.new( "Property, #{e.message}, not set for, #{handler.class.name}" )
-          rescue StandardError => e
-            puts "E #{e.message}"
-            log 'An error occurred in Handler: ' + handler.class.name
-            raise e
-          end
+          log "Handler, #{handler.class.name}, Started"
+          handler.handle(@msg.msg)
+          log "Handler, #{handler.class.name}, Finished"
+        rescue PropertyNotSet => e
+          raise PropertyNotSet, "Property, #{e.message}, not set for, #{handler.class.name}"
+        rescue StandardError => e
+          puts "E #{e.message}"
+          log "An error occurred in Handler: #{handler.class.name}"
+          raise e
         end
 
-        if @saga_manager.handle(@msg) == false && handler_list.length == 0
-          fail NoHandlerFound, msg_name
-        end
+        raise NoHandlerFound, msg_name if @saga_manager.handle(@msg) == false && handler_list.length.zero?
+
         @resource_manager.commit(msg_name)
 
         send_queued_msgs
         log "Finished processing msg, #{msg_name}"
-
       rescue StandardError => e
         @resource_manager.rollback(msg_name)
         @queue_for_msgs_to_be_sent_on_complete = nil
@@ -381,9 +322,7 @@ module RServiceBus2
     def _send_already_wrapped_and_serialised(serialized_object, queue_name)
       RServiceBus2.rlog 'Bus._send_already_wrapped_and_serialised'
 
-      unless @config.forward_sent_messages_to.nil?
-        @mq.send(@config.forward_sent_messages_to, serialized_object)
-      end
+      @mq.send(@config.forward_sent_messages_to, serialized_object) unless @config.forward_sent_messages_to.nil?
 
       @mq.send(queue_name, serialized_object)
     end
@@ -422,8 +361,9 @@ module RServiceBus2
 
     def queue_msg_for_send_on_complete(msg, queue_name, timestamp = nil)
       correlation_id = @saga_data.nil? ? nil : @saga_data.correlation_id
-      correlation_id = (!@msg.nil? && !@msg.correlation_id.nil?) ? @msg.correlation_id : correlation_id
-      @queue_for_msgs_to_be_sent_on_complete << Hash['msg', msg, 'queue_name', queue_name, 'correlation_id', correlation_id, 'timestamp',timestamp ]
+      correlation_id = !@msg.nil? && !@msg.correlation_id.nil? ? @msg.correlation_id : correlation_id
+      @queue_for_msgs_to_be_sent_on_complete <<
+        Hash['msg', msg, 'queue_name', queue_name, 'correlation_id', correlation_id, 'timestamp', timestamp]
     end
 
     # Sends a msg back across the bus
@@ -431,7 +371,7 @@ module RServiceBus2
     # email, where the reply address can actually be anywhere
     # @param [RServiceBus2::Message] msg msg to be sent
     def reply(msg)
-      RServiceBus2.rlog 'Reply with: ' + msg.class.name + ' To: ' + @msg.return_address
+      RServiceBus2.rlog "Reply with: #{msg.class.name} To: #{@msg.return_address}"
       @stats.inc_total_reply
 
       queue_msg_for_send_on_complete(msg, @msg.return_address)
@@ -443,21 +383,20 @@ module RServiceBus2
 
       return @mq.local_queue_name if @handler_manager.can_msg_be_handled_locally(msg_name)
 
-      log 'No end point mapping found for: ' + msg_name
-      log '**** Check environment variable MessageEndpointMappings contains an entry named : ' + msg_name
-      raise 'No end point mapping found for: ' + msg_name
+      log "No end point mapping found for: #{msg_name}"
+      log "**** Check environment variable MessageEndpointMappings contains an entry named: #{msg_name}"
+      raise "No end point mapping found for: #{msg_name}"
     end
 
-
     # Send a msg across the bus
     # msg destination is specified at the infrastructure level
     # @param [RServiceBus2::Message] msg msg to be sent
-    def send( msg, timestamp=nil )
+    def send(msg, timestamp = nil)
       RServiceBus2.rlog 'Bus.Send'
       @stats.inc_total_sent
 
       msg_name = msg.class.name
-      queue_name = self.get_endpoint_for_msg(msg_name)
+      queue_name = get_endpoint_for_msg(msg_name)
       queue_msg_for_send_on_complete(msg, queue_name, timestamp)
     end
 
@@ -476,7 +415,7 @@ module RServiceBus2
     # Sends a subscription request across the Bus
     # @param [String] eventName event to be subscribes to
     def subscribe(event_name)
-      RServiceBus2.rlog 'Bus.Subscribe: ' + event_name
+      RServiceBus2.rlog "Bus.Subscribe: #{event_name}"
 
       queue_name = get_endpoint_for_msg(event_name)
       subscription = MessageSubscription.new(event_name)

data/lib/rservicebus2/monitor_configure.rb

@@ -26,7 +26,7 @@ module RServiceBus2
       @resource_manager.all.each do |k, v|
         next unless monitor.class.method_defined?(k)
 
-        monitor.instance_variable_set("@#{k}", v.get_resource)
+        monitor.instance_variable_set("@#{k}", v.resource)
         @resource_list[monitor.class.name] = [] if
           @resource_list[monitor.class.name].nil?
         @resource_list[monitor.class.name] << v

data/lib/rservicebus2/saga/manager.rb

@@ -76,7 +76,7 @@ module RServiceBus2
       return if @resource_list_by_saga_name[saga.class.name].nil?
 
       @resource_list_by_saga_name[saga.class.name].each do |k, v|
-        saga.instance_variable_set("@#{k}", @resource_manager.get(k).get_resource)
+        saga.instance_variable_set("@#{k}", @resource_manager.get(k).resource)
         RServiceBus2.rlog "App resource attribute, #{k}, set for: " + saga.class.name
       end
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rservicebus2
 version: !ruby/object:Gem::Version
-  version: 0.2.8
+  version: 0.2.9
 platform: ruby
 authors:
 - Guy Irvine
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-05-08 00:00:00.000000000 Z
+date: 2021-05-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: beanstalk-client