fix: respect novalidate

Author: calebdwilliams

Diff for: src/utils.ts

@@ -132,6 +132,13 @@ export const formSubmitCallback = (event: Event) => {
   const form = event.target as HTMLFormElement;
   const elements = formElementsMap.get(form);
 
+  /**
+   * If this form does not validate then we're done
+   */
+  if (form.noValidate) {
+    return;
+  }
+
   /** If the Set has items, continue */
   if (elements.size) {
     const nodes = Array.from(elements);

Diff for: test/ElementInternals.test.js

@@ -510,4 +510,25 @@ describe('The ElementInternals polyfill', () => {
       expect(callCount).to.equal(1);
     });
   });
+
+  describe('Forms with novalidate', () => {
+    it('will not block submit', async () => {
+      let submitCount = 0;
+      const onSubmit = (event) => {
+        event.preventDefault();
+        submitCount += 1;
+      }
+      const form = await fixture(html`<form novalidate @submit="${onSubmit}">
+        <test-el name="foo" id="foo"></test-el>
+        <button type="submit">Submit</button>
+      </form>`);
+      const testEl = form.querySelector('test-el');
+      const button = form.querySelector('button');
+      testEl.internals.setValidity({
+        customError: true
+      }, 'error message');
+      button.click();
+      expect(submitCount).to.equal(1);
+    });
+  });
 });