fix(test): serial for prevent race condition

src/puppet-web/browser-driver.ts

@@ -23,7 +23,9 @@ import {
 export class BrowserDriver {
   private driver: WebDriver
 
-  constructor(private head: HeadName) {
+  constructor(
+    private head: HeadName,
+  ) {
     log.verbose('PuppetWebBrowserDriver', 'constructor(%s)', head)
   }
 

src/puppet-web/browser.spec.ts

@@ -5,18 +5,11 @@
  * https://github.com/wechaty/wechaty
  *
  */
-import { test } from 'ava'
+import { test }     from 'ava'
 
-// import {
-  //   Config
-  // , log
-// } from '../config'
+import { Browser }  from './browser'
 
-import {
-    Browser,
-} from './browser'
-
-test('quit()', async t => {
+test.serial('quit()', async t => {
   const browser = new Browser()
   await browser.driver.init() // init driver, not init browser
 
@@ -32,7 +25,7 @@ test('quit()', async t => {
 
 })
 
-test('init()', async t => {
+test.serial('init()', async t => {
   const browser = new Browser()
   browser.state.target('open')
 

src/puppet-web/browser.ts

@@ -42,10 +42,12 @@ export class Browser extends EventEmitter {
 
   public state = new StateSwitch<'open', 'close'>('Browser', 'close', log)
 
-  constructor(private setting: BrowserSetting = {
-    head: Config.head,
-    sessionFile: '',
-  }) {
+  constructor(
+    private setting: BrowserSetting = {
+      head: Config.head,
+      sessionFile: '',
+    },
+  ) {
     super()
     log.verbose('PuppetWebBrowser', 'constructor() with head(%s) sessionFile(%s)', setting.head, setting.sessionFile)
 
@@ -247,9 +249,10 @@ export class Browser extends EventEmitter {
       else                            { log.warn('PuppetWebBrowser', 'driver.quit() exception: %s', e.message) }
 
       /* fail safe */
-    }
 
-    this.state.current('close')
+    } finally {
+      this.state.current('close')
+    }
 
     return
   }