[Fix #8704] Fix an error for Lint/AmbiguousOperator

Fixes #8704. This PR fixes an error for `Lint/AmbiguousOperator` when using safe navigation operator with a unary operator.
rubocop · Sep 12, 2020 · 134068f · 134068f
1 parent 3686984
commit 134068f
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -30,6 +30,7 @@
 * [#8683](https://github.com/rubocop-hq/rubocop/issues/8683): Make naming cops work with non-ascii characters. ([@tejasbubane][])
 * [#8626](https://github.com/rubocop-hq/rubocop/issues/8626): Fix false negatives for `Lint/UselessMethodDefinition`. ([@marcandre][])
 * [#8698](https://github.com/rubocop-hq/rubocop/pull/8698): Fix cache to avoid encoding exception. ([@marcandre][])
+* [#8704](https://github.com/rubocop-hq/rubocop/issues/8704): Fix an error for `Lint/AmbiguousOperator` when using safe navigation operator with a unary operator. ([@koic][])
 
 ### Changes
 

diff --git a/lib/rubocop/cop/lint/ambiguous_operator.rb b/lib/rubocop/cop/lint/ambiguous_operator.rb
@@ -43,6 +43,8 @@ def on_new_investigation
             next unless diagnostic.reason == :ambiguous_prefix
 
             offense_node = find_offense_node_by(diagnostic)
+            next unless offense_node
+
             message = message(diagnostic)
 
             add_offense(

diff --git a/spec/rubocop/cop/lint/ambiguous_operator_spec.rb b/spec/rubocop/cop/lint/ambiguous_operator_spec.rb
@@ -194,4 +194,12 @@
       end
     end
   end
+
+  context 'when using safe navigation operator with a unary operator' do
+    it 'does not register an offense' do
+      expect_no_offenses(<<~RUBY)
+        do_something&.* -1
+      RUBY
+    end
+  end
 end