Merge pull request #8591 from koic/fix_an_error_when_autocorrecting

[Fix #8590] Fix an error when auto-correcting encoding mismatch file

CHANGELOG.md

@@ -28,6 +28,7 @@
 * [#8518](https://github.com/rubocop-hq/rubocop/issues/8518): Fix `Lint/ConstantResolution` cop reporting offense for `module` and `class` definitions. ([@tejasbubane][])
 * [#8158](https://github.com/rubocop-hq/rubocop/issues/8158): Fix `Style/MultilineWhenThen` cop to correctly handle cases with multiline body. ([@dsavochkin][])
 * [#7705](https://github.com/rubocop-hq/rubocop/issues/7705): Fix `Style/OneLineConditional` cop to handle if/then/elsif/then/else/end cases. Add `AlwaysCorrectToMultiline` config option to this cop to always convert offenses to the multi-line form (false by default). ([@Lykos][], [@dsavochkin][])
+* [#8590](https://github.com/rubocop-hq/rubocop/issues/8590): Fix an error when auto-correcting encoding mismatch file. ([@koic][])
 
 ### Changes
 

lib/rubocop/cop/team.rb

@@ -117,6 +117,7 @@ def external_dependency_checksum
       def autocorrect(processed_source, report)
         @updated_source_file = false
         return unless autocorrect?
+        return if report.processed_source.parser_error
 
         new_source = autocorrect_report(report)
 

spec/rubocop/cop/team_spec.rb

@@ -156,6 +156,24 @@ def a
       end
     end
 
+    context 'when autocorrection is enabled and file encoding is mismatch' do
+      let(:options) { { auto_correct: true } }
+
+      before do
+        create_file(file_path, <<~RUBY)
+          # encoding: Shift_JIS
+          puts 'Ｔｈｉｓ ｆｉｌｅ ｅｎｃｏｄｉｎｇ ｉｓ ＵＴＦ－８．'
+        RUBY
+      end
+
+      it 'no error occurs' do
+        source = RuboCop::ProcessedSource.from_file(file_path, ruby_version)
+        team.inspect_file(source)
+
+        expect(team.errors.empty?).to be(true)
+      end
+    end
+
     context 'when Cop#on_* raises an error' do
       include_context 'mock console output'
       before do