fix(polyfill): ensure promise polyfill applied for ie11

Closes #1188

Author: adamdbradley

src/client/core-esm.ts

@@ -35,59 +35,55 @@ export function defineCustomElement(win: Window, cmpData: d.ComponentHostData |
     createComponentOnReadyPrototype(win, namespace, (win as any).HTMLElement.prototype);
   }
 
-  return new Promise(resolve => {
+  return applyPolyfills(win).then(() => {
+
+    if (!pltMap[namespace]) {
+      const Context: d.CoreContext = {};
+      const resourcesUrl = opts.resourcesUrl || './';
+
+      appGlobal(namespace, Context, win, doc, resourcesUrl, hydratedCssClass);
+
+      // create a platform for this namespace
+      pltMap[namespace] = createPlatformMain(
+        namespace,
+        Context,
+        win,
+        doc,
+        resourcesUrl,
+        hydratedCssClass,
+        cmpDataArray
+      );
+    }
 
-    applyPolyfills(win, () => {
+    // polyfills have been applied if need be
+    (cmpData as d.ComponentHostData[]).forEach(c => {
+      let HostElementConstructor: any;
 
-      if (!pltMap[namespace]) {
-        const Context: d.CoreContext = {};
-        const resourcesUrl = opts.resourcesUrl || './';
+      if (isNative(win.customElements.define)) {
+        // native custom elements supported
+        const createHostConstructor = new Function('w', 'return class extends w.HTMLElement{}');
+        HostElementConstructor = createHostConstructor(win);
 
-        appGlobal(namespace, Context, win, doc, resourcesUrl, hydratedCssClass);
+      } else {
+        // using polyfilled custom elements
+        HostElementConstructor = function(self: any) {
+          return (win as any).HTMLElement.call(this, self);
+        };
 
-        // create a platform for this namespace
-        pltMap[namespace] = createPlatformMain(
-          namespace,
-          Context,
-          win,
-          doc,
-          resourcesUrl,
-          hydratedCssClass,
-          cmpDataArray
+        HostElementConstructor.prototype = Object.create(
+          (win as any).HTMLElement.prototype,
+          { constructor: { value: HostElementConstructor, configurable: true } }
         );
       }
 
-      // polyfills have been applied if need be
-      (cmpData as d.ComponentHostData[]).forEach(c => {
-        let HostElementConstructor: any;
-
-        if (isNative(win.customElements.define)) {
-          // native custom elements supported
-          const createHostConstructor = new Function('w', 'return class extends w.HTMLElement{}');
-          HostElementConstructor = createHostConstructor(win);
-
-        } else {
-          // using polyfilled custom elements
-          HostElementConstructor = function(self: any) {
-            return (win as any).HTMLElement.call(this, self);
-          };
-
-          HostElementConstructor.prototype = Object.create(
-            (win as any).HTMLElement.prototype,
-            { constructor: { value: HostElementConstructor, configurable: true } }
-          );
-        }
-
-        // convert the static constructor data to cmp metadata
-        // define the component as a custom element
-        pltMap[namespace].defineComponent(
-          buildComponentLoader(c),
-          HostElementConstructor
-        );
-      });
-
-      resolve();
+      // convert the static constructor data to cmp metadata
+      // define the component as a custom element
+      pltMap[namespace].defineComponent(
+        buildComponentLoader(c),
+        HostElementConstructor
+      );
     });
+
   });
 }
 

src/client/polyfills/index.js

@@ -12,6 +12,8 @@ describe('esm-webpack', () => {
 
   it('webpack', async () => {
     await testEsmImport(app);
+    const hydratedElm = app.querySelector('esm-import.hydrated')
+    expect(hydratedElm).not.toBe(null);
   });
 
 });