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

Unified Diff: runtime/observatory/tests/service/test_helper.dart

Issue 1090293003: Kill service_test.cc tests in favor of dart tests for the service protocol. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: fix tests Created 5 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: runtime/observatory/tests/service/test_helper.dart
diff --git a/runtime/observatory/tests/service/test_helper.dart b/runtime/observatory/tests/service/test_helper.dart
index 3dddedd351cc0760ff8cbd4df2c7b3a6bf1d72b9..34b7a431c2f83bedc8836385ae842d9de325d3e5 100644
--- a/runtime/observatory/tests/service/test_helper.dart
+++ b/runtime/observatory/tests/service/test_helper.dart
@@ -106,8 +106,8 @@ void runIsolateTests(List<String> mainArgs,
var process = new _TestLauncher();
process.launch(pause_on_exit).then((port) {
String addr = 'ws://localhost:$port/ws';
- var testIndex = 0;
- var totalTests = tests.length - 1;
+ var testIndex = 1;
+ var totalTests = tests.length;
var name = Platform.script.pathSegments.last;
runZoned(() {
new WebSocketVM(new WebSocketVMTarget(addr)).load()
@@ -167,8 +167,8 @@ Future runVMTests(List<String> mainArgs,
var process = new _TestLauncher();
process.launch(pause_on_exit).then((port) async {
String addr = 'ws://localhost:$port/ws';
- var testIndex = 0;
- var totalTests = tests.length - 1;
+ var testIndex = 1;
+ var totalTests = tests.length;
var name = Platform.script.pathSegments.last;
runZoned(() {
new WebSocketVM(new WebSocketVMTarget(addr)).load()

Powered by Google App Engine
This is Rietveld 408576698