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

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

Issue 2694013003: Add tests of line numbers to causal async stack tests (Closed)
Patch Set: rmacnak review Created 3 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/causal_async_star_stack_contents_test.dart
diff --git a/runtime/observatory/tests/service/causal_async_star_stack_contents_test.dart b/runtime/observatory/tests/service/causal_async_star_stack_contents_test.dart
index 0a32ace82d8745eae1f0e10f43f0bf1062e212e8..47c3131bd0ebac213d1364514c0a1e58e6e0e48f 100644
--- a/runtime/observatory/tests/service/causal_async_star_stack_contents_test.dart
+++ b/runtime/observatory/tests/service/causal_async_star_stack_contents_test.dart
@@ -75,6 +75,12 @@ var tests = [
expect(asyncStack[2].toString(), contains('helper'));
expect(asyncStack[3].kind, equals(M.FrameKind.asyncSuspensionMarker));
expect(asyncStack[4].toString(), contains('testMain'));
+ // Line 21.
+ expect(await asyncStack[0].location.toUserString(), contains('.dart:21'));
+ // Line 27.
+ expect(await asyncStack[2].location.toUserString(), contains('.dart:27'));
+ // Line 30.
+ expect(await asyncStack[4].location.toUserString(), contains('.dart:33'));
},
];

Powered by Google App Engine
This is Rietveld 408576698