Skip to content

Commit d88461c

Browse files
committed
Fix menu idle state after failing update
1 parent bcd9f44 commit d88461c

File tree

2 files changed

+4
-12
lines changed

2 files changed

+4
-12
lines changed

packages/compass/src/main/menu.spec.ts

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -163,6 +163,7 @@ describe('CompassMenu', function () {
163163
AutoUpdateManagerState.UpdateDismissed,
164164
AutoUpdateManagerState.DownloadingError,
165165
AutoUpdateManagerState.Restarting,
166+
AutoUpdateManagerState.OutdatedOperatingSystem,
166167
];
167168
for (const state of idleStates) {
168169
App.emit('auto-updater:new-state', state);
@@ -233,6 +234,7 @@ describe('CompassMenu', function () {
233234
AutoUpdateManagerState.UpdateDismissed,
234235
AutoUpdateManagerState.DownloadingError,
235236
AutoUpdateManagerState.Restarting,
237+
AutoUpdateManagerState.OutdatedOperatingSystem,
236238
];
237239
for (const state of idleStates) {
238240
App.emit('auto-updater:new-state', state);

packages/compass/src/main/menu.ts

Lines changed: 2 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -516,24 +516,14 @@ class CompassMenu {
516516
app.on('auto-updater:new-state', (state) => {
517517
const updateManagerState = ((): UpdateManagerState => {
518518
switch (state) {
519-
case AutoUpdateManagerState.Initial:
520-
case AutoUpdateManagerState.Disabled:
521-
case AutoUpdateManagerState.UserPromptedManualCheck:
522-
case AutoUpdateManagerState.CheckingForUpdatesForManualCheck:
523-
case AutoUpdateManagerState.CheckingForUpdatesForAutomaticCheck:
524-
case AutoUpdateManagerState.NoUpdateAvailable:
525-
case AutoUpdateManagerState.UpdateAvailable:
526-
case AutoUpdateManagerState.UpdateDismissed:
527-
case AutoUpdateManagerState.DownloadingError:
528-
case AutoUpdateManagerState.PromptToUpdateExternally:
529-
case AutoUpdateManagerState.Restarting:
530-
return 'idle';
531519
case AutoUpdateManagerState.ManualDownload:
532520
case AutoUpdateManagerState.DownloadingUpdate:
533521
return 'installing updates';
534522
case AutoUpdateManagerState.RestartDismissed:
535523
case AutoUpdateManagerState.PromptForRestart:
536524
return 'ready to restart';
525+
default:
526+
return 'idle';
537527
}
538528
})();
539529
this.updateMenu({ updateManagerState });

0 commit comments

Comments
 (0)