Merge pull request rubocop#614 from rubocop-rspec/fix-aligh-brace-with-empty-file

Darhazer · web-flow · commit aca8aaf8ed77 · 2018-05-23T17:00:26.000+03:00
Fix investigation of empty files
diff --git a/lib/rubocop/cop/rspec/align_left_let_brace.rb b/lib/rubocop/cop/rspec/align_left_let_brace.rb
@@ -25,6 +25,8 @@ def self.autocorrect_incompatible_with
         end
 
         def investigate(_processed_source)
+          return if processed_source.blank?
+
           token_aligner.offending_tokens.each do |let|
             add_offense(let, location: :begin)
           end
diff --git a/lib/rubocop/cop/rspec/align_right_let_brace.rb b/lib/rubocop/cop/rspec/align_right_let_brace.rb
@@ -25,6 +25,8 @@ def self.autocorrect_incompatible_with
         end
 
         def investigate(_processed_source)
+          return if processed_source.blank?
+
           token_aligner.offending_tokens.each do |let|
             add_offense(let, location: :end)
           end
diff --git a/spec/rubocop/cop/rspec/align_left_let_brace_spec.rb b/spec/rubocop/cop/rspec/align_left_let_brace_spec.rb
@@ -24,6 +24,10 @@
     RUBY
   end
 
+  it 'works with empty file' do
+    expect_no_offenses('')
+  end
+
   offensive_source = <<-RUBY
     let(:foo) { bar }
     let(:hi) { baz }
diff --git a/spec/rubocop/cop/rspec/align_right_let_brace_spec.rb b/spec/rubocop/cop/rspec/align_right_let_brace_spec.rb
@@ -24,6 +24,10 @@
     RUBY
   end
 
+  it 'works with empty file' do
+    expect_no_offenses('')
+  end
+
   offensive_source = <<-RUBY
     let(:foo)      { a }
     let(:hi)       { ab }
