mirror of
https://github.com/samsonjs/grape_logging.git
synced 2026-03-25 08:55:47 +00:00
AutoFix RuboCop Layout/SpaceInsideHashLiteralBraces
This commit is contained in:
parent
8b1f794e78
commit
433ace3101
5 changed files with 7 additions and 19 deletions
|
|
@ -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.
|
||||
|
|
|
|||
|
|
@ -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' } })
|
||||
)
|
||||
)
|
||||
)
|
||||
|
|
|
|||
|
|
@ -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
|
||||
|
|
|
|||
|
|
@ -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
|
||||
|
|
|
|||
|
|
@ -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' }
|
||||
|
|
|
|||
Loading…
Reference in a new issue