fix(race condition): fire login event before bridge inited (#772)

src/puppet-web/browser.ts

@@ -501,6 +501,11 @@ export class Browser extends EventEmitter {
     log.verbose('PuppetWebBrowser', 'clickSwitchAccount()')
 
     try {
+      // TODO
+      // Promise.race([
+      //   english
+      //   chinese
+      // ])
       const button = await this.driver.driver.findElement(By.linkText('Switch Account'))
       button.click()
       log.silly('PuppetWebBrowser', 'clickSwitchAccount() clicked!')

src/puppet-web/event.ts

@@ -240,6 +240,15 @@ function onServerLog(data) {
 async function onServerLogin(this: PuppetWeb, data, attempt = 0): Promise<void> {
   log.verbose('PuppetWebEvent', 'onServerLogin(%s, %d)', data, attempt)
 
+  // issue #772
+  // if `login` event fired before this.bridge inited, we delay the event for 1 second.
+  if (!this.bridge) {
+    setTimeout(() => {
+      onServerLogin.apply(this, arguments)
+    }, 1000)
+    return
+  }
+
   this.scan = null
 
   if (this.userId) {
@@ -248,6 +257,8 @@ async function onServerLogin(this: PuppetWeb, data, attempt = 0): Promise<void>
   try {
     /**
      * save login user id to this.userId
+     *
+     * issue #772: this.bridge might not inited if the 'login' event fired too fast(because of auto login)
      */
     this.userId = await this.bridge.getUserName()
 

src/puppet-web/puppet-web.ts

@@ -51,9 +51,9 @@ import * as bl from 'bl'
 type MediaType = 'pic' | 'video' | 'doc'
 
 const enum UploadMediaType {
-  IMAGE = 1,
-  VIDEO = 2,
-  AUDIO = 3,
+  IMAGE      = 1,
+  VIDEO      = 2,
+  AUDIO      = 3,
   ATTACHMENT = 4,
 }
 export interface PuppetWebSetting {