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

Side by Side Diff: sdk/lib/_internal/pub/test/serve/missing_file_test.dart

Issue 122573003: Don't test for file differences in the polling watcher. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Revise shared test. Created 6 years, 11 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 | Annotate | Revision Log
« no previous file with comments | « sdk/lib/_internal/pub/pub.status ('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) 2013, the Dart project authors. Please see the AUTHORS d.file 1 // Copyright (c) 2013, the Dart project authors. Please see the AUTHORS d.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 library pub_tests; 5 library pub_tests;
6 6
7 import 'package:path/path.dart' as path; 7 import 'package:path/path.dart' as path;
8 import 'package:scheduled_test/scheduled_test.dart'; 8 import 'package:scheduled_test/scheduled_test.dart';
9 9
10 import '../../lib/src/io.dart'; 10 import '../../lib/src/io.dart';
11 import '../descriptor.dart' as d; 11 import '../descriptor.dart' as d;
12 import '../test_pub.dart'; 12 import '../test_pub.dart';
13 import 'utils.dart'; 13 import 'utils.dart';
14 14
15 main() { 15 main() {
16 initConfig(); 16 initConfig();
17 integration("responds with a 404 for missing files", () { 17 integration("responds with a 404 for missing source files", () {
18 d.dir(appPath, [ 18 d.dir(appPath, [
19 d.appPubspec(), 19 d.appPubspec(),
20 d.dir("asset", [ 20 d.dir("asset", [
21 d.file("nope.png", "nope") 21 d.file("nope.png", "nope")
22 ]), 22 ]),
23 d.dir("lib", [ 23 d.dir("lib", [
24 d.file("nope.dart", "nope") 24 d.file("nope.dart", "nope")
25 ]), 25 ]),
26 d.dir("web", [ 26 d.dir("web", [
27 d.file("index.html", "<body>"), 27 d.file("index.html", "<body>"),
28 ]) 28 ])
29 ]).create(); 29 ]).create();
30 30
31 // Start the server with the files present so that it creates barback 31 // Start the server with the files present so that it creates barback
32 // assets for them. 32 // assets for them.
33 pubServe(); 33 pubServe();
34 34
35 // TODO(rnystrom): When pub serve supports file watching, we'll have to do
36 // something here to specifically disable that so that we can get barback
37 // into the inconsistent state of thinking there is an asset but where the
38 // underlying file does not exist. One option would be configure barback
39 // with an insanely long delay between polling to ensure a poll doesn't
40 // happen.
41
42 // Now delete them. 35 // Now delete them.
43 schedule(() { 36 schedule(() {
44 deleteEntry(path.join(sandboxDir, appPath, "asset", "nope.png")); 37 deleteEntry(path.join(sandboxDir, appPath, "asset", "nope.png"));
45 deleteEntry(path.join(sandboxDir, appPath, "lib", "nope.dart")); 38 deleteEntry(path.join(sandboxDir, appPath, "lib", "nope.dart"));
46 deleteEntry(path.join(sandboxDir, appPath, "web", "index.html")); 39 deleteEntry(path.join(sandboxDir, appPath, "web", "index.html"));
47 }, "delete files"); 40 }, "delete files");
48 41
42 // Now request them.
43 // TODO(rnystrom): It's possible for these requests to happen quickly
44 // enough that the file system hasn't notified for the deletions yet. If
45 // that happens, we can probably just add a short delay here.
46
49 requestShould404("index.html"); 47 requestShould404("index.html");
50 requestShould404("packages/myapp/nope.dart"); 48 requestShould404("packages/myapp/nope.dart");
51 requestShould404("assets/myapp/nope.png"); 49 requestShould404("assets/myapp/nope.png");
52 requestShould404("dir/packages/myapp/nope.dart"); 50 requestShould404("dir/packages/myapp/nope.dart");
53 requestShould404("dir/assets/myapp/nope.png"); 51 requestShould404("dir/assets/myapp/nope.png");
54 endPubServe(); 52 endPubServe();
55 }); 53 });
56 } 54 }
OLDNEW
« no previous file with comments | « sdk/lib/_internal/pub/pub.status ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698