railway-ipc 0.1.5 → 1.1.0

data/lib/railway_ipc/railtie.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module RailwayIpc
   class Railtie < Rails::Railtie
     railtie_name :railway_ipc

data/lib/railway_ipc/responder.rb

@@ -1,17 +1,20 @@
+# frozen_string_literal: true
+
 module RailwayIpc
   class Responder
     def self.respond(&block)
       @block = block
     end
 
-    def self.block
-      @block
+    class << self
+      attr_reader :block
     end
 
     def respond(request)
-      RailwayIpc.logger.info(request, "Responding to request")
+      RailwayIpc.logger.info(request, 'Responding to request')
       response = self.class.block.call(request)
       raise ResponseTypeError.new(response.class) unless response.is_a?(Google::Protobuf::MessageExts)
+
       response
     end
 

data/lib/railway_ipc/response.rb

@@ -1,7 +1,10 @@
+# frozen_string_literal: true
+
 module RailwayIpc
   class Response
     attr_reader :body, :success
-    def initialize(message, success:true)
+
+    def initialize(message, success: true)
       @body = message
       @success = success
     end

data/lib/railway_ipc/rpc/client/client.rb

@@ -1,12 +1,15 @@
-require "railway_ipc/rpc/client/client_response_handlers"
-require "railway_ipc/rpc/concerns/publish_location_configurable"
-require "railway_ipc/rpc/concerns/error_adapter_configurable"
-require "railway_ipc/rpc/client/errors/timeout_error"
+# frozen_string_literal: true
+
+require 'railway_ipc/rpc/client/client_response_handlers'
+require 'railway_ipc/rpc/concerns/publish_location_configurable'
+require 'railway_ipc/rpc/concerns/error_adapter_configurable'
+require 'railway_ipc/rpc/client/errors/timeout_error'
 
 module RailwayIpc
   class Client
     attr_accessor :response_message, :request_message
     attr_reader :rabbit_connection, :message
+
     extend RailwayIpc::RPC::PublishLocationConfigurable
     extend RailwayIpc::RPC::ErrorAdapterConfigurable
 
@@ -18,12 +21,12 @@ module RailwayIpc
       RPC::ClientResponseHandlers.instance.register(response_type)
     end
 
-    def initialize(request_message, opts = { automatic_recovery: false }, rabbit_adapter: RailwayIpc::Rabbitmq::Adapter)
+    def initialize(request_message, opts={ automatic_recovery: false }, rabbit_adapter: RailwayIpc::Rabbitmq::Adapter)
       @rabbit_connection = rabbit_adapter.new(exchange_name: self.class.exchange_name, options: opts)
       @request_message = request_message
     end
 
-    def request(timeout = 10)
+    def request(timeout=10)
       setup_rabbit_connection
       attach_reply_queue_to_message
       publish_message
@@ -35,18 +38,20 @@ module RailwayIpc
       RailwayIpc::RPC::ClientResponseHandlers.instance.registered
     end
 
+    # rubocop:disable Metrics/AbcSize
     def process_payload(response)
       decoded_payload = decode_payload(response)
       case decoded_payload.type
       when *registered_handlers
         @message = get_message_class(decoded_payload).decode(decoded_payload.message)
-        RailwayIpc.logger.info(message, "Handling response")
+        RailwayIpc.logger.info(message, 'Handling response')
         RailwayIpc::Response.new(message, success: true)
       else
         @message = LearnIpc::ErrorMessage.decode(decoded_payload.message)
-        raise RailwayIpc::UnhandledMessageError, "#{self.class} does not know how to handle #{decoded_payload.type}"
+        raise RailwayIpc::UnhandledMessageError.new("#{self.class} does not know how to handle #{decoded_payload.type}")
       end
     end
+    # rubocop:enable Metrics/AbcSize
 
     def setup_rabbit_connection
       rabbit_connection
@@ -60,7 +65,9 @@ module RailwayIpc
         self.response_message = process_payload(payload)
       end
     rescue RailwayIpc::Rabbitmq::Adapter::TimeoutError
+      # rubocop:disable Style/RedundantSelf
       error = self.class.rpc_error_adapter_class.error_message(TimeoutError.new, self.request_message)
+      # rubocop:enable Style/RedundantSelf
       self.response_message = RailwayIpc::Response.new(error, success: false)
     rescue StandardError
       self.response_message = RailwayIpc::Response.new(message, success: false)
@@ -70,12 +77,12 @@ module RailwayIpc
 
     private
 
