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

Unified Diff: tests/standalone/no_assert_test.dart

Issue 1305183010: - Reorder the priority of flags in the test harness when passed to the VM: (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: Created 5 years, 3 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/standalone/no_assert_test.dart
diff --git a/tests/language/regress_17382_test.dart b/tests/standalone/no_assert_test.dart
similarity index 50%
copy from tests/language/regress_17382_test.dart
copy to tests/standalone/no_assert_test.dart
index 749bcf0e0ea583303f2bf01c8cf6dc6b0dd596db..825578b590d24725b8353618a638e26370655dcc 100644
--- a/tests/language/regress_17382_test.dart
+++ b/tests/standalone/no_assert_test.dart
@@ -1,20 +1,16 @@
// Copyright (c) 2015, 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.
+// VMOptions=--no-enable_asserts --enable_type_checks
-// Regression test for issue 17382.
-
-import 'package:expect/expect.dart';
-
-var mCalled = false;
-
-m(x) { mCalled = true; return x; }
+// Ensure that enabling of type checks does not automatically enable asserts.
main() {
+ assert(false);
try {
- tl(m(0));
- } catch (e) {
+ int i = "String";
+ throw "FAIL";
+ } on TypeError catch (e) {
+ print("PASS");
}
- Expect.isTrue(mCalled);
}
-

Powered by Google App Engine
This is Rietveld 408576698