Override and allow the hidden option to be set on MigrateCommand

`hidden` is implemented in the same manner that we do for flutter daemon, see https://github.com/flutter/flutter/blob/master/packages/flutter_tools/lib/src/commands/daemon.dart#L49

Change-Id: I6dcb4a4f5e1a71ba7e90956cd15b4b95524dd2bb
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/138106
Reviewed-by: Devon Carew <devoncarew@google.com>
Commit-Queue: Jaime Wren <jwren@google.com>

Author: jwren

pkg/dartdev/lib/dartdev.dart

@@ -27,7 +27,7 @@ class DartdevRunner<int> extends CommandRunner {
     addCommand(AnalyzeCommand(verbose: verbose));
     addCommand(CreateCommand(verbose: verbose));
     addCommand(FormatCommand(verbose: verbose));
-    addCommand(MigrateCommand(logProvider: () => log));
+    addCommand(MigrateCommand(logProvider: () => log, hidden: !verbose));
     addCommand(PubCommand(verbose: verbose));
     addCommand(TestCommand(verbose: verbose));
   }

pkg/dartdev/test/commands/flag_test.dart

@@ -27,6 +27,26 @@ void help() {
     expect(result.stdout, contains('Usage: dart <command> [arguments]'));
     expect(result.stdout, contains('Global options:'));
     expect(result.stdout, contains('Available commands:'));
+    expect(result.stdout, contains('analyze '));
+    expect(result.stdout, contains('create '));
+    expect(result.stdout, contains('format '));
+    expect(result.stdout, isNot(contains('migrate ')));
+  });
+
+  test('--help --verbose', () {
+    p = project();
+    var result = p.runSync('--help', ['--verbose']);
+
+    expect(result.exitCode, 0);
+    expect(result.stdout, contains('migrate '));
+  });
+
+  test('--help -v', () {
+    p = project();
+    var result = p.runSync('--help', ['-v']);
+
+    expect(result.exitCode, 0);
+    expect(result.stdout, contains('migrate '));
   });
 
   // For each command description, assert that the values are not empty, don't

pkg/dartfix/lib/src/migrate/migrate.dart

@@ -26,6 +26,9 @@ typedef LogProvider = Logger Function();
 class MigrateCommand extends Command {
   final LogProvider logProvider;
 
+  @override
+  final bool hidden;
+
   @override
   String get name => 'migrate';
 
@@ -38,7 +41,7 @@ class MigrateCommand extends Command {
     return '${super.invocation} [project or directory]';
   }
 
-  MigrateCommand({this.logProvider}) {
+  MigrateCommand({this.logProvider, this.hidden = false}) {
     MigrateOptions.defineOptions(argParser);
   }