Report total time for multiphase tests, in test script's --test option, not time of last phase.

Report total time for multiphase tests, in test script's --test option, not time of last phase.


BUG=
TEST=test.py --time

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

[Bill Hesse, 2012-05-14 12:15:45]

As seen in the following before-and-after comparison, reporting total times for
tests helps more in finding long-running tests:


$ tools/test.py --compiler=dart2js --runtime=d8 --time utils
Test configuration: dart2js_d8_debug_ia32
[02:02 | 100% | +    4 | -    0]


--- Total time: 02:02 ---
0:00:05.271 - dart2js-d8 debug_ia32 utils/utf8_test
0:00:00.382 - dart2js-d8 debug_ia32 utils/uri_test
0:00:00.259 - dart2js-d8 debug_ia32 utils/dummy_compiler_test
0:00:00.036 - dart2js-d8 debug_ia32 utils/json_test

$ tools/test.py --compiler=dart2js --runtime=d8 --time utils
Test configuration: dart2js_d8_debug_ia32
[01:58 | 100% | +    4 | -    0]


--- Total time: 01:58 ---
0:01:58.175 - dart2js-d8 debug_ia32 utils/dummy_compiler_test
0:00:19.501 - dart2js-d8 debug_ia32 utils/json_test
0:00:15.232 - dart2js-d8 debug_ia32 utils/uri_test
0:00:04.422 - dart2js-d8 debug_ia32 utils/utf8_test

[Mads Ager (google), 2012-05-14 12:20:16]

lgtm

http://codereview.chromium.org/10392078/diff/3/tools/testing/dart/test_runner...
File tools/testing/dart/test_runner.dart (right):

http://codereview.chromium.org/10392078/diff/3/tools/testing/dart/test_runner...
tools/testing/dart/test_runner.dart:435: Cannot have both @static-clean and ///
static type warning annotations.""");
Why not just split the line and use string concatenation by juxtaposition?

"Cannot have both @static-clean and "
"/// static type warning annotations."

I think that would be easier to read.

http://codereview.chromium.org/10392078/diff/3/tools/testing/dart/test_runner...
tools/testing/dart/test_runner.dart:448: //The test did not 'fail' in the way
Space after //

[ahe, 2012-05-14 12:23:26]

LGTM, thank you!