Add analytics to analyzer-cli and analysis server.

pkg/analysis_server/test/domain_server_test.dart

 // Copyright (c) 2014, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
+import 'dart:async';
+
 import 'package:analysis_server/protocol/protocol.dart';
 import 'package:analysis_server/protocol/protocol_constants.dart';
 import 'package:analysis_server/protocol/protocol_generated.dart';
 import 'package:analysis_server/src/analysis_server.dart';
 import 'package:analysis_server/src/constants.dart';
 import 'package:analysis_server/src/domain_server.dart';
 import 'package:analysis_server/src/plugin/server_plugin.dart';
 import 'package:analyzer/file_system/memory_file_system.dart';
 import 'package:analyzer/instrumentation/instrumentation.dart';
 import 'package:analyzer/src/generated/sdk.dart';
@@ skipping 10 matching lines @@
     var serverChannel = new MockServerChannel();
     var resourceProvider = new MemoryResourceProvider();
     ExtensionManager manager = new ExtensionManager();
     ServerPlugin serverPlugin = new ServerPlugin();
     manager.processPlugins([serverPlugin]);
     server = new AnalysisServer(
         serverChannel,
         resourceProvider,
         new MockPackageMapProvider(),
         serverPlugin,
-        new AnalysisServerOptions(),
+        new AnalysisServerOptions()..addMocks(),
         new DartSdkManager('', false),
         InstrumentationService.NULL_SERVICE);
     handler = new ServerDomainHandler(server);
   });
 
   group('ServerDomainHandler', () {
     test('getVersion', () {
       var request = new ServerGetVersionParams().toRequest('0');
       var response = handler.handleRequest(request);
       expect(
@@ skipping 19 matching lines @@
         Request request =
             new ServerSetSubscriptionsParams([ServerService.STATUS])
                 .toRequest('0');
         var response = handler.handleRequest(request);
         expect(response, isResponseSuccess('0'));
         // set of services has been changed
         expect(server.serverServices, contains(ServerService.STATUS));
       });
     });
 
-    test('shutdown', () {
+    test('shutdown', () async {
       expect(server.running, isTrue);
       // send request
       var request = new ServerShutdownParams().toRequest('0');
       var response = handler.handleRequest(request);
       expect(response, isResponseSuccess('0'));
+
+      // delay a bit
+      await new Future.delayed(new Duration(seconds: 1));

[scheglov, 2017/06/30 23:36:30]

Please don't add unconditional delays.
We run these these tests many times each days, and waiting the whole second is
not fun :-(

It is also not obvious why we should wait at all.
The comment does not explain this :-(

[devoncarew, 2017/07/01 22:18:05]

This has been re-worked to not use a delay.

+
       // server is down
       expect(server.running, isFalse);
     });
   });
 }