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

Unified Diff: tests/standalone/io/file_system_watcher_test.dart

Issue 23506025: Make multi-test less strict, to avoid oddly-delayed errors on Windows. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 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
« no previous file with comments | « no previous file | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tests/standalone/io/file_system_watcher_test.dart
diff --git a/tests/standalone/io/file_system_watcher_test.dart b/tests/standalone/io/file_system_watcher_test.dart
index 37cb0a8ec3d73c8c6e3e8432c3c8e69ed50470ad..453b4b5f4a55466ec93290fab948bb05253d3f1d 100644
--- a/tests/standalone/io/file_system_watcher_test.dart
+++ b/tests/standalone/io/file_system_watcher_test.dart
@@ -144,38 +144,37 @@ void testMultipleEvents() {
int state = 0;
var sub;
sub = watcher.listen((event) {
- switch (state) {
- case 0:
- Expect.isTrue(event is FileSystemCreateEvent);
- state = 1;
+ int newState = 0;
+ switch (event.type) {
+ case FileSystemEvent.CREATE:
+ newState = 1;
break;
- case 1:
- if (event is FileSystemCreateEvent) break;
- Expect.isTrue(event is FileSystemModifyEvent);
- state = 2;
+ case FileSystemEvent.MODIFY:
+ newState = 2;
break;
- case 2:
- if (event is FileSystemModifyEvent) break;
- Expect.isTrue(event is FileSystemMoveEvent);
- state = 3;
+ case FileSystemEvent.MOVE:
+ newState = 3;
break;
- case 3:
- if (event is FileSystemMoveEvent) break;
- Expect.isTrue(event is FileSystemDeleteEvent);
+ case FileSystemEvent.DELETE:
+ newState = 4;
sub.cancel();
asyncEnd();
dir.deleteSync();
break;
}
+ if (newState < state) throw "Bad state";
+ state = newState;
});
- file.createSync();
- file.writeAsStringSync('a');
- file.renameSync(file2.path);
- file2.deleteSync();
+ runAsync(() {
+ file.createSync();
+ file.writeAsStringSync('a');
+ file.renameSync(file2.path);
+ file2.deleteSync();
+ });
}
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698