[Fix #7993] Fix a false positive for Migration/DepartmentName

CHANGELOG.md

@@ -16,6 +16,7 @@
 * [#7972](https://github.com/rubocop-hq/rubocop/issues/7972): Fix an incorrect autocrrect for `Style/HashSyntax` when using a return value uses `return`. ([@koic][])
 * [#7886](https://github.com/rubocop-hq/rubocop/issues/7886): Fix a bug in `AllowComments` logic in `Lint/SuppressedException`. ([@jonas054][])
 * [#7991](https://github.com/rubocop-hq/rubocop/issues/7991): Fix an error for `Layout/EmptyLinesAroundAttributeAccessor` when attribute method is method chained. ([@koic][])
+* [#7993](https://github.com/rubocop-hq/rubocop/issues/7993): Fix a false positive for `Migration/DepartmentName` when a disable comment contains an unexpected character for department name. ([@koic][])
 
 ### Changes
 

lib/rubocop/cop/migration/department_name.rb

@@ -27,10 +27,10 @@ def investigate(processed_source)
             Regexp.last_match(4).scan(/[^,]+|[\W]+/) do |name|
               trimmed_name = name.strip
 
-              break if contain_plain_comment?(trimmed_name)
-
               check_cop_name(trimmed_name, comment, offset) unless valid_content_token?(trimmed_name)
 
+              break if contain_unexpected_character_for_department_name?(name)
+
               offset += name.length
             end
           end
@@ -64,8 +64,8 @@ def valid_content_token?(content_token)
             !DISABLING_COPS_CONTENT_TOKEN.match(content_token).nil?
         end
 
-        def contain_plain_comment?(name)
-          name == '#'
+        def contain_unexpected_character_for_department_name?(name)
+          name.match?(%r{[^A-z/, ]})
         end
 
         def qualified_legacy_cop_name(cop_name)

manual/configuration.md

@@ -629,6 +629,12 @@ Running `rubocop --[safe-]auto-correct --disable-uncorrectable` will
 create comments to disable all offenses that can't be automatically
 corrected.
 
+Do not write anything other than cop name in the disabling comment. E.g.:
+
+```ruby
+# rubocop:disable Layout/LineLength --This is a bad comment that includes other than cop name.
+```
+
 ## Setting the style guide URL
 
 You can specify the base URL of the style guide using `StyleGuideBaseURL`.

spec/rubocop/cop/migration/department_name_spec.rb

@@ -90,6 +90,15 @@
     end
   end
 
+  context 'when a disable comment contains an unexpected character for department name' do
+    it 'accepts' do
+      expect_no_offenses(<<~RUBY)
+        # rubocop:disable Style/Alias -- because something, something, and something
+        alias :ala :bala
+      RUBY
+    end
+  end
+
   # `Migration/DepartmentName` cop's role is to complement a department name.
   # The role would be simple if another feature could detect unexpected
   # disable comment format.