Skip to content

Commit 1b962c9

Browse files
committed
Fix menu idle state after failing update
1 parent 95f4c48 commit 1b962c9

File tree

2 files changed

+4
-12
lines changed

2 files changed

+4
-12
lines changed

Diff for: packages/compass/src/main/menu.spec.ts

+2
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);

Diff for: packages/compass/src/main/menu.ts

+2-12
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)