Fix Style/HashSyntax offenses

Author: mklocek

lib/mailtrap/client.rb

@@ -35,7 +35,7 @@ def initialize( # rubocop:disable Metrics/ParameterLists
       raise ArgumentError, 'api_key is required' if api_key.nil?
       raise ArgumentError, 'api_port is required' if api_port.nil?
 
-      api_host ||= select_api_host(bulk: bulk, sandbox: sandbox)
+      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

lib/mailtrap/mail/base.rb

@@ -67,11 +67,11 @@ def attachments=(attachments)
 
       def add_attachment(content:, filename:, type: nil, disposition: nil, content_id: nil)
         attachment = Mailtrap::Attachment.new(
-          content: content,
-          filename: filename,
-          type: type,
-          disposition: disposition,
-          content_id: content_id
+          content:,
+          filename:,
+          type:,
+          disposition:,
+          content_id:
         )
         attachments << attachment
 

lib/mailtrap/mail/from_template.rb

@@ -18,14 +18,14 @@ def initialize( # rubocop:disable Metrics/ParameterLists, Metrics/MethodLength
         template_variables: {}
       )
         super(
-          from: from,
-          to: to,
-          reply_to: reply_to,
-          cc: cc,
-          bcc: bcc,
-          attachments: attachments,
-          headers: headers,
-          custom_variables: custom_variables
+          from:,
+          to:,
+          reply_to:,
+          cc:,
+          bcc:,
+          attachments:,
+          headers:,
+          custom_variables:
         )
         @template_uuid = template_uuid
         @template_variables = template_variables

spec/mailtrap/attachment_spec.rb

@@ -6,11 +6,11 @@
 RSpec.describe Mailtrap::Attachment do
   subject(:attachment) do
     described_class.new(
-      content: content,
-      filename: filename,
-      type: type,
-      disposition: disposition,
-      content_id: content_id
+      content:,
+      filename:,
+      type:,
+      disposition:,
+      content_id:
     )
   end
 

spec/mailtrap/client_spec.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 RSpec.describe Mailtrap::Client do
-  subject(:client) { described_class.new(api_key: api_key) }
+  subject(:client) { described_class.new(api_key:) }
 
   let(:api_key) { 'correct-api-key' }
 
@@ -65,7 +65,7 @@
 
       context 'with an alternative host' do
         let(:client) do
-          described_class.new(api_key: api_key, api_host: 'alternative.host.mailtrap.io', api_port: 8080)
+          described_class.new(api_key:, api_host: 'alternative.host.mailtrap.io', api_port: 8080)
         end
 
         it 'sending is successful' do
@@ -75,7 +75,7 @@
 
       context 'with bulk flag' do
         let(:client) do
-          described_class.new(api_key: api_key, bulk: true)
+          described_class.new(api_key:, bulk: true)
         end
 
         it 'chooses host for bulk sending' do
@@ -85,7 +85,7 @@
 
       context 'with bulk flag and alternative host' do
         let(:client) do
-          described_class.new(api_key: api_key, bulk: true, api_host: 'alternative.host.mailtrap.io', api_port: 8080)
+          described_class.new(api_key:, bulk: true, api_host: 'alternative.host.mailtrap.io', api_port: 8080)
         end
 
         it 'chooses alternative host' do
@@ -95,7 +95,7 @@
 
       context 'with sandbox flag' do
         let(:client) do
-          described_class.new(api_key: api_key, sandbox: true, inbox_id: 12)
+          described_class.new(api_key:, sandbox: true, inbox_id: 12)
         end
 
         it 'chooses host for sandbox sending' do
@@ -105,15 +105,15 @@
 
       context 'with sandbox flag without inbox id' do
         let(:client) do
-          described_class.new(api_key: api_key, sandbox: true)
+          described_class.new(api_key:, sandbox: true)
         end
 
         it { expect { send }.to raise_error(ArgumentError, 'inbox_id is required for sandbox API') }
       end
 
       context 'with bulk and sandbox flag' do
         let(:client) do
-          described_class.new(api_key: api_key, bulk: true, sandbox: true)
+          described_class.new(api_key:, bulk: true, sandbox: true)
         end
 
         it { expect { send }.to raise_error(ArgumentError, 'bulk mode is not applicable for sandbox API') }
@@ -155,7 +155,7 @@
 
       context 'when using sandbox' do
         let(:client) do
-          described_class.new(api_key: api_key, sandbox: true, inbox_id: 13)
+          described_class.new(api_key:, sandbox: true, inbox_id: 13)
         end
 
         it 'sending is successful' do
@@ -178,7 +178,7 @@
     end
 
     def stub_api_send(status, body = nil)
-      stub = stub_request(:post, %r{/api/send}).to_return(status: status, body: body)
+      stub = stub_request(:post, %r{/api/send}).to_return(status:, body:)
       yield
       expect(stub).to have_been_requested
     end

spec/mailtrap/mail/base_spec.rb

@@ -6,18 +6,18 @@
 RSpec.describe Mailtrap::Mail::Base do
   subject(:mail) do
     described_class.new(
-      from: from,
-      to: to,
-      reply_to: reply_to,
-      cc: cc,
-      bcc: bcc,
+      from:,
+      to:,
+      reply_to:,
+      cc:,
+      bcc:,
       subject: mail_subject,
-      text: text,
-      html: html,
-      attachments: attachments,
-      headers: headers,
-      category: category,
-      custom_variables: custom_variables
+      text:,
+      html:,
+      attachments:,
+      headers:,
+      category:,
+      custom_variables:
     )
   end
 

spec/mailtrap/mail/from_template_spec.rb

@@ -6,16 +6,16 @@
 RSpec.describe Mailtrap::Mail::FromTemplate do
   subject(:mail) do
     described_class.new(
-      from: from,
-      to: to,
-      reply_to: reply_to,
-      cc: cc,
-      bcc: bcc,
-      attachments: attachments,
-      headers: headers,
-      custom_variables: custom_variables,
-      template_uuid: template_uuid,
-      template_variables: template_variables
+      from:,
+      to:,
+      reply_to:,
+      cc:,
+      bcc:,
+      attachments:,
+      headers:,
+      custom_variables:,
+      template_uuid:,
+      template_variables:
     )
   end