feat(manage): show progress in 'update all' button

lib/snapd.dart

@@ -1,4 +1,5 @@
 export 'src/snapd/snap_category_enum.dart';
+export 'src/snapd/snap_l10n.dart';
 export 'src/snapd/snap_launcher.dart';
 export 'src/snapd/snap_model.dart';
 export 'src/snapd/snapd_service.dart';

lib/src/detail/detail_page.dart

@@ -509,20 +509,3 @@ class _ChannelDropdownEntry extends StatelessWidget {
     );
   }
 }
-
-extension SnapdChangeL10n on SnapdChange {
-  String? localize(AppLocalizations l10n) => switch (kind) {
-        'install-snap' => l10n.snapActionInstallingLabel,
-        'remove-snap' => l10n.snapActionRemovingLabel,
-        _ => null,
-      };
-}
-
-extension SnapConfinementL10n on SnapConfinement {
-  String localize(AppLocalizations l10n) => switch (this) {
-        SnapConfinement.classic => l10n.snapConfinementClassic,
-        SnapConfinement.devmode => l10n.snapConfinementDevmode,
-        SnapConfinement.strict => l10n.snapConfinementStrict,
-        _ => name,
-      };
-}

lib/src/l10n/app_en.arb

@@ -92,6 +92,7 @@
     "snapActionRemovingLabel": "Uninstalling",
     "snapActionSwitchChannelLabel": "Switch Channel",
     "snapActionUpdateLabel": "Update",
+    "snapActionUpdatingLabel": "Updating",
     "snapCategoryArtAndDesign": "Art and Design",
     "snapCategoryBooksAndReference": "Books and Reference",
     "snapCategoryDefaultButtonLabel": "Discover more",

lib/src/manage/manage_page.dart

@@ -127,6 +127,7 @@ class _ManageView extends ConsumerWidget {
   }
 }
 
+// TODO: refactor/generalize - similar to `_SnapActionButtons`
 class _ActionButtons extends ConsumerWidget {
   const _ActionButtons();
 
@@ -152,7 +153,8 @@ class _ActionButtons extends ConsumerWidget {
       mainAxisSize: MainAxisSize.min,
       children: [
         PushButton.outlined(
-          onPressed: ref.read(updatesModelProvider).refresh,
+          onPressed:
+              updatesModel.activeChangeId != null ? null : updatesModel.refresh,
           child: Row(
             mainAxisSize: MainAxisSize.min,
             children: [
@@ -170,23 +172,53 @@ class _ActionButtons extends ConsumerWidget {
         ),
         const SizedBox(width: 8),
         PushButton.elevated(
-          onPressed: updatesModel.refreshableSnapNames.isNotEmpty
+          onPressed: updatesModel.refreshableSnapNames.isNotEmpty &&
+                  !updatesModel.state.isLoading
               ? ref.read(updatesModelProvider).updateAll
               : null,
-          child: Row(
-            mainAxisSize: MainAxisSize.min,
-            children: [
-              const Icon(YaruIcons.download),
-              const SizedBox(width: 8),
-              Flexible(
-                child: Text(
-                  l10n.managePageUpdateAllLabel,
-                  maxLines: 1,
-                  overflow: TextOverflow.ellipsis,
+          child: updatesModel.activeChangeId != null
+              ? Consumer(
+                  builder: (context, ref, child) {
+                    final change = ref
+                        .watch(changeProvider(updatesModel.activeChangeId))
+                        .whenOrNull(data: (data) => data);
+                    return Row(
+                      children: [
+                        SizedBox.square(
+                          dimension: 16,
+                          child: YaruCircularProgressIndicator(
+                            value: change?.progress,
+                            strokeWidth: 2,
+                          ),
+                        ),
+                        if (change != null) ...[
+                          const SizedBox(width: 8),
+                          Flexible(
+                            child: Text(
+                              change.localize(l10n) ?? '',
+                              maxLines: 1,
+                              overflow: TextOverflow.ellipsis,
+                            ),
+                          ),
+                        ]
+                      ],
+                    );
+                  },
+                )
+              : Row(
+                  mainAxisSize: MainAxisSize.min,
+                  children: [
+                    const Icon(YaruIcons.download),
+                    const SizedBox(width: 8),
+                    Flexible(
+                      child: Text(
+                        l10n.managePageUpdateAllLabel,
+                        maxLines: 1,
+                        overflow: TextOverflow.ellipsis,
+                      ),
+                    ),
+                  ],
                 ),
-              ),
-            ],
-          ),
         ),
       ],
     );

lib/src/snapd/snap_l10n.dart

@@ -0,0 +1,21 @@
+import 'package:snapd/snapd.dart';
+
+import '/l10n.dart';
+
+extension SnapdChangeL10n on SnapdChange {
+  String? localize(AppLocalizations l10n) => switch (kind) {
+        'install-snap' => l10n.snapActionInstallingLabel,
+        'refresh-snap' => l10n.snapActionUpdatingLabel,
+        'remove-snap' => l10n.snapActionRemovingLabel,
+        _ => null,
+      };
+}
+
+extension SnapConfinementL10n on SnapConfinement {
+  String localize(AppLocalizations l10n) => switch (this) {
+        SnapConfinement.classic => l10n.snapConfinementClassic,
+        SnapConfinement.devmode => l10n.snapConfinementDevmode,
+        SnapConfinement.strict => l10n.snapConfinementStrict,
+        _ => name,
+      };
+}

test/detail_page_test.dart

@@ -228,4 +228,6 @@ void main() {
     expect(find.text(tester.l10n.snapActionInstallLabel), findsNothing);
     expect(find.byType(YaruCircularProgressIndicator), findsOneWidget);
   });
+
+  // TODO: test loading states with snap change in progress
 }

test/manage_page_test.dart

@@ -99,6 +99,8 @@ void main() {
     await tester.tap(openButton);
     verify(snapLauncher.open()).called(1);
   });
+
+  // TODO: test loading states with snap change in progress
 }
 
 extension on CommonFinders {

test/test_utils.dart

@@ -125,6 +125,7 @@ MockUpdatesModel createMockUpdatesModel(
   when(model.hasUpdate(any)).thenAnswer((i) =>
       refreshableSnapNames?.contains(i.positionalArguments.single) ?? false);
   when(model.state).thenReturn(AsyncValue.data(() {}()));
+  when(model.activeChangeId).thenReturn(null);
   return model;
 }