Minor fixes to test.dart.

Minor fixes to test.dart.

Fix some switches, remove interfaces, and add 'incomplete' to
the TestOutput abstract class.

R=ricow@google.com,whesse@google.com
BUG=

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

[ricow1, 2012-10-03 10:39:02]

LGTM with one question

https://codereview.chromium.org/11047013/diff/1/tools/testing/dart/test_suite...
File tools/testing/dart/test_suite.dart (left):

https://codereview.chromium.org/11047013/diff/1/tools/testing/dart/test_suite...
tools/testing/dart/test_suite.dart:836: Expect.fail("configuration['runtime'] is
not set");
if we don't pass runtime in on the command line don't we hit this?

[Mads Ager (google), 2012-10-03 10:44:10]

https://codereview.chromium.org/11047013/diff/1/tools/testing/dart/test_suite...
File tools/testing/dart/test_suite.dart (left):

https://codereview.chromium.org/11047013/diff/1/tools/testing/dart/test_suite...
tools/testing/dart/test_suite.dart:836: Expect.fail("configuration['runtime'] is
not set");
On 2012/10/03 10:39:03, ricow1 wrote:
> if we don't pass runtime in on the command line don't we hit this?

At this point I'm just cleaning this up. 'runtime' is always set. The option has
a default value 'vm' which you can overwrite, so this method is a no-op. I was
just doing this in passing, but this hasRuntime check should go away.

[Mads Ager (google), 2012-10-03 10:46:29]

On 2012/10/03 10:44:10, Mads Ager wrote:
>
https://codereview.chromium.org/11047013/diff/1/tools/testing/dart/test_suite...
> File tools/testing/dart/test_suite.dart (left):
> 
>
https://codereview.chromium.org/11047013/diff/1/tools/testing/dart/test_suite...
> tools/testing/dart/test_suite.dart:836: Expect.fail("configuration['runtime']
is
> not set");
> On 2012/10/03 10:39:03, ricow1 wrote:
> > if we don't pass runtime in on the command line don't we hit this?
> 
> At this point I'm just cleaning this up. 'runtime' is always set. The option
has
> a default value 'vm' which you can overwrite, so this method is a no-op. I was
> just doing this in passing, but this hasRuntime check should go away.

That is not true, it shouldn't go away. What I did was the right thing. It will
never be null but it could be 'none'. :-)