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

Side by Side Diff: runtime/observatory/tests/service/caching_test.dart

Issue 1584173003: Use getSourceReport for coverage/callsites in Observatory. (Closed) Base URL: git@github.com:dart-lang/sdk.git@master
Patch Set: cleanups Created 4 years, 11 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 unified diff | Download patch
OLDNEW
1 // Copyright (c) 2015, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2015, the Dart project authors. Please see the AUTHORS file
2 // for details. All rights reserved. Use of this source code is governed by a 2 // for details. All rights reserved. Use of this source code is governed by a
3 // BSD-style license that can be found in the LICENSE file. 3 // BSD-style license that can be found in the LICENSE file.
4 // VMOptions=--error_on_bad_type --error_on_bad_override 4 // VMOptions=--error_on_bad_type --error_on_bad_override
5 5
6 // If caching is working properly, the coverage data will go into the same 6 // If caching is working properly, the coverage data will go into the same
7 // Script object from which we requested coverage data, instead of a new 7 // Script object from which we requested coverage data, instead of a new
8 // Script object. 8 // Script object.
9 9
10 library caching_test; 10 library caching_test;
11 11
12 import 'package:observatory/service_io.dart'; 12 import 'package:observatory/service_io.dart';
13 import 'package:unittest/unittest.dart'; 13 import 'package:unittest/unittest.dart';
14 import 'test_helper.dart'; 14 import 'test_helper.dart';
15 15
16 script() {
17 print("This executed");
18 }
19
20 hasSomeCoverageData(Script script) {
21 for (var line in script.lines) {
22 if (line.hits != null) return true;
23 }
24 return false;
25 }
26
27 var tests = [ 16 var tests = [
28 (Isolate isolate) async { 17 (Isolate isolate) async {
29 Library lib = await isolate.rootLibrary.load(); 18 Library lib = await isolate.rootLibrary.load();
30 Script script = await lib.scripts.single.load(); 19 Script script = await lib.scripts.single.load();
31 expect(hasSomeCoverageData(script), isFalse); 20 Script script2 = await isolate.getObject(script.id);
32 Script script2 = await script.refreshCoverage();
33 expect(identical(script, script2), isTrue); 21 expect(identical(script, script2), isTrue);
34 expect(hasSomeCoverageData(script), isTrue);
35 }, 22 },
36 23
37 ]; 24 ];
38 25
39 main(args) => runIsolateTests(args, tests, testeeBefore: script); 26 main(args) => runIsolateTests(args, tests);
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698