feat: modified collection progress message on check license (#855)

VeryGoodOpenSource · Oct 18, 2023 · 973eb41 · 973eb41
1 parent 0662587
commit 973eb41
Show file tree

Hide file tree

Showing 2 changed files with 52 additions and 18 deletions.
diff --git a/lib/src/commands/packages/commands/check/commands/licenses.dart b/lib/src/commands/packages/commands/check/commands/licenses.dart
@@ -162,7 +162,7 @@ class PackagesCheckLicensesCommand extends Command<int> {
     final licenses = <String, Set<String>?>{};
     for (final dependency in filteredDependencies) {
       progress.update(
-        'Collecting licenses of ${licenses.length}/${filteredDependencies.length} packages.',
+        '''Collecting licenses from ${licenses.length + 1} out of ${filteredDependencies.length} ${filteredDependencies.length == 1 ? 'package' : 'packages'}''',
       );
 
       final dependencyName = dependency.package();

diff --git a/test/src/commands/packages/commands/check/commands/licenses_test.dart b/test/src/commands/packages/commands/check/commands/licenses_test.dart
@@ -113,7 +113,9 @@ void main() {
             );
 
             verify(
-              () => progress.update('Collecting licenses of 0/1 packages.'),
+              () => progress.update(
+                'Collecting licenses from 1 out of 1 package',
+              ),
             ).called(1);
             verify(
               () => progress.complete(
@@ -145,10 +147,14 @@ void main() {
             );
 
             verify(
-              () => progress.update('Collecting licenses of 0/2 packages.'),
+              () => progress.update(
+                'Collecting licenses from 1 out of 2 packages',
+              ),
             ).called(1);
             verify(
-              () => progress.update('Collecting licenses of 1/2 packages.'),
+              () => progress.update(
+                'Collecting licenses from 2 out of 2 packages',
+              ),
             ).called(1);
             verify(
               () => progress.complete(
@@ -196,10 +202,14 @@ void main() {
             verify(() => logger.err(errorMessage)).called(1);
 
             verify(
-              () => progress.update('Collecting licenses of 0/2 packages.'),
+              () => progress.update(
+                'Collecting licenses from 1 out of 2 packages',
+              ),
             ).called(1);
             verify(
-              () => progress.update('Collecting licenses of 1/2 packages.'),
+              () => progress.update(
+                'Collecting licenses from 2 out of 2 packages',
+              ),
             ).called(1);
             verify(
               () => progress.complete(
@@ -241,10 +251,14 @@ void main() {
             verify(() => logger.err(errorMessage)).called(1);
 
             verify(
-              () => progress.update('Collecting licenses of 0/2 packages.'),
+              () => progress.update(
+                'Collecting licenses from 1 out of 2 packages',
+              ),
             ).called(1);
             verify(
-              () => progress.update('Collecting licenses of 1/2 packages.'),
+              () => progress.update(
+                'Collecting licenses from 2 out of 2 packages',
+              ),
             ).called(1);
             verify(
               () => progress.complete(
@@ -296,10 +310,14 @@ void main() {
           ).called(1);
 
           verify(
-            () => progress.update('Collecting licenses of 0/2 packages.'),
+            () => progress.update(
+              'Collecting licenses from 1 out of 2 packages',
+            ),
           ).called(1);
           verify(
-            () => progress.update('Collecting licenses of 1/2 packages.'),
+            () => progress.update(
+              'Collecting licenses from 2 out of 2 packages',
+            ),
           ).called(1);
           verify(
             () => progress.complete(
@@ -382,7 +400,9 @@ void main() {
                 );
 
                 verify(
-                  () => progress.update('Collecting licenses of 0/1 packages.'),
+                  () => progress.update(
+                    'Collecting licenses from 1 out of 1 package',
+                  ),
                 ).called(1);
                 verify(
                   () => progress.complete(
@@ -431,7 +451,9 @@ void main() {
                 );
 
                 verify(
-                  () => progress.update('Collecting licenses of 0/1 packages.'),
+                  () => progress.update(
+                    'Collecting licenses from 1 out of 1 package',
+                  ),
                 ).called(1);
                 verify(
                   () => progress.complete(
@@ -481,7 +503,9 @@ void main() {
               );
 
               verify(
-                () => progress.update('Collecting licenses of 0/1 packages.'),
+                () => progress.update(
+                  'Collecting licenses from 1 out of 1 package',
+                ),
               ).called(1);
               verify(
                 () => progress.complete(
@@ -530,7 +554,9 @@ void main() {
               );
 
               verify(
-                () => progress.update('Collecting licenses of 0/1 packages.'),
+                () => progress.update(
+                  'Collecting licenses from 1 out of 1 package',
+                ),
               ).called(1);
               verify(
                 () => progress.complete(
@@ -583,13 +609,19 @@ void main() {
               );
 
               verify(
-                () => progress.update('Collecting licenses of 0/3 packages.'),
+                () => progress.update(
+                  'Collecting licenses from 1 out of 3 packages',
+                ),
               ).called(1);
               verify(
-                () => progress.update('Collecting licenses of 1/3 packages.'),
+                () => progress.update(
+                  'Collecting licenses from 2 out of 3 packages',
+                ),
               ).called(1);
               verify(
-                () => progress.update('Collecting licenses of 2/3 packages.'),
+                () => progress.update(
+                  'Collecting licenses from 3 out of 3 packages',
+                ),
               ).called(1);
               verify(
                 () => progress.complete(
@@ -936,7 +968,9 @@ void main() {
             );
 
             verify(
-              () => progress.update('Collecting licenses of 0/1 packages.'),
+              () => progress.update(
+                'Collecting licenses from 1 out of 1 package',
+              ),
             ).called(1);
             verify(
               () => progress.complete(