Add logging to figure out failures on BrowserWindow.moveTop()

John Kleinschmidt · John Kleinschmidt · commit e21ace30dc24 · 2019-09-04T10:49:24.000-04:00
diff --git a/spec/api-browser-window-spec.js b/spec/api-browser-window-spec.js
@@ -516,43 +516,46 @@ describe('BrowserWindow module', () => {
 
   describe('BrowserWindow.moveTop()', () => {
     it('should not steal focus', async () => {
+      console.log('should not steal focus 1')
       const posDelta = 50
       const wShownInactive = emittedOnce(w, 'show')
       w.showInactive()
       await wShownInactive
       assert(!w.isFocused())
-
+      console.log('should not steal focus 2')
       const otherWindow = new BrowserWindow({ show: false, title: 'otherWindow' })
       const otherWindowShown = emittedOnce(otherWindow, 'show')
       otherWindow.loadURL('data:text/html,<html><body background-color: rgba(255,255,255,0)></body></html>')
       otherWindow.show()
       await otherWindowShown
       assert(otherWindow.isFocused())
-
+      console.log('should not steal focus 3') 
       w.moveTop()
       const wPos = w.getPosition()
       const wMoving = emittedOnce(w, 'move')
       w.setPosition(wPos[0] + posDelta, wPos[1] + posDelta)
       await wMoving
       assert(!w.isFocused())
       assert(otherWindow.isFocused())
-
+      console.log('should not steal focus 4') 
       const wFocused = emittedOnce(w, 'focus')
       w.focus()
       await wFocused
       assert(w.isFocused())
-
+      console.log('should not steal focus 5') 
       otherWindow.moveTop()
       const otherWindowPos = otherWindow.getPosition()
       const otherWindowMoving = emittedOnce(otherWindow, 'move')
       otherWindow.setPosition(otherWindowPos[0] + posDelta, otherWindowPos[1] + posDelta)
       await otherWindowMoving
       assert(!otherWindow.isFocused())
       assert(w.isFocused())
-
+      console.log('should not steal focus 6') 
       await closeWindow(otherWindow, { assertSingleWindow: false }).then(() => {
+        console.log('should not steal focus 7') 
         assert.strictEqual(BrowserWindow.getAllWindows().length, 2) // Test window + w
       })
+      console.log('should not steal focus 8') 
     })
   })
 
