Reformat tests/dartdevc.

Change-Id: I043b7df217e5a479fa15375458bc643e2b38f3d0
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/399629
Reviewed-by: Srujan Gaddam <[email protected]>
Auto-Submit: Bob Nystrom <[email protected]>
Commit-Queue: Srujan Gaddam <[email protected]>

Author: munificent

tests/dartdevc/cast_error/library_uri_in_message_test.dart

@@ -15,13 +15,15 @@ void main() {
   } on TypeError catch (error) {
     var message = error.toString();
     expectStringContains(
-        "Expected a value of type 'Animal' "
-        "(in org-dartlang-app:/tests/dartdevc/cast_error/lib_b.dart)",
-        message);
+      "Expected a value of type 'Animal' "
+      "(in org-dartlang-app:/tests/dartdevc/cast_error/lib_b.dart)",
+      message,
+    );
     expectStringContains(
-        "but got one of type 'Animal' "
-        "(in org-dartlang-app:/tests/dartdevc/cast_error/lib_a.dart)",
-        message);
+      "but got one of type 'Animal' "
+      "(in org-dartlang-app:/tests/dartdevc/cast_error/lib_a.dart)",
+      message,
+    );
   }
   // Verify the libraries are properly ordered.
   var b = libB.Animal();
@@ -30,13 +32,15 @@ void main() {
   } on TypeError catch (error) {
     var message = error.toString();
     expectStringContains(
-        "Expected a value of type 'Animal' "
-        "(in org-dartlang-app:/tests/dartdevc/cast_error/lib_a.dart)",
-        message);
+      "Expected a value of type 'Animal' "
+      "(in org-dartlang-app:/tests/dartdevc/cast_error/lib_a.dart)",
+      message,
+    );
     expectStringContains(
-        "but got one of type 'Animal' "
-        "(in org-dartlang-app:/tests/dartdevc/cast_error/lib_b.dart)",
-        message);
+      "but got one of type 'Animal' "
+      "(in org-dartlang-app:/tests/dartdevc/cast_error/lib_b.dart)",
+      message,
+    );
   }
 
   // Shows library URIs when one of the types is nullable.
@@ -45,13 +49,15 @@ void main() {
   } on TypeError catch (error) {
     var message = error.toString();
     expectStringContains(
-        "Expected a value of type 'Animal?' "
-        "(in org-dartlang-app:/tests/dartdevc/cast_error/lib_a.dart)",
-        message);
+      "Expected a value of type 'Animal?' "
+      "(in org-dartlang-app:/tests/dartdevc/cast_error/lib_a.dart)",
+      message,
+    );
     expectStringContains(
-        "but got one of type 'Animal' "
-        "(in org-dartlang-app:/tests/dartdevc/cast_error/lib_b.dart)",
-        message);
+      "but got one of type 'Animal' "
+      "(in org-dartlang-app:/tests/dartdevc/cast_error/lib_b.dart)",
+      message,
+    );
   }
 
   // URIs are not displayed when the class names are different.
@@ -60,7 +66,8 @@ void main() {
   } on TypeError catch (error) {
     var message = error.toString();
     expectStringContains(
-        "Expected a value of type 'String', but got one of type 'Animal'",
-        message);
+      "Expected a value of type 'String', but got one of type 'Animal'",
+      message,
+    );
   }
 }

tests/dartdevc/debugger/debugger_test.dart

@@ -155,7 +155,8 @@ JSObject getCurrentLibrary() =>
     // We can't use interop for this, as the lowering would be emitted as
     // `dart.global.eval('this')`, which does not evaluate to the same value as
     // `eval('this')`.
