starting to work on unit test config interface

starting to work on unit test config interface

R=gram@google.com

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

[kevmoo-old, 2013-08-15 17:55:39]

Configuration introduces a lot of members that aren't used by the core test
system but basically exist to support subclasses.

It also has a relatively opinionated implementation.

This leaves folks who wish to create a radically different Configuration model
to implement a lot of superfluous members from the base class.

This CL breaks Configuration into:

A minimal abstract class Configuration that has only the members needed by the
test system

The 'legacy' implementation moved to SimpleConfiguration.

[gram, 2013-08-15 20:38:00]

lgtm, but I think you should add a todo to check whether each of the various
configurations we have like vm, html, etc need to inherit from
SimpleConfiguration or whether they would be cleaner inheriting just from
Configuration.

https://codereview.chromium.org/22883008/diff/9001/pkg/unittest/lib/src/confi...
File pkg/unittest/lib/src/config.dart (left):

https://codereview.chromium.org/22883008/diff/9001/pkg/unittest/lib/src/confi...
pkg/unittest/lib/src/config.dart:2: // for details. All rights reserved. Use of
this source code is governed by a
Why empty? Shouldn't it be deleted?

[kevmoo-old, 2013-08-15 20:46:16]

I'll open a bug (CC you and me) to look into the subclasses

https://codereview.chromium.org/22883008/diff/9001/pkg/unittest/lib/src/confi...
File pkg/unittest/lib/src/config.dart (left):

https://codereview.chromium.org/22883008/diff/9001/pkg/unittest/lib/src/confi...
pkg/unittest/lib/src/config.dart:2: // for details. All rights reserved. Use of
this source code is governed by a
On 2013/08/15 20:38:00, gram wrote:
> Why empty? Shouldn't it be deleted?

It's deleted in my world.

Was it still around (empty?) when you applied the patch?

[kevmoo-old, 2013-08-15 20:48:41]

Committed patchset #4 manually as r26215 (presubmit successful).