fix(emulators): wait for doc element

Author: blakebyrnes

emulator-plugins/shared/injected-scripts/index.ts

@@ -22,10 +22,13 @@ function buildExecutionScript(name: string, script: string, args?: any) {
     ${utilsScript}
    
     // documentElement is not loaded  
-    if ('${name}' === 'polyfill') {  
-      while (!document.documentElement) {
-          await new Promise(resolve => setTimeout(resolve, 10));
-      }
+    if ('${name}' === 'polyfill' && !document.documentElement) {
+      await new Promise(resolve => {
+        new MutationObserver((list, observer) => {
+          resolve();
+          observer.disconnect();
+        }).observe(document, {childList: true, subtree: true});
+      });
     }
     
     (function ${name}(args) {

emulator-plugins/shared/injected-scripts/polyfill.js

@@ -1,4 +1,4 @@
-for (const remove of args.removals) {
+for (const remove of args.removals || []) {
   try {
     const parts = getParentAndProperty(remove);
     delete parts.parent[parts.property];
@@ -7,7 +7,7 @@ for (const remove of args.removals) {
   }
 }
 
-for (const addition of args.additions) {
+for (const addition of args.additions || []) {
   try {
     if (addition.propertyName === 'getVideoPlaybackQuality') {
       addition.property['_value()'] = function() {
@@ -28,7 +28,7 @@ for (const addition of args.additions) {
   }
 }
 
-for (const change of args.changes) {
+for (const change of args.changes || []) {
   try {
     if (change.propertyName === '_function') {
       const func = getObjectAtPath(change.path);
@@ -67,7 +67,7 @@ for (const change of args.changes) {
 }
 
 function reorder() {
-  for (const { propertyName, prevProperty, throughProperty, path } of args.order) {
+  for (const { propertyName, prevProperty, throughProperty, path } of args.order || []) {
     try {
       if (!path.includes('.prototype')) {
         reorderOnWindow(path, propertyName, prevProperty, throughProperty);

jest.teardown.js

@@ -1,7 +1,12 @@
 const fs = require('fs');
+const os = require('os');
+const { execSync } = require('child_process');
 
 module.exports = async () => {
   try {
+    if (os.platform() === 'win32') {
+      execSync(`TASKKILL /IM chrome.exe /F`);
+    }
     fs.rmdirSync(`${__dirname}/.cache-test`, { recursive: true });
   } catch (err) {}
 };