code clean

src/puppet-web/browser.js

@@ -54,8 +54,7 @@ class Browser extends EventEmitter {
     })
   }
 
-  open(url) {
-    url = url || 'https://wx.qq.com'
+  open(url = 'https://wx.qq.com') {
     log.verbose('PuppetWebBrowser', `open(${url})`)
 
     // TODO: set a timer to guard driver.get timeout, then retry 3 times 201607
@@ -324,7 +323,7 @@ this.onResourceRequested = function(request, net) {
      */
     // if (cookie.expiry) { cookie.expiry = cookie.expiry * 1000 /* XXX: be aware of new version of webdriver */}
 
-    log.silly('PuppetWebBrowser', 'addCookies("%s")', JSON.stringify(cookie))
+    log.silly('PuppetWebBrowser', 'addCookies(%s)', JSON.stringify(cookie))
 
     return this.driver.manage()
     // this is old webdriver format

test/puppet-web/browser.spec.js

@@ -60,6 +60,7 @@ test('Browser class cookie smoking tests', async t => {
     const cookies0 = cookies.filter(c => { return RegExp(EXPECTED_COOKIES[0].name).test(c.name) })
     t.is(cookies0[0].name, EXPECTED_COOKIES[0].name, 'getCookies() should filter out the cookie named wechaty0')
     const cookies1 = cookies.filter(c => { return RegExp(EXPECTED_COOKIES[1].name).test(c.name) })
+    t.truthy(cookies1, 'should get cookies1')
     t.is(cookies1[0].name, EXPECTED_COOKIES[1].name, 'getCookies() should filter out the cookie named wechaty1')
 
     await b.open()