Profile fetching failure toast (#1108)

* Toast safety

* Show failure toast if resource or external resource queries fail

* Make sure external resource loading indicator is reset properly

* Make toasts test safe

---------

Co-authored-by: Dan Delany <dan.delany@gmail.com>

src/routes/plans/[id]/+page.svelte

@@ -85,6 +85,7 @@
     externalResourceNames,
     externalResources,
     fetchingResources,
+    fetchingResourcesExternal,
     resetSimulationStores,
     resourceTypes,
     resources,
@@ -341,6 +342,7 @@
   } else {
     simulationDataAbortController?.abort();
     fetchingResources.set(false);
+    fetchingResourcesExternal.set(false);
     $resources = [];
     $spans = [];
   }

src/utilities/effects.ts

@@ -2882,6 +2882,7 @@ const effects = {
       const error = e as Error;
       if (error.name !== 'AbortError') {
         catchError(error);
+        showFailureToast('Failed to fetch profiles');
         fetchingResources.set(false);
       }
       return [];
@@ -2935,6 +2936,7 @@ const effects = {
       const error = e as Error;
       if (error.name !== 'AbortError') {
         catchError(error);
+        showFailureToast('Failed to fetch external profiles');
         fetchingResourcesExternal.set(false);
       }
       return [];

src/utilities/toast.ts

@@ -16,6 +16,11 @@ function findToast(toastText: string, checkIsClosed: boolean = false) {
 }
 
 function hideToast(toast: Toast) {
+  // Only hide toast if in browser context
+  if (typeof document === 'undefined') {
+    return;
+  }
+
   toast.options.close = true;
   toast.hideToast();
 }
@@ -35,6 +40,10 @@ function toastCallback(this: Element) {
 }
 
 function showToast(toast: Toast) {
+  // Only show toast if in browser context
+  if (typeof document === 'undefined') {
+    return;
+  }
   const toastText = toast.options.text;
   const existingToastIndex = toastText !== undefined ? findToast(toastText, true) : -1;