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

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

Issue 2759973004: Fix observatory tests broken by running dartfmt. Temporarily reverted formatting for evaluate_activ… (Closed)
Patch Set: Created 3 years, 9 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 import 'package:observatory/service_io.dart'; 6 import 'package:observatory/service_io.dart';
7 import 'package:unittest/unittest.dart'; 7 import 'package:unittest/unittest.dart';
8 import 'test_helper.dart'; 8 import 'test_helper.dart';
9 9
10 import 'dart:developer'; 10 import 'dart:developer';
11 11
12 void script() { 12 void script() {
13 var counter = new Counter('a.b.c', 'description'); 13 var counter = new Counter('a.b.c', 'description');
14 Metrics.register(counter); 14 Metrics.register(counter);
15 counter.value = 1234.5; 15 counter.value = 1234.5;
16 } 16 }
17 17
18 var tests = [ 18 var tests = [
19
20 (Isolate isolate) async { 19 (Isolate isolate) async {
21 Map metrics = await isolate.refreshNativeMetrics(); 20 Map metrics = await isolate.refreshNativeMetrics();
22 expect(metrics.length, greaterThan(1)); 21 expect(metrics.length, greaterThan(1));
23 expect(metrics.length, greaterThan(1)); 22 expect(metrics.length, greaterThan(1));
24 var foundOldHeapCapacity = metrics.values.any( 23 var foundOldHeapCapacity =
25 (m) => m.name == 'heap.old.capacity'); 24 metrics.values.any((m) => m.name == 'heap.old.capacity');
26 expect(foundOldHeapCapacity, equals(true)); 25 expect(foundOldHeapCapacity, equals(true));
27 }, 26 },
28
29 (Isolate isolate) async { 27 (Isolate isolate) async {
30 var params = { 'metricId': 'metrics/native/heap.old.used' }; 28 var params = {'metricId': 'metrics/native/heap.old.used'};
31 ServiceMetric counter = 29 ServiceMetric counter =
32 await isolate.invokeRpc('_getIsolateMetric', params); 30 await isolate.invokeRpc('_getIsolateMetric', params);
33 expect(counter.type, equals('Counter')); 31 expect(counter.type, equals('Counter'));
34 expect(counter.name, equals('heap.old.used')); 32 expect(counter.name, equals('heap.old.used'));
35 }, 33 },
36
37 (Isolate isolate) async { 34 (Isolate isolate) async {
38 bool caughtException; 35 bool caughtException;
39 try { 36 try {
40 await isolate.invokeRpc('_getIsolateMetric', 37 await isolate.invokeRpc(
41 { 'metricId': 'metrics/native/doesnotexist' }); 38 '_getIsolateMetric', {'metricId': 'metrics/native/doesnotexist'});
42 expect(false, isTrue, reason:'Unreachable'); 39 expect(false, isTrue, reason: 'Unreachable');
43 } on ServerRpcException catch (e) { 40 } on ServerRpcException catch (e) {
44 caughtException = true; 41 caughtException = true;
45 expect(e.code, equals(ServerRpcException.kInvalidParams)); 42 expect(e.code, equals(ServerRpcException.kInvalidParams));
46 expect(e.message, 43 expect(
47 "_getIsolateMetric: invalid 'metricId' " 44 e.message,
48 "parameter: metrics/native/doesnotexist"); 45 "_getIsolateMetric: invalid 'metricId' "
46 "parameter: metrics/native/doesnotexist");
49 } 47 }
50 expect(caughtException, isTrue); 48 expect(caughtException, isTrue);
51 }, 49 },
52
53 ]; 50 ];
54 51
55 main(args) => runIsolateTests(args, tests, testeeBefore: script); 52 main(args) => runIsolateTests(args, tests, testeeBefore: script);
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698