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

Side by Side Diff: runtime/observatory/test/coverage_test.dart

Issue 1031393003: Make sure Observatory tests run for each commit. Fix broken tests. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 5 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 | Annotate | Revision Log
« no previous file with comments | « runtime/observatory/test/contexts_test.dart ('k') | tools/testing/dart/test_options.dart » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
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 4
5 import 'package:observatory/service_io.dart'; 5 import 'package:observatory/service_io.dart';
6 import 'package:unittest/unittest.dart'; 6 import 'package:unittest/unittest.dart';
7 import 'test_helper.dart'; 7 import 'test_helper.dart';
8 import 'dart:async'; 8 import 'dart:async';
9 9
10 int globalVar = 100; 10 int globalVar = 100;
(...skipping 103 matching lines...) Expand 10 before | Expand all | Expand 10 after
114 // Library 114 // Library
115 tests.add(isolate.invokeRpcNoUpgrade('getCoverage', 115 tests.add(isolate.invokeRpcNoUpgrade('getCoverage',
116 { 'targetId': lib.id }) 116 { 'targetId': lib.id })
117 .then((Map coverage) { 117 .then((Map coverage) {
118 expect(coverage['type'], equals('CodeCoverage')); 118 expect(coverage['type'], equals('CodeCoverage'));
119 expect(coverage['coverage'].length, equals(3)); 119 expect(coverage['coverage'].length, equals(3));
120 expect(normalize(coverage['coverage'][0]['hits']), 120 expect(normalize(coverage['coverage'][0]['hits']),
121 equals([14, 1, 15, 1, 16, 0, 18, 1, 121 equals([14, 1, 15, 1, 16, 0, 18, 1,
122 23, 1, 24, 1, 26, 0])); 122 23, 1, 24, 1, 26, 0]));
123 expect(normalize(coverage['coverage'][1]['hits']).take(12), 123 expect(normalize(coverage['coverage'][1]['hits']).take(12),
124 equals([32, 0, 35, 0, 36, 0, 32, 1, 35, 1, 36, 0])); 124 equals([32, 1, 35, 1, 36, 0, 31, 1, 44, 0, 45, 0]));
125 })); 125 }));
126 // Script 126 // Script
127 tests.add(cls.load().then((_) { 127 tests.add(cls.load().then((_) {
128 return isolate.invokeRpcNoUpgrade('getCoverage', 128 return isolate.invokeRpcNoUpgrade('getCoverage',
129 { 'targetId': cls.script.id }) 129 { 'targetId': cls.script.id })
130 .then((Map coverage) { 130 .then((Map coverage) {
131 expect(coverage['type'], equals('CodeCoverage')); 131 expect(coverage['type'], equals('CodeCoverage'));
132 expect(coverage['coverage'].length, equals(3)); 132 expect(coverage['coverage'].length, equals(3));
133 expect(normalize(coverage['coverage'][0]['hits']), 133 expect(normalize(coverage['coverage'][0]['hits']),
134 equals([14, 1, 15, 1, 16, 0, 18, 1, 134 equals([14, 1, 15, 1, 16, 0, 18, 1,
135 23, 1, 24, 1, 26, 0])); 135 23, 1, 24, 1, 26, 0]));
136 expect(normalize(coverage['coverage'][1]['hits']).take(12), 136 expect(normalize(coverage['coverage'][1]['hits']).take(12),
137 equals([32, 0, 35, 0, 36, 0, 32, 1, 35, 1, 36, 0])); 137 equals([32, 1, 35, 1, 36, 0, 31, 1, 44, 0, 45, 0]));
138 }); 138 });
139 })); 139 }));
140 // Isolate 140 // Isolate
141 tests.add(cls.load().then((_) { 141 tests.add(cls.load().then((_) {
142 return isolate.invokeRpcNoUpgrade('getCoverage', {}) 142 return isolate.invokeRpcNoUpgrade('getCoverage', {})
143 .then((Map coverage) { 143 .then((Map coverage) {
144 expect(coverage['type'], equals('CodeCoverage')); 144 expect(coverage['type'], equals('CodeCoverage'));
145 expect(coverage['coverage'].length, greaterThan(100)); 145 expect(coverage['coverage'].length, greaterThan(100));
146 }); 146 });
147 })); 147 }));
148 return Future.wait(tests); 148 return Future.wait(tests);
149 }); 149 });
150 }, 150 },
151 151
152 ]; 152 ];
153 153
154 main(args) => runIsolateTests(args, tests, testeeConcurrent: testFunction); 154 main(args) => runIsolateTests(args, tests, testeeConcurrent: testFunction);
OLDNEW
« no previous file with comments | « runtime/observatory/test/contexts_test.dart ('k') | tools/testing/dart/test_options.dart » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698