fix: LEAP-740: Don't use destroy = null

e2e/tests/helpers.js

@@ -157,14 +157,16 @@ const createAddEventListenerScript = (eventName, callback) => {
  * Wait for the main Image object to be loaded
  */
 const waitForImage = () => {
-  return new Promise((resolve) => {
+  return new Promise((resolve, reject) => {
     const img = document.querySelector('[alt=LS]');
 
     if (!img || img.complete) return resolve();
     // this should be rewritten to isReady when it is ready
     img.onload = () => {
       setTimeout(resolve, 100);
     };
+    // if image is not loaded in 10 seconds, reject
+    setTimeout(reject, 10000);
   });
 };
 

src/LabelStudio.tsx

@@ -53,7 +53,7 @@ export class LabelStudio {
   root: Element | string;
   store: any;
 
-  destroy: (() => void) | null = null;
+  destroy: (() => void) | null = () => {};
   events = new EventInvoker();
 
   getRootElement(root: Element | string) {