Merge pull request #21 from pipefy/eh-main/feature/pidfile-path

refact: remove validation Aws.config

bin/pipefymessage

@@ -13,8 +13,10 @@ module PipefyMessage
     desc "start", "Starts PipefyMessage consumer"
     method_option :worker, aliases: "-w", type: :string, desc: "Worker to load"
     method_option :rails, aliases: "-R", type: :boolean, desc: "Load Rails", default: false
+    method_option :pidfile, aliases: "-P", type: :string, desc: "Path to pidfile"
     def start
       PipefyMessage::Runner.instance.initialize_rails if options[:rails]
+      PipefyMessage::Runner.instance.write_pid(options) if options[:pidfile]
       PipefyMessage::Runner.instance.run(options[:worker]) if options[:worker]
     end
   end
@@ -35,6 +37,12 @@ module PipefyMessage
       end
     end
 
+    def write_pid(options)
+      return unless (path = options[:pidfile])
+
+      File.open(path, "w") { |f| f.puts(Process.pid) }
+    end
+
     def run(worker)
       worker.constantize.process_message
     end

lib/pipefy_message/providers/aws_client/aws_client.rb

@@ -14,10 +14,7 @@ module AwsClient
       ##
       # Sets up AWS options the first time an AWS service is used.
       def self.aws_setup
-        return unless Aws.config.empty?
-
         logger.info({ message_text: "AWS configurations set" })
-
         Aws.config.update(retrieve_config)
       end
 

spec/providers/aws/aws_client_spec.rb

@@ -48,6 +48,7 @@ def initialize
       TestClient.new
 
       ENV["AWS_CLI_STUB_RESPONSE"] = "false"
+      ENV["ENABLE_AWS_CLIENT_CONFIG"] = "false"
       TestClient.new
 
       expect(Aws.config).to eq aws_opts