fix rubocop errors, updated Maitrap::Client#initialize

Author: sarco3t

lib/mailtrap/client.rb

@@ -37,12 +37,9 @@ def initialize( # rubocop:disable Metrics/ParameterLists
       sandbox: false,
       inbox_id: nil
     )
-      raise ArgumentError, 'api_key is required' if api_key.nil?
-      raise ArgumentError, 'api_port is required' if api_port.nil?
-      raise ArgumentError, 'bulk mode is not applicable for sandbox API' if sandbox && bulk
+      validate_args!(api_key, api_port, bulk, sandbox, inbox_id)
 
       api_host ||= select_api_host(bulk:, sandbox:)
-      raise ArgumentError, 'inbox_id is required for sandbox API' if sandbox && inbox_id.nil?
 
       @api_key = api_key
       @api_host = api_host
@@ -215,5 +212,12 @@ def json_response(body)
     def compact_with_empty_arrays(hash)
       hash.reject { |_, v| v.nil? || (v.is_a?(Array) && v.empty?) }
     end
+
+    def validate_args!(api_key, api_port, bulk, sandbox, inbox_id)
+      raise ArgumentError, 'api_key is required' if api_key.nil?
+      raise ArgumentError, 'api_port is required' if api_port.nil?
+      raise ArgumentError, 'bulk mode is not applicable for sandbox API' if bulk && sandbox
+      raise ArgumentError, 'inbox_id is required for sandbox API' if sandbox && inbox_id.nil?
+    end
   end
 end

…successfully_sends_a_batch_of_emails.yml …successfully_sends_a_batch_of_emails.ymlspec/fixtures/vcr_cassettes/Mailtrap_Client/_send_batch/in_bulk_mode/successfully_sends_a_batch_of_emails.yml renamed to spec/fixtures/vcr_cassettes/Mailtrap_Client/_send_batch/when_in_bulk_mode/successfully_sends_a_batch_of_emails.yml spec/fixtures/vcr_cassettes/Mailtrap_Client/_send_batch/in_bulk_mode/successfully_sends_a_batch_of_emails.yml renamed to spec/fixtures/vcr_cassettes/Mailtrap_Client/_send_batch/when_in_bulk_mode/successfully_sends_a_batch_of_emails.yml

@@ -300,7 +300,7 @@ def stub_post(path, status, body)
       end
     end
 
-    context 'in bulk mode' do
+    context 'when in bulk mode' do
       let(:client) { described_class.new(api_key:, bulk: true) }
 
       it 'successfully sends a batch of emails', :vcr do
@@ -317,7 +317,7 @@ def stub_post(path, status, body)
       end
     end
 
-    context 'in sandbox mode' do
+    context 'when in sandbox mode' do
       let(:client) { described_class.new(api_key:, sandbox: true, inbox_id: 3_809_768) }
 
       it 'successfully sends a batch of emails', :vcr do