refactor: ensure start order (#322)

apps/electron/main.js

@@ -29,14 +29,14 @@ let win;
 let splash;
 let tray = null;
 
-(async () => {
+async function startBackend() {
+  await (async () => {
     // in dev mode, we expect both UI and server to be running
     if (!isProduction) {
       return;
     }
 
-  try {
+    const ontimeServer = require(nodePath);
-    const ontimeServer = require(nodePath)
     const { initAssets, startServer, startOSCServer, startIntegrations } = ontimeServer;
 
     await initAssets();
@@ -44,10 +44,8 @@ let tray = null;
     loaded = await startServer();
     await startOSCServer();
     await startIntegrations();
-  } catch (error) {
-    loaded = error;
-  }
   })();
+}
 
 /**
  * @description utility function to create a notification
@@ -158,12 +156,14 @@ app.whenReady().then(() => {
     bringToFront();
   });
 
-  // cheat to schedule process
+  startBackend()
-  setTimeout(() => {
+    .then(() => {
       // Load page served by node or use React dev run
       const clientUrl = isProduction ? electronConfig.reactAppUrl.production : electronConfig.reactAppUrl.development;
 
-    win.loadURL(clientUrl).then(() => {
+      win
+        .loadURL(clientUrl)
+        .then(() => {
           win.webContents.setBackgroundThrottling(false);
 
           win.show();
@@ -176,8 +176,14 @@ app.whenReady().then(() => {
           } else {
             tray.setToolTip('Initialising error: please restart Ontime');
           }
+        })
+        .catch((error) => {
+          dialog.showErrorBox('Ontime failed to reach server', error);
+        });
+    })
+    .catch((error) => {
+      dialog.showErrorBox('Ontime failed to start', error);
     });
-  }, 0);
 
   // recreate window if no others open
   app.on('activate', () => {