utility for manually running all analysis_server tests

utility for manually running all analysis_server tests

BUG=
R=brianwilkerson@google.com

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

[Brian Wilkerson, 2014-03-22 14:37:20]

https://codereview.chromium.org/209003005/diff/1/pkg/analysis_server/test/cha...
File pkg/analysis_server/test/channel_test.dart (right):

https://codereview.chromium.org/209003005/diff/1/pkg/analysis_server/test/cha...
pkg/analysis_server/test/channel_test.dart:16: main(List<String> args) {
I'm not sure what the value of adding 'args' is when we're not using them.

https://codereview.chromium.org/209003005/diff/1/pkg/analysis_server/test/tes...
File pkg/analysis_server/test/test_runner.dart (right):

https://codereview.chromium.org/209003005/diff/1/pkg/analysis_server/test/tes...
pkg/analysis_server/test/test_runner.dart:1: // Run all tests
Missing standard copyright (and IMO the comment is redundant with the comment on
'main').

Traditionally I have called these files "TestAll". I plan to continue to use
that convention in analyzer and would prefer to use that convention here.

https://codereview.chromium.org/209003005/diff/1/pkg/analysis_server/test/tes...
pkg/analysis_server/test/test_runner.dart:8: // Utility for manually running all
tests
"//" --> "///"

https://codereview.chromium.org/209003005/diff/1/pkg/analysis_server/test/tes...
pkg/analysis_server/test/test_runner.dart:10: print("Running all Analysi Server
cmdline tests...");
"Analysi" --> "Analysis"

Also, rather than 'print' I would use 'group'? Then these files can nest nicely
when we organize them into subdirectories.

[danrubel, 2014-03-25 19:03:58]

https://codereview.chromium.org/209003005/diff/1/pkg/analysis_server/test/cha...
File pkg/analysis_server/test/channel_test.dart (right):

https://codereview.chromium.org/209003005/diff/1/pkg/analysis_server/test/cha...
pkg/analysis_server/test/channel_test.dart:16: main(List<String> args) {
On 2014/03/22 14:37:20, Brian Wilkerson wrote:
> I'm not sure what the value of adding 'args' is when we're not using them.

Good point. Removed here and elsewhere.

https://codereview.chromium.org/209003005/diff/1/pkg/analysis_server/test/tes...
File pkg/analysis_server/test/test_runner.dart (right):

https://codereview.chromium.org/209003005/diff/1/pkg/analysis_server/test/tes...
pkg/analysis_server/test/test_runner.dart:1: // Run all tests
On 2014/03/22 14:37:20, Brian Wilkerson wrote:
> Missing standard copyright (and IMO the comment is redundant with the comment
on
> 'main').
> 
> Traditionally I have called these files "TestAll". I plan to continue to use
> that convention in analyzer and would prefer to use that convention here.

Done.

https://codereview.chromium.org/209003005/diff/1/pkg/analysis_server/test/tes...
pkg/analysis_server/test/test_runner.dart:8: // Utility for manually running all
tests
On 2014/03/22 14:37:20, Brian Wilkerson wrote:
> "//" --> "///"

Done.

https://codereview.chromium.org/209003005/diff/1/pkg/analysis_server/test/tes...
pkg/analysis_server/test/test_runner.dart:10: print("Running all Analysi Server
cmdline tests...");
On 2014/03/22 14:37:20, Brian Wilkerson wrote:
> "Analysi" --> "Analysis"
> 
> Also, rather than 'print' I would use 'group'? Then these files can nest
nicely
> when we organize them into subdirectories.

Good idea. Done.

[Brian Wilkerson, 2014-03-28 17:53:00]

Sorry about the delay! LGTM

[danrubel, 2014-03-28 20:29:34]

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