Reformat tests/language/c*.

Change-Id: I94dfcea40566190c6bf153c108b0d0d5dbf27a4d Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/400148 Commit-Queue: Bob Nystrom <[email protected]> Reviewed-by: Lasse Nielsen <[email protected]> Auto-Submit: Bob Nystrom <[email protected]>
dart-lang · Dec 12, 2024 · 6cc9efe · 6cc9efe
1 parent 7190cb0
commit 6cc9efe
Show file tree

Hide file tree

Showing 248 changed files with 42,717 additions and 42,738 deletions.
diff --git a/tests/language/call/function_test.dart b/tests/language/call/function_test.dart
@@ -44,6 +44,8 @@ void main() {
   var s = (FooType).toString();
   var minified = s != 'FooType'; // dart2js --minify has minified names.
   dynamic d = null;
-  Expect.throws(() => d(),
-      (e) => e is NoSuchMethodError && (minified || '$e'.contains('call')));
+  Expect.throws(
+    () => d(),
+    (e) => e is NoSuchMethodError && (minified || '$e'.contains('call')),
+  );
 }
diff --git a/tests/language/call/implicit_tearoff_assignment_test.dart b/tests/language/call/implicit_tearoff_assignment_test.dart
@@ -48,8 +48,9 @@ class Derived extends Base {
   }
 
   void testSuperPropertySet() {
-    Expect.type<void Function()>((super.baseProperty = C())
-      ..expectStaticType<Exactly<void Function()>>());
+    Expect.type<void Function()>(
+      (super.baseProperty = C())..expectStaticType<Exactly<void Function()>>(),
+    );
     Expect.type<void Function()>(super._basePropertySetValue);
   }
 }
@@ -72,56 +73,64 @@ extension on Extended {
 void testExtensionIndexSet() {
   Extended e = Extended();
   Expect.type<void Function()>(
-      (e[0] = C())..expectStaticType<Exactly<void Function()>>());
+    (e[0] = C())..expectStaticType<Exactly<void Function()>>(),
+  );
   Expect.type<void Function()>(e._extensionIndexSetValue);
 }
 
 void testExtensionSet() {
   Extended e = Extended();
-  Expect.type<void Function()>((e.extensionProperty = C())
-    ..expectStaticType<Exactly<void Function()>>());
+  Expect.type<void Function()>(
+    (e.extensionProperty = C())..expectStaticType<Exactly<void Function()>>(),
+  );
   Expect.type<void Function()>(e._extensionPropertySetValue);
 }
 
 void testIndexSet() {
   Derived d = Derived();
   Expect.type<void Function()>(
-      (d[0] = C())..expectStaticType<Exactly<void Function()>>());
+    (d[0] = C())..expectStaticType<Exactly<void Function()>>(),
+  );
   Expect.type<void Function()>(d._indexSetValue);
 }
 
 void testInstanceSet() {
   Derived d = Derived();
   Expect.type<void Function()>(
-      (d.instanceProperty = C())..expectStaticType<Exactly<void Function()>>());
+    (d.instanceProperty = C())..expectStaticType<Exactly<void Function()>>(),
+  );
   Expect.type<void Function()>(d._instanceSetValue);
 }
 
 void testNullAwarePropertySet() {
   Derived? d = Derived() as Derived?; // ignore: unnecessary_cast
-  Expect.type<void Function()>((d?.instanceProperty = C())
-    ..expectStaticType<Exactly<void Function()?>>());
+  Expect.type<void Function()>(
+    (d?.instanceProperty = C())..expectStaticType<Exactly<void Function()?>>(),
+  );
   Expect.type<void Function()>(d!._instanceSetValue);
 }
 
 void testStaticSet() {
   C._staticPropertySetValue = null;
   Expect.type<void Function()>(
-      (C.staticProperty = C())..expectStaticType<Exactly<void Function()>>());
+    (C.staticProperty = C())..expectStaticType<Exactly<void Function()>>(),
+  );
   Expect.type<void Function()>(C._staticPropertySetValue);
 }
 
 void testTopLevelSet() {
   _topLevelPropertySetValue = null;
   Expect.type<void Function()>(
-      (topLevelProperty = C())..expectStaticType<Exactly<void Function()>>());
+    (topLevelProperty = C())..expectStaticType<Exactly<void Function()>>(),
+  );
   Expect.type<void Function()>(_topLevelPropertySetValue);
 }
 
 void testVariableSet() {
   void Function() f;
   Expect.type<void Function()>(
-      (f = C())..expectStaticType<Exactly<void Function()>>());
+    (f = C())..expectStaticType<Exactly<void Function()>>(),
+  );
   Expect.type<void Function()>(f);
 }
 

