fix(client): do not throw on SASS compile errors

client/src/client/workers/sass-compile.js

@@ -19,7 +19,7 @@ self.onmessage = e => {
     if (result.status === 0) {
       self.postMessage(result.text);
     } else {
-      throw result.formatted;
+      self.postMessage({ type: 'error', data: { message: result.formatted } });
     }
   });
 };

client/src/templates/Challenges/utils/worker-executor.js

@@ -39,31 +39,48 @@ class WorkerExecutor {
       // Handle timeout
       const timeoutId = setTimeout(() => {
         this.killWorker();
-        reject('timeout');
+        done('timeout');
       }, timeout);
 
+      const done = (err, data) => {
+        clearTimeout(timeoutId);
+        this.remove('error', handleError);
+        if (err) {
+          reject(err);
+        } else {
+          resolve(data);
+        }
+      };
+
+      const handleError = e => {
+        done(e.message);
+      };
+      this.on('error', handleError);
+
       worker.postMessage(data);
 
       // Handle result
       worker.onmessage = e => {
         if (e.data && e.data.type) {
-          const observers = this.observers[e.data.type] || [];
+          this.handleEvent(e.data.type, e.data.data);
-          for (const observer of observers) {
-            observer(e.data.data);
-          }
           return;
         }
-        clearTimeout(timeoutId);
+        done(null, e.data);
-        resolve(e.data);
       };
 
       worker.onerror = e => {
-        clearTimeout(timeoutId);
+        this.handleEvent('error', { message: e.message });
-        reject(e.message);
       };
     });
   }
 
+  handleEvent(type, data) {
+    const observers = this.observers[type] || [];
+    for (const observer of observers) {
+      observer(data);
+    }
+  }
+
   on(type, callback) {
     const observers = this.observers[type] || [];
     observers.push(callback);