Revert "[Form] Default to use POST as the method (#2066)" (#2117)

This reverts commit e2b7ece.

Author: amrocha

UNRELEASED.md

@@ -10,7 +10,6 @@ Use [the changelog guidelines](https://git.io/polaris-changelog-guidelines) to f
 
 ### Enhancements
 
-- Changed `Form` to default the `method` to `post` in order to prevent accidental leaking of form details ([#2066](https://github.com/Shopify/polaris-react/pull/2066))
 - Added support for boolean type on Choice error prop ([#2085](https://github.com/shopify/polaris-react/pull/2085))
 
 ### Documentation

src/components/Form/Form.tsx

@@ -46,7 +46,7 @@ export default class Form extends React.PureComponent<Props> {
       children,
       encType,
       implicitSubmit = true,
-      method = 'post',
+      method,
       name,
       noValidate,
       target,

src/components/Form/tests/Form.test.tsx

@@ -18,7 +18,7 @@ describe('<Form />', () => {
       const wrapper = mountWithAppProvider(
         <Form acceptCharset={acceptCharset} onSubmit={noop} />,
       );
-      expect(wrapper.find('form').prop('acceptCharset')).toBe(acceptCharset);
+      expect(wrapper.prop('acceptCharset')).toBe(acceptCharset);
     });
   });
 
@@ -27,16 +27,16 @@ describe('<Form />', () => {
       const wrapper = mountWithAppProvider(
         <Form action={action} onSubmit={noop} />,
       );
-      expect(wrapper.find('form').prop('action')).toBe(action);
+      expect(wrapper.prop('action')).toBe(action);
     });
   });
 
   describe('autoComplete', () => {
-    it('sets the autocomplete attribute to "off" when provided as false', () => {
+    it('sets the autocomplete attribute when provided', () => {
       const wrapper = mountWithAppProvider(
         <Form autoComplete={false} onSubmit={noop} />,
       );
-      expect(wrapper.find('form').prop('autoComplete')).toBe('off');
+      expect(wrapper.prop('autoComplete')).toBe(false);
     });
   });
 
@@ -45,7 +45,7 @@ describe('<Form />', () => {
       const wrapper = mountWithAppProvider(
         <Form encType={encType} onSubmit={noop} />,
       );
-      expect(wrapper.find('form').prop('encType')).toBe(encType);
+      expect(wrapper.prop('encType')).toBe(encType);
     });
   });
 
@@ -68,12 +68,7 @@ describe('<Form />', () => {
       const wrapper = mountWithAppProvider(
         <Form method={method} onSubmit={noop} />,
       );
-      expect(wrapper.find('form').prop('method')).toBe(method);
-    });
-
-    it('defaults to post when no method is set', () => {
-      const wrapper = mountWithAppProvider(<Form onSubmit={noop} />);
-      expect(wrapper.find('form').prop('method')).toBe('post');
+      expect(wrapper.prop('method')).toBe(method);
     });
   });
 
@@ -82,7 +77,7 @@ describe('<Form />', () => {
       const wrapper = mountWithAppProvider(
         <Form name={name} onSubmit={noop} />,
       );
-      expect(wrapper.find('form').prop('name')).toBe(name);
+      expect(wrapper.prop('name')).toBe(name);
     });
   });
 
@@ -91,7 +86,7 @@ describe('<Form />', () => {
       const wrapper = mountWithAppProvider(
         <Form noValidate={noValidate} onSubmit={noop} />,
       );
-      expect(wrapper.find('form').prop('noValidate')).toBe(noValidate);
+      expect(wrapper.prop('noValidate')).toBe(noValidate);
     });
   });
 
@@ -155,7 +150,7 @@ describe('<Form />', () => {
       const wrapper = mountWithAppProvider(
         <Form target={target} onSubmit={noop} />,
       );
-      expect(wrapper.find('form').prop('target')).toBe(target);
+      expect(wrapper.prop('target')).toBe(target);
     });
   });
 });