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

Unified Diff: tests/isolate/src/ConstructorTest.dart

Issue 10153005: unittest step 3 and 4: remove TestFramework.dart, make test.dart use (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 8 years, 8 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/isolate/src/ConstructorTest.dart
diff --git a/tests/isolate/src/ConstructorTest.dart b/tests/isolate/src/ConstructorTest.dart
index c467e427fc297c6a20fd9a20083eb40eba3acb5a..9829cfadb7c2e06f84a32579bc4edd5fa33f851f 100644
--- a/tests/isolate/src/ConstructorTest.dart
+++ b/tests/isolate/src/ConstructorTest.dart
@@ -4,7 +4,7 @@
#library("ConstructorTest");
#import("dart:isolate");
-#import("TestFramework.dart");
+#import('../../../lib/unittest/unittest.dart');
class ConstructorTest extends Isolate {
final int field;
@@ -18,16 +18,14 @@ class ConstructorTest extends Isolate {
}
}
-void test(TestExpectation expect) {
- ConstructorTest test = new ConstructorTest();
- expect.completes(test.spawn()).then((SendPort port) {
- port.call("ignored").then(expect.runs1((message) {
- Expect.equals(499, message);
- expect.succeeded();
- }));
- });
-}
Emily Fortuna 2012/04/20 20:06:19 Extra whitespace?
Siggi Cherem (dart-lang) 2012/04/21 00:03:43 empty line. Removed.
main() {
- runTests([test]);
+ test("constructor initialized correctly child isolate", () {
Bob Nystrom 2012/04/20 22:31:36 "constructor correctly initialized child isolate"
Siggi Cherem (dart-lang) 2012/04/21 00:03:43 Done.
+ ConstructorTest test = new ConstructorTest();
+ test.spawn().then(later1((SendPort port) {
+ port.call("ignored").then(later1((message) {
+ Expect.equals(499, message);
+ }));
+ }));
+ });
}

Powered by Google App Engine
This is Rietveld 408576698