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

Unified Diff: tests/language/vm/regress_27201_test.dart

Issue 2537253002: Fixed assert causing crash when attempting to load a library from an invalid path. (Closed)
Patch Set: Addressed comments from initial upload Created 4 years, 1 month 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/vm/regress_27201_lib.dart ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/language/vm/regress_27201_test.dart
diff --git a/tests/language/vm/regress_27201_test.dart b/tests/language/vm/regress_27201_test.dart
new file mode 100644
index 0000000000000000000000000000000000000000..251a1adcc5c66443e804279acb61a9f71a3e768b
--- /dev/null
+++ b/tests/language/vm/regress_27201_test.dart
@@ -0,0 +1,56 @@
+/*
+ * Copyright (c) 2016, 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.
+ */
+import "dart:async";
+import "package:expect/expect.dart";
+import "regress_27201_lib.dart" deferred as p;
+import "regress_27201_bad_lib_path.dart" deferred as q;
+
+test_loaded() {
+ try {
+ p.someFunc();
+ } catch (e) {
+ Expect.fail("Should not be here");
+ }
+ try {
+ p.someGetter;
+ } catch (e) {
+ Expect.fail("Should not be here");
+ }
+ try {
+ p.someSetter = 1;
+ } catch (e) {
+ Expect.fail("Should not be here");
+ }
+ try {
+ p.Func;
+ } catch (e) {
+ Expect.fail("Should not be here");
+ }
+ try {
+ Expect.isTrue(p.loadLibrary() is Future);
+ } catch (e) {
+ Expect.fail("Should not be here");
+ }
+}
+
+
+main() {
+ p.loadLibrary().then((v) {
+ test_loaded();
+ },
+ onError: (e) {
+ Expect.fail("Should have loaded library!");
+ });
+
+ // Ensure bad library import is handled correctly.
+ q.loadLibrary().then((v) {
+ Expect.fail("Should have failed");
+ },
+ onError: (e) {
+ Expect.throws(() => q.x);
+ });
+}
+
« no previous file with comments | « tests/language/vm/regress_27201_lib.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698