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

Side by Side Diff: tools/testing/dart/test_runner.dart

Issue 23368003: Cleanup of isolate.status (first step) (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 7 years, 4 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
OLDNEW
1 // Copyright (c) 2012, the Dart project authors. Please see the AUTHORS file 1 // Copyright (c) 2012, 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 /** 5 /**
6 * Classes and methods for executing tests. 6 * Classes and methods for executing tests.
7 * 7 *
8 * This module includes: 8 * This module includes:
9 * - Managing parallel execution of tests, including timeout checks. 9 * - Managing parallel execution of tests, including timeout checks.
10 * - Evaluating the output of each test as pass/fail/crash/timeout. 10 * - Evaluating the output of each test as pass/fail/crash/timeout.
(...skipping 522 matching lines...) Expand 10 before | Expand all | Expand 10 after
533 final arch = configuration['arch']; 533 final arch = configuration['arch'];
534 final checked = configuration['checked'] ? '-checked' : ''; 534 final checked = configuration['checked'] ? '-checked' : '';
535 return "$compiler-$runtime$checked ${mode}_$arch"; 535 return "$compiler-$runtime$checked ${mode}_$arch";
536 } 536 }
537 537
538 List<String> get batchTestArguments => commands.last.arguments; 538 List<String> get batchTestArguments => commands.last.arguments;
539 539
540 bool get usesWebDriver => TestUtils.usesWebDriver(configuration['runtime']); 540 bool get usesWebDriver => TestUtils.usesWebDriver(configuration['runtime']);
541 541
542 bool get isFlaky { 542 bool get isFlaky {
543 if (expectedOutcomes.contains(SKIP)) { 543 if (expectedOutcomes.contains(SKIP) ||
544 expectedOutcomes.contains(SKIP_BY_DESIGN)) {
544 return false; 545 return false;
545 } 546 }
546 547
547 var flags = new Set.from(expectedOutcomes); 548 var flags = new Set.from(expectedOutcomes);
548 flags..remove(TIMEOUT) 549 flags..remove(TIMEOUT)
549 ..remove(SLOW); 550 ..remove(SLOW);
550 return flags.contains(PASS) && flags.length > 1; 551 return flags.contains(PASS) && flags.length > 1;
551 } 552 }
552 553
553 bool get isFinished { 554 bool get isFinished {
(...skipping 1588 matching lines...) Expand 10 before | Expand all | Expand 10 after
2142 } 2143 }
2143 } 2144 }
2144 2145
2145 void eventAllTestsDone() { 2146 void eventAllTestsDone() {
2146 for (var listener in _eventListener) { 2147 for (var listener in _eventListener) {
2147 listener.allDone(); 2148 listener.allDone();
2148 } 2149 }
2149 _allDone(); 2150 _allDone();
2150 } 2151 }
2151 } 2152 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698