-    def log_exception(e, payload)
+    def log_exception(exception, payload)
       RailwayIpc.logger.log_exception(
-        feature: "railway_consumer",
-        error: e.class,
-        error_message: e.message,
-        payload: decode_for_error(e, payload),
+        feature: 'railway_consumer',
+        error: exception.class,
+        error_message: exception.message,
+        payload: decode_for_error(exception, payload)
       )
     end
 
@@ -92,13 +99,16 @@ module RailwayIpc
     end
 
     def publish_message
-      RailwayIpc.logger.info(request_message, "Sending request")
-      rabbit_connection.publish(RailwayIpc::Rabbitmq::Payload.encode(request_message), routing_key: "")
+      RailwayIpc.logger.info(request_message, 'Sending request')
+      rabbit_connection.publish(RailwayIpc::Rabbitmq::Payload.encode(request_message), routing_key: '')
     end
 
-    def decode_for_error(e, payload)
-      return e.message unless payload
+    def decode_for_error(exception, payload)
+      return exception.message unless payload
+
+      # rubocop:disable Style/RedundantSelf
       self.class.rpc_error_adapter_class.error_message(payload, self.request_message)
+      # rubocop:enable Style/RedundantSelf
     end
   end
 end

data/lib/railway_ipc/rpc/client/client_response_handlers.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module RailwayIpc
   module RPC
     class ClientResponseHandlers

data/lib/railway_ipc/rpc/client/errors/timeout_error.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module RailwayIpc
   class Client
     class TimeoutError < StandardError; end

data/lib/railway_ipc/rpc/concerns/error_adapter_configurable.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module RailwayIpc
   module RPC
     module ErrorAdapterConfigurable

data/lib/railway_ipc/rpc/concerns/message_observation_configurable.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module RailwayIpc
   module RPC
     module MessageObservationConfigurable

data/lib/railway_ipc/rpc/concerns/publish_location_configurable.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module RailwayIpc
   module RPC
     module PublishLocationConfigurable

data/lib/railway_ipc/rpc/rpc.rb

@@ -1,2 +1,4 @@
+# frozen_string_literal: true
+
 require 'railway_ipc/rpc/client/client'
 require 'railway_ipc/rpc/server/server'

data/lib/railway_ipc/rpc/server/server.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'railway_ipc/rpc/server/server_response_handlers'
 require 'railway_ipc/rpc/concerns/error_adapter_configurable'
 require 'railway_ipc/rpc/concerns/message_observation_configurable'
@@ -12,7 +14,7 @@ module RailwayIpc
       RailwayIpc::RPC::ServerResponseHandlers.instance.register(handler: with, message: message_type)
     end
 
