[Fix rubocop#11792] Fix an error for Lint/DuplicateMatchPattern

Fixes rubocop#11792. This PR fixes an error for `Lint/DuplicateMatchPattern` when using hash pattern with `if` guard.
owst · Apr 12, 2023 · c59e349 · c59e349
1 parent 5befa36
commit c59e349
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 1 deletion.
diff --git a/changelog/fix_an_error_for_lint_duplicate_match_pattern.md b/changelog/fix_an_error_for_lint_duplicate_match_pattern.md
@@ -0,0 +1 @@
+* [#11792](https://github.com/rubocop/rubocop/issues/11792): Fix an error for `Lint/DuplicateMatchPattern` when using hash pattern with `if` guard. ([@koic][])
diff --git a/lib/rubocop/cop/lint/duplicate_match_pattern.rb b/lib/rubocop/cop/lint/duplicate_match_pattern.rb
@@ -107,7 +107,7 @@ def on_case_match(case_node)
 
         def pattern_identity(pattern)
           pattern_source = if pattern.hash_pattern_type? || pattern.match_alt_type?
-                             pattern.children.map(&:source).sort
+                             pattern.children.map(&:source).sort.to_s
                            else
                              pattern.source
                            end

diff --git a/spec/rubocop/cop/lint/duplicate_match_pattern_spec.rb b/spec/rubocop/cop/lint/duplicate_match_pattern_spec.rb
@@ -214,4 +214,12 @@
       end
     RUBY
   end
+
+  it 'does not crash when using hash pattern with `if` guard' do
+    expect_no_offenses(<<~RUBY)
+      case x
+      in { key: value } if condition
+      end
+    RUBY
+  end
 end