Provides a descriptive error when simulating non-existent events, fixes

#79
enzymejs · Jan 12, 2016 · 6389d93 · 6389d93
1 parent ba911a0
commit 6389d93
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/src/ReactWrapper.js b/src/ReactWrapper.js
@@ -294,7 +294,14 @@ export default class ReactWrapper {
    * @returns {ReactWrapper}
    */
   simulate(event, ...args) {
-    this.single(n => Simulate[event](findDOMNode(n), ...args));
+    this.single(n => {
+      const eventFn = Simulate[event];
+      if (!eventFn) {
+        throw new TypeError(`ReactWrapper::simulate() event '${event}' does not exist`);
+      }
+
+      eventFn(findDOMNode(n), ...args);
+    });
     return this;
   }
 

diff --git a/src/__tests__/ReactWrapper-spec.js b/src/__tests__/ReactWrapper-spec.js
@@ -341,7 +341,6 @@ describeWithDOM('mount', () => {
 
     });
 
-
     it('should pass in event data', () => {
       const spy = sinon.spy();
       class Foo extends React.Component {
@@ -359,6 +358,11 @@ describeWithDOM('mount', () => {
       expect(spy.args[0][0].someSpecialData).to.equal('foo');
     });
 
+    it('should throw a descriptive error for invalid events', () => {
+      const wrapper = mount(<div>foo</div>);
+      expect(wrapper.simulate.bind(wrapper, 'invalidEvent'))
+        .to.throw(TypeError, "ReactWrapper::simulate() event 'invalidEvent' does not exist");
+    });
   });
 
   describe('.setState(newState)', () => {