feat: add skip-packages to check licenses (#854)

Author: alestiago

lib/src/commands/packages/commands/check/commands/licenses.dart

@@ -65,6 +65,10 @@ class PackagesCheckLicensesCommand extends Command<int> {
       ..addMultiOption(
         'forbidden',
         help: 'Deny the use of certain licenses.',
+      )
+      ..addMultiOption(
+        'skip-packages',
+        help: 'Skip packages from having their licenses checked.',
       );
   }
 
@@ -94,6 +98,7 @@ class PackagesCheckLicensesCommand extends Command<int> {
     final dependencyTypes = _argResults['dependency-type'] as List<String>;
     final allowedLicenses = _argResults['allowed'] as List<String>;
     final forbiddenLicenses = _argResults['forbidden'] as List<String>;
+    final skippedPackages = _argResults['skip-packages'] as List<String>;
 
     if (allowedLicenses.isNotEmpty && forbiddenLicenses.isNotEmpty) {
       usageException(
@@ -135,6 +140,8 @@ class PackagesCheckLicensesCommand extends Command<int> {
       final isPubHosted = dependency.hosted != null;
       if (!isPubHosted) return false;
 
+      if (skippedPackages.contains(dependency.package())) return false;
+
       final dependencyType = dependency.type();
       return (dependencyTypes.contains('direct-main') &&
               dependencyType == DependencyType.direct) ||

test/src/commands/packages/commands/check/commands/licenses_test.dart

@@ -26,7 +26,8 @@ const _expectedPackagesCheckLicensesUsage = [
       '''          [transitive]               Check for transitive dependencies.\n'''
       '\n'
       '''    --allowed                        Only allow the use of certain licenses.\n'''
-      '''    --forbidden                      Deny the use of certain licenses.\n'''
+      '    --forbidden                      Deny the use of certain licenses.\n'
+      '''    --skip-packages                  Skip packages from having their licenses checked.\n'''
       '\n'
       'Run "very_good help" to see global options.'
 ];
@@ -909,6 +910,78 @@ void main() {
       });
     });
 
+    group('skip-packages', () {
+      const skipPackagesArgument = '--skip-packages';
+
+      group('skips', () {
+        test(
+          'a single package by name',
+          withRunner(
+              (commandRunner, logger, pubUpdater, pubLicense, printLogs) async {
+            final tempDirectory = Directory.systemTemp.createTempSync();
+            addTearDown(() => tempDirectory.deleteSync(recursive: true));
+
+            File(path.join(tempDirectory.path, pubspecLockBasename))
+                .writeAsStringSync(_validMultiplePubspecLockContent);
+
+            when(() => logger.progress(any())).thenReturn(progress);
+
+            final result = await commandRunner.run(
+              [
+                ...commandArguments,
+                skipPackagesArgument,
+                'cli_completion',
+                tempDirectory.path,
+              ],
+            );
+
+            verify(
+              () => progress.update('Collecting licenses of 0/1 packages.'),
+            ).called(1);
+            verify(
+              () => progress.complete(
+                '''Retrieved 1 license from 1 package of type: MIT (1).''',
+              ),
+            ).called(1);
+            expect(result, equals(ExitCode.success.code));
+          }),
+        );
+
+        test(
+          'multiple packages by name',
+          withRunner(
+              (commandRunner, logger, pubUpdater, pubLicense, printLogs) async {
+            final tempDirectory = Directory.systemTemp.createTempSync();
+            addTearDown(() => tempDirectory.deleteSync(recursive: true));
+
+            File(path.join(tempDirectory.path, pubspecLockBasename))
+                .writeAsStringSync(_validMultiplePubspecLockContent);
+
+            when(() => logger.progress(any())).thenReturn(progress);
+
+            final result = await commandRunner.run(
+              [
+                ...commandArguments,
+                skipPackagesArgument,
+                'cli_completion',
+                skipPackagesArgument,
+                'very_good_test_runner',
+                tempDirectory.path,
+              ],
+            );
+
+            final errorMessage =
+                '''No hosted dependencies found in ${tempDirectory.path} of type: direct-main.''';
+            verify(() => logger.err(errorMessage)).called(1);
+
+            verify(() => progress.cancel()).called(1);
+
+            expect(result, equals(ExitCode.usage.code));
+          }),
+        );
+      });
+    });
+
     group('exits with error', () {
       test(
         'when it did not find a pubspec.lock file at the target path',