[dart fix] failing test for fixes in parts

Bug: #59572
Change-Id: I6d70f5801b7e7f27ad944a1335b7f0aeba857e2b
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/401180
Commit-Queue: Phil Quitslund <[email protected]>
Reviewed-by: Brian Wilkerson <[email protected]>

Author: pq

pkg/analysis_server/test/integration/edit/bulk_fixes_test.dart

@@ -36,4 +36,43 @@ class B extends A {
     var result = await sendEditBulkFixes([sourceDirectory.path]);
     expect(result.edits, hasLength(1));
   }
+
+  @FailingTest(issue: 'https://github.com/dart-lang/sdk/issues/59572')
+  Future<void> test_bulk_fix_with_parts() async {
+    writeFile(sourcePath(file_paths.analysisOptionsYaml), '''
+linter:
+  rules:
+    - empty_statements
+    - prefer_const_constructors
+''');
+    writeFile(sourcePath('part.dart'), '''
+part of 'test.dart';
+
+class C {
+  const C();
+}
+
+C b() {
+  // dart fix should only add a single const
+  return C();
+}
+''');
+    writeFile(sourcePath('test.dart'), '''
+part 'part.dart';
+
+void a() {
+  // need to trigger a lint in main.dart for the bug to happen
+  ;
+  b();
+}
+''');
+    await standardAnalysisSetup();
+    await analysisFinished;
+
+    var result = await sendEditBulkFixes([sourceDirectory.path]);
+    var edits = result.edits;
+    expect(edits, hasLength(2));
+    expect(edits[0].edits, hasLength(1));
+    expect(edits[1].edits, hasLength(1));
+  }
 }

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

@@ -146,6 +146,41 @@ var c = new C();
     expect(processor.changeMap.libraryMap.length, 3);
   }
 
+  /// https://github.com/dart-lang/sdk/issues/59572
+  Future<void> test_hasFixes_in_part_and_library2() async {
+    createAnalysisOptionsFile(
+      experiments: experiments,
+      lints: [LintNames.empty_statements, LintNames.prefer_const_constructors],
+    );
+
+    newFile('$testPackageLibPath/part.dart', '''
+part of 'test.dart';
+
+class C {
+  const C();
+}
+
+C b() {
+  // dart fix should only add a single const
+  return C();
+}
+''');
+
+    await resolveTestCode('''
+part 'part.dart';
+
+void a() {
+  // need to trigger a lint in main.dart for the bug to happen
+  ;
+  b();
+}
+''');
+
+    expect(await computeHasFixes(), isTrue);
+    expect(processor.changeMap.libraryMap.length, 2);
+    expect(processor.fixDetails.length, 2);
+  }
+
   Future<void> test_hasFixes_stoppedAfterFirst() async {
     createAnalysisOptionsFile(
       experiments: experiments,

pkg/dartdev/test/commands/fix_test.dart

@@ -403,6 +403,59 @@ linter:
           ]));
     });
 
+    test(
+      '--apply --code=(multiple) [part file]',
+      () async {
+        p = project(
+          mainSrc: '''
+part 'part.dart';
+
+void a() {
+  // need to trigger a lint in main.dart for the bug to happen
+  ;
+  b();
+}
+''',
+          analysisOptions: '''
+linter:
+  rules:
+    - empty_statements
+    - prefer_const_constructors
+''',
+        );
+        p!.file('lib/part.dart', '''
+part of 'main.dart';
+
+Stream<String> b() {
+  // dart fix should only add a single const
+  return Stream.empty();
+}
+''');
+        var result = await p!.runFix([
+          '--apply',
+          '--code',
+          'empty_statements',
+          '--code',
+          'prefer_const_constructors',
+          '.'
+        ], workingDir: p!.dirPath);
+
+        expect(result.exitCode, 0);
+        expect(result.stderr, isEmpty);
+        expect(
+            result.stdout,
+            stringContainsInOrderWithVariableBullets([
+              'Applying fixes...',
+              'lib${Platform.pathSeparator}main.dart',
+              '  empty_statements $bullet 1 fix',
+              'lib${Platform.pathSeparator}part.dart',
+              '  prefer_const_constructors $bullet 1 fix',
+              '2 fixes made in 2 files.',
+            ]));
+      },
+      skip: 'Failing: https://github.com/dart-lang/sdk/issues/59572',
+    );
+
     test('--apply --code=(multiple: comma-delimited)', () async {
       p = project(
         mainSrc: '''