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

Unified Diff: tests/compiler/dart2js/type_inference5_test.dart

Issue 326913002: Make dart2js unittests async. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Updated cf. comments. Created 6 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/compiler/dart2js/type_inference4_test.dart ('k') | tests/compiler/dart2js/union_type_test.dart » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/compiler/dart2js/type_inference5_test.dart
diff --git a/tests/compiler/dart2js/type_inference5_test.dart b/tests/compiler/dart2js/type_inference5_test.dart
index ba4645673b2787aedaf1fc068d245697f1a69d97..4bf5caa7e6c7fc66a2b96aacc037e74eee372a7d 100644
--- a/tests/compiler/dart2js/type_inference5_test.dart
+++ b/tests/compiler/dart2js/type_inference5_test.dart
@@ -2,7 +2,8 @@
// 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 "package:expect/expect.dart";
+import 'package:expect/expect.dart';
+import 'package:async_helper/async_helper.dart';
import 'compiler_helper.dart';
const String TEST_ONE = r"""
@@ -15,16 +16,18 @@ foo(j) {
""";
main() {
- String generated = compile(TEST_ONE, entry: 'foo');
+ asyncTest(() => compile(TEST_ONE, entry: 'foo', check: (String generated) {
- // Test for absence of an illegal argument exception. This means that the
- // arguments are known to be integers.
- Expect.isFalse(generated.contains('iae'));
- // Also make sure that we are not just in bailout mode without speculative
- // types by grepping for the integer-bailout check on argument j.
- var argname =
- new RegExp(r'function(?: [a-z]+)?\(([a-zA-Z0-9_]+)\)').firstMatch(generated)[1];
- print(argname);
- RegExp regexp = new RegExp(getIntTypeCheck("(i|$argname)"));
- Expect.isTrue(regexp.hasMatch(generated));
+ // Test for absence of an illegal argument exception. This means that the
+ // arguments are known to be integers.
+ Expect.isFalse(generated.contains('iae'));
+ // Also make sure that we are not just in bailout mode without speculative
+ // types by grepping for the integer-bailout check on argument j.
+ var argname =
+ new RegExp(r'function(?: [a-z]+)?\(([a-zA-Z0-9_]+)\)')
+ .firstMatch(generated)[1];
+ print(argname);
+ RegExp regexp = new RegExp(getIntTypeCheck("(i|$argname)"));
+ Expect.isTrue(regexp.hasMatch(generated));
+ }));
}
« no previous file with comments | « tests/compiler/dart2js/type_inference4_test.dart ('k') | tests/compiler/dart2js/union_type_test.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698