Add specs

github · May 28, 2024 · 1d5282c · 1d5282c
1 parent c47e474
commit 1d5282c
Show file tree

Hide file tree

Showing 3 changed files with 33 additions and 1 deletion.
diff --git a/src/auto-check-element.ts b/src/auto-check-element.ts
@@ -219,6 +219,9 @@ function makeAbortController() {
 }
 
 async function fetchWithNetworkEvents(el: Element, url: string, options: RequestInit): Promise<Response> {
+  if (options.method === 'GET') {
+    delete options.body
+  }
   try {
     const response = await fetch(url, options)
     el.dispatchEvent(new Event('load'))

diff --git a/test/auto-check.js b/test/auto-check.js
@@ -114,6 +114,35 @@ describe('auto-check element', function () {
     })
   })
 
+  describe('using HTTP GET', function () {
+    let checker
+    let input
+
+    beforeEach(function () {
+      const container = document.createElement('div')
+      container.innerHTML = `
+        <auto-check src="/success" http-method="GET" required>
+          <input>
+        </auto-check>`
+      document.body.append(container)
+
+      checker = document.querySelector('auto-check')
+      input = checker.querySelector('input')
+    })
+
+    afterEach(function () {
+      document.body.innerHTML = ''
+      checker = null
+      input = null
+    })
+
+    it('validates input with successful server response with GET', async function () {
+      triggerInput(input, 'hub')
+      await once(input, 'auto-check-complete')
+      assert.isTrue(input.checkValidity())
+    })
+  })
+
   describe('network lifecycle events', function () {
     let checker
     let input

diff --git a/web-test-runner.config.js b/web-test-runner.config.js
@@ -19,7 +19,7 @@ export default {
   middleware: [
     async ({request, response}, next) => {
       const {method, path} = request
-      if (method === 'POST') {
+      if (method === 'POST' || method === 'GET') {
         if (path.startsWith('/fail')) {
           response.status = 422
           // eslint-disable-next-line i18n-text/no-en