Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(524)

Unified Diff: tests/lib_2/lib_2.status

Issue 2999943002: Migrated test block 168 to Dart 2.0. (Closed)
Patch Set: Fixed async/future_test.dart Created 3 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: tests/lib_2/lib_2.status
diff --git a/tests/lib_2/lib_2.status b/tests/lib_2/lib_2.status
index 8ed3c7d6556c23f5e3b60dfadec572575953480c..e8e362d4e9229980b9e2f151619579d5315b5cc2 100644
--- a/tests/lib_2/lib_2.status
+++ b/tests/lib_2/lib_2.status
@@ -1,3 +1,20 @@
# Copyright (c) 2017, 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.
+
+[ $strong && $compiler != dart2analyzer && $compiler != dartdevc ]
Bob Nystrom 2017/08/16 22:27:20 Is this basically to match "strong mode on the VM"
bkonyi 2017/08/16 23:44:13 Done.
+# async/future_or_bad_type_test/implements: CompileTimeError
Bob Nystrom 2017/08/16 22:27:20 Should this be commented out?
bkonyi 2017/08/16 23:44:13 This was temporary since adding that line actually
+async/future_or_only_in_async_test/00: MissingCompileTimeError
+
+[ !$strong && $compiler != dartdevc && $checked ]
+async/future_or_bad_type_test/implements: CompileTimeError
+async/future_or_only_in_async_test/00: MissingCompileTimeError
+
+[ !$strong && !$checked]
+async/future_or_bad_type_test/implements: CompileTimeError
+async/future_or_only_in_async_test/00: MissingCompileTimeError
+
+[ $compiler != dart2js ]
+async/dart2js_uncaught_error_test: Skip # JS-integration only test
Bob Nystrom 2017/08/16 22:27:20 Ah, the migration script doesn't handle "!=" (beca
bkonyi 2017/08/16 23:44:13 Added the TODO. Done.
+
+

Powered by Google App Engine
This is Rietveld 408576698