Try specs without setTimeout calls
authorKevin Sawicki <kevinsawicki@gmail.com>
Tue, 22 Nov 2016 21:13:30 +0000 (13:13 -0800)
committerKevin Sawicki <kevinsawicki@gmail.com>
Tue, 22 Nov 2016 21:14:17 +0000 (13:14 -0800)
spec/api-browser-window-spec.js

index d5d5b00..75e4dab 100644 (file)
@@ -1755,7 +1755,7 @@ describe('browser-window module', function () {
     })
   })
 
-  describe('minWidth', function () {
+  describe('Restoring the window', function () {
     beforeEach(function () {
       if (w != null) w.destroy()
       w = new BrowserWindow({
@@ -1764,30 +1764,24 @@ describe('browser-window module', function () {
       })
     })
 
-    it('should not affect the bounds when restoring the window', function (done) {
+    it('should set the correct width when minWidth is used', function () {
       w.minimize()
-      setTimeout(() => {
-        w.restore()
-        assert.equal(w.getSize()[0], 800)
-        done()
-      }, 200)
+      w.restore()
+      assert.equal(w.getSize()[0], 800)
     })
   })
 
-  describe('window position', function () {
+  describe('Unmaximizing the window', function () {
     beforeEach(function () {
       if (w != null) w.destroy()
       w = new BrowserWindow()
     })
 
-    it('should not affect the bounds when restoring the window', function (done) {
-      const originalPos = w.getPosition()
+    it('should set the correct position', function () {
+      const initialPosition = w.getPosition()
       w.maximize()
-      setTimeout(() => {
-        w.unmaximize()
-        assertBoundsEqual(originalPos, w.getPosition())
-        done()
-      }, 200)
+      w.unmaximize()
+      assertBoundsEqual(initialPosition, w.getPosition())
     })
   })
 })