[Fix #1406] Allow a newline in SpaceInsideRangeLiteral

CHANGELOG.md

@@ -14,6 +14,7 @@
 * [#1415](https://github.com/bbatsov/rubocop/issues/1415): String literals concatenated with backslashes are now handled correctly by `StringLiteralsInInterpolation`. ([@jonas054][])
 * [#1416](https://github.com/bbatsov/rubocop/issues/1416): Fix handling of `begin/rescue/else/end` in `ElseAlignment`. ([@jonas054][])
 * [#1413](https://github.com/bbatsov/rubocop/issues/1413): Support empty elsif branches in `MultilineIfThen`. ([@janraasch][], [@jonas054][])
+* [#1406](https://github.com/bbatsov/rubocop/issues/1406): Allow a newline in `SpaceInsideRangeLiteral`. ([@bbatsov][])
 
 ## 0.27.0 (30/10/2014)
 

lib/rubocop/cop/style/space_inside_range_literal.rb

@@ -4,6 +4,7 @@ module RuboCop
   module Cop
     module Style
       # Checks for spaces inside range literals.
+      #
       # @example
       #   # bad
       #   1 .. 3
@@ -31,9 +32,13 @@ def on_erange(node)
 
         def check(node)
           expression = node.loc.expression.source
-          operator = node.loc.operator.source.gsub(/\./, '\.')
+          op = node.loc.operator.source
+          escaped_op = op.gsub(/\./, '\.')
+
+          # account for multiline range literals
+          expression.sub!(/#{escaped_op}\n\s*/, op)
 
-          return unless expression =~ /(\s#{operator})|(#{operator}\s)/
+          return unless expression =~ /(\s#{escaped_op})|(#{escaped_op}\s)/
 
           add_offense(node, :expression)
         end

spec/rubocop/cop/style/space_inside_range_literal_spec.rb

@@ -40,6 +40,18 @@
     expect(cop.offenses).to be_empty
   end
 
+  it 'accepts multiline range literal with no space in it' do
+    inspect_source(cop, ['x = 0..',
+                         '    10'])
+    expect(cop.offenses).to be_empty
+  end
+
+  it 'registers an offense in multiline range literal with space in it' do
+    inspect_source(cop, ['x = 0 ..',
+                         '    10'])
+    expect(cop.offenses.size).to eq(1)
+  end
+
   it 'autocorrects space around .. literal' do
     corrected = autocorrect_source(cop, ['1  .. 2'])
     expect(corrected).to eq '1..2'