Reformat tests/language/abstract.

Change-Id: I18bbc65a798e79fb5e69a9a4134c10a9f6cdbb3f
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/399860
Auto-Submit: Bob Nystrom <[email protected]>
Reviewed-by: Lasse Nielsen <[email protected]>
Commit-Queue: Lasse Nielsen <[email protected]>

Author: munificent

tests/language/abstract/factory_constructor_runtime_test.dart

@@ -1,6 +1,3 @@
-// TODO(multitest): This was automatically migrated from a multitest and may
-// contain strange or dead code.
-
 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
@@ -22,13 +19,6 @@ abstract class A1 {
   }
 }
 
-class A2 {
-  // Intentionally abstract method.
-
-  A2.make() {}
-}
-
 main() {
   new A1.make();
-
 }

tests/language/abstract/factory_constructor_test.dart

@@ -20,13 +20,13 @@ abstract class A1 {
 }
 
 class A2 {
-//    ^
-// [cfe] The non-abstract class 'A2' is missing implementations for these members:
+  //  ^
+  // [cfe] The non-abstract class 'A2' is missing implementations for these members:
 
   // Intentionally abstract method.
   method();
-//^^^^^^^^^
-// [analyzer] COMPILE_TIME_ERROR.CONCRETE_CLASS_WITH_ABSTRACT_MEMBER
+  // [error column 3, length 9]
+  // [analyzer] COMPILE_TIME_ERROR.CONCRETE_CLASS_WITH_ABSTRACT_MEMBER
   A2.make() {}
 }
 

tests/language/abstract/getter2_test.dart

@@ -52,15 +52,17 @@ void main() {
   /// Tests that overriding either the getter or setter with an abstract member
   /// has no effect.
   /// Regression test for https://github.com/dart-lang/sdk/issues/29914
-  var c1 = AbstractGetterOverride1()
-    ..foo = 123
-    ..bar = 456;
+  var c1 =
+      AbstractGetterOverride1()
+        ..foo = 123
+        ..bar = 456;
   Expect.equals(c1.foo, 123);
   Expect.equals(c1.bar, 456);
 
-  var c2 = AbstractGetterOverride2()
-    ..foo = 123
-    ..bar = 456;
+  var c2 =
+      AbstractGetterOverride2()
+        ..foo = 123
+        ..bar = 456;
   Expect.equals(c2.foo, 123);
   Expect.equals(c2.bar, 456);
 }

tests/language/abstract/getter_test.dart renamed to tests/language/abstract/getter_error_test.dart

@@ -7,17 +7,15 @@ import "package:expect/expect.dart";
 // Test to ensure that an abstract getter is not mistaken for a field.
 
 class Foo {
-//    ^
-// [cfe] The non-abstract class 'Foo' is missing implementations for these members:
+  //  ^
+  // [cfe] The non-abstract class 'Foo' is missing implementations for these members:
 
   // Intentionally abstract:
   get i;
-//^^^^^^
-// [analyzer] COMPILE_TIME_ERROR.CONCRETE_CLASS_WITH_ABSTRACT_MEMBER
+  // [error column 3, length 6]
+  // [analyzer] COMPILE_TIME_ERROR.CONCRETE_CLASS_WITH_ABSTRACT_MEMBER
 }
 
-class Bar {}
-
 checkIt(f) {
   Expect.throwsNoSuchMethodError(() => f.i = 'hi');
   Expect.throwsNoSuchMethodError(() => print(f.i));
@@ -26,5 +24,4 @@ checkIt(f) {
 
 main() {
   checkIt(new Foo());
-  checkIt(new Bar());
 }

tests/language/abstract/override_adds_optional_args_concrete_test.dart renamed to tests/language/abstract/override_adds_optional_args_concrete_error_test.dart

@@ -7,9 +7,9 @@ class A {
 }
 
 class B extends A {
-//    ^
-// [analyzer] COMPILE_TIME_ERROR.INVALID_IMPLEMENTATION_OVERRIDE
-// [cfe] The implementation of 'foo' in the non-abstract class 'B' does not conform to its interface.
+  //  ^
+  // [analyzer] COMPILE_TIME_ERROR.INVALID_IMPLEMENTATION_OVERRIDE
+  // [cfe] The implementation of 'foo' in the non-abstract class 'B' does not conform to its interface.
 
   // This class declaration violates soundness, since it allows `new
   // B().foo(42)`, which would lead to invalid arguments being passed to A.foo.

tests/language/abstract/override_adds_optional_args_concrete_subclass_test.dart renamed to tests/language/abstract/override_adds_optional_args_concrete_subclass_error_test.dart

@@ -15,12 +15,9 @@ abstract class B extends A {
 }
 
 class C extends B {
-//    ^
-// [analyzer] COMPILE_TIME_ERROR.INVALID_IMPLEMENTATION_OVERRIDE
-// [cfe] The implementation of 'foo' in the non-abstract class 'C' does not conform to its interface.
-
-  // However, there is a problem here because this class is concrete and doesn't
-  // override foo.
+  //  ^
+  // [analyzer] COMPILE_TIME_ERROR.INVALID_IMPLEMENTATION_OVERRIDE
+  // [cfe] The implementation of 'foo' in the non-abstract class 'C' does not conform to its interface.
 }
 
 void f(B b) {