From 7b0fbc66dfa3acc2ab927bc0ae88fdb9b5bee63c Mon Sep 17 00:00:00 2001 From: ayacai115 Date: Mon, 18 Nov 2019 08:43:42 +0900 Subject: [PATCH] [Fix #7509] Layout/SpaceInsideArrayLiteralBrackets to correct empty lines --- lib/rubocop/cop/correctors/space_corrector.rb | 3 +-- .../cop/layout/space_inside_array_literal_brackets_spec.rb | 5 +++++ 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/lib/rubocop/cop/correctors/space_corrector.rb b/lib/rubocop/cop/correctors/space_corrector.rb index c3542669537..11f4688c568 100644 --- a/lib/rubocop/cop/correctors/space_corrector.rb +++ b/lib/rubocop/cop/correctors/space_corrector.rb @@ -12,12 +12,11 @@ class << self def empty_corrections(processed_source, corrector, empty_config, left_token, right_token) @processed_source = processed_source + range = range_between(left_token.end_pos, right_token.begin_pos) if offending_empty_space?(empty_config, left_token, right_token) - range = side_space_range(range: left_token.pos, side: :right) corrector.remove(range) corrector.insert_after(left_token.pos, ' ') elsif offending_empty_no_space?(empty_config, left_token, right_token) - range = side_space_range(range: left_token.pos, side: :right) corrector.remove(range) end end diff --git a/spec/rubocop/cop/layout/space_inside_array_literal_brackets_spec.rb b/spec/rubocop/cop/layout/space_inside_array_literal_brackets_spec.rb index 0926303be1c..8cefc5e8ca6 100644 --- a/spec/rubocop/cop/layout/space_inside_array_literal_brackets_spec.rb +++ b/spec/rubocop/cop/layout/space_inside_array_literal_brackets_spec.rb @@ -42,6 +42,11 @@ new_source = autocorrect_source('a = [ ]') expect(new_source).to eq('a = []') end + + it 'auto-corrects multiline spaces' do + new_source = autocorrect_source("a = [\n]") + expect(new_source).to eq('a = []') + end end context 'with space inside empty braces allowed' do