From 433ace31017f4f8699b56a2c6a7aed293f1776b7 Mon Sep 17 00:00:00 2001 From: Pieter Oliver Date: Mon, 30 Jun 2025 12:40:56 +0100 Subject: [PATCH] AutoFix RuboCop Layout/SpaceInsideHashLiteralBraces --- .rubocop_todo.yml | 12 ------------ .../grape_logging/loggers/filter_parameters_spec.rb | 2 +- .../grape_logging/loggers/request_headers_spec.rb | 8 ++++---- spec/lib/grape_logging/loggers/response_spec.rb | 2 +- .../grape_logging/middleware/request_logger_spec.rb | 2 +- 5 files changed, 7 insertions(+), 19 deletions(-) diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml index b00d8c6..6f6f8d2 100644 --- a/.rubocop_todo.yml +++ b/.rubocop_todo.yml @@ -29,18 +29,6 @@ Gemspec/RequiredRubyVersion: Exclude: - 'grape_logging.gemspec' -# Offense count: 16 -# This cop supports safe autocorrection (--autocorrect). -# Configuration parameters: EnforcedStyle, EnforcedStyleForEmptyBraces. -# SupportedStyles: space, no_space, compact -# SupportedStylesForEmptyBraces: space, no_space -Layout/SpaceInsideHashLiteralBraces: - Exclude: - - 'spec/lib/grape_logging/loggers/filter_parameters_spec.rb' - - 'spec/lib/grape_logging/loggers/request_headers_spec.rb' - - 'spec/lib/grape_logging/loggers/response_spec.rb' - - 'spec/lib/grape_logging/middleware/request_logger_spec.rb' - # Offense count: 3 # This cop supports safe autocorrection (--autocorrect). # Configuration parameters: EnforcedStyle. diff --git a/spec/lib/grape_logging/loggers/filter_parameters_spec.rb b/spec/lib/grape_logging/loggers/filter_parameters_spec.rb index 67b7705..3c2475d 100644 --- a/spec/lib/grape_logging/loggers/filter_parameters_spec.rb +++ b/spec/lib/grape_logging/loggers/filter_parameters_spec.rb @@ -20,7 +20,7 @@ describe GrapeLogging::Loggers::FilterParameters do OpenStruct.new( params: deep_clone.call.merge( 'five' => deep_clone.call.merge( - deep_clone.call.merge({'six' => {'seven' => 'seven', 'eight' => 'eight', 'one' => 'another one'}}) + deep_clone.call.merge({ 'six' => { 'seven' => 'seven', 'eight' => 'eight', 'one' => 'another one' } }) ) ) ) diff --git a/spec/lib/grape_logging/loggers/request_headers_spec.rb b/spec/lib/grape_logging/loggers/request_headers_spec.rb index 6a33f4c..2d00843 100644 --- a/spec/lib/grape_logging/loggers/request_headers_spec.rb +++ b/spec/lib/grape_logging/loggers/request_headers_spec.rb @@ -3,7 +3,7 @@ require 'ostruct' describe GrapeLogging::Loggers::RequestHeaders do let(:mock_request) do - OpenStruct.new(env: {HTTP_REFERER: 'http://example.com', HTTP_ACCEPT: 'text/plain'}) + OpenStruct.new(env: { HTTP_REFERER: 'http://example.com', HTTP_ACCEPT: 'text/plain' }) end let(:mock_request_with_unhandled_headers) do @@ -22,19 +22,19 @@ describe GrapeLogging::Loggers::RequestHeaders do it 'strips HTTP_ from the parameter' do expect(subject.parameters(mock_request, nil)).to eq({ - headers: {'Referer' => 'http://example.com', 'Accept' => 'text/plain'} + headers: { 'Referer' => 'http://example.com', 'Accept' => 'text/plain' } }) end it 'only handle things which start with HTTP_' do expect(subject.parameters(mock_request_with_unhandled_headers, nil)).to eq({ - headers: {'Referer' => 'http://example.com' } + headers: { 'Referer' => 'http://example.com' } }) end it 'substitutes _ with -' do expect(subject.parameters(mock_request_with_long_headers, nil)).to eq({ - headers: {'Referer' => 'http://example.com', 'User-Agent' => 'Mozilla/5.0' } + headers: { 'Referer' => 'http://example.com', 'User-Agent' => 'Mozilla/5.0' } }) end end diff --git a/spec/lib/grape_logging/loggers/response_spec.rb b/spec/lib/grape_logging/loggers/response_spec.rb index ce2e7b5..64b229d 100644 --- a/spec/lib/grape_logging/loggers/response_spec.rb +++ b/spec/lib/grape_logging/loggers/response_spec.rb @@ -4,7 +4,7 @@ require 'ostruct' describe GrapeLogging::Loggers::Response do context 'with a parseable JSON body' do let(:response) do - OpenStruct.new(body: [{"one": "two", "three": {"four": 5}}]) + OpenStruct.new(body: [{ "one": "two", "three": { "four": 5 } }]) end it 'returns an array of parsed JSON objects' do diff --git a/spec/lib/grape_logging/middleware/request_logger_spec.rb b/spec/lib/grape_logging/middleware/request_logger_spec.rb index 652a63f..b3876fc 100644 --- a/spec/lib/grape_logging/middleware/request_logger_spec.rb +++ b/spec/lib/grape_logging/middleware/request_logger_spec.rb @@ -6,7 +6,7 @@ describe GrapeLogging::Middleware::RequestLogger do let(:app) { proc { [status, {}, ['response body']] } } let(:stack) { described_class.new app, options } let(:request) { Rack::MockRequest.new(stack) } - let(:options) { {include: [], logger: logger} } + let(:options) { { include: [], logger: logger } } let(:logger) { double('logger') } let(:path) { '/' } let(:request_method) { 'get' }