commit 9c895fabf2c08686ad8f051f1c69b0a812bae6da
author Victor Choueiri <victor.oc@gmail.com> Wed Feb 28 01:19:32 2018 +0200
committer Devon Carew <devoncarew@google.com> Tue Feb 27 15:19:32 2018 -0800
tree 813bdf756af0a426a07d73f21fbef4f89ce92b06
parent a5d3a9eb4bd1229f8cebfe003a4c2595e5ebcdd2

Add intelliJ path to doctor output (#14795)

packages/flutter_tools/lib/src/doctor.dart

diff --git a/packages/flutter_tools/lib/src/doctor.dart b/packages/flutter_tools/lib/src/doctor.dart
index d4f8a15..dcb12b8 100644
--- a/packages/flutter_tools/lib/src/doctor.dart
+++ b/packages/flutter_tools/lib/src/doctor.dart
@@ -276,7 +276,9 @@
 }
 
 abstract class IntelliJValidator extends DoctorValidator {
-  IntelliJValidator(String title) : super(title);
+  final String installPath;
+
+  IntelliJValidator(String title, this.installPath) : super(title);
 
   String get version;
   String get pluginsPath;
@@ -301,6 +303,8 @@
   Future<ValidationResult> validate() async {
     final List<ValidationMessage> messages = <ValidationMessage>[];
 
+    messages.add(new ValidationMessage('IntelliJ at $installPath'));
+
     _validatePackage(messages, <String>['flutter-intellij', 'flutter-intellij.jar'],
         'Flutter', minVersion: kMinFlutterPluginVersion);
     _validatePackage(messages, <String>['Dart'], 'Dart');
@@ -397,13 +401,11 @@
 }
 
 class IntelliJValidatorOnLinuxAndWindows extends IntelliJValidator {
-  IntelliJValidatorOnLinuxAndWindows(String title, this.version, this.installPath, this.pluginsPath) : super(title);
+  IntelliJValidatorOnLinuxAndWindows(String title, this.version, String installPath, this.pluginsPath) : super(title, installPath);
 
   @override
   String version;
 
-  final String installPath;
-
   @override
   String pluginsPath;
 
@@ -451,10 +453,9 @@
 }
 
 class IntelliJValidatorOnMac extends IntelliJValidator {
-  IntelliJValidatorOnMac(String title, this.id, this.installPath) : super(title);
+  IntelliJValidatorOnMac(String title, this.id, String installPath) : super(title, installPath);
 
   final String id;
-  final String installPath;
 
   static final Map<String, String> _dirNameToId = <String, String>{
     'IntelliJ IDEA.app' : 'IntelliJIdea',

packages/flutter_tools/test/commands/doctor_test.dart

diff --git a/packages/flutter_tools/test/commands/doctor_test.dart b/packages/flutter_tools/test/commands/doctor_test.dart
index 69405a2b..ea2a3c9 100644
--- a/packages/flutter_tools/test/commands/doctor_test.dart
+++ b/packages/flutter_tools/test/commands/doctor_test.dart
@@ -15,12 +15,17 @@
 void main() {
   group('doctor', () {
     testUsingContext('intellij validator', () async {
-      final ValidationResult result = await new IntelliJValidatorTestTarget('Test').validate();
+      final String installPath = '/path/to/intelliJ';
+      final ValidationResult result = await new IntelliJValidatorTestTarget('Test', installPath).validate();
       expect(result.type, ValidationType.partial);
       expect(result.statusInfo, 'version test.test.test');
-      expect(result.messages, hasLength(3));
+      expect(result.messages, hasLength(4));
 
       ValidationMessage message = result.messages
+          .firstWhere((ValidationMessage m) => m.message.startsWith('IntelliJ '));
+      expect(message.message, 'IntelliJ at $installPath');
+
+      message = result.messages
           .firstWhere((ValidationMessage m) => m.message.startsWith('Dart '));
       expect(message.message, 'Dart plugin version 162.2485');
 
@@ -148,7 +153,7 @@
 }
 
 class IntelliJValidatorTestTarget extends IntelliJValidator {
-  IntelliJValidatorTestTarget(String title) : super(title);
+  IntelliJValidatorTestTarget(String title, String installPath) : super(title, installPath);
 
   @override
   String get pluginsPath => fs.path.join('test', 'data', 'intellij', 'plugins');