Added correct name of remove handler method

Matt Fehskens · Matt Fehskens · commit bd15ca98be12 · 2019-07-01T16:54:22.000-04:00
- Added  test for checking that a handler was added
diff --git a/src/__tests__/react-plotly.test.js b/src/__tests__/react-plotly.test.js
@@ -164,5 +164,20 @@ describe('<Plotly/>', () => {
           .catch(err => done.fail(err));
       });
     });
+
+    describe('manging event handlers', () => {
+      test('should add an event handler when one does not already exist', (done) => {
+        const onRelayout = () => {};
+
+        createPlot({onRelayout}).then((plot) => {
+          const { handlers } = plot.instance();
+
+          expect(plot.prop('onRelayout')).toBe(onRelayout);
+          expect(handlers.Relayout).toBe(onRelayout);
+
+          done();
+        });
+      });
+    });
   });
 });
diff --git a/src/factory.js b/src/factory.js
@@ -214,10 +214,10 @@ export default function plotComponentFactory(Plotly) {
           this.addEventHandler(eventName, prop);
         } else if (!prop && hasHandler) {
           // Needs to be removed:
-          this.removeHandler(eventName);
+          this.removeEventHandler(eventName);
         } else if (prop && hasHandler && prop !== handler) {
           // replace the handler
-          this.removeHandler(eventName);
+          this.removeEventHandler(eventName);
           this.addEventHandler(eventName, prop);
         }
       });
