From 19259bf537ba106e4138aeedb6017f4fea71f2dc Mon Sep 17 00:00:00 2001 From: Jim Ehrismann Date: Fri, 24 Jun 2022 19:37:42 -0400 Subject: [PATCH] removed update broadcasting/listening, updated snapshots Signed-off-by: Jim Ehrismann --- ...acters-in-page-registrations.test.tsx.snap | 20 +- .../navigate-to-extension-page.test.tsx.snap | 50 +- .../navigating-between-routes.test.tsx.snap | 20 +- ...ation-using-application-menu.test.tsx.snap | 20 +- .../installing-update-using-tray.test.ts.snap | 66 ++- .../installing-update.test.ts.snap | 66 ++- ...eriodical-checking-of-updates.test.ts.snap | 10 +- ...selection-of-update-stability.test.ts.snap | 10 +- ...gation-using-application-menu.test.ts.snap | 20 +- ...elm-repository-in-preferences.test.ts.snap | 100 +++- ...tory-from-list-in-preferences.test.ts.snap | 100 +++- ...m-repositories-in-preferences.test.ts.snap | 100 +++- ...ive-repository-in-preferences.test.ts.snap | 40 +- .../closing-preferences.test.tsx.snap | 80 +++- ...on-to-application-preferences.test.ts.snap | 20 +- ...igation-to-editor-preferences.test.ts.snap | 20 +- ...tension-specific-preferences.test.tsx.snap | 100 +++- ...ion-to-kubernetes-preferences.test.ts.snap | 20 +- ...vigation-to-proxy-preferences.test.ts.snap | 20 +- ...ion-to-telemetry-preferences.test.tsx.snap | 50 +- ...ation-to-terminal-preferences.test.ts.snap | 20 +- ...gation-using-application-menu.test.ts.snap | 20 +- .../navigation-using-tray.test.ts.snap | 20 +- ...gation-using-application-menu.test.ts.snap | 20 +- ...cation-update-status-channel.injectable.ts | 30 -- ...st-change-in-updating-status.injectable.ts | 23 - ...process-checking-for-updates.injectable.ts | 14 - .../__snapshots__/ask-boolean.test.ts.snap | 444 ++++++++++++++++++ ...ation-update-status-listener.injectable.ts | 62 --- .../auto-update-status-bar-item.tsx | 1 + 30 files changed, 1355 insertions(+), 231 deletions(-) delete mode 100644 src/common/application-update/application-update-status-channel.injectable.ts delete mode 100644 src/main/application-update/check-for-updates/broadcast-change-in-updating-status.injectable.ts create mode 100644 src/main/ask-boolean/__snapshots__/ask-boolean.test.ts.snap delete mode 100644 src/renderer/application-update/application-update-status-listener.injectable.ts diff --git a/src/behaviours/__snapshots__/extension-special-characters-in-page-registrations.test.tsx.snap b/src/behaviours/__snapshots__/extension-special-characters-in-page-registrations.test.tsx.snap index 3a3d32317c..723a7dcb3a 100644 --- a/src/behaviours/__snapshots__/extension-special-characters-in-page-registrations.test.tsx.snap +++ b/src/behaviours/__snapshots__/extension-special-characters-in-page-registrations.test.tsx.snap @@ -52,7 +52,15 @@ exports[`extension special characters in page registrations renders 1`] = ` >
+ > +
+
+ No new updates available +
+
+
@@ -115,7 +123,15 @@ exports[`extension special characters in page registrations when navigating to r >
+ > +
+
+ No new updates available +
+
+
diff --git a/src/behaviours/__snapshots__/navigate-to-extension-page.test.tsx.snap b/src/behaviours/__snapshots__/navigate-to-extension-page.test.tsx.snap index c99c770ef1..deb712fdda 100644 --- a/src/behaviours/__snapshots__/navigate-to-extension-page.test.tsx.snap +++ b/src/behaviours/__snapshots__/navigate-to-extension-page.test.tsx.snap @@ -52,7 +52,15 @@ exports[`navigate to extension page renders 1`] = ` >
+ > +
+
+ No new updates available +
+
+
@@ -115,7 +123,15 @@ exports[`navigate to extension page when extension navigates to child route rend >
+ > +
+
+ No new updates available +
+
+
@@ -181,7 +197,15 @@ exports[`navigate to extension page when extension navigates to route with param >
+ > +
+
+ No new updates available +
+
+
@@ -263,7 +287,15 @@ exports[`navigate to extension page when extension navigates to route without pa >
+ > +
+
+ No new updates available +
+
+
@@ -345,7 +377,15 @@ exports[`navigate to extension page when extension navigates to route without pa >
+ > +
+
+ No new updates available +
+
+
diff --git a/src/behaviours/__snapshots__/navigating-between-routes.test.tsx.snap b/src/behaviours/__snapshots__/navigating-between-routes.test.tsx.snap index 921adfec18..774c596c10 100644 --- a/src/behaviours/__snapshots__/navigating-between-routes.test.tsx.snap +++ b/src/behaviours/__snapshots__/navigating-between-routes.test.tsx.snap @@ -52,7 +52,15 @@ exports[`navigating between routes given route with optional path parameters whe >
+ > +
+
+ No new updates available +
+
+
@@ -121,7 +129,15 @@ exports[`navigating between routes given route without path parameters when navi >
+ > +
+
+ No new updates available +
+
+
diff --git a/src/behaviours/add-cluster/__snapshots__/navigation-using-application-menu.test.tsx.snap b/src/behaviours/add-cluster/__snapshots__/navigation-using-application-menu.test.tsx.snap index 1da17bdc47..3fecea0b6b 100644 --- a/src/behaviours/add-cluster/__snapshots__/navigation-using-application-menu.test.tsx.snap +++ b/src/behaviours/add-cluster/__snapshots__/navigation-using-application-menu.test.tsx.snap @@ -52,7 +52,15 @@ exports[`add-cluster - navigation using application menu renders 1`] = ` >
+ > +
+
+ No new updates available +
+
+
@@ -115,7 +123,15 @@ exports[`add-cluster - navigation using application menu when navigating to add >
+ > +
+
+ No new updates available +
+
+
diff --git a/src/behaviours/application-update/__snapshots__/installing-update-using-tray.test.ts.snap b/src/behaviours/application-update/__snapshots__/installing-update-using-tray.test.ts.snap index bb362ea546..ae04ddac4f 100644 --- a/src/behaviours/application-update/__snapshots__/installing-update-using-tray.test.ts.snap +++ b/src/behaviours/application-update/__snapshots__/installing-update-using-tray.test.ts.snap @@ -53,7 +53,15 @@ exports[`installing update using tray when started renders 1`] = ` >
+ > +
+
+ No new updates available +
+
+
@@ -118,7 +126,18 @@ exports[`installing update using tray when started when user checks for updates >
+ > +
+
+
+ Checking for updates... +
+
+
@@ -224,7 +243,18 @@ exports[`installing update using tray when started when user checks for updates >
+ > +
+
+ Download for version some-version started +
+
+
+
@@ -388,7 +418,15 @@ exports[`installing update using tray when started when user checks for updates >
+ > +
+
+ No new updates available +
+
+
@@ -592,7 +630,15 @@ exports[`installing update using tray when started when user checks for updates >
+ > +
+
+ some-version is available +
+
+
@@ -738,7 +784,15 @@ exports[`installing update using tray when started when user checks for updates >
+ > +
+
+ No new updates available +
+
+
diff --git a/src/behaviours/application-update/__snapshots__/installing-update.test.ts.snap b/src/behaviours/application-update/__snapshots__/installing-update.test.ts.snap index d865b6ce06..9228858235 100644 --- a/src/behaviours/application-update/__snapshots__/installing-update.test.ts.snap +++ b/src/behaviours/application-update/__snapshots__/installing-update.test.ts.snap @@ -53,7 +53,15 @@ exports[`installing update when started renders 1`] = ` >
+ > +
+
+ No new updates available +
+
+
@@ -118,7 +126,18 @@ exports[`installing update when started when user checks for updates renders 1`] >
+ > +
+
+
+ Checking for updates... +
+
+
@@ -183,7 +202,18 @@ exports[`installing update when started when user checks for updates when new up >
+ > +
+
+ Download for version some-version started +
+
+
+
@@ -266,7 +296,15 @@ exports[`installing update when started when user checks for updates when new up >
+ > +
+
+ No new updates available +
+
+
@@ -349,7 +387,15 @@ exports[`installing update when started when user checks for updates when new up >
+ > +
+
+ some-version is available +
+
+
@@ -414,7 +460,15 @@ exports[`installing update when started when user checks for updates when no new >
+ > +
+
+ No new updates available +
+
+
diff --git a/src/behaviours/application-update/__snapshots__/periodical-checking-of-updates.test.ts.snap b/src/behaviours/application-update/__snapshots__/periodical-checking-of-updates.test.ts.snap index 84259158ef..28f2bcb59a 100644 --- a/src/behaviours/application-update/__snapshots__/periodical-checking-of-updates.test.ts.snap +++ b/src/behaviours/application-update/__snapshots__/periodical-checking-of-updates.test.ts.snap @@ -53,7 +53,15 @@ exports[`periodical checking of updates given updater is enabled and configurati >
+ > +
+
+ No new updates available +
+
+
diff --git a/src/behaviours/application-update/__snapshots__/selection-of-update-stability.test.ts.snap b/src/behaviours/application-update/__snapshots__/selection-of-update-stability.test.ts.snap index c0ffb43590..4882b5c6d5 100644 --- a/src/behaviours/application-update/__snapshots__/selection-of-update-stability.test.ts.snap +++ b/src/behaviours/application-update/__snapshots__/selection-of-update-stability.test.ts.snap @@ -53,7 +53,15 @@ exports[`selection of update stability when started renders 1`] = ` >
+ > +
+
+ No new updates available +
+
+
diff --git a/src/behaviours/extensions/__snapshots__/navigation-using-application-menu.test.ts.snap b/src/behaviours/extensions/__snapshots__/navigation-using-application-menu.test.ts.snap index b8c6804f4d..dc164f44ca 100644 --- a/src/behaviours/extensions/__snapshots__/navigation-using-application-menu.test.ts.snap +++ b/src/behaviours/extensions/__snapshots__/navigation-using-application-menu.test.ts.snap @@ -52,7 +52,15 @@ exports[`extensions - navigation using application menu renders 1`] = ` >
+ > +
+
+ No new updates available +
+
+
@@ -115,7 +123,15 @@ exports[`extensions - navigation using application menu when navigating to exten >
+ > +
+
+ No new updates available +
+
+
diff --git a/src/behaviours/helm-charts/__snapshots__/add-custom-helm-repository-in-preferences.test.ts.snap b/src/behaviours/helm-charts/__snapshots__/add-custom-helm-repository-in-preferences.test.ts.snap index ff6be7d818..9be28c54f4 100644 --- a/src/behaviours/helm-charts/__snapshots__/add-custom-helm-repository-in-preferences.test.ts.snap +++ b/src/behaviours/helm-charts/__snapshots__/add-custom-helm-repository-in-preferences.test.ts.snap @@ -53,7 +53,15 @@ exports[`add custom helm repository in preferences when navigating to preference >
+ > +
+
+ No new updates available +
+
+
@@ -577,7 +585,15 @@ exports[`add custom helm repository in preferences when navigating to preference >
+ > +
+
+ No new updates available +
+
+
@@ -1108,7 +1124,15 @@ exports[`add custom helm repository in preferences when navigating to preference >
+ > +
+
+ No new updates available +
+
+
@@ -1748,7 +1772,15 @@ exports[`add custom helm repository in preferences when navigating to preference >
+ > +
+
+ No new updates available +
+
+
@@ -2287,7 +2319,15 @@ exports[`add custom helm repository in preferences when navigating to preference >
+ > +
+
+ No new updates available +
+
+
@@ -2927,7 +2967,15 @@ exports[`add custom helm repository in preferences when navigating to preference >
+ > +
+
+ No new updates available +
+
+
@@ -3743,7 +3791,15 @@ exports[`add custom helm repository in preferences when navigating to preference >
+ > +
+
+ No new updates available +
+
+
@@ -4383,7 +4439,11 @@ exports[`add custom helm repository in preferences when navigating to preference >
+ > +
+
@@ -5199,7 +5259,11 @@ exports[`add custom helm repository in preferences when navigating to preference >
+ > +
+
@@ -5839,7 +5903,11 @@ exports[`add custom helm repository in preferences when navigating to preference >
+ > +
+
@@ -6378,7 +6446,11 @@ exports[`add custom helm repository in preferences when navigating to preference >
+ > +
+
@@ -6910,7 +6982,11 @@ exports[`add custom helm repository in preferences when navigating to preference >
+ > +
+
diff --git a/src/behaviours/helm-charts/__snapshots__/add-helm-repository-from-list-in-preferences.test.ts.snap b/src/behaviours/helm-charts/__snapshots__/add-helm-repository-from-list-in-preferences.test.ts.snap index 776755635f..e9d72c8aac 100644 --- a/src/behaviours/helm-charts/__snapshots__/add-helm-repository-from-list-in-preferences.test.ts.snap +++ b/src/behaviours/helm-charts/__snapshots__/add-helm-repository-from-list-in-preferences.test.ts.snap @@ -53,7 +53,15 @@ exports[`add helm repository from list in preferences when navigating to prefere >
+ > +
+
+ No new updates available +
+
+
@@ -577,7 +585,15 @@ exports[`add helm repository from list in preferences when navigating to prefere >
+ > +
+
+ No new updates available +
+
+
@@ -1108,7 +1124,15 @@ exports[`add helm repository from list in preferences when navigating to prefere >
+ > +
+
+ No new updates available +
+
+
@@ -1692,7 +1716,15 @@ exports[`add helm repository from list in preferences when navigating to prefere >
+ > +
+
+ No new updates available +
+
+
@@ -2223,7 +2255,15 @@ exports[`add helm repository from list in preferences when navigating to prefere >
+ > +
+
+ No new updates available +
+
+
@@ -2754,7 +2794,15 @@ exports[`add helm repository from list in preferences when navigating to prefere >
+ > +
+
+ No new updates available +
+
+
@@ -3278,7 +3326,15 @@ exports[`add helm repository from list in preferences when navigating to prefere >
+ > +
+
+ No new updates available +
+
+
@@ -3839,7 +3895,15 @@ exports[`add helm repository from list in preferences when navigating to prefere >
+ > +
+
+ No new updates available +
+
+
@@ -4463,7 +4527,15 @@ exports[`add helm repository from list in preferences when navigating to prefere >
+ > +
+
+ No new updates available +
+
+
@@ -5024,7 +5096,15 @@ exports[`add helm repository from list in preferences when navigating to prefere >
+ > +
+
+ No new updates available +
+
+
diff --git a/src/behaviours/helm-charts/__snapshots__/listing-active-helm-repositories-in-preferences.test.ts.snap b/src/behaviours/helm-charts/__snapshots__/listing-active-helm-repositories-in-preferences.test.ts.snap index 83666bdfcc..5756a20e51 100644 --- a/src/behaviours/helm-charts/__snapshots__/listing-active-helm-repositories-in-preferences.test.ts.snap +++ b/src/behaviours/helm-charts/__snapshots__/listing-active-helm-repositories-in-preferences.test.ts.snap @@ -53,7 +53,15 @@ exports[`listing active helm repositories in preferences when navigating to pref >
+ > +
+
+ No new updates available +
+
+
@@ -577,7 +585,15 @@ exports[`listing active helm repositories in preferences when navigating to pref >
+ > +
+
+ No new updates available +
+
+
@@ -1101,7 +1117,15 @@ exports[`listing active helm repositories in preferences when navigating to pref >
+ > +
+
+ No new updates available +
+
+
@@ -1517,7 +1541,15 @@ exports[`listing active helm repositories in preferences when navigating to pref >
+ > +
+
+ No new updates available +
+
+
@@ -2041,7 +2073,15 @@ exports[`listing active helm repositories in preferences when navigating to pref >
+ > +
+
+ No new updates available +
+
+
@@ -2457,7 +2497,15 @@ exports[`listing active helm repositories in preferences when navigating to pref >
+ > +
+
+ No new updates available +
+
+
@@ -2981,7 +3029,15 @@ exports[`listing active helm repositories in preferences when navigating to pref >
+ > +
+
+ No new updates available +
+
+
@@ -3542,7 +3598,15 @@ exports[`listing active helm repositories in preferences when navigating to pref >
+ > +
+
+ No new updates available +
+
+
@@ -3958,7 +4022,15 @@ exports[`listing active helm repositories in preferences when navigating to pref >
+ > +
+
+ No new updates available +
+
+
@@ -4374,7 +4446,15 @@ exports[`listing active helm repositories in preferences when navigating to pref >
+ > +
+
+ No new updates available +
+
+
diff --git a/src/behaviours/helm-charts/__snapshots__/remove-helm-repository-from-list-of-active-repository-in-preferences.test.ts.snap b/src/behaviours/helm-charts/__snapshots__/remove-helm-repository-from-list-of-active-repository-in-preferences.test.ts.snap index 0823b7d1f1..96de0d6df3 100644 --- a/src/behaviours/helm-charts/__snapshots__/remove-helm-repository-from-list-of-active-repository-in-preferences.test.ts.snap +++ b/src/behaviours/helm-charts/__snapshots__/remove-helm-repository-from-list-of-active-repository-in-preferences.test.ts.snap @@ -53,7 +53,15 @@ exports[`remove helm repository from list of active repositories in preferences >
+ > +
+
+ No new updates available +
+
+
@@ -577,7 +585,15 @@ exports[`remove helm repository from list of active repositories in preferences >
+ > +
+
+ No new updates available +
+
+
@@ -1108,7 +1124,15 @@ exports[`remove helm repository from list of active repositories in preferences >
+ > +
+
+ No new updates available +
+
+
@@ -1639,7 +1663,15 @@ exports[`remove helm repository from list of active repositories in preferences >
+ > +
+
+ No new updates available +
+
+
diff --git a/src/behaviours/preferences/__snapshots__/closing-preferences.test.tsx.snap b/src/behaviours/preferences/__snapshots__/closing-preferences.test.tsx.snap index b84f16d647..c2f3e8010c 100644 --- a/src/behaviours/preferences/__snapshots__/closing-preferences.test.tsx.snap +++ b/src/behaviours/preferences/__snapshots__/closing-preferences.test.tsx.snap @@ -52,7 +52,15 @@ exports[`preferences - closing-preferences given accessing preferences directly >
+ > +
+
+ No new updates available +
+
+
@@ -648,7 +656,15 @@ exports[`preferences - closing-preferences given accessing preferences directly >
+ > +
+
+ No new updates available +
+
+
@@ -848,7 +864,15 @@ exports[`preferences - closing-preferences given accessing preferences directly >
+ > +
+
+ No new updates available +
+
+
@@ -914,7 +938,15 @@ exports[`preferences - closing-preferences given accessing preferences directly >
+ > +
+
+ No new updates available +
+
+
@@ -980,7 +1012,15 @@ exports[`preferences - closing-preferences given already in a page and then navi >
+ > +
+
+ No new updates available +
+
+
@@ -1576,7 +1616,15 @@ exports[`preferences - closing-preferences given already in a page and then navi >
+ > +
+
+ No new updates available +
+
+
@@ -1776,7 +1824,15 @@ exports[`preferences - closing-preferences given already in a page and then navi >
+ > +
+
+ No new updates available +
+
+
@@ -1976,7 +2032,15 @@ exports[`preferences - closing-preferences given already in a page and then navi >
+ > +
+
+ No new updates available +
+
+
diff --git a/src/behaviours/preferences/__snapshots__/navigation-to-application-preferences.test.ts.snap b/src/behaviours/preferences/__snapshots__/navigation-to-application-preferences.test.ts.snap index b4b2807e5c..a53afd0f9e 100644 --- a/src/behaviours/preferences/__snapshots__/navigation-to-application-preferences.test.ts.snap +++ b/src/behaviours/preferences/__snapshots__/navigation-to-application-preferences.test.ts.snap @@ -52,7 +52,15 @@ exports[`preferences - navigation to application preferences given in some child >
+ > +
+
+ No new updates available +
+
+
@@ -312,7 +320,15 @@ exports[`preferences - navigation to application preferences given in some child >
+ > +
+
+ No new updates available +
+
+
diff --git a/src/behaviours/preferences/__snapshots__/navigation-to-editor-preferences.test.ts.snap b/src/behaviours/preferences/__snapshots__/navigation-to-editor-preferences.test.ts.snap index d40c0086d6..3a16047bfd 100644 --- a/src/behaviours/preferences/__snapshots__/navigation-to-editor-preferences.test.ts.snap +++ b/src/behaviours/preferences/__snapshots__/navigation-to-editor-preferences.test.ts.snap @@ -52,7 +52,15 @@ exports[`preferences - navigation to editor preferences given in preferences, wh >
+ > +
+
+ No new updates available +
+
+
@@ -636,7 +644,15 @@ exports[`preferences - navigation to editor preferences given in preferences, wh >
+ > +
+
+ No new updates available +
+
+
diff --git a/src/behaviours/preferences/__snapshots__/navigation-to-extension-specific-preferences.test.tsx.snap b/src/behaviours/preferences/__snapshots__/navigation-to-extension-specific-preferences.test.tsx.snap index 41039b5881..9bee95dd99 100644 --- a/src/behaviours/preferences/__snapshots__/navigation-to-extension-specific-preferences.test.tsx.snap +++ b/src/behaviours/preferences/__snapshots__/navigation-to-extension-specific-preferences.test.tsx.snap @@ -67,7 +67,15 @@ exports[`preferences - navigation to extension specific preferences given in pre >
+ > +
+
+ No new updates available +
+
+
@@ -301,7 +309,15 @@ exports[`preferences - navigation to extension specific preferences given in pre >
+ > +
+
+ No new updates available +
+
+
@@ -547,7 +563,15 @@ exports[`preferences - navigation to extension specific preferences given in pre >
+ > +
+
+ No new updates available +
+
+
@@ -793,7 +817,15 @@ exports[`preferences - navigation to extension specific preferences given in pre >
+ > +
+
+ No new updates available +
+
+
@@ -1062,7 +1094,15 @@ exports[`preferences - navigation to extension specific preferences given in pre >
+ > +
+
+ No new updates available +
+
+
@@ -1646,7 +1686,15 @@ exports[`preferences - navigation to extension specific preferences given in pre >
+ > +
+
+ No new updates available +
+
+
@@ -2265,7 +2313,15 @@ exports[`preferences - navigation to extension specific preferences given in pre >
+ > +
+
+ No new updates available +
+
+
@@ -2523,7 +2579,15 @@ exports[`preferences - navigation to extension specific preferences given in pre >
+ > +
+
+ No new updates available +
+
+
@@ -2726,7 +2790,15 @@ exports[`preferences - navigation to extension specific preferences when navigat >
+ > +
+
+ No new updates available +
+
+
@@ -2960,7 +3032,15 @@ exports[`preferences - navigation to extension specific preferences when navigat >
+ > +
+
+ No new updates available +
+
+
diff --git a/src/behaviours/preferences/__snapshots__/navigation-to-kubernetes-preferences.test.ts.snap b/src/behaviours/preferences/__snapshots__/navigation-to-kubernetes-preferences.test.ts.snap index 67796e88af..944aaec393 100644 --- a/src/behaviours/preferences/__snapshots__/navigation-to-kubernetes-preferences.test.ts.snap +++ b/src/behaviours/preferences/__snapshots__/navigation-to-kubernetes-preferences.test.ts.snap @@ -52,7 +52,15 @@ exports[`preferences - navigation to kubernetes preferences given in preferences >
+ > +
+
+ No new updates available +
+
+
@@ -636,7 +644,15 @@ exports[`preferences - navigation to kubernetes preferences given in preferences >
+ > +
+
+ No new updates available +
+
+
diff --git a/src/behaviours/preferences/__snapshots__/navigation-to-proxy-preferences.test.ts.snap b/src/behaviours/preferences/__snapshots__/navigation-to-proxy-preferences.test.ts.snap index 0ee9cc76b1..5cdf47c381 100644 --- a/src/behaviours/preferences/__snapshots__/navigation-to-proxy-preferences.test.ts.snap +++ b/src/behaviours/preferences/__snapshots__/navigation-to-proxy-preferences.test.ts.snap @@ -52,7 +52,15 @@ exports[`preferences - navigation to proxy preferences given in preferences, whe >
+ > +
+
+ No new updates available +
+
+
@@ -636,7 +644,15 @@ exports[`preferences - navigation to proxy preferences given in preferences, whe >
+ > +
+
+ No new updates available +
+
+
diff --git a/src/behaviours/preferences/__snapshots__/navigation-to-telemetry-preferences.test.tsx.snap b/src/behaviours/preferences/__snapshots__/navigation-to-telemetry-preferences.test.tsx.snap index 5915f12efd..25781cfaf1 100644 --- a/src/behaviours/preferences/__snapshots__/navigation-to-telemetry-preferences.test.tsx.snap +++ b/src/behaviours/preferences/__snapshots__/navigation-to-telemetry-preferences.test.tsx.snap @@ -52,7 +52,15 @@ exports[`preferences - navigation to telemetry preferences given URL for Sentry >
+ > +
+
+ No new updates available +
+
+
@@ -298,7 +306,15 @@ exports[`preferences - navigation to telemetry preferences given in preferences, >
+ > +
+
+ No new updates available +
+
+
@@ -882,7 +898,15 @@ exports[`preferences - navigation to telemetry preferences given in preferences, >
+ > +
+
+ No new updates available +
+
+
@@ -1478,7 +1502,15 @@ exports[`preferences - navigation to telemetry preferences given in preferences, >
+ > +
+
+ No new updates available +
+
+
@@ -1712,7 +1744,15 @@ exports[`preferences - navigation to telemetry preferences given no URL for Sent >
+ > +
+
+ No new updates available +
+
+
diff --git a/src/behaviours/preferences/__snapshots__/navigation-to-terminal-preferences.test.ts.snap b/src/behaviours/preferences/__snapshots__/navigation-to-terminal-preferences.test.ts.snap index 3d1b26f668..3290e2fd68 100644 --- a/src/behaviours/preferences/__snapshots__/navigation-to-terminal-preferences.test.ts.snap +++ b/src/behaviours/preferences/__snapshots__/navigation-to-terminal-preferences.test.ts.snap @@ -52,7 +52,15 @@ exports[`preferences - navigation to terminal preferences given in preferences, >
+ > +
+
+ No new updates available +
+
+
@@ -636,7 +644,15 @@ exports[`preferences - navigation to terminal preferences given in preferences, >
+ > +
+
+ No new updates available +
+
+
diff --git a/src/behaviours/preferences/__snapshots__/navigation-using-application-menu.test.ts.snap b/src/behaviours/preferences/__snapshots__/navigation-using-application-menu.test.ts.snap index 339e746af6..ec8c7f22c1 100644 --- a/src/behaviours/preferences/__snapshots__/navigation-using-application-menu.test.ts.snap +++ b/src/behaviours/preferences/__snapshots__/navigation-using-application-menu.test.ts.snap @@ -52,7 +52,15 @@ exports[`preferences - navigation using application menu renders 1`] = ` >
+ > +
+
+ No new updates available +
+
+
@@ -115,7 +123,15 @@ exports[`preferences - navigation using application menu when navigating to pref >
+ > +
+
+ No new updates available +
+
+
diff --git a/src/behaviours/preferences/__snapshots__/navigation-using-tray.test.ts.snap b/src/behaviours/preferences/__snapshots__/navigation-using-tray.test.ts.snap index c395fb5a18..036e1cc9fe 100644 --- a/src/behaviours/preferences/__snapshots__/navigation-using-tray.test.ts.snap +++ b/src/behaviours/preferences/__snapshots__/navigation-using-tray.test.ts.snap @@ -53,7 +53,15 @@ exports[`show-about-using-tray renders 1`] = ` >
+ > +
+
+ No new updates available +
+
+
@@ -118,7 +126,15 @@ exports[`show-about-using-tray when navigating using tray renders 1`] = ` >
+ > +
+
+ No new updates available +
+
+
diff --git a/src/behaviours/welcome/__snapshots__/navigation-using-application-menu.test.ts.snap b/src/behaviours/welcome/__snapshots__/navigation-using-application-menu.test.ts.snap index 47b75b4303..babd50badd 100644 --- a/src/behaviours/welcome/__snapshots__/navigation-using-application-menu.test.ts.snap +++ b/src/behaviours/welcome/__snapshots__/navigation-using-application-menu.test.ts.snap @@ -52,7 +52,15 @@ exports[`welcome - navigation using application menu renders 1`] = ` >
+ > +
+
+ No new updates available +
+
+
@@ -115,7 +123,15 @@ exports[`welcome - navigation using application menu when navigating to welcome >
+ > +
+
+ No new updates available +
+
+
diff --git a/src/common/application-update/application-update-status-channel.injectable.ts b/src/common/application-update/application-update-status-channel.injectable.ts deleted file mode 100644 index e87849e3a7..0000000000 --- a/src/common/application-update/application-update-status-channel.injectable.ts +++ /dev/null @@ -1,30 +0,0 @@ -/** - * Copyright (c) OpenLens Authors. All rights reserved. - * Licensed under MIT License. See LICENSE in root directory for more information. - */ -import { getInjectable } from "@ogre-tools/injectable"; -import type { MessageChannel } from "../utils/channel/message-channel-injection-token"; -import { messageChannelInjectionToken } from "../utils/channel/message-channel-injection-token"; - -export type ApplicationUpdateStatusEventId = - | "checking-for-updates" - | "no-updates-available" - | "download-for-update-started" - | "download-for-update-failed" - | "download-for-update-succeeded"; - -// eslint-disable-next-line @typescript-eslint/consistent-type-definitions -export type ApplicationUpdateStatusChannelMessage = { eventId: ApplicationUpdateStatusEventId; version?: string }; -export type ApplicationUpdateStatusChannel = MessageChannel; - -const applicationUpdateStatusChannelInjectable = getInjectable({ - id: "application-update-status-channel", - - instantiate: (): ApplicationUpdateStatusChannel => ({ - id: "application-update-status-channel", - }), - - injectionToken: messageChannelInjectionToken, -}); - -export default applicationUpdateStatusChannelInjectable; diff --git a/src/main/application-update/check-for-updates/broadcast-change-in-updating-status.injectable.ts b/src/main/application-update/check-for-updates/broadcast-change-in-updating-status.injectable.ts deleted file mode 100644 index 7e9257e966..0000000000 --- a/src/main/application-update/check-for-updates/broadcast-change-in-updating-status.injectable.ts +++ /dev/null @@ -1,23 +0,0 @@ -/** - * Copyright (c) OpenLens Authors. All rights reserved. - * Licensed under MIT License. See LICENSE in root directory for more information. - */ -import { getInjectable } from "@ogre-tools/injectable"; -import type { ApplicationUpdateStatusChannelMessage } from "../../../common/application-update/application-update-status-channel.injectable"; -import { messageToChannelInjectionToken } from "../../../common/utils/channel/message-to-channel-injection-token"; -import applicationUpdateStatusChannelInjectable from "../../../common/application-update/application-update-status-channel.injectable"; - -const broadcastChangeInUpdatingStatusInjectable = getInjectable({ - id: "broadcast-change-in-updating-status", - - instantiate: (di) => { - const messageToChannel = di.inject(messageToChannelInjectionToken); - const applicationUpdateStatusChannel = di.inject(applicationUpdateStatusChannelInjectable); - - return (data: ApplicationUpdateStatusChannelMessage) => { - messageToChannel(applicationUpdateStatusChannel, data); - }; - }, -}); - -export default broadcastChangeInUpdatingStatusInjectable; diff --git a/src/main/application-update/check-for-updates/process-checking-for-updates.injectable.ts b/src/main/application-update/check-for-updates/process-checking-for-updates.injectable.ts index bd3501b5d8..5b17dbe1ac 100644 --- a/src/main/application-update/check-for-updates/process-checking-for-updates.injectable.ts +++ b/src/main/application-update/check-for-updates/process-checking-for-updates.injectable.ts @@ -8,7 +8,6 @@ import updatesAreBeingDiscoveredInjectable from "../../../common/application-upd import discoveredUpdateVersionInjectable from "../../../common/application-update/discovered-update-version/discovered-update-version.injectable"; import { runInAction } from "mobx"; import downloadUpdateInjectable from "../download-update/download-update.injectable"; -import broadcastChangeInUpdatingStatusInjectable from "./broadcast-change-in-updating-status.injectable"; import checkForUpdatesStartingFromChannelInjectable from "./check-for-updates-starting-from-channel.injectable"; import withOrphanPromiseInjectable from "../../../common/utils/with-orphan-promise/with-orphan-promise.injectable"; import emitEventInjectable from "../../../common/app-event-bus/emit-event.injectable"; @@ -20,7 +19,6 @@ const processCheckingForUpdatesInjectable = getInjectable({ instantiate: (di) => { const downloadUpdate = di.inject(downloadUpdateInjectable); const selectedUpdateChannel = di.inject(selectedUpdateChannelInjectable); - const broadcastChangeInUpdatingStatus = di.inject(broadcastChangeInUpdatingStatusInjectable); const checkingForUpdatesState = di.inject(updatesAreBeingDiscoveredInjectable); const discoveredVersionState = di.inject(discoveredUpdateVersionInjectable); const checkForUpdatesStartingFromChannel = di.inject(checkForUpdatesStartingFromChannelInjectable); @@ -34,8 +32,6 @@ const processCheckingForUpdatesInjectable = getInjectable({ params: { currentDateTime: getCurrentDateTime(), source }, }); - broadcastChangeInUpdatingStatus({ eventId: "checking-for-updates" }); - runInAction(() => { checkingForUpdatesState.set(true); }); @@ -43,8 +39,6 @@ const processCheckingForUpdatesInjectable = getInjectable({ const result = await checkForUpdatesStartingFromChannel(selectedUpdateChannel.value.get()); if (!result.updateWasDiscovered) { - broadcastChangeInUpdatingStatus({ eventId: "no-updates-available" }); - runInAction(() => { discoveredVersionState.set(null); checkingForUpdatesState.set(false); @@ -61,11 +55,6 @@ const processCheckingForUpdatesInjectable = getInjectable({ params: { version, currentDateTime: getCurrentDateTime() }, }); - broadcastChangeInUpdatingStatus({ - eventId: "download-for-update-started", - version, - }); - runInAction(() => { discoveredVersionState.set({ version, @@ -79,9 +68,6 @@ const processCheckingForUpdatesInjectable = getInjectable({ const { downloadWasSuccessful } = await downloadUpdate(); if (!downloadWasSuccessful) { - broadcastChangeInUpdatingStatus({ - eventId: "download-for-update-failed", - }); } })(); }; diff --git a/src/main/ask-boolean/__snapshots__/ask-boolean.test.ts.snap b/src/main/ask-boolean/__snapshots__/ask-boolean.test.ts.snap new file mode 100644 index 0000000000..f6e54b9b2d --- /dev/null +++ b/src/main/ask-boolean/__snapshots__/ask-boolean.test.ts.snap @@ -0,0 +1,444 @@ +// Jest Snapshot v1, https://goo.gl/fbAQLP + +exports[`ask-boolean given started when asking multiple questions renders 1`] = ` + +
+
+
+
+
+ No new updates available +
+
+
+
+
+
+
+
+ + + info_outline + + +
+
+
+ + some-title + +

+ Some question +

+
+ + +
+
+
+
+ + + close + + +
+
+
+
+ + + info_outline + + +
+
+
+ + some-other-title + +

+ Some other question +

+
+ + +
+
+
+
+ + + close + + +
+
+
+
+ +`; + +exports[`ask-boolean given started when asking multiple questions when answering to first question renders 1`] = ` + +
+
+
+
+
+ No new updates available +
+
+
+
+
+
+
+
+ + + info_outline + + +
+
+
+ + some-other-title + +

+ Some other question +

+
+ + +
+
+
+
+ + + close + + +
+
+
+
+ +`; + +exports[`ask-boolean given started when asking question renders 1`] = ` + +
+
+
+
+
+ No new updates available +
+
+
+
+
+
+
+
+ + + info_outline + + +
+
+
+ + some-title + +

+ Some question +

+
+ + +
+
+
+
+ + + close + + +
+
+
+
+ +`; + +exports[`ask-boolean given started when asking question when user answers "no" renders 1`] = ` + +
+
+
+
+
+ No new updates available +
+
+
+
+
+
+
+ +`; + +exports[`ask-boolean given started when asking question when user answers "yes" renders 1`] = ` + +
+
+
+
+
+ No new updates available +
+
+
+
+
+
+
+ +`; + +exports[`ask-boolean given started when asking question when user closes notification without answering the question renders 1`] = ` + +
+
+
+
+
+ No new updates available +
+
+
+
+
+
+
+ +`; diff --git a/src/renderer/application-update/application-update-status-listener.injectable.ts b/src/renderer/application-update/application-update-status-listener.injectable.ts deleted file mode 100644 index 1e48c8f66c..0000000000 --- a/src/renderer/application-update/application-update-status-listener.injectable.ts +++ /dev/null @@ -1,62 +0,0 @@ -/** - * Copyright (c) OpenLens Authors. All rights reserved. - * Licensed under MIT License. See LICENSE in root directory for more information. - */ -import { getInjectable } from "@ogre-tools/injectable"; -import type { ApplicationUpdateStatusChannel, ApplicationUpdateStatusEventId } from "../../common/application-update/application-update-status-channel.injectable"; -import applicationUpdateStatusChannelInjectable from "../../common/application-update/application-update-status-channel.injectable"; -//import showInfoNotificationInjectable from "../components/notifications/show-info-notification.injectable"; -import type { MessageChannelListener } from "../../common/utils/channel/message-channel-listener-injection-token"; -import { messageChannelListenerInjectionToken } from "../../common/utils/channel/message-channel-listener-injection-token"; - -const applicationUpdateStatusListenerInjectable = getInjectable({ - id: "application-update-status-listener", - - instantiate: (di): MessageChannelListener => { - const channel = di.inject(applicationUpdateStatusChannelInjectable); - //const showInfoNotification = di.inject(showInfoNotificationInjectable); - - const eventHandlers: Record void }> = { - "checking-for-updates": { - handle: () => { - //showInfoNotification("Checking for updates..."); - }, - }, - - "no-updates-available": { - handle: () => { - //showInfoNotification("No new updates available"); - }, - }, - - "download-for-update-started": { - handle: (/*version*/) => { - //showInfoNotification(`Download for version ${version} started...`); - }, - }, - - "download-for-update-failed": { - handle: () => { - //showInfoNotification("Download of update failed"); - }, - }, - - "download-for-update-succeeded": { - handle: () => { - }, - }, - }; - - return { - channel, - - handler: ({ eventId, version }) => { - eventHandlers[eventId].handle(version); - }, - }; - }, - - injectionToken: messageChannelListenerInjectionToken, -}); - -export default applicationUpdateStatusListenerInjectable; diff --git a/src/renderer/components/status-bar/auto-update-status-bar-item.tsx b/src/renderer/components/status-bar/auto-update-status-bar-item.tsx index 19e246e190..a07140d4cd 100644 --- a/src/renderer/components/status-bar/auto-update-status-bar-item.tsx +++ b/src/renderer/components/status-bar/auto-update-status-bar-item.tsx @@ -72,6 +72,7 @@ const downloadSucceeded = (version: string) =>
{`Download for version ${ver const idle = () => <>; + export const NonInjectedAutoUpdateComponent = observer(({ progressOfUpdateDownload, discoveredVersionState, downloadingUpdateState, checkingForUpdatesState }: Dependencies) => { const discoveredVersion = discoveredVersionState.value.get();