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

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

Issue 1726773002: Refactor service tests in preparation of running on sky_shell (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: Created 4 years, 10 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/debugger_location_test.dart
diff --git a/runtime/observatory/tests/service/debugger_location_test.dart b/runtime/observatory/tests/service/debugger_location_test.dart
index ab50f4d136ee943acd5c8561e2d6175ccd65fe0a..53ed4fb97c1fc497cdb3f4e7b65347f8e5066d9a 100644
--- a/runtime/observatory/tests/service/debugger_location_test.dart
+++ b/runtime/observatory/tests/service/debugger_location_test.dart
@@ -6,15 +6,20 @@
import 'package:observatory/service_io.dart';
import 'package:observatory/debugger.dart';
import 'package:unittest/unittest.dart';
+import 'service_test_common.dart';
import 'test_helper.dart';
import 'dart:async';
import 'dart:developer';
+const int LINE_A = 22;
+const int LINE_B = 112;
+const int LINE_C = 12;
+
void testFunction() {
int i = 0;
while (i == 0) {
debugger();
- print('loop');
+ print('loop'); // Line A.
print('loop');
}
}
@@ -59,7 +64,7 @@ hasStoppedAtBreakpoint,
var debugger = await initDebugger(isolate);
var loc = await DebuggerLocation.parse(debugger, '');
expect(loc.valid, isTrue);
- expect(loc.toString(), equals('debugger_location_test.dart:17:5'));
+ expect(loc.toString(), equals('debugger_location_test.dart:$LINE_A:5'));
},
// Parse line
@@ -236,22 +241,18 @@ hasStoppedAtBreakpoint,
await DebuggerLocation.complete(debugger,
'debugger_location_test.dart:11');
expect(completions.toString(), equals(
- '[debugger_location_test.dart:110 ,'
- ' debugger_location_test.dart:110:,'
- ' debugger_location_test.dart:111 ,'
- ' debugger_location_test.dart:111:,'
- ' debugger_location_test.dart:112 ,'
- ' debugger_location_test.dart:112:,'
- ' debugger_location_test.dart:115 ,'
- ' debugger_location_test.dart:115:,'
- ' debugger_location_test.dart:116 ,'
- ' debugger_location_test.dart:116:,'
- ' debugger_location_test.dart:117 ,'
- ' debugger_location_test.dart:117:,'
- ' debugger_location_test.dart:118 ,'
- ' debugger_location_test.dart:118:,'
- ' debugger_location_test.dart:119 ,'
- ' debugger_location_test.dart:119:]'));
+ '[debugger_location_test.dart:${LINE_B + 0} ,'
+ ' debugger_location_test.dart:${LINE_B + 0}:,'
+ ' debugger_location_test.dart:${LINE_B + 1} ,'
+ ' debugger_location_test.dart:${LINE_B + 1}:,'
+ ' debugger_location_test.dart:${LINE_B + 2} ,'
+ ' debugger_location_test.dart:${LINE_B + 2}:,'
+ ' debugger_location_test.dart:${LINE_B + 3} ,'
+ ' debugger_location_test.dart:${LINE_B + 3}:,'
+ ' debugger_location_test.dart:${LINE_B + 4} ,'
+ ' debugger_location_test.dart:${LINE_B + 4}:,'
+ ' debugger_location_test.dart:${LINE_B + 5} ,'
+ ' debugger_location_test.dart:${LINE_B + 5}:]'));
},
// Complete script:line:col
@@ -259,27 +260,27 @@ hasStoppedAtBreakpoint,
var debugger = await initDebugger(isolate);
var completions =
await DebuggerLocation.complete(debugger,
- 'debugger_location_test.dart:11:2');
+ 'debugger_location_test.dart:$LINE_C:2');
expect(completions.toString(), equals(
- '[debugger_location_test.dart:11:2 ,'
- ' debugger_location_test.dart:11:20 ,'
- ' debugger_location_test.dart:11:21 ,'
- ' debugger_location_test.dart:11:22 ,'
- ' debugger_location_test.dart:11:23 ,'
- ' debugger_location_test.dart:11:24 ]'));
+ '[debugger_location_test.dart:$LINE_C:2 ,'
+ ' debugger_location_test.dart:$LINE_C:20 ,'
+ ' debugger_location_test.dart:$LINE_C:21 ,'
+ ' debugger_location_test.dart:$LINE_C:22 ,'
+ ' debugger_location_test.dart:$LINE_C:23 ,'
+ ' debugger_location_test.dart:$LINE_C:24 ]'));
},
// Complete without the script name.
(Isolate isolate) async {
var debugger = await initDebugger(isolate);
- var completions = await DebuggerLocation.complete(debugger, '11:2');
+ var completions = await DebuggerLocation.complete(debugger, '$LINE_C:2');
expect(completions.toString(), equals(
- '[debugger_location_test.dart:11:2 ,'
- ' debugger_location_test.dart:11:20 ,'
- ' debugger_location_test.dart:11:21 ,'
- ' debugger_location_test.dart:11:22 ,'
- ' debugger_location_test.dart:11:23 ,'
- ' debugger_location_test.dart:11:24 ]'));
+ '[debugger_location_test.dart:$LINE_C:2 ,'
+ ' debugger_location_test.dart:$LINE_C:20 ,'
+ ' debugger_location_test.dart:$LINE_C:21 ,'
+ ' debugger_location_test.dart:$LINE_C:22 ,'
+ ' debugger_location_test.dart:$LINE_C:23 ,'
+ ' debugger_location_test.dart:$LINE_C:24 ]'));
},
];

Powered by Google App Engine
This is Rietveld 408576698