Opt multitests out of formatting in standalone/.
The new formatter supports opting a region of code out from being
formatted. I'm applying this marker to all of the multitests since
those tests are often very sensitive to formatting and easily broken.
This way, anyone touching a multitest (including me when I reformat
the tests) doesn't have to remember to not run the formatter on it.
Change-Id: I06928e08530ad658affd23e594702cff658d8035
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/396181
Commit-Queue: Bob Nystrom <rnystrom@google.com>
Commit-Queue: Brian Quinlan <bquinlan@google.com>
Auto-Submit: Bob Nystrom <rnystrom@google.com>
Reviewed-by: Brian Quinlan <bquinlan@google.com>
diff --git a/tests/standalone/deny_listed_test.dart b/tests/standalone/deny_listed_test.dart
index e802c6d..51bccec 100644
--- a/tests/standalone/deny_listed_test.dart
+++ b/tests/standalone/deny_listed_test.dart
@@ -3,6 +3,9 @@
// BSD-style license that can be found in the LICENSE file.
// Dart test checking that static/instance field shadowing do not conflict.
+// Formatting can break multitests, so don't format them.
+// dart format off
+
// Test that certain interfaces/classes are denylisted from being
// implemented or extended (VM corelib only).
diff --git a/tests/standalone/io/dart_std_io_pipe_test.dart b/tests/standalone/io/dart_std_io_pipe_test.dart
index b1ab075..d27b880 100644
--- a/tests/standalone/io/dart_std_io_pipe_test.dart
+++ b/tests/standalone/io/dart_std_io_pipe_test.dart
@@ -10,6 +10,9 @@
// VMOptions=--short_socket_write
// VMOptions=--short_socket_read --short_socket_write
+// Formatting can break multitests, so don't format them.
+// dart format off
+
import "package:expect/expect.dart";
import "dart:io";
import "process_test_util.dart";
diff --git a/tests/standalone/io/platform_resolved_executable_test.dart b/tests/standalone/io/platform_resolved_executable_test.dart
index b6e6fc6..958742a 100644
--- a/tests/standalone/io/platform_resolved_executable_test.dart
+++ b/tests/standalone/io/platform_resolved_executable_test.dart
@@ -4,6 +4,9 @@
//
// Process test program to test process communication.
+// Formatting can break multitests, so don't format them.
+// dart format off
+
library PlatformExecutableTest;
import "dart:io";
diff --git a/tests/standalone/io/signals_exception_test.dart b/tests/standalone/io/signals_exception_test.dart
index 28f552e..d9f8717 100644
--- a/tests/standalone/io/signals_exception_test.dart
+++ b/tests/standalone/io/signals_exception_test.dart
@@ -2,6 +2,9 @@
// 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.
+// Formatting can break multitests, so don't format them.
+// dart format off
+
import "dart:io";
import "package:expect/expect.dart";
diff --git a/tests/standalone/io/socket_ipv6_test.dart b/tests/standalone/io/socket_ipv6_test.dart
index 3262ec7..4335750 100644
--- a/tests/standalone/io/socket_ipv6_test.dart
+++ b/tests/standalone/io/socket_ipv6_test.dart
@@ -2,6 +2,9 @@
// 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.
+// Formatting can break multitests, so don't format them.
+// dart format off
+
import 'dart:io';
import 'dart:async';
diff --git a/tests/standalone/package/invalid_uri_test.dart b/tests/standalone/package/invalid_uri_test.dart
index c081860..95d2542 100644
--- a/tests/standalone/package/invalid_uri_test.dart
+++ b/tests/standalone/package/invalid_uri_test.dart
@@ -2,6 +2,9 @@
// 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.
+// Formatting can break multitests, so don't format them.
+// dart format off
+
library invalid_uri_test;
// Should not contain "//".
diff --git a/tests/standalone/package/scenarios/empty_packages_file/empty_packages_file_discovery_test.dart b/tests/standalone/package/scenarios/empty_packages_file/empty_packages_file_discovery_test.dart
index 898a519..3900a87 100644
--- a/tests/standalone/package/scenarios/empty_packages_file/empty_packages_file_discovery_test.dart
+++ b/tests/standalone/package/scenarios/empty_packages_file/empty_packages_file_discovery_test.dart
@@ -2,6 +2,9 @@
// 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.
+// Formatting can break multitests, so don't format them.
+// dart format off
+
// Packages=none
library empty_packages_file_discovery_test;
diff --git a/tests/standalone/package/scenarios/empty_packages_file/empty_packages_file_option_test.dart b/tests/standalone/package/scenarios/empty_packages_file/empty_packages_file_option_test.dart
index 93a1612..1490166 100644
--- a/tests/standalone/package/scenarios/empty_packages_file/empty_packages_file_option_test.dart
+++ b/tests/standalone/package/scenarios/empty_packages_file/empty_packages_file_option_test.dart
@@ -2,6 +2,9 @@
// 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.
+// Formatting can break multitests, so don't format them.
+// dart format off
+
// Packages=.dart_tool/package_config.json
library empty_packages_file_option_test;
diff --git a/tests/standalone/package/scenarios/packages_file_strange_formatting/empty_package_dir_test.dart b/tests/standalone/package/scenarios/packages_file_strange_formatting/empty_package_dir_test.dart
index b908cbf..0dd8b09 100644
--- a/tests/standalone/package/scenarios/packages_file_strange_formatting/empty_package_dir_test.dart
+++ b/tests/standalone/package/scenarios/packages_file_strange_formatting/empty_package_dir_test.dart
@@ -2,6 +2,9 @@
// 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.
+// Formatting can break multitests, so don't format them.
+// dart format off
+
// Packages=empty_package_dir.packages
// This test verifies handling of legacy .packages file and cannot be run in
diff --git a/tests/standalone/regress_29350_test.dart b/tests/standalone/regress_29350_test.dart
index fe3b534..83fb8e0 100644
--- a/tests/standalone/regress_29350_test.dart
+++ b/tests/standalone/regress_29350_test.dart
@@ -4,6 +4,9 @@
* BSD-style license that can be found in the LICENSE file.
*/
+// Formatting can break multitests, so don't format them.
+// dart format off
+
// 'covariant' when used incorrectly should report an error and not crash
// the VM.
diff --git a/tests/standalone/regress_52715_test.dart b/tests/standalone/regress_52715_test.dart
index 7ac3aa6..f797356 100644
--- a/tests/standalone/regress_52715_test.dart
+++ b/tests/standalone/regress_52715_test.dart
@@ -2,6 +2,9 @@
// 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.
+// Formatting can break multitests, so don't format them.
+// dart format off
+
// Verify that cancelled [Directory.watch] subscriptions do not waste memory.
import 'dart:io';
diff --git a/tests/standalone/stack_trace/rethrow_error_test.dart b/tests/standalone/stack_trace/rethrow_error_test.dart
index a32b7d1..0229d8a 100644
--- a/tests/standalone/stack_trace/rethrow_error_test.dart
+++ b/tests/standalone/stack_trace/rethrow_error_test.dart
@@ -2,6 +2,9 @@
// 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.
+// Formatting can break multitests, so don't format them.
+// dart format off
+
class SubclassOfError extends Error {}
fail() => throw "Fail";
