Changed: TestCase.output -> TestCase.commandOutputs[]

Changed: TestCase.output -> TestCase.commandOutputs[]

Currently the TestCase has exactly one TestOutput. This worked
until now, since we were only interested in the output of the
last command.

This change introduces one CommandOutput for each command which
has been run. Doing so enables us to fetch output (stdin,stderr)
information as well as exitCode ... for all commands.

See also 11293278.

Committed: https://code.google.com/p/dart/source/detail?r=15006

[kustermann, 2012-11-16 10:08:58]

PTAL. Since the old issue '11293278' was closed, so I created a new one.

A bug in "RunningProcess.stepExitHandler/commandComplete" is fixed now: in the
else branch, the command output didn't get created.

[ricow1, 2012-11-16 10:36:56]

lgtm

https://codereview.chromium.org/11348088/diff/1/tools/testing/dart/test_progr...
File tools/testing/dart/test_progress.dart (right):

https://codereview.chromium.org/11348088/diff/1/tools/testing/dart/test_progr...
tools/testing/dart/test_progress.dart:78: _tests.sort((a, b) {
add a todo here, we should take all the commands into acount

[Bill Hesse, 2012-11-16 10:37:33]

LGTM.

[kustermann, 2012-11-16 11:14:25]

https://codereview.chromium.org/11348088/diff/1/tools/testing/dart/test_progr...
File tools/testing/dart/test_progress.dart (right):

https://codereview.chromium.org/11348088/diff/1/tools/testing/dart/test_progr...
tools/testing/dart/test_progress.dart:78: _tests.sort((a, b) {
On 2012/11/16 10:36:56, ricow1 wrote:
> add a todo here, we should take all the commands into acount

Done.