fix two test failures in analyzer_cli/test/options_test.dart

pkg/analyzer_cli/test/options_test.dart

 // Copyright (c) 2015, 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.
 
 library analyzer_cli.test.options;
 
 import 'dart:io';
 
 import 'package:analyzer_cli/src/driver.dart';
 import 'package:analyzer_cli/src/options.dart';
+import 'package:telemetry/telemetry.dart' as telemetry;
 import 'package:test/test.dart';
+import 'package:test_reflective_loader/test_reflective_loader.dart';
 import 'package:usage/usage.dart';
-import 'package:test_reflective_loader/test_reflective_loader.dart';
 
 main() {
   group('CommandLineOptions', () {
     group('parse', () {
       int lastExitHandlerCode;
       StringBuffer outStringBuffer = new StringBuffer();
       StringBuffer errorStringBuffer = new StringBuffer();
 
       StringSink savedOutSink, savedErrorSink;
       int savedExitCode;
@@ skipping 203 matching lines @@
             equals("Cannot specify both '--package-root' and '--packages."));
       });
 
       test("bad SDK dir", () {
         var failureMessage;
         CommandLineOptions.parse(['--dart-sdk', '&&&&&', 'foo.dart'],
             printAndFail: (msg) => failureMessage = msg);
         expect(failureMessage, equals('Invalid Dart SDK path: &&&&&'));
       });
 
-      test('--analytics', () {
-        AnalyticsMock mock = new AnalyticsMock()..enabled = false;
-        setAnalytics(mock);
-        CommandLineOptions.parse(['--analytics']);
-        expect(mock.enabled, true);
-        expect(lastExitHandlerCode, 0);
-        expect(outStringBuffer.toString(), contains('Analytics are currently'));
-      });
+      if (telemetry.SHOW_ANALYTICS_UI) {
+        test('--analytics', () {
+          AnalyticsMock mock = new AnalyticsMock()..enabled = false;
+          setAnalytics(mock);
+          CommandLineOptions.parse(['--analytics']);
+          expect(mock.enabled, true);
+          expect(lastExitHandlerCode, 0);
+          expect(
+              outStringBuffer.toString(), contains('Analytics are currently'));
+        });
 
-      test('--no-analytics', () {
-        AnalyticsMock mock = new AnalyticsMock()..enabled = false;
-        setAnalytics(mock);
-        CommandLineOptions.parse(['--no-analytics']);
-        expect(mock.enabled, false);
-        expect(lastExitHandlerCode, 0);
-        expect(outStringBuffer.toString(), contains('Analytics are currently'));
-      });
+        test('--no-analytics', () {
+          AnalyticsMock mock = new AnalyticsMock()..enabled = false;
+          setAnalytics(mock);
+          CommandLineOptions.parse(['--no-analytics']);
+          expect(mock.enabled, false);
+          expect(lastExitHandlerCode, 0);
+          expect(
+              outStringBuffer.toString(), contains('Analytics are currently'));
+        });
+      }
     });
   });
   defineReflectiveTests(CommandLineOptionsTest);
 }
 
 @reflectiveTest
 class AbstractStatusTest {
   int lastExitHandlerCode;
   StringBuffer outStringBuffer = new StringBuffer();
   StringBuffer errorStringBuffer = new StringBuffer();
@@ skipping 142 matching lines @@
       'package:p/foo.dart|/path/to/p/lib/foo.dart'
     ]);
     expect(options.buildMode, isTrue);
     expect(options.buildSuppressExitCode, isTrue);
   }
 
   void _parse(List<String> args) {
     options = CommandLineOptions.parse(args);
   }
 }