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

Unified Diff: tests/language/deferred_not_loaded_check_test.dart

Issue 2416513003: Fix issues with argument evaluation order and noSuchMethod errors. (Closed)
Patch Set: Created 4 years, 2 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 | « runtime/vm/parser.cc ('k') | tests/language/language_dart2js.status » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/language/deferred_not_loaded_check_test.dart
diff --git a/tests/language/deferred_not_loaded_check_test.dart b/tests/language/deferred_not_loaded_check_test.dart
index 5c48da3bca1f5a1255abbc85b75558e74667afcd..70be80fc076b7e8c3fa21e3b41597496b1d8becf 100644
--- a/tests/language/deferred_not_loaded_check_test.dart
+++ b/tests/language/deferred_not_loaded_check_test.dart
@@ -12,6 +12,12 @@ import "deferred_not_loaded_check_lib.dart" deferred as lib;
var c;
+expectSideEffect(test) {
+ c = 0;
+ test();
+ Expect.isTrue(c == 1);
+}
+
expectNoSideEffect(test) {
c = 0;
test();
@@ -28,7 +34,7 @@ int sideEffect() {
}
void main() {
- expectNoSideEffect(() {
+ expectSideEffect(() {
expectThrowsNotLoaded(() {
lib.foo(sideEffect());
});
@@ -38,7 +44,7 @@ void main() {
lib.C.foo(sideEffect());
});
});
- expectNoSideEffect(() {
+ expectSideEffect(() {
expectThrowsNotLoaded(() {
new lib.C(sideEffect());
});
@@ -46,7 +52,7 @@ void main() {
expectThrowsNotLoaded(() {
lib.a;
});
- expectNoSideEffect(() {
+ expectSideEffect(() {
expectThrowsNotLoaded(() {
lib.a = sideEffect();
});
@@ -54,7 +60,7 @@ void main() {
expectThrowsNotLoaded(() {
lib.getter;
});
- expectNoSideEffect(() {
+ expectSideEffect(() {
expectThrowsNotLoaded(() {
lib.setter = sideEffect();
});
@@ -64,7 +70,7 @@ void main() {
lib.list[sideEffect()] = sideEffect();
});
});
- expectNoSideEffect(() {
+ expectSideEffect(() {
expectThrowsNotLoaded(() {
lib.closure(sideEffect());
});
« no previous file with comments | « runtime/vm/parser.cc ('k') | tests/language/language_dart2js.status » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698