invalid_case_patterns fix

See: #49960

Change-Id: I5ada583d5353521fe863d639bbad85691ca953ee
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/281870
Reviewed-by: Brian Wilkerson <[email protected]>
Commit-Queue: Phil Quitslund <[email protected]>
Reviewed-by: Keerti Parthasarathy <[email protected]>

Author: pq

pkg/analysis_server/lib/src/services/correction/error_fix_status.yaml

@@ -1813,9 +1813,7 @@ LintCode.implementation_imports:
 LintCode.implicit_call_tearoffs:
   status: hasFix
 LintCode.invalid_case_patterns:
-  status: needsFix
-  notes: |-
-    https://github.com/dart-lang/sdk/issues/49960
+  status: hasFix
 LintCode.invariant_booleans:
   status: needsEvaluation
 LintCode.iterable_contains_unrelated_type:

pkg/analysis_server/lib/src/services/correction/fix_internal.dart

@@ -503,6 +503,9 @@ class FixProcessor extends BaseProcessor {
     LintNames.implicit_call_tearoffs: [
       AddExplicitCall.new,
     ],
+    LintNames.invalid_case_patterns: [
+      AddConst.new,
+    ],
     LintNames.leading_newlines_in_multiline_strings: [
       AddLeadingNewlineToString.new,
     ],

pkg/analysis_server/test/src/services/correction/fix/add_const_test.dart

@@ -11,6 +11,8 @@ import 'fix_processor.dart';
 
 void main() {
   defineReflectiveSuite(() {
+    defineReflectiveTests(AddConst_InvalidCasePatternsTest);
+    defineReflectiveTests(AddConst_InvalidCasePatternsBulkTest);
     defineReflectiveTests(AddConst_NonConstGenerativeEnumConstructorTest);
     defineReflectiveTests(AddConst_PatternExpressionMustBeValidConst);
     defineReflectiveTests(AddConst_PreferConstConstructorsInImmutablesBulkTest);
@@ -23,6 +25,134 @@ void main() {
   });
 }
 
+@reflectiveTest
+class AddConst_InvalidCasePatternsBulkTest extends BulkFixProcessorTest {
+  @override
+  String get lintCode => LintNames.invalid_case_patterns;
+
+  Future<void> test_singleFile() async {
+    await resolveTestCode(r'''
+//@dart=2.19
+class C {
+  const C();
+}
+
+void f(Object c) {
+  switch (c) {
+    case C():
+    case [1, 2]:
+  }
+}
+''');
+    await assertHasFix(r'''
+//@dart=2.19
+class C {
+  const C();
+}
+
+void f(Object c) {
+  switch (c) {
+    case const C():
+    case const [1, 2]:
+  }
+}
+''');
+  }
+}
+
+@reflectiveTest
+class AddConst_InvalidCasePatternsTest extends FixProcessorLintTest {
+  @override
+  FixKind get kind => DartFixKind.ADD_CONST;
+
+  @override
+  String get lintCode => LintNames.invalid_case_patterns;
+
+  Future<void> test_constructorCall() async {
+    await resolveTestCode(r'''
+//@dart=2.19
+class C {
+  const C();
+}
+
+void f(Object c) {
+  switch (c) {
+    case C():
+  }
+}
+''');
+    await assertHasFix(r'''
+//@dart=2.19
+class C {
+  const C();
+}
+
+void f(Object c) {
+  switch (c) {
+    case const C():
+  }
+}
+''');
+  }
+
+  Future<void> test_listLiteral() async {
+    await resolveTestCode(r'''
+//@dart=2.19
+void f(Object o) {
+  switch (o) {
+    case [1, 2]:
+  }
+}
+''');
+    await assertHasFix(r'''
+//@dart=2.19
+void f(Object o) {
+  switch (o) {
+    case const [1, 2]:
+  }
+}
+''');
+  }
+
+  Future<void> test_mapLiteral() async {
+    await resolveTestCode(r'''
+//@dart=2.19
+void f(Object o) {
+  switch (o) {
+   case {'k': 'v'}:
+  }
+}
+''');
+    await assertHasFix(r'''
+//@dart=2.19
+void f(Object o) {
+  switch (o) {
+   case const {'k': 'v'}:
+  }
+}
+''');
+  }
+
+  Future<void> test_setLiteral() async {
+    await resolveTestCode(r'''
+//@dart=2.19
+void f(Object o) {
+  switch (o) {
+    case {1}:
+  }
+}
+''');
+    await assertHasFix(r'''
+//@dart=2.19
+void f(Object o) {
+  switch (o) {
+    case const {1}:
+  }
+}
+''');
+  }
+}
+
 @reflectiveTest
 class AddConst_NonConstGenerativeEnumConstructorTest extends FixProcessorTest {
   @override