chore: handle more cases where methods can be mocked

Signed-off-by: Florent Benoit <[email protected]>
podman-desktop · Dec 18, 2024 · eca9082 · eca9082
1 parent 40dab1a
commit eca9082
Show file tree

Hide file tree

Showing 4 changed files with 6 additions and 6 deletions.
diff --git a/packages/renderer/src/TelemetryService.ts b/packages/renderer/src/TelemetryService.ts
@@ -37,7 +37,7 @@ export class TelemetryService {
 
     this.handlerFlusher = setTimeout(() => {
       if (window.telemetryPage) {
-        window.telemetryPage(pagePath).catch((error: unknown) => {
+        window.telemetryPage(pagePath)?.catch((error: unknown) => {
           console.error('Failed to send page event', error);
         });
       }

diff --git a/packages/renderer/src/stores/appearance.ts b/packages/renderer/src/stores/appearance.ts
@@ -26,13 +26,13 @@ export const isDark: Writable<boolean> = writable(false);
 configurationProperties.subscribe(() => {
   if (window?.getConfigurationValue) {
     window
-      .getConfigurationValue<string>(AppearanceSettings.SectionName + '.' + AppearanceSettings.Appearance)
-      .then(value => {
+      ?.getConfigurationValue<string>(AppearanceSettings.SectionName + '.' + AppearanceSettings.Appearance)
+      ?.then(value => {
         if (value) {
           updateIsDark(value);
         }
       })
-      .catch((err: unknown) =>
+      ?.catch((err: unknown) =>
         console.error(
           `Error getting configuration value ${AppearanceSettings.SectionName + '.' + AppearanceSettings.Appearance}`,
           err,

diff --git a/packages/renderer/src/stores/context.ts b/packages/renderer/src/stores/context.ts
@@ -46,7 +46,7 @@ export function setup(): Writable<ContextUI> {
     // this function can be undefined during tests
     window
       .contextCollectAllValues?.()
-      .then(values => {
+      ?.then(values => {
         const currentContext = Object.entries(values).reduce((result, [key, value]) => {
           result.setValue(key, value);
           return result;

diff --git a/packages/renderer/src/stores/navigation/navigation-registry.ts b/packages/renderer/src/stores/navigation/navigation-registry.ts
@@ -154,7 +154,7 @@ configurationProperties.subscribe(() => {
   if (window.getConfigurationValue) {
     window
       .getConfigurationValue<string[]>('navbar.disabledItems')
-      .then(value => {
+      ?.then(value => {
         if (value) {
           hiddenItems = value;
         }