-    def initialize(opts = {automatic_recovery: true}, rabbit_adapter: RailwayIpc::Rabbitmq::Adapter)
+    def initialize(opts={ automatic_recovery: true }, rabbit_adapter: RailwayIpc::Rabbitmq::Adapter)
       @rabbit_connection = rabbit_adapter.new(
         queue_name: self.class.queue_name,
         exchange_name: self.class.exchange_name,
@@ -29,6 +31,8 @@ module RailwayIpc
       subscribe_to_queue
     end
 
+    # rubocop:disable Metrics/AbcSize
+    # rubocop:disable Metrics/MethodLength
     def work(payload)
       decoded_payload = RailwayIpc::Rabbitmq::Payload.decode(payload)
       case decoded_payload.type
@@ -38,7 +42,7 @@ module RailwayIpc
         responder.respond(message)
       else
         @message = LearnIpc::ErrorMessage.decode(decoded_payload.message)
-        raise RailwayIpc::UnhandledMessageError, "#{self.class} does not know how to handle #{decoded_payload.type}"
+        raise RailwayIpc::UnhandledMessageError.new("#{self.class} does not know how to handle #{decoded_payload.type}")
       end
     rescue StandardError => e
       RailwayIpc.logger.log_exception(
@@ -49,6 +53,8 @@ module RailwayIpc
       )
       raise e
     end
+    # rubocop:enable Metrics/AbcSize
+    # rubocop:enable Metrics/MethodLength
 
     def handle_request(payload)
       response = work(payload)
@@ -84,6 +90,5 @@ module RailwayIpc
         handle_request(payload)
       end
     end
-
   end
 end

data/lib/railway_ipc/rpc/server/server_response_handlers.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'railway_ipc/handler_store'
 module RailwayIpc
   module RPC

data/lib/railway_ipc/tasks/generate_migrations.rake

@@ -1,25 +1,25 @@
+# frozen_string_literal: true
+
 require 'fileutils'
 
 namespace :railway_ipc do
   namespace :generate do
-    desc "Generates migrations to store Railway messages"
+    desc 'Generates migrations to store Railway messages'
     task :migrations do
-      if defined?(ActiveRecord::Base)
-        puts "generating Railway IPC table migrations"
-        seconds = 0
-        gem_path = Gem.loaded_specs['railway-ipc'].full_gem_path
-        folder_dest = "#{Rails.root.to_s}/db/migrate"
-        FileUtils.mkdir_p(folder_dest)
+      raise 'Migration generation requires active record' unless defined?(ActiveRecord::Base)
+
+      puts 'generating Railway IPC table migrations'
+      seconds = 0
+      gem_path = Gem.loaded_specs['railway-ipc'].full_gem_path
+      folder_dest = "#{Rails.root}/db/migrate"
+      FileUtils.mkdir_p(folder_dest)
 
-        Dir.glob("#{gem_path}/priv/migrations/*.rb").each do |file_path|
-          file_name = File.basename(file_path)
-          migration_timestamp = (Time.now + seconds).utc.strftime("%Y%m%d%H%M%S") % "%.14d"
-          new_file_name = "#{migration_timestamp}_#{file_name}"
-          FileUtils.copy_file(file_path, "#{folder_dest}/#{new_file_name}")
-          seconds += 1
-        end
-      else
-        raise "Migration generation requires active record"
+      Dir.glob("#{gem_path}/priv/migrations/*.rb").each do |file_path|
+        file_name = File.basename(file_path)
+        migration_timestamp = (Time.now + seconds).utc.strftime('%Y%m%d%H%M%S') % '%.14d'
+        new_file_name = "#{migration_timestamp}_#{file_name}"
+        FileUtils.copy_file(file_path, "#{folder_dest}/#{new_file_name}")
+        seconds += 1
       end
     end
   end

data/lib/railway_ipc/tasks/start_consumers.rake

@@ -1,7 +1,9 @@
+# frozen_string_literal: true
+
 namespace :railway_ipc do
   namespace :consumers do
     task :start do
-      ENV["WORKERS"] = ENV["CONSUMERS"]
+      ENV['WORKERS'] = ENV['CONSUMERS']
       RailwayIpc.start
     end
   end

data/lib/railway_ipc/tasks/start_servers.rake

@@ -1,7 +1,9 @@
+# frozen_string_literal: true
+
 namespace :railway_ipc do
   namespace :servers do
     task :start do
-      ENV["WORKERS"] = ENV["SERVERS"]
+      ENV['WORKERS'] = ENV['SERVERS']
       RailwayIpc.start
     end
   end

data/lib/railway_ipc/unhandled_message_error.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module RailwayIpc
   class UnhandledMessageError < StandardError
   end

data/lib/railway_ipc/unknown_message.pb.rb

@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+require 'google/protobuf'
+
+Google::Protobuf::DescriptorPool.generated_pool.build do
+  add_message 'railway_ipc.messages.Unknown' do
+    optional :user_uuid, :string, 1
+    optional :correlation_id, :string, 2
+    optional :uuid, :string, 3
+    map :context, :string, :string, 4
+  end
+end
+
+module RailwayIpc
+  module Messages
+    Unknown = Google::Protobuf::DescriptorPool.generated_pool.lookup('railway_ipc.messages.Unknown').msgclass
+  end
+end

data/lib/railway_ipc/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module RailwayIpc
-  VERSION = "0.1.5"
+  VERSION = '1.1.0'
 end

data/priv/migrations/add_railway_ipc_consumed_messages.rb

@@ -1,19 +1,21 @@
+# frozen_string_literal: true
+
 class AddRailwayIpcConsumedMessages < ActiveRecord::Migration
   def change
-    create_table :railway_ipc_consumed_messages, id: false do | t |
+    create_table :railway_ipc_consumed_messages do |t|
       t.uuid :uuid, null: false
       t.string :message_type
       t.uuid :user_uuid
       t.uuid :correlation_id
       t.text :encoded_message
       t.string :status, null: false
-      t.string :queue
+      t.string :queue, null: false
       t.string :exchange
 
       t.datetime :updated_at
       t.datetime :inserted_at
     end
 
-    add_index :railway_ipc_consumed_messages, :uuid, unique: true
+    add_index :railway_ipc_consumed_messages, %i[uuid queue], unique: true
   end
 end

data/priv/migrations/add_railway_ipc_published_messages.rb

@@ -1,6 +1,8 @@
+# frozen_string_literal: true
+
 class AddRailwayIpcPublishedMessages < ActiveRecord::Migration
   def change
-    create_table :railway_ipc_published_messages, id: false do | t |
+    create_table :railway_ipc_published_messages, id: false do |t|
       t.uuid :uuid, null: false
       t.string :message_type
       t.uuid :user_uuid

data/railway_ipc.gemspec

@@ -1,50 +1,54 @@
-lib = File.expand_path("../lib", __FILE__)
+# frozen_string_literal: true
+
+lib = File.expand_path('lib', __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
-require "railway_ipc/version"
+require 'railway_ipc/version'
 
 Gem::Specification.new do |spec|
-  spec.name = "railway-ipc"
+  spec.name = 'railway-ipc'
   spec.version = RailwayIpc::VERSION
-  spec.authors = ""
-  spec.email = ""
+  spec.authors = ''
+  spec.email = ''
+  spec.required_ruby_version = '>= 2.5'
 
-  spec.summary = %q{IPC components for Rails}
-  spec.description = %q{IPC components for Rails}
-  spec.homepage = "http://learn.co"
-  spec.license = "MIT"
+  spec.summary = 'IPC components for Rails'
+  spec.description = 'IPC components for Rails'
+  spec.homepage = 'http://learn.co'
+  spec.license = 'MIT'
 
   # Prevent pushing this gem to RubyGems.org. To allow pushes either set the 'allowed_push_host'
   # to allow pushing to a single host or delete this section to allow pushing to any host.
   if spec.respond_to?(:metadata)
-    spec.metadata["allowed_push_host"] = "https://rubygems.org/"
+    spec.metadata['allowed_push_host'] = 'https://rubygems.org/'
   else
-    raise "RubyGems 2.0 or newer is required to protect against " \
-          "public gem pushes."
+    raise 'RubyGems 2.0 or newer is required to protect against ' \
+          'public gem pushes.'
   end
 
   # Specify which files should be added to the gem when it is released.
   # The `git ls-files -z` loads the files in the RubyGem that have been added into git.
-  spec.files = Dir.chdir(File.expand_path("..", __FILE__)) do
-    `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
+  spec.files = Dir.chdir(File.expand_path(__dir__)) do
+    `git ls-files -z`.split("\x0").reject { |f| f.match(/^(.circleci|.rspec|.rubocop.yml|test|spec|features)/) }
   end
-  spec.bindir = "exe"
+  spec.bindir = 'exe'
   spec.executables = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
-  spec.require_paths = ["lib"]
+  spec.require_paths = ['lib']
+
+  spec.add_development_dependency 'bundler', '2.0.1'
+  spec.add_development_dependency 'factory_bot', '~> 5.1'
+  spec.add_development_dependency 'google-protobuf', '~> 3.9'
+  spec.add_development_dependency 'rake', '>= 10.0.0'
+  spec.add_development_dependency 'rspec', '~> 3.0'
+  spec.add_development_dependency 'rubocop', '~> 0.86'
 
-  spec.add_development_dependency "rake", ">= 10.0.0"
-  spec.add_development_dependency "bundler", "2.0.1"
-  spec.add_development_dependency "rspec", "~> 3.0"
-  spec.add_development_dependency "factory_bot", "~> 5.1"
-  spec.add_development_dependency "pry-byebug", "3.4.2"
-  spec.add_development_dependency "google-protobuf", "~> 3.9"
-  spec.add_dependency "sneakers", "~> 2.3.5"
-  spec.add_dependency "bunny", "~> 2.2.0"
+  spec.add_dependency 'bunny', '~> 2.2.0'
+  spec.add_dependency 'sneakers', '~> 2.3.5'
 
   # Setup for testing Rails type code within mock Rails app
-  spec.add_development_dependency "rails", "~> 5.0.7"
-  spec.add_development_dependency "rspec-rails"
-  spec.add_development_dependency "pg", "~> 0.18"
-  spec.add_development_dependency "shoulda-matchers", "~> 4.2"
-  spec.add_development_dependency "database_cleaner", "~> 1.7"
-  spec.add_development_dependency "listen", "~> 3.0.5"
+  spec.add_development_dependency 'database_cleaner', '~> 1.7'
+  spec.add_development_dependency 'listen', '~> 3.0.5'
+  spec.add_development_dependency 'pg', '~> 0.18'
+  spec.add_development_dependency 'rails', '~> 5.0.7'
+  spec.add_development_dependency 'rspec-rails'
+  spec.add_development_dependency 'shoulda-matchers', '~> 4.2'
 end