diff --git a/tests/language/call/implicit_tearoff_exceptions_test.dart b/tests/language/call/implicit_tearoff_exceptions_test.dart
@@ -62,21 +62,39 @@ void testConditional() {
   // `(bTrue ? c : a)..expectStaticType<...>()`, no implicit tearoff of `c`
   // occurs, and the subexpression `bTrue ? c : a` gets assigned a static type
   // of `A`.
-  Expect.throws(() => context<void Function()>(
-      bFalse ? d : ((bTrue ? c : a)..expectStaticType<Exactly<A>>())));
-  Expect.throws(() => context<void Function()>(
-      bFalse ? d : ((bFalse ? a : c)..expectStaticType<Exactly<A>>())));
+  Expect.throws(
+    () => context<void Function()>(
+      bFalse ? d : ((bTrue ? c : a)..expectStaticType<Exactly<A>>()),
+    ),
+  );
+  Expect.throws(
+    () => context<void Function()>(
+      bFalse ? d : ((bFalse ? a : c)..expectStaticType<Exactly<A>>()),
+    ),
+  );
 
   // Same as above, but confirm that extra parens around `c` don't change the
   // behavior.
-  Expect.throws(() => context<void Function()>(
-      bFalse ? d : ((bTrue ? (c) : a)..expectStaticType<Exactly<A>>())));
-  Expect.throws(() => context<void Function()>(
-      bFalse ? d : ((bFalse ? a : (c))..expectStaticType<Exactly<A>>())));
-  Expect.throws(() => context<void Function()>(
-      bFalse ? d : ((bTrue ? ((c)) : a)..expectStaticType<Exactly<A>>())));
-  Expect.throws(() => context<void Function()>(
-      bFalse ? d : ((bFalse ? a : ((c)))..expectStaticType<Exactly<A>>())));
+  Expect.throws(
+    () => context<void Function()>(
+      bFalse ? d : ((bTrue ? (c) : a)..expectStaticType<Exactly<A>>()),
+    ),
+  );
+  Expect.throws(
+    () => context<void Function()>(
+      bFalse ? d : ((bFalse ? a : (c))..expectStaticType<Exactly<A>>()),
+    ),
+  );
+  Expect.throws(
+    () => context<void Function()>(
+      bFalse ? d : ((bTrue ? ((c)) : a)..expectStaticType<Exactly<A>>()),
+    ),
+  );
+  Expect.throws(
+    () => context<void Function()>(
+      bFalse ? d : ((bFalse ? a : ((c)))..expectStaticType<Exactly<A>>()),
+    ),
+  );
 }
 
 void testIfNull() {
@@ -90,30 +108,51 @@ void testIfNull() {
   // appropriate).  So, in
   // `(c ?? a)..expectStaticType<...>()`, no implicit tearoff of `c` occurs, and
   // the subexpression `c ?? a` gets assigned a static type of `A`.
-  Expect.throws(() => context<void Function()>(bFalse
-      ? d
-      : ((c ?? a) // ignore: dead_null_aware_expression
-        ..expectStaticType<Exactly<A>>())));
+  Expect.throws(
+    () => context<void Function()>(
+      bFalse
+          ? d
+          : ((c ?? a) // ignore: dead_null_aware_expression
+            ..expectStaticType<Exactly<A>>()),
+    ),
+  );
 
   // In `(aq ?? c)..expectStaticType<...>()`, the situation is similar, but the
   // context for `c` is (non-nullable) `void Function()`.
-  Expect.throws(() => context<void Function()>(
-      bFalse ? d : ((aq ?? c)..expectStaticType<Exactly<A>>())));
+  Expect.throws(
+    () => context<void Function()>(
+      bFalse ? d : ((aq ?? c)..expectStaticType<Exactly<A>>()),
+    ),
+  );
 
   // Same as above, but confirm that extra parens around `c` don't change the
   // behavior.
-  Expect.throws(() => context<void Function()>(bFalse
-      ? d
-      : (((c) ?? a) // ignore: dead_null_aware_expression
-        ..expectStaticType<Exactly<A>>())));
-  Expect.throws(() => context<void Function()>(
-      bFalse ? d : ((aq ?? (c))..expectStaticType<Exactly<A>>())));
-  Expect.throws(() => context<void Function()>(bFalse
-      ? d
-      : ((((c)) ?? a) // ignore: dead_null_aware_expression
-        ..expectStaticType<Exactly<A>>())));
-  Expect.throws(() => context<void Function()>(
-      bFalse ? d : ((aq ?? ((c)))..expectStaticType<Exactly<A>>())));
+  Expect.throws(
+    () => context<void Function()>(
+      bFalse
+          ? d
+          : (((c) ?? a) // ignore: dead_null_aware_expression
+            ..expectStaticType<Exactly<A>>()),
+    ),
+  );
+  Expect.throws(
+    () => context<void Function()>(
+      bFalse ? d : ((aq ?? (c))..expectStaticType<Exactly<A>>()),
+    ),
+  );
+  Expect.throws(
+    () => context<void Function()>(
+      bFalse
+          ? d
+          : ((((c)) ?? a) // ignore: dead_null_aware_expression
+            ..expectStaticType<Exactly<A>>()),
+    ),
+  );
+  Expect.throws(
+    () => context<void Function()>(
+      bFalse ? d : ((aq ?? ((c)))..expectStaticType<Exactly<A>>()),
+    ),
+  );
 }
 
 main() {

diff --git a/tests/language/call/implicit_tearoff_type_variable_error_test.dart b/tests/language/call/implicit_tearoff_type_variable_error_test.dart
@@ -36,7 +36,8 @@ void testTypeVariablePromotedToNullableCallableClass<T>(T t) {
 }
 
 void testTypeVariableExtendsNullableTypeVariable<T extends U?, U extends C>(
-    T t) {
+  T t,
+) {
   context<void Function()>(t);
   //                       ^
   // [analyzer] COMPILE_TIME_ERROR.ARGUMENT_TYPE_NOT_ASSIGNABLE

diff --git a/tests/language/call/method_implicit_invoke_local_legacy_test.dart b/tests/language/call/method_implicit_invoke_local_legacy_test.dart
@@ -32,13 +32,13 @@ main() {
   Expect.equals(d2(1), 2);
   // Cannot invoke with the wrong signature.
   c2();
+  //^
+  // [cfe] Too few positional arguments: 1 required, 0 given.
   // ^
   // [analyzer] COMPILE_TIME_ERROR.NOT_ENOUGH_POSITIONAL_ARGUMENTS
-  //^^
-  // [cfe] Too few positional arguments: 1 required, 0 given.
   c2(3, 4);
+  //^
+  // [cfe] Too many positional arguments: 1 allowed, but 2 found.
   //    ^
   // [analyzer] COMPILE_TIME_ERROR.EXTRA_POSITIONAL_ARGUMENTS
-  //^^^^^^
-  // [cfe] Too many positional arguments: 1 allowed, but 2 found.
 }
diff --git a/tests/language/call/method_implicit_invoke_local_runtime_1_legacy_test.dart b/tests/language/call/method_implicit_invoke_local_runtime_1_legacy_test.dart
@@ -34,6 +34,4 @@ main() {
   // Implicitly invokes d2.call(1)
 
   // Cannot invoke with the wrong signature.
-
-
 }
diff --git a/tests/language/call/method_implicit_invoke_local_runtime_1_test.dart b/tests/language/call/method_implicit_invoke_local_runtime_1_test.dart
@@ -31,6 +31,4 @@ main() {
   // Implicitly invokes d2.call(1)
 
   // Cannot invoke with the wrong signature.
-
-
 }
diff --git a/tests/language/call/method_implicit_invoke_local_runtime_2_legacy_test.dart b/tests/language/call/method_implicit_invoke_local_runtime_2_legacy_test.dart
@@ -34,6 +34,4 @@ main() {
   // Implicitly invokes d2.call(1)
 
   // Cannot invoke with the wrong signature.
-
-
 }
diff --git a/tests/language/call/method_implicit_invoke_local_runtime_2_test.dart b/tests/language/call/method_implicit_invoke_local_runtime_2_test.dart
@@ -31,6 +31,4 @@ main() {
   // Implicitly invokes d2.call(1)
 
   // Cannot invoke with the wrong signature.
-
-
 }
diff --git a/tests/language/call/method_implicit_invoke_local_runtime_3_legacy_test.dart b/tests/language/call/method_implicit_invoke_local_runtime_3_legacy_test.dart
@@ -34,6 +34,4 @@ main() {
   // Implicitly invokes d2.call(1)
 
   // Cannot invoke with the wrong signature.
-
-
 }
diff --git a/tests/language/call/method_implicit_invoke_local_runtime_3_test.dart b/tests/language/call/method_implicit_invoke_local_runtime_3_test.dart
@@ -31,6 +31,4 @@ main() {
   // Implicitly invokes d2.call(1)
 
   // Cannot invoke with the wrong signature.
-
-
 }
diff --git a/tests/language/call/method_implicit_invoke_local_runtime_4_legacy_test.dart b/tests/language/call/method_implicit_invoke_local_runtime_4_legacy_test.dart
@@ -34,6 +34,4 @@ main() {
   // Implicitly invokes d2.call(1)
   Expect.equals(d2(1), 2);
   // Cannot invoke with the wrong signature.
-
-
 }
diff --git a/tests/language/call/method_implicit_invoke_local_runtime_4_test.dart b/tests/language/call/method_implicit_invoke_local_runtime_4_test.dart
@@ -31,6 +31,4 @@ main() {
   // Implicitly invokes d2.call(1)
   Expect.equals(d2(1), 2);
   // Cannot invoke with the wrong signature.
-
-
 }
diff --git a/tests/language/call/method_implicit_invoke_local_runtime_legacy_test.dart b/tests/language/call/method_implicit_invoke_local_runtime_legacy_test.dart
@@ -34,6 +34,4 @@ main() {
   // Implicitly invokes d2.call(1)
 
   // Cannot invoke with the wrong signature.
-
-
 }
diff --git a/tests/language/call/method_implicit_invoke_local_runtime_test.dart b/tests/language/call/method_implicit_invoke_local_runtime_test.dart
@@ -31,6 +31,4 @@ main() {
   // Implicitly invokes d2.call(1)
 
   // Cannot invoke with the wrong signature.
-
-
 }
diff --git a/tests/language/call/method_implicit_invoke_local_test.dart b/tests/language/call/method_implicit_invoke_local_test.dart
@@ -19,13 +19,13 @@ main() {
   Expect.equals(d1(1), 2);
   // Cannot invoke with the wrong signature.
   c1();
+  //^
+  // [cfe] Too few positional arguments: 1 required, 0 given.
   // ^
   // [analyzer] COMPILE_TIME_ERROR.NOT_ENOUGH_POSITIONAL_ARGUMENTS
-  //^^
-  // [cfe] Too few positional arguments: 1 required, 0 given.
   c1(3, 4);
+  //^
+  // [cfe] Too many positional arguments: 1 allowed, but 2 found.
   //    ^
   // [analyzer] COMPILE_TIME_ERROR.EXTRA_POSITIONAL_ARGUMENTS
-  //^^^^^^
-  // [cfe] Too many positional arguments: 1 allowed, but 2 found.
 }
diff --git a/tests/language/call/method_implicit_tear_off_assignable_test.dart b/tests/language/call/method_implicit_tear_off_assignable_test.dart
@@ -16,7 +16,8 @@ main() {
   // supertype of the target type. Since we no longer allow implicit downcasts,
   // this is a static error.
   void Function([int]) f = i;
+  //                   ^
+  // [cfe] A value of type 'void Function()' can't be assigned to a variable of type 'void Function([int])'.
   //                       ^
   // [analyzer] COMPILE_TIME_ERROR.INVALID_ASSIGNMENT
-  // [cfe] unspecified
 }
diff --git a/tests/language/call/method_implicit_tear_off_nullable_test.dart b/tests/language/call/method_implicit_tear_off_nullable_test.dart
@@ -20,5 +20,5 @@ main() {
   check(c);
   //    ^
   // [analyzer] COMPILE_TIME_ERROR.ARGUMENT_TYPE_NOT_ASSIGNABLE
-  // [cfe] unspecified
+  // [cfe] Can't tear off method 'call' from a potentially null value.
 }
diff --git a/tests/language/call/non_method_field_error_test.dart b/tests/language/call/non_method_field_error_test.dart
@@ -13,15 +13,15 @@ class Hest extends Fisk {}
 main() {
   Fisk x1 = new Fisk();
   x1.i();
-//^^^^
-// [analyzer] COMPILE_TIME_ERROR.INVOCATION_OF_NON_FUNCTION_EXPRESSION
-//    ^
-// [cfe] 'i' isn't a function or method and can't be invoked.
+  // [error column 3, length 4]
+  // [analyzer] COMPILE_TIME_ERROR.INVOCATION_OF_NON_FUNCTION_EXPRESSION
+  //  ^
+  // [cfe] 'i' isn't a function or method and can't be invoked.
 
   Hest x2 = new Hest();
   x2.i();
-//^^^^
-// [analyzer] COMPILE_TIME_ERROR.INVOCATION_OF_NON_FUNCTION_EXPRESSION
-//    ^
-// [cfe] 'i' isn't a function or method and can't be invoked.
+  // [error column 3, length 4]
+  // [analyzer] COMPILE_TIME_ERROR.INVOCATION_OF_NON_FUNCTION_EXPRESSION
+  //  ^
+  // [cfe] 'i' isn't a function or method and can't be invoked.
 }
diff --git a/tests/language/call/object_has_no_call_method_error_test.dart b/tests/language/call/object_has_no_call_method_error_test.dart
@@ -5,10 +5,10 @@
 void test(dynamic d, Object o, Function f) {
   d();
   o();
-//^
-// [analyzer] COMPILE_TIME_ERROR.INVOCATION_OF_NON_FUNCTION_EXPRESSION
-// ^
-// [cfe] The method 'call' isn't defined for the class 'Object'.
+  // [error column 3, length 1]
+  // [analyzer] COMPILE_TIME_ERROR.INVOCATION_OF_NON_FUNCTION_EXPRESSION
+  // [error column 4]
+  // [cfe] The method 'call' isn't defined for the class 'Object'.
   f();
   d.call;
   o.call;
-Original file line number
+Diff line change
@@ Expand Up @@
     }
     void testTypeVariableExtendsNullableTypeVariable<T extends U?, U extends C>(
-        T t) {
+      T t,
+    ) {
       context<void Function()>(t);
       //                       ^
       // [analyzer] COMPILE_TIME_ERROR.ARGUMENT_TYPE_NOT_ASSIGNABLE
@@ Expand Down @@
Original file line number	Diff line number	Diff line change
Expand Up		@@ -34,6 +34,4 @@ main() {
		// Implicitly invokes d2.call(1)

		// Cannot invoke with the wrong signature.


		}
Original file line number	Diff line number	Diff line change
Expand Up		@@ -31,6 +31,4 @@ main() {
		// Implicitly invokes d2.call(1)

		// Cannot invoke with the wrong signature.


		}