implement all

Author: yusukebe

src/pico.ts

@@ -1,7 +1,7 @@
 import type { Handler } from './types'
 
-export const METHODS = ['get', 'post', 'put', 'delete', 'head', 'options', 'patch'] as const
-export function defineDynamicClass(): {
+const METHODS = ['all', 'get', 'post', 'put', 'delete', 'head', 'options', 'patch'] as const
+function defineDynamicClass(): {
   new (): {
     [K in typeof METHODS[number]]: (path: string, handler: Handler) => Pico
   }
@@ -23,7 +23,7 @@ class Pico extends defineDynamicClass() {
           pattern: new URLPattern({
             pathname: path,
           }),
-          method: method.toLocaleUpperCase(),
+          method: method.toUpperCase(),
           handler,
         }
         this.routes.push(route)
@@ -35,7 +35,8 @@ class Pico extends defineDynamicClass() {
   match(method: string, url: string): { handler: Handler; result: URLPatternURLPatternResult } {
     method = method.toUpperCase()
     for (const route of this.routes) {
-      if (route.pattern.test(url) && route.method === method) {
+      const matched = route.pattern.test(url)
+      if ((matched && route.method === 'ALL') || (matched && route.method === method)) {
         return { handler: route.handler, result: route.pattern.exec(url) }
       }
     }

test/pico.test.ts

@@ -1,6 +1,6 @@
 import { Pico } from '../src/pico'
 
-describe('Test Pico', () => {
+describe('Basic', () => {
   const app = new Pico()
   app.get('/', () => 'Hi')
   app.get('/json', () => ({
@@ -33,3 +33,20 @@ describe('Test Pico', () => {
     expect(await res.text()).toBe('Custom Not Found')
   })
 })
+
+describe('All', () => {
+  const app = new Pico()
+  app.all('/abc', () => 'Hi')
+
+  it('Should return 200 response with GET request', async () => {
+    const req = new Request('http://localhost/abc')
+    const res = app.fetch(req)
+    expect(res.status).toBe(200)
+  })
+
+  it('Should return 200 response with POST request', async () => {
+    const req = new Request('http://localhost/abc', { method: 'POST' })
+    const res = app.fetch(req)
+    expect(res.status).toBe(200)
+  })
+})