Run outer tearDown()s even if inner ones fail.

Run outer tearDown()s even if inner ones fail.

R=kevmoo@google.com

Committed: https://github.com/dart-lang/test/commit/2c25c1ff64059574524413613512a6669fd35c26

[nweiz, 2015-09-24 20:23:58]

nweiz@google.com changed reviewers:
+ rnystrom@google.com

[kevmoo, 2015-09-24 20:39:40]

kevmoo@google.com changed reviewers:
+ kevmoo@google.com

[kevmoo, 2015-09-24 20:39:40]

lgtm

[nweiz, 2015-09-24 21:05:27]

I uploaded a change to make this more robust. PTAL

[kevmoo, 2015-09-24 21:07:30]

lgtm

LGTM

https://codereview.chromium.org/1364893004/diff/20001/lib/src/backend/group.dart
File lib/src/backend/group.dart (right):

https://codereview.chromium.org/1364893004/diff/20001/lib/src/backend/group.d...
lib/src/backend/group.dart:89: /// Runs [body] with special error-handling
behavior.
Is the case this addresses already covered in the tests?

[nweiz, 2015-09-24 21:13:07]

https://codereview.chromium.org/1364893004/diff/20001/lib/src/backend/group.dart
File lib/src/backend/group.dart (right):

https://codereview.chromium.org/1364893004/diff/20001/lib/src/backend/group.d...
lib/src/backend/group.dart:89: /// Runs [body] with special error-handling
behavior.
On 2015/09/24 21:07:30, kevmoo wrote:
> Is the case this addresses already covered in the tests?

Yeah; the test coincidentally passed because of the specific way this used to
work, but it does exercise this behavior.

[nweiz, 2015-09-24 21:13:42]

Committed patchset #2 (id:20001) manually as
2c25c1ff64059574524413613512a6669fd35c26 (presubmit successful).