Kernel debugging; service tests

runtime/observatory/tests/service/add_breakpoint_rpc_kernel_test.dart

Index: runtime/observatory/tests/service/add_breakpoint_rpc_kernel_test.dart
diff --git a/runtime/observatory/tests/service/add_breakpoint_rpc_kernel_test.dart b/runtime/observatory/tests/service/add_breakpoint_rpc_kernel_test.dart
new file mode 100644
index 0000000000000000000000000000000000000000..d3b88c3298875108e039489c62ffbdfd6405f6a1
--- /dev/null
+++ b/runtime/observatory/tests/service/add_breakpoint_rpc_kernel_test.dart
@@ -0,0 +1,141 @@
+// Copyright (c) 2015, the Dart project authors.  Please see the AUTHORS file

[Kevin Millikin (Google), 2017/02/08 15:37:51]

2017

[jensj, 2017/02/13 14:04:15]

Done.

+// for details. All rights reserved. Use of this source code is governed by a
+// BSD-style license that can be found in the LICENSE file.
+// VMOptions=--error_on_bad_type --error_on_bad_override
+
+import 'package:observatory/service_io.dart';
+import 'package:unittest/unittest.dart';
+import 'service_test_common.dart';
+import 'test_helper.dart';
+
+import 'dart:async';
+
+const int LINE_A = 24;
+const int LINE_B = 26;
+
+int value = 0;
+
+int incValue(int amount) {
+  value += amount;
+  return amount;
+}
+
+Future testMain() async {
+  incValue(incValue(1));  // line A.
+
+  incValue(incValue(1));  // line B.
+}
+
+var tests = [
+  hasPausedAtStart,
+
+  // Test future breakpoints.
+  (Isolate isolate) async {
+    var rootLib = isolate.rootLibrary;
+    await rootLib.load();
+    var script = rootLib.scripts[0];
+
+    // Future breakpoint.
+    var futureBpt1 = await isolate.addBreakpoint(script, LINE_A);
+    expect(futureBpt1.number, equals(1));
+    expect(futureBpt1.resolved, isFalse);
+    expect(await futureBpt1.location.getLine(), equals(LINE_A));
+    expect(await futureBpt1.location.getColumn(), equals(null));
+
+    // Future breakpoint with specific column.
+    var futureBpt2 = await isolate.addBreakpoint(script, LINE_A, 3);
+    expect(futureBpt2.number, equals(2));
+    expect(futureBpt2.resolved, isFalse);
+    expect(await futureBpt2.location.getLine(), equals(LINE_A));
+    expect(await futureBpt2.location.getColumn(), equals(3));
+
+    var stream = await isolate.vm.getEventStream(VM.kDebugStream);

[Cutch, 2017/02/08 16:48:19]

Please factor this code into a helper method.

[jensj, 2017/02/13 14:04:15]

Done.

+    Completer completer = new Completer();
+    var subscription;
+    var resolvedCount = 0;
+    subscription = stream.listen((ServiceEvent event) async {
+      if (event.kind == ServiceEvent.kBreakpointResolved) {
+        resolvedCount++;
+      }
+      if (event.kind == ServiceEvent.kPauseBreakpoint) {

[Cutch, 2017/02/08 16:48:19]

this is hasStoppedAtBreakpoint

[jensj, 2017/02/13 14:04:15]

Acknowledged.

+        subscription.cancel();
+        completer.complete(null);
+      }
+    });
+    await isolate.resume();
+    await completer.future;
+
+    // After resolution the breakpoints have assigned line & column.
+    expect(resolvedCount, equals(2));
+    expect(futureBpt1.resolved, isTrue);
+    expect(await futureBpt1.location.getLine(), equals(LINE_A));
+    expect(await futureBpt1.location.getColumn(), equals(12));
+    expect(futureBpt2.resolved, isTrue);
+    expect(await futureBpt2.location.getLine(), equals(LINE_A));
+    expect(await futureBpt2.location.getColumn(), equals(3));
+
+    // The first breakpoint hits before value is modified.
+    expect((await rootLib.evaluate('value')).valueAsString, equals('0'));
+
+    stream = await isolate.vm.getEventStream(VM.kDebugStream);
+    completer = new Completer();

[Cutch, 2017/02/08 16:48:20]

(here and elsewhere) Please use the helper: hasStoppedAtBreakpoint

[jensj, 2017/02/13 14:04:15]

I've refactored this file and the file it was copied from.

+    subscription = stream.listen((ServiceEvent event) async {
+      if (event.kind == ServiceEvent.kPauseBreakpoint) {
+        subscription.cancel();
+        completer.complete(null);
+      }
+    });
+    await isolate.resume();
+    await completer.future;
+
+    // The second breakpoint hits after value has been modified once.
+    expect((await rootLib.evaluate('value')).valueAsString, equals('1'));
+
+    // Remove the breakpoints.
+    expect((await isolate.removeBreakpoint(futureBpt1)).type,
+           equals('Success'));
+    expect((await isolate.removeBreakpoint(futureBpt2)).type,
+           equals('Success'));
+  },
+
+  // Test resolution of column breakpoints.
+  (Isolate isolate) async {
+    var script = isolate.rootLibrary.scripts[0];
+    // Try all columns, including some columns that are too big.
+    for (int col = 1; col <= 50; col++) {
+      var bpt = await isolate.addBreakpoint(script, LINE_A, col);
+      expect(bpt.resolved, isTrue);
+      int resolvedLine = await bpt.location.getLine();
+      int resolvedCol = await bpt.location.getColumn();
+      print('$LINE_A:${col} -> ${resolvedLine}:${resolvedCol}');
+      if (col <= 10) {
+        expect(resolvedLine, equals(LINE_A));
+        expect(resolvedCol, equals(3));
+      } else if (col <= 19) {
+        expect(resolvedLine, equals(LINE_A));
+        expect(resolvedCol, equals(12));
+      } else {
+        expect(resolvedLine, equals(LINE_B));
+        expect(resolvedCol, equals(12));
+      }
+      expect((await isolate.removeBreakpoint(bpt)).type, equals('Success'));
+    }
+
+    // Make sure that a zero column is an error.
+    var caughtException = false;
+    try {
+      await isolate.addBreakpoint(script, 20, 0);
+      expect(false, isTrue, reason:'Unreachable');
+    } on ServerRpcException catch(e) {
+      caughtException = true;
+      expect(e.code, equals(ServerRpcException.kInvalidParams));
+      expect(e.message,
+             "addBreakpoint: invalid 'column' parameter: 0");
+    }
+    expect(caughtException, isTrue);
+  },
+];
+
+main(args) => runIsolateTests(args, tests,
+                              testeeConcurrent: testMain,
+                              pause_on_start: true);