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

Unified Diff: test/serve/web_socket/serve_directory_arg_errors_test.dart

Issue 1282533003: Don't implicitly run "pub get". (Closed) Base URL: git@github.com:dart-lang/pub.git@master
Patch Set: Code review changes Created 5 years, 4 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: test/serve/web_socket/serve_directory_arg_errors_test.dart
diff --git a/test/serve/web_socket/serve_directory_arg_errors_test.dart b/test/serve/web_socket/serve_directory_arg_errors_test.dart
index 458a5ade024cad48a3eb895ae0f7ce0d504a196b..2f788f9638025a77fdb498a4fd478e4a73729d74 100644
--- a/test/serve/web_socket/serve_directory_arg_errors_test.dart
+++ b/test/serve/web_socket/serve_directory_arg_errors_test.dart
@@ -19,10 +19,11 @@ main() {
d.file("index.html", "<body>")
])
]).create();
+ pubGet();
});
integration("responds with an error if 'path' is not a string", () {
- pubServe(shouldGetFirst: true);
+ pubServe();
expectWebSocketError("serveDirectory", {"path": 123},
rpc_error_code.INVALID_PARAMS,
'Parameter "path" for method "serveDirectory" must be a string, but '
@@ -31,7 +32,7 @@ main() {
});
integration("responds with an error if 'path' is absolute", () {
- pubServe(shouldGetFirst: true);
+ pubServe();
expectWebSocketError("serveDirectory", {"path": "/absolute.txt"},
rpc_error_code.INVALID_PARAMS,
'"path" must be a relative path. Got "/absolute.txt".');
@@ -39,7 +40,7 @@ main() {
});
integration("responds with an error if 'path' reaches out", () {
- pubServe(shouldGetFirst: true);
+ pubServe();
expectWebSocketError("serveDirectory", {"path": "a/../../bad.txt"},
rpc_error_code.INVALID_PARAMS,
'"path" cannot reach out of its containing directory. Got '

Powered by Google App Engine
This is Rietveld 408576698