Code completion build fix following https://codereview.chromium.org/1860213002, fix test_all.dart t…

pkg/analysis_server/test/test_all.dart

Index: pkg/analysis_server/test/test_all.dart
diff --git a/pkg/analysis_server/test/test_all.dart b/pkg/analysis_server/test/test_all.dart
index 0a4a883e4a4add98909f77fd18980c5019bdddbd..994f5ef620a4b2dbe3a920bd9243af4586df49f1 100644
--- a/pkg/analysis_server/test/test_all.dart
+++ b/pkg/analysis_server/test/test_all.dart
@@ -7,6 +7,7 @@ import 'package:unittest/unittest.dart';
 import 'analysis/test_all.dart' as analysis_all;
 import 'analysis_server_test.dart' as analysis_server_test;
 import 'channel/test_all.dart' as channel_test;
+import 'completion_test.dart' as completion_test;
 import 'context_manager_test.dart' as context_manager_test;
 import 'domain_analysis_test.dart' as domain_analysis_test;
 import 'domain_completion_test.dart' as domain_completion_test;
@@ -35,6 +36,7 @@ main() {
     analysis_all.main();
     analysis_server_test.main();
     channel_test.main();
+    completion_test.main();

[danrubel, 2016/04/06 20:16:03]

I believe that the bots run this test separately, so I don't think it belongs
here. Ok to leave it here and run the tests twice if Brian et al are ok with
that.

[Paul Berry, 2016/04/06 20:22:35]

Actually, this won't cause the bots to run the test twice, because the bots
don't run test_all.dart.  They only run tests ending with "_test.dart".  This
file exists merely for the convenience of analyzer developers, some of whom
prefer to run all the tests in a single invocation (e.g. with an IDE) rather
than via test.sh.

So I think it makes sense to add this line, since it will ensure that when
developers run tests in this way, they don't miss out on failures in
completion_test.

     context_manager_test.main();
     domain_analysis_test.main();
     domain_completion_test.main();