-    _foreign_helper.JS('', 'this');
+    _foreign_helper
+    .JS('', 'this');
 
 main() async {
   asyncStart();
@@ -167,7 +168,8 @@ main() async {
   // Cache blocker is a workaround for:
   // https://code.google.com/p/dart/issues/detail?id=11834
   var cacheBlocker = new DateTime.now().millisecondsSinceEpoch;
-  var goldenUrl = '/root_dart/tests/dartdevc/debugger/'
+  var goldenUrl =
+      '/root_dart/tests/dartdevc/debugger/'
       'debugger_test_golden.txt?cacheBlock=$cacheBlocker';
 
   String? golden;
@@ -177,17 +179,20 @@ main() async {
     print("Warning: couldn't load golden file from $goldenUrl");
   }
 
-  document.body!.append(new ScriptElement()
-    ..type = 'text/javascript'
-    ..innerHtml = r"""
+  document.body!.append(
+    new ScriptElement()
+      ..type = 'text/javascript'
+      ..innerHtml = r"""
 window.PackageJSClass = function PackageJSClass(x) {
   this.x = x;
 };
-""");
+""",
+  );
 
-  var _devtoolsFormatter = (devtoolsFormatters![0]
-          as ExternalDartReference<_debugger.JsonMLFormatter>)
-      .toDartObject;
+  var _devtoolsFormatter =
+      (devtoolsFormatters![0]
+              as ExternalDartReference<_debugger.JsonMLFormatter>)
+          .toDartObject;
 
   var actual = new StringBuffer();
 
@@ -200,15 +205,19 @@ window.PackageJSClass = function PackageJSClass(x) {
   // to expected output.
   void addGolden(String name, JSAny value) {
     var text = format(value);
-    actual.write('Test: $name\n'
-        'Value:\n'
-        '$text\n'
-        '-----------------------------------\n');
+    actual.write(
+      'Test: $name\n'
+      'Value:\n'
+      '$text\n'
+      '-----------------------------------\n',
+    );
   }
 
   void addFormatterGoldens(String name, Object? object, [Object? config]) {
     addGolden(
-        '$name formatting header', _devtoolsFormatter.header(object, config));
+      '$name formatting header',
+      _devtoolsFormatter.header(object, config),
+    );
     addGolden('$name formatting body', _devtoolsFormatter.body(object, config));
   }
 
@@ -258,10 +267,11 @@ window.PackageJSClass = function PackageJSClass(x) {
 
     addNestedFormatterGoldens('Iterable', iterable);
 
-    var s = new Set()
-      ..add("foo")
-      ..add(42)
-      ..add(true);
+    var s =
+        new Set()
+          ..add("foo")
+          ..add(42)
+          ..add(true);
     addNestedFormatterGoldens('Set', s);
   });
 
@@ -332,13 +342,14 @@ window.PackageJSClass = function PackageJSClass(x) {
       lib['link'] = () {}.toJS;
     }
     addFormatterGoldens(
-        'Test library',
-        // TODO(srujzs): We have to construct a `Library` manually here,
-        // whereas the `LibraryModuleFormatter` does that for us automatically
-        // when we format the module. We should add properties to libraries so
-        // that we can detect them as a library. Once we have support for that,
-        // revisit this and the formatter code.
-        _debugger.Library('debugger_test', lib));
+      'Test library',
+      // TODO(srujzs): We have to construct a `Library` manually here,
+      // whereas the `LibraryModuleFormatter` does that for us automatically
+      // when we format the module. We should add properties to libraries so
+      // that we can detect them as a library. Once we have support for that,
+      // revisit this and the formatter code.
+      _debugger.Library('debugger_test', lib),
+    );
   });
 
   group('StackTrace formatting', () {
@@ -364,11 +375,15 @@ window.PackageJSClass = function PackageJSClass(x) {
 
   group('Generics formatting', () {
     addNestedFormatterGoldens(
-        'TestGenericClass', new TestGenericClass<int, List>(42));
+      'TestGenericClass',
+      new TestGenericClass<int, List>(42),
+    );
     addNestedFormatterGoldens(
-        'TestGenericClassJSInterop',
-        new TestGenericClass<PackageJSClass<JSString>, int>(
-            new PackageJSClass("Hello".toJS)));
+      'TestGenericClassJSInterop',
+      new TestGenericClass<PackageJSClass<JSString>, int>(
+        new PackageJSClass("Hello".toJS),
+      ),
+    );
   });
 
   test('verify golden match', () {
@@ -394,8 +409,9 @@ window.PackageJSClass = function PackageJSClass(x) {
       textField.style
         ..width = '800px'
         ..height = '400px';
-      document.body!.append(new Element.tag('h3')
-        ..innerHtml = helpMessage.replaceAll('\n', '<br>'));
+      document.body!.append(
+        new Element.tag('h3')..innerHtml = helpMessage.replaceAll('\n', '<br>'),
+      );
       document.body!.append(textField);
       document.body!.onClick.listen((_) {
         textField.select();

tests/dartdevc/developer_events_test.dart

@@ -83,8 +83,9 @@ void testRegisterExtensionWarningMessage() {
 
     consoleWarnLog.clear();
 
-    var testHandler = (String s, Map<String, String> m) async =>
-        ServiceExtensionResponse.result('test result');
+    var testHandler =
+        (String s, Map<String, String> m) async =>
+            ServiceExtensionResponse.result('test result');
 
     expect(consoleWarnLog.isEmpty, true);
 
@@ -93,9 +94,9 @@ void testRegisterExtensionWarningMessage() {
     // A warning message was issued about calling `registerExtension()` from
     // dart:developer.
     expect(
-        consoleWarnLog.single
-            .contains('registerExtension() from dart:developer'),
-        true);
+      consoleWarnLog.single.contains('registerExtension() from dart:developer'),
+      true,
+    );
 
     registerExtension('ext.method1', testHandler);
     registerExtension('ext.method2', testHandler);
@@ -169,8 +170,9 @@ void testRegisterExtension() {
     dwdsVersion = '1.0.0-for-test';
     expect(registerEventLog.isEmpty, true);
 
-    var testHandler = (String s, Map<String, String> m) async =>
-        ServiceExtensionResponse.result('test result');
+    var testHandler =
+        (String s, Map<String, String> m) async =>
+            ServiceExtensionResponse.result('test result');
     registerExtension('ext.method0', testHandler);
 
     expect(registerEventLog.single, 'ext.method0');

tests/dartdevc/js_interop_test.dart

@@ -163,10 +163,11 @@ void main() {
 
   // Assignment to instance field.
   _someClass = helper.JS(
-      '',
-      '{"jsInstanceMethod": function(f) {return f();}, '
-          '"jsNonFunctionField": {"stringField":"hello js"}, '
-          '"jsFunctionFieldGetter": function(f) {return f();}}');
+    '',
+    '{"jsInstanceMethod": function(f) {return f();}, '
+        '"jsNonFunctionField": {"stringField":"hello js"}, '
+        '"jsFunctionFieldGetter": function(f) {return f();}}',
+  );
   Expect.throws((() {
     someClass.jsFunctionFieldSetter = () => 'hello';
   }));
@@ -328,23 +329,30 @@ void main() {
   method(wrappedDartStaticMethod);
 
   // Non-function fields
-  Expect.equals('hello js', someClass.jsNonFunctionField.stringField,
-      'Does not wrap access to a field');
+  Expect.equals(
+    'hello js',
+    someClass.jsNonFunctionField.stringField,
+    'Does not wrap access to a field',
+  );
 
   // No such method errors from interop calls.
   // The current behavior is that DDC does not treat these errors from the
   // JavaScript side as a LegacyJavaScriptObject.
   Expect.throwsNoSuchMethodError(
-      () => context.callMethod('eval', ['self.foo()']));
+    () => context.callMethod('eval', ['self.foo()']),
+  );
   Expect.throws<interceptors.JSNoSuchMethodError>(
-      () => context.callMethod('eval', ['self.foo()']));
+    () => context.callMethod('eval', ['self.foo()']),
+  );
   var error = Expect.throws(() => context.callMethod('eval', ['self.foo()']));
   Expect.notType<interceptors.LegacyJavaScriptObject>(error);
   Expect.notEquals(interceptors.LegacyJavaScriptObject, error.runtimeType);
   Expect.throwsNoSuchMethodError(
-      () => context.callMethod('eval', ['self.foo.bar()']));
+    () => context.callMethod('eval', ['self.foo.bar()']),
+  );
   Expect.throws<interceptors.JSNoSuchMethodError>(
-      () => context.callMethod('eval', ['self.foo.bar()']));
+    () => context.callMethod('eval', ['self.foo.bar()']),
+  );
   error = Expect.throws(() => context.callMethod('eval', ['self.foo.bar()']));
   Expect.notType<interceptors.LegacyJavaScriptObject>(error);
   Expect.notEquals(interceptors.LegacyJavaScriptObject, error.runtimeType);