diff --git a/packages/shorebird_tools/lib/src/command_runner.dart b/packages/shorebird_tools/lib/src/command_runner.dart index e4a4cb1eb1814..25c5fc4ac0ad9 100644 --- a/packages/shorebird_tools/lib/src/command_runner.dart +++ b/packages/shorebird_tools/lib/src/command_runner.dart @@ -35,7 +35,6 @@ class ShorebirdToolsCommandRunner extends CommandRunner { ); // Add sub commands - addCommand(SampleCommand(logger: _logger)); addCommand(PackageCommand(logger: _logger)); } diff --git a/packages/shorebird_tools/lib/src/commands/commands.dart b/packages/shorebird_tools/lib/src/commands/commands.dart index 34ceda22503ab..046111b80c91f 100644 --- a/packages/shorebird_tools/lib/src/commands/commands.dart +++ b/packages/shorebird_tools/lib/src/commands/commands.dart @@ -1,2 +1 @@ export 'package_command.dart'; -export 'sample_command.dart'; diff --git a/packages/shorebird_tools/lib/src/commands/sample_command.dart b/packages/shorebird_tools/lib/src/commands/sample_command.dart deleted file mode 100644 index b332cc914a0b4..0000000000000 --- a/packages/shorebird_tools/lib/src/commands/sample_command.dart +++ /dev/null @@ -1,39 +0,0 @@ -import 'package:args/command_runner.dart'; -import 'package:mason_logger/mason_logger.dart'; - -/// {@template sample_command} -/// -/// `shorebird_tools sample` -/// A [Command] to exemplify a sub command -/// {@endtemplate} -class SampleCommand extends Command { - /// {@macro sample_command} - SampleCommand({ - required Logger logger, - }) : _logger = logger { - argParser.addFlag( - 'cyan', - abbr: 'c', - help: 'Prints the same joke, but in cyan', - negatable: false, - ); - } - - @override - String get description => 'A sample sub command that just prints one joke'; - - @override - String get name => 'sample'; - - final Logger _logger; - - @override - Future run() async { - var output = 'Which unicorn has a cold? The Achoo-nicorn!'; - if (argResults?['cyan'] == true) { - output = lightCyan.wrap(output)!; - } - _logger.info(output); - return ExitCode.success.code; - } -} diff --git a/packages/shorebird_tools/test/src/command_runner_test.dart b/packages/shorebird_tools/test/src/command_runner_test.dart index 6de70d8d68556..ac334fed94e41 100644 --- a/packages/shorebird_tools/test/src/command_runner_test.dart +++ b/packages/shorebird_tools/test/src/command_runner_test.dart @@ -76,22 +76,6 @@ void main() { verify(() => logger.detail(' - verbose: true')).called(1); verifyNever(() => logger.detail(' Command options:')); }); - - test('enables verbose logging for sub commands', () async { - final result = await commandRunner.run([ - '--verbose', - 'sample', - '--cyan', - ]); - expect(result, equals(ExitCode.success.code)); - - verify(() => logger.detail('Argument information:')).called(1); - verify(() => logger.detail(' Top level options:')).called(1); - verify(() => logger.detail(' - verbose: true')).called(1); - verify(() => logger.detail(' Command: sample')).called(1); - verify(() => logger.detail(' Command options:')).called(1); - verify(() => logger.detail(' - cyan: true')).called(1); - }); }); }); } diff --git a/packages/shorebird_tools/test/src/commands/sample_command_test.dart b/packages/shorebird_tools/test/src/commands/sample_command_test.dart deleted file mode 100644 index f0a55a5ad31e8..0000000000000 --- a/packages/shorebird_tools/test/src/commands/sample_command_test.dart +++ /dev/null @@ -1,58 +0,0 @@ -import 'package:mason_logger/mason_logger.dart'; -import 'package:mocktail/mocktail.dart'; -import 'package:shorebird_tools/src/command_runner.dart'; -import 'package:test/test.dart'; - -class _MockLogger extends Mock implements Logger {} - -void main() { - group('sample', () { - late Logger logger; - late ShorebirdToolsCommandRunner commandRunner; - - setUp(() { - logger = _MockLogger(); - commandRunner = ShorebirdToolsCommandRunner(logger: logger); - }); - - test('tells a joke', () async { - final exitCode = await commandRunner.run(['sample']); - - expect(exitCode, ExitCode.success.code); - - verify( - () => logger.info('Which unicorn has a cold? The Achoo-nicorn!'), - ).called(1); - }); - test('tells a joke in cyan', () async { - final exitCode = await commandRunner.run(['sample', '-c']); - - expect(exitCode, ExitCode.success.code); - - verify( - () => logger.info( - lightCyan.wrap('Which unicorn has a cold? The Achoo-nicorn!'), - ), - ).called(1); - }); - - test('wrong usage', () async { - final exitCode = await commandRunner.run(['sample', '-p']); - - expect(exitCode, ExitCode.usage.code); - - verify(() => logger.err('Could not find an option or flag "-p".')) - .called(1); - verify( - () => logger.info( - ''' -Usage: $executableName sample [arguments] --h, --help Print this usage information. --c, --cyan Prints the same joke, but in cyan - -Run "$executableName help" to see global options.''', - ), - ).called(1); - }); - }); -}