commit da375d6cd56b467d90b3395b758a9d4ab1a72ea0
author Konstantin Shcheglov <scheglov@google.com> Sat Aug 03 00:06:28 2024 +0000
committer Commit Queue <dart-scoped@luci-project-accounts.iam.gserviceaccount.com> Sat Aug 03 00:06:28 2024 +0000
tree a90fa42f041d3bd05183288867629283fcae1f64
parent 488b63e2b3bb64fc1168dd4432e3c38530085eb5

CQ. Work around updating reused elements text in MacroIncrementalTest.

Change-Id: Ic8cc02e38c24012ee738099962bbde374af26579
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/378765
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Konstantin Shcheglov <scheglov@google.com>

pkg/analyzer/test/src/summary/macro_test.dart

diff --git a/pkg/analyzer/test/src/summary/macro_test.dart b/pkg/analyzer/test/src/summary/macro_test.dart
index c1b2c52..bc2c16f 100644
--- a/pkg/analyzer/test/src/summary/macro_test.dart
+++ b/pkg/analyzer/test/src/summary/macro_test.dart
@@ -12168,7 +12168,11 @@
 class X {}
 ''');
 
-    var expectedLibraryText = r'''
+    void assertElementText(LibraryElementImpl element) {
+      configuration
+        ..withConstructors = false
+        ..withMetadata = false;
+      checkElementText(element, r'''
 library
   reference: <testLibrary>
   libraryImports
@@ -12223,12 +12227,10 @@
               reference: <testLibrary>::@fragment::package:test/test.macro.dart::@classAugmentation::X::@method::foo
               enclosingElement: <testLibrary>::@fragment::package:test/test.macro.dart::@classAugmentation::X
               returnType: void
-''';
+''');
+    }
 
-    configuration
-      ..withConstructors = false
-      ..withMetadata = false;
-    checkElementText(library1, expectedLibraryText);
+    assertElementText(library1);
 
     // Generated and put into the cache.
     _assertMacroCachedGenerated(testFile, r'''
@@ -12245,7 +12247,7 @@
     await contextFor(testFile).applyPendingFileChanges();
 
     var library2 = await buildFileLibrary(testFile);
-    checkElementText(library2, expectedLibraryText);
+    assertElementText(library2);
 
     // Used cached, no new generated.
     _assertMacroCachedGenerated(testFile, r'''
@@ -12270,7 +12272,11 @@
 class X {}
 ''');
 
-    var expectedLibraryText = r'''
+    void assertElementText(LibraryElementImpl element) {
+      configuration
+        ..withConstructors = false
+        ..withMetadata = false;
+      checkElementText(element, r'''
 library
   reference: <testLibrary>
   libraryImports
@@ -12345,12 +12351,10 @@
               reference: <testLibrary>::@fragment::package:test/test.macro.dart::@classAugmentation::X::@method::foo
               enclosingElement: <testLibrary>::@fragment::package:test/test.macro.dart::@classAugmentation::X
               returnType: A
-''';
+''');
+    }
 
-    configuration
-      ..withConstructors = false
-      ..withMetadata = false;
-    checkElementText(library1, expectedLibraryText);
+    assertElementText(library1);
 
     // Generated and put into the cache.
     _assertMacroCachedGenerated(testFile, r'''
@@ -12368,7 +12372,7 @@
     await contextFor(testFile).applyPendingFileChanges();
 
     var library2 = await buildFileLibrary(testFile);
-    checkElementText(library2, expectedLibraryText);
+    assertElementText(library2);
 
     // Cannot prove that `package:test/test.dart@A` is still there.
     // So, not used cached.