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

Unified Diff: tests/language/language.status

Issue 2930673002: Implement initializer asserts. (Closed)
Patch Set: Address comments and tweak error recovery. Created 3 years, 6 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
« no previous file with comments | « tests/language/assert_initializer_test.dart ('k') | tests/language/language_kernel.status » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/language/language.status
diff --git a/tests/language/language.status b/tests/language/language.status
index bb8a0b8a80dc539a20eede5e5d2c2cffcec82141..92cbca7aab31915b6f54088dea6d6c4f54a50c42 100644
--- a/tests/language/language.status
+++ b/tests/language/language.status
@@ -295,12 +295,8 @@ conditional_import_string_test: Crash # Requires deferred libraries
vm/regress_27201_test: Pass, Crash # Requires deferred libraries
vm/optimized_stacktrace_test: Pass, Slow
-[$runtime != vm || $compiler != none]
-assert_initializer_test: SKIP # not implemented yet, experiment is VM only.
-
-[$runtime == vm && $compiler == none && ! $checked]
-assert_initializer_test/*: MissingCompileTimeError, OK # asserts shouldn't fail in unchecked mode.
-assert_initializer_test/none: pass
+[ ($runtime != vm && $compiler != dartk && $compiler != dartkp ) || ($compiler != none && $compiler != dartk && $compiler != dartkp )]
+assert_initializer_test/*: SKIP # not implemented yet, experiment is VM only.
[($runtime == vm || $runtime == flutter) && $compiler == none && $checked]
# The VM doesn't enforce that potentially const expressions are actually
« no previous file with comments | « tests/language/assert_initializer_test.dart ('k') | tests/language/language_kernel.status » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698