[ReactFinalForm.js]: Add a test for 100% test coverage (final-form#307)

* chore(contributing.md): Fix word spacing * test(ReactFinalForm): Test events without preventDefault or stopPropagation
eJayYoung · Jul 23, 2018 · 90459e7 · 90459e7
1 parent dec9c7e
commit 90459e7
Show file tree

Hide file tree

Showing 2 changed files with 23 additions and 1 deletion.
diff --git a/.github/CONTRIBUTING.md b/.github/CONTRIBUTING.md
@@ -8,7 +8,7 @@ We are open to, and grateful for, any contributions made by the community.
 ## Reporting issues and asking questions
 
 Before opening an issue, please search
-the[issue tracker](https://github.com/final-form/react-final-form/issues) to
+the [issue tracker](https://github.com/final-form/react-final-form/issues) to
 make sure your issue hasn’t already been reported.
 
 **We use the issue tracker to keep track of bugs and improvements** to 🏁 React

diff --git a/src/ReactFinalForm.test.js b/src/ReactFinalForm.test.js
@@ -202,6 +202,28 @@ describe('ReactFinalForm', () => {
     TestUtils.Simulate.submit(form)
     expect(onSubmit).toHaveBeenCalled()
   })
+  it('does not throw if handleSubmit event preventDefault or stopPropagation are not functions', () => {
+    const onSubmit = jest.fn()
+    const dom = TestUtils.renderIntoDocument(
+      <Form onSubmit={onSubmit}>
+        {({ handleSubmit }) => (
+          <form
+            onSubmit={() => {
+              handleSubmit({
+                preventDefault: undefined,
+                stopPropagation: undefined
+              })
+            }}
+          >
+            <Field name="foo" component="input" />
+          </form>
+        )}
+      </Form>
+    )
+    const form = TestUtils.findRenderedDOMComponentWithTag(dom, 'form')
+    TestUtils.Simulate.submit(form)
+    expect(onSubmit).toHaveBeenCalled()
+  })
   it('should reinitialize when initialValues prop changes', () => {
     const renderInput = jest.fn(({ input }) => <input {...input} />)
     class Container extends React.Component {