Chromium Code Reviews| 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 1580 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 1591 // The test is not yet ready to run. Put the test back in | 1591 // The test is not yet ready to run. Put the test back in |
| 1592 // the queue. Avoid spin-polling by using a timeout. | 1592 // the queue. Avoid spin-polling by using a timeout. |
| 1593 _tests.add(test); | 1593 _tests.add(test); |
| 1594 new Timer(100, (_) => _tryRunTest()); // Don't lose a process. | 1594 new Timer(100, (_) => _tryRunTest()); // Don't lose a process. |
| 1595 return; | 1595 return; |
| 1596 } | 1596 } |
| 1597 if (_verbose) { | 1597 if (_verbose) { |
| 1598 int i = 1; | 1598 int i = 1; |
| 1599 if (test is BrowserTestCase) { | 1599 if (test is BrowserTestCase) { |
| 1600 // Additional command for rerunning the steps locally after the fact. | 1600 // Additional command for rerunning the steps locally after the fact. |
| 1601 print('$i. ${TestUtils.dartTestExecutable.toNativePath()} ' | 1601 print('$i. ${TestUtils.dartTestExecutable.toNativePath()} ' |
|
ricow1
2013/02/01 10:19:12
The fact that we do the exact same thing two place
kustermann
2013/02/01 10:22:17
That is true. It actually belongs IMHO to http_ser
| |
| 1602 '${TestUtils.dartDir().toNativePath()}/tools/testing/dart/' | 1602 '${TestUtils.dartDir().toNativePath()}/tools/testing/dart/' |
| 1603 'http_server.dart -m ${test.configuration["mode"]} ' | 1603 'http_server.dart -m ${test.configuration["mode"]} ' |
| 1604 '-a ${test.configuration["arch"]} ' | 1604 '-a ${test.configuration["arch"]} ' |
|
ahe
2013/02/01 10:28:59
-m and -a unused.
| |
| 1605 '-p ${http_server.TestingServerRunner.serverList[0].port} ' | 1605 '-p ${http_server.TestingServerRunner.serverList[0].port} ' |
| 1606 '-c ${http_server.TestingServerRunner.serverList[1].port}'); | 1606 '-c ${http_server.TestingServerRunner.serverList[1].port} ' |
| 1607 '--package-root=' | |
| 1608 '${http_server.TestingServerRunner.packageRootDir}'); | |
| 1607 i++; | 1609 i++; |
| 1608 } | 1610 } |
| 1609 for (Command command in test.commands) { | 1611 for (Command command in test.commands) { |
| 1610 print('$i. ${command.commandLine}'); | 1612 print('$i. ${command.commandLine}'); |
| 1611 i++; | 1613 i++; |
| 1612 } | 1614 } |
| 1613 } | 1615 } |
| 1614 | 1616 |
| 1615 var isLastCommand = | 1617 var isLastCommand = |
| 1616 ((test.commands.length-1) == test.commandOutputs.length); | 1618 ((test.commands.length-1) == test.commandOutputs.length); |
| (...skipping 132 matching lines...) Expand 10 before | Expand all | Expand 10 after Loading... | |
| 1749 completer.complete(testCase); | 1751 completer.complete(testCase); |
| 1750 } | 1752 } |
| 1751 }); | 1753 }); |
| 1752 } | 1754 } |
| 1753 runCommand(); | 1755 runCommand(); |
| 1754 | 1756 |
| 1755 return completer.future; | 1757 return completer.future; |
| 1756 } | 1758 } |
| 1757 } | 1759 } |
| 1758 | 1760 |
| OLD | NEW |