Make the unittest framework more robust wrt asynchronous tests.

Make the unittest framework more robust wrt asynchronous tests.

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

[floitsch, 2012-12-05 19:39:50]

https://codereview.chromium.org/11451017/diff/1/pkg/unittest/lib/src/config.dart
File pkg/unittest/lib/src/config.dart (right):

https://codereview.chromium.org/11451017/diff/1/pkg/unittest/lib/src/config.d...
pkg/unittest/lib/src/config.dart:103: var resultString =
"${t.result}".toUpperCase();
I got null-pointer exceptions on "toUpperCase". Haven't investigated further,
but at least it shouldn't throw anymore.

[Bob Nystrom, 2012-12-05 21:21:54]

Adding Gram and taking myself off. The change looks fine to me, but it's been a
while since I've poked at unittest. Gram is more up to speed on it than I am.

[gram, 2012-12-05 21:39:04]

lgtm

[siva, 2012-12-20 21:27:20]

https://codereview.chromium.org/11451017/diff/1/pkg/unittest/lib/src/config.dart
File pkg/unittest/lib/src/config.dart (right):

https://codereview.chromium.org/11451017/diff/1/pkg/unittest/lib/src/config.d...
pkg/unittest/lib/src/config.dart:136: throw new Exception('Some tests failed.');
The _receivePort should be closed on the exception path too?

[floitsch, 2012-12-21 08:25:42]

https://codereview.chromium.org/11451017/diff/1/pkg/unittest/lib/src/config.dart
File pkg/unittest/lib/src/config.dart (right):

https://codereview.chromium.org/11451017/diff/1/pkg/unittest/lib/src/config.d...
pkg/unittest/lib/src/config.dart:136: throw new Exception('Some tests failed.');
On 2012/12/20 21:27:20, siva wrote:
> The _receivePort should be closed on the exception path too?

https://codereview.chromium.org/11570065
thanks.