OLD | NEW |
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 1472 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
1483 } | 1483 } |
1484 | 1484 |
1485 /** | 1485 /** |
1486 * Perform any cleanup needed once all tests in a TestSuite have completed | 1486 * Perform any cleanup needed once all tests in a TestSuite have completed |
1487 * and notify our progress indicator that we are done. | 1487 * and notify our progress indicator that we are done. |
1488 */ | 1488 */ |
1489 void _cleanupAndMarkDone() { | 1489 void _cleanupAndMarkDone() { |
1490 _allDone(); | 1490 _allDone(); |
1491 if (browserUsed != '' && _seleniumServer != null) { | 1491 if (browserUsed != '' && _seleniumServer != null) { |
1492 _seleniumServer.kill(); | 1492 _seleniumServer.kill(); |
1493 } else { | |
1494 _progress.allDone(); | |
1495 } | 1493 } |
| 1494 _progress.allDone(); |
1496 } | 1495 } |
1497 | 1496 |
1498 void _checkDone() { | 1497 void _checkDone() { |
1499 if (_allTestsWereEnqueued && _tests.isEmpty && _numProcesses == 0) { | 1498 if (_allTestsWereEnqueued && _tests.isEmpty && _numProcesses == 0) { |
1500 _terminateBatchRunners().then((_) => _cleanupAndMarkDone()); | 1499 _terminateBatchRunners().then((_) => _cleanupAndMarkDone()); |
1501 } | 1500 } |
1502 } | 1501 } |
1503 | 1502 |
1504 void _runTests(List<TestSuite> testSuites) { | 1503 void _runTests(List<TestSuite> testSuites) { |
1505 // FIXME: For some reason we cannot call this method on all test suites | 1504 // FIXME: For some reason we cannot call this method on all test suites |
(...skipping 241 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... |
1747 // the developer doesn't waste his or her time trying to fix a bunch of | 1746 // the developer doesn't waste his or her time trying to fix a bunch of |
1748 // tests that appear to be broken but were actually just flakes that | 1747 // tests that appear to be broken but were actually just flakes that |
1749 // didn't get retried because there had already been one failure. | 1748 // didn't get retried because there had already been one failure. |
1750 bool allowRetry = _MAX_FAILED_NO_RETRY > _progress.numFailedTests; | 1749 bool allowRetry = _MAX_FAILED_NO_RETRY > _progress.numFailedTests; |
1751 new RunningProcess(test, allowRetry, this).start(); | 1750 new RunningProcess(test, allowRetry, this).start(); |
1752 } | 1751 } |
1753 _numProcesses++; | 1752 _numProcesses++; |
1754 } | 1753 } |
1755 } | 1754 } |
1756 } | 1755 } |
OLD | NEW |