fix spec

react-component · May 23, 2016 · 239c2cd · 239c2cd
1 parent f08a521
commit 239c2cd
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 7 deletions.
diff --git a/src/BannerAnim.jsx b/src/BannerAnim.jsx
@@ -63,7 +63,6 @@ class BannerAnim extends Component {
     } else {
       window.detachEvent('onresize', this.getDomDataSetToState);
     }
-    this.cancelRequestAnimationFrame();
   }
 
   onMouseEnter() {

diff --git a/src/Element.jsx b/src/Element.jsx
@@ -430,7 +430,7 @@ Element.propTypes = {
   callBack: PropTypes.func,
   bgParallax: PropTypes.object,
   followParallax: PropTypes.object,
-  show: PropTypes.string,
+  show: PropTypes.bool,
 };
 Element.defaultProps = {
   component: 'div',

diff --git a/tests/index.spec.js b/tests/index.spec.js
@@ -71,8 +71,10 @@ describe('rc-banner-anim', function() {
       type: 'across',
     });
     const children = TestUtils.scryRenderedDOMComponentsWithClass(instance, 'banner-anim-elem');
-    expect(children[0].style.display).to.be('none');
-    expect(children[1].style.display).to.not.be.ok();
+    setTimeout(()=> {
+      expect(children[0].style.display).to.be('none');
+      expect(children[1].style.display).to.be('block');
+    });
   });
 
   it('banner animation autoplay', function(done) {
@@ -82,10 +84,12 @@ describe('rc-banner-anim', function() {
       type: 'across',
     });
     const children = TestUtils.scryRenderedDOMComponentsWithClass(instance, 'banner-anim-elem');
-    expect(children[0].style.display).to.not.be.ok();
+    expect(children[0].style.display).to.be('block');
+    expect(children[1].style.display).to.be('none');
     setTimeout(()=> {
-      expect(children[1].style.display).to.not.be.ok();
+      expect(children[0].style.display).to.be('block');
+      expect(children[1].style.display).to.be('block');
       done();
-    }, 1650);
+    }, 1030);
   });
 });