fix(Container): Fix Promise handling on #load(), change event namespa…

…ce from 'open' to 'load' as spec
bdfoster · Nov 21, 2017 · 1f40417 · 1f40417
1 parent 0389925
commit 1f40417
Showing 1 changed file with 4 additions and 3 deletions.
diff --git a/src/Container.ts b/src/Container.ts
@@ -223,7 +223,7 @@ export class Container extends AsyncEventEmitter {
 
         if (this._isLoading) {
             return new Promise((resolve) => {
-                this.on('open', (self) => {
+                this.on('load', (self) => {
                     resolve(self);
                 });
             });
@@ -239,10 +239,11 @@ export class Container extends AsyncEventEmitter {
             }
 
             return Promise.all(results).then(() => {
-                this.emit('open', this);
                 this._isLoaded = true;
                 this._isLoading = false;
-                return this;
+                return this.emit('load', this).then(() => {
+                    return this;
+                });
             });
         });
     }