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

Side by Side Diff: test/directory_watcher/shared.dart

Issue 995623002: Fix bugs where events could be added after watchers were closed. (Closed) Base URL: git@github.com:dart-lang/watcher@master
Patch Set: Created 5 years, 9 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 unified diff | Download patch
« no previous file with comments | « pubspec.yaml ('k') | no next file » | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 4
5 import 'dart:async';
6
5 import 'package:scheduled_test/scheduled_test.dart'; 7 import 'package:scheduled_test/scheduled_test.dart';
6 import 'package:watcher/src/utils.dart'; 8 import 'package:watcher/src/utils.dart';
7 9
8 import '../utils.dart'; 10 import '../utils.dart';
9 11
10 void sharedTests() { 12 void sharedTests() {
11 test('does not notify for files that already exist when started', () { 13 test('does not notify for files that already exist when started', () {
12 // Make some pre-existing files. 14 // Make some pre-existing files.
13 writeFile("a.txt"); 15 writeFile("a.txt");
14 writeFile("b.txt"); 16 writeFile("b.txt");
(...skipping 274 matching lines...) Expand 10 before | Expand all | Expand 10 after
289 deleteFile("dir/sub"); 291 deleteFile("dir/sub");
290 renameDir("old", "dir/sub"); 292 renameDir("old", "dir/sub");
291 293
292 var events = withPermutations((i, j, k) => 294 var events = withPermutations((i, j, k) =>
293 isAddEvent("dir/sub/sub-$i/sub-$j/file-$k.txt")); 295 isAddEvent("dir/sub/sub-$i/sub-$j/file-$k.txt"));
294 events.add(isRemoveEvent("dir/sub")); 296 events.add(isRemoveEvent("dir/sub"));
295 inAnyOrder(events); 297 inAnyOrder(events);
296 }); 298 });
297 }); 299 });
298 } 300 }
OLDNEW
« no previous file with comments | « pubspec.yaml ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698