Fix specs

Author: jeffcarbs

src/addons/Portal/Portal.js

@@ -258,10 +258,10 @@ class Portal extends Component {
     this.trySetState({ open: true })
   }
 
-  openWithTimeout = (e, delay = 0) => {
+  openWithTimeout = (e, delay) => {
     // React wipes certain props (e.g. currentTarget) so we need to clone.
     const eventClone = { ...e }
-    return setTimeout(() => this.open(eventClone), delay)
+    return setTimeout(() => this.open(eventClone), delay || 0)
   }
 
   close = (e) => {
@@ -273,10 +273,10 @@ class Portal extends Component {
     this.trySetState({ open: false })
   }
 
-  closeWithTimeout = (e, delay = 0) => {
+  closeWithTimeout = (e, delay) => {
     // React wipes certain props (e.g. currentTarget) so we need to clone.
     const eventClone = { ...e }
-    return setTimeout(() => this.close(eventClone), delay)
+    return setTimeout(() => this.close(eventClone), delay || 0)
   }
 
   renderPortal() {

test/specs/addons/Portal/Portal-test.js

@@ -265,7 +265,6 @@ describe('Portal', () => {
       setTimeout(() => {
         document.body.childElementCount.should.equal(0)
         spy.should.have.been.calledOnce()
-        done()
       }, mouseOverDelay - 1)
 
       setTimeout(() => {
@@ -304,7 +303,6 @@ describe('Portal', () => {
       setTimeout(() => {
         document.body.lastElementChild.should.equal(wrapper.instance().node)
         spy.should.have.been.calledOnce()
-        done()
       }, mouseLeaveDelay - 1)
 
       setTimeout(() => {
@@ -314,18 +312,17 @@ describe('Portal', () => {
       }, mouseLeaveDelay + 1)
     })
 
-    it('should close portal on mouseleave of portal when set', (done) => {
+    it('should close portal on mouseleave *of portal* when set', (done) => {
       const trigger = <button>button</button>
       const mouseLeaveDelay = 100
       wrapperMount(
         <Portal trigger={trigger} defaultOpen closeOnMouseLeave mouseLeaveDelay={mouseLeaveDelay}><p>Hi</p></Portal>
       )
 
-      domEvent.mouseOver(wrapper.instance().node.firstElementChild)
+      domEvent.mouseLeave(wrapper.instance().node.firstElementChild)
 
       setTimeout(() => {
         document.body.lastElementChild.should.equal(wrapper.instance().node)
-        done()
       }, mouseLeaveDelay - 1)
 
       setTimeout(() => {
@@ -346,7 +343,6 @@ describe('Portal', () => {
       // Fire a mouseOver on the portal within the time limit
       setTimeout(() => {
         domEvent.mouseOver(wrapper.instance().node.firstElementChild)
-        done()
       }, mouseLeaveDelay - 1)
 
       // The portal should not have closed