acrylblends 0.6.3 → 0.7.0

package/plugins/acryl-router/src/core/middleware/logger/index.test.ts

@@ -1,204 +0,0 @@
-import { logger } from './index.ts'
-import { Acryl } from '../../app.ts'
-
-describe('Logger by Middleware', () => {
-  let app: Acryl
-  let log: string
-
-  beforeEach(() => {
-    function sleep(time: number) {
-      return new Promise((resolve) => setTimeout(resolve, time))
-    }
-
-    app = new Acryl()
-
-    const logFn = (str: string) => {
-      log = str
-    }
-
-    const shortRandomString = 'acryl'
-    const longRandomString = 'acryl'.repeat(1000)
-
-    app.use('*', logger(logFn))
-    app.get('/short', (c) => c.text(shortRandomString))
-    app.get('/long', (c) => c.text(longRandomString))
-    app.get('/seconds', async (c) => {
-      await sleep(1000)
-
-      return c.text(longRandomString)
-    })
-    app.get('/empty', (c) => c.text(''))
-    app.get('/redirect', (c) => {
-      return c.redirect('/empty', 301)
-    })
-    app.get('/server-error', (c) => {
-      const res = new Response('', { status: 511 })
-      if (c.req.query('status')) {
-        // test status code not yet supported by runtime `Response` object
-        Object.defineProperty(res, 'status', { value: parseInt(c.req.query('status') as string) })
-      }
-      return res
-    })
-  })
-
-  it('Log status 200 with empty body', async () => {
-    const res = await app.request('http://localhost/empty')
-    expect(res).not.toBeNull()
-    expect(res.status).toBe(200)
-    expect(log.startsWith('--> GET /empty \x1b[32m200\x1b[0m')).toBe(true)
-    expect(log).toMatch(/m?s$/)
-  })
-
-  it('Log status 200 with small body', async () => {
-    const res = await app.request('http://localhost/short')
-    expect(res).not.toBeNull()
-    expect(res.status).toBe(200)
-    expect(log.startsWith('--> GET /short \x1b[32m200\x1b[0m')).toBe(true)
-    expect(log).toMatch(/m?s$/)
-  })
-
-  it('Log status 200 with small body and query param', async () => {
-    const res = await app.request('http://localhost/short?foo=bar')
-    expect(res).not.toBeNull()
-    expect(res.status).toBe(200)
-    expect(log.startsWith('--> GET /short?foo=bar \x1b[32m200\x1b[0m')).toBe(true)
-    expect(log).toMatch(/m?s$/)
-  })
-
-  it('Log status 200 with big body', async () => {
-    const res = await app.request('http://localhost/long')
-    expect(res).not.toBeNull()
-    expect(res.status).toBe(200)
-    expect(log.startsWith('--> GET /long \x1b[32m200\x1b[0m')).toBe(true)
-    expect(log).toMatch(/m?s$/)
-  })
-
-  it('Time in seconds', async () => {
-    const res = await app.request('http://localhost/seconds')
-    expect(res).not.toBeNull()
-    expect(res.status).toBe(200)
-    expect(log.startsWith('--> GET /seconds \x1b[32m200\x1b[0m')).toBe(true)
-    expect(log).toMatch(/1s/)
-  })
-
-  it('Log status 301 with empty body', async () => {
-    const res = await app.request('http://localhost/redirect')
-    expect(res).not.toBeNull()
-    expect(res.status).toBe(301)
-    expect(log.startsWith('--> GET /redirect \x1b[36m301\x1b[0m')).toBe(true)
-    expect(log).toMatch(/m?s$/)
-  })
-
-  it('Log status 404', async () => {
-    const msg = 'Default 404 Not Found'
-    app.all('*', (c) => {
-      return c.text(msg, 404)
-    })
-    const res = await app.request('http://localhost/notfound')
-    expect(res).not.toBeNull()
-    expect(res.status).toBe(404)
-    expect(log.startsWith('--> GET /notfound \x1b[33m404\x1b[0m')).toBe(true)
-    expect(log).toMatch(/m?s$/)
-  })
-
-  it('Log status 511 with empty body', async () => {
-    const res = await app.request('http://localhost/server-error')
-    expect(res).not.toBeNull()
-    expect(res.status).toBe(511)
-    expect(log.startsWith('--> GET /server-error \x1b[31m511\x1b[0m')).toBe(true)
-    expect(log).toMatch(/m?s$/)
-  })
-
-  it('Log status 100', async () => {
-    const res = await app.request('http://localhost/server-error?status=100')
-    expect(res).not.toBeNull()
-    expect(res.status).toBe(100)
-    expect(log.startsWith('--> GET /server-error?status=100 100')).toBe(true)
-    expect(log).toMatch(/m?s$/)
-  })
-
-  it('Log status 700', async () => {
-    const res = await app.request('http://localhost/server-error?status=700')
-    expect(res).not.toBeNull()
-    expect(res.status).toBe(700)
-    expect(log.startsWith('--> GET /server-error?status=700 700')).toBe(true)
-    expect(log).toMatch(/m?s$/)
-  })
-})
-
-describe('Logger by Middleware in NO_COLOR', () => {
-  let app: Acryl
-  let log: string
-
-  beforeEach(() => {
-    vi.stubEnv('NO_COLOR', '1')
-    function sleep(time: number) {
-      return new Promise((resolve) => setTimeout(resolve, time))
-    }
-
-    app = new Acryl()
-
-    const logFn = (str: string) => {
-      log = str
-    }
-
-    const shortRandomString = 'acryl'
-    const longRandomString = 'acryl'.repeat(1000)
-
-    app.use('*', logger(logFn))
-    app.get('/short', (c) => c.text(shortRandomString))
-    app.get('/long', (c) => c.text(longRandomString))
-    app.get('/seconds', async (c) => {
-      await sleep(1000)
-
-      return c.text(longRandomString)
-    })
-    app.get('/empty', (c) => c.text(''))
-  })
-  afterAll(() => {
-    vi.unstubAllEnvs()
-  })
-  it('Log status 200 with empty body', async () => {
-    const res = await app.request('http://localhost/empty')
-    expect(res).not.toBeNull()
-    expect(res.status).toBe(200)
-    expect(log.startsWith('--> GET /empty 200')).toBe(true)
-    expect(log).toMatch(/m?s$/)
-  })
-
-  it('Log status 200 with small body', async () => {
-    const res = await app.request('http://localhost/short')
-    expect(res).not.toBeNull()
-    expect(res.status).toBe(200)
-    expect(log.startsWith('--> GET /short 200')).toBe(true)
-    expect(log).toMatch(/m?s$/)
-  })
-
-  it('Log status 200 with big body', async () => {
-    const res = await app.request('http://localhost/long')
-    expect(res).not.toBeNull()
-    expect(res.status).toBe(200)
-    expect(log.startsWith('--> GET /long 200')).toBe(true)
-    expect(log).toMatch(/m?s$/)
-  })
-
-  it('Time in seconds', async () => {
-    const res = await app.request('http://localhost/seconds')
-    expect(res).not.toBeNull()
-    expect(res.status).toBe(200)
-    expect(log.startsWith('--> GET /seconds 200')).toBe(true)
-    expect(log).toMatch(/1s/)
-  })
-
-  it('Log status 404', async () => {
-    const msg = 'Default 404 Not Found'
-    app.all('*', (c) => {
-      return c.text(msg, 404)
-    })
-    const res = await app.request('http://localhost/notfound')
-    expect(res).not.toBeNull()
-    expect(res.status).toBe(404)
-    expect(log.startsWith('--> GET /notfound 404')).toBe(true)
-    expect(log).toMatch(/m?s$/)
-  })
-})

package/plugins/acryl-router/src/core/middleware/method-not-allowed/index.test.ts

@@ -1,292 +0,0 @@
-import { methodNotAllowed } from './index.ts'
-import { Acryl } from '../../app.ts'
-
-describe('Method Not Allowed Middleware', () => {
-  it('infers the environment from the app', () => {
-    const app = new Acryl<{ Variables: { requestId: string } }>()
-    const middleware = methodNotAllowed({
-      app,
-      onMethodNotAllowed: (c) => c.text(c.var.requestId, 405),
-    })
-
-    expect(middleware).toBeTypeOf('function')
-  })
-
-  it('returns 405 with every allowed method for an existing path', async () => {
-    const app = new Acryl()
-    app.use(methodNotAllowed({ app }))
-    app.get('/resource', (c) => c.text('GET'))
-    app.post('/resource', (c) => c.text('POST'))
-    app.delete('/resource', (c) => c.text('DELETE'))
-
-    const res = await app.request('/resource', { method: 'PUT' })
-
-    expect(res.status).toBe(405)
-    expect(res.headers.get('Allow')).toBe('GET, HEAD, POST, DELETE')
-    expect(await res.text()).toBe('Method Not Allowed')
-
-    const resUsingCachedRouter = await app.request('/resource', { method: 'OPTIONS' })
-    expect(resUsingCachedRouter.status).toBe(405)
-    expect(resUsingCachedRouter.headers.get('Allow')).toBe('GET, HEAD, POST, DELETE')
-  })
-
-  it('does not affect an allowed method', async () => {
-    const app = new Acryl()
-    app.use(methodNotAllowed({ app }))
-    app.get('/resource', (c) => c.text('GET'))
-
-    const res = await app.request('/resource')
-
-    expect(res.status).toBe(200)
-    expect(res.headers.has('Allow')).toBe(false)
-    expect(await res.text()).toBe('GET')
-  })
-
-  it('returns 404 when the path does not exist', async () => {
-    const app = new Acryl()
-    app.use(methodNotAllowed({ app }))
-    app.get('/resource', (c) => c.text('GET'))
-
-    const res = await app.request('/missing', { method: 'POST' })
-
-    expect(res.status).toBe(404)
-    expect(res.headers.has('Allow')).toBe(false)
-  })
-
-  it('matches parameterized and overlapping routes without duplicate methods', async () => {
-    const app = new Acryl()
-    app.use(methodNotAllowed({ app }))
-    app.get('/users/:id', (_c, next) => next())
-    app.get('/users/me', (c) => c.text('me'))
-    app.patch('/users/:id', (c) => c.text(c.req.param('id')))
-
-    const res = await app.request('/users/me', { method: 'POST' })
-
-    expect(res.status).toBe(405)
-    expect(res.headers.get('Allow')).toBe('GET, HEAD, PATCH')
-  })
-
-  it('preserves an intentional 404 from a handler registered for the request method', async () => {
-    const app = new Acryl()
-    app.use(methodNotAllowed({ app }))
-    app.get('/users/:id', (c) => c.notFound())
-    app.post('/users/:id', (c) => c.text('Created'))
-
-    const res = await app.request('/users/123')
-
-    expect(res.status).toBe(404)
-    expect(res.headers.has('Allow')).toBe(false)
-  })
-
-  it('preserves an intentional 404 for HEAD when a GET route is registered', async () => {
-    const app = new Acryl()
-    app.use(methodNotAllowed({ app }))
-    app.get('/users/:id', (c) => c.notFound())
-    app.post('/users/:id', (c) => c.text('Created'))
-
-    const res = await app.request('/users/123', { method: 'HEAD' })
-
-    expect(res.status).toBe(404)
-    expect(res.headers.has('Allow')).toBe(false)
-  })
-
-  it('preserves a 404 returned by the error handler', async () => {
-    const app = new Acryl()
-    app.onError((_error, c) => c.text('Handled error', 404))
-    app.use(methodNotAllowed({ app }))
-    app.use('/resource', () => {
-      throw new Error('boom')
-    })
-    app.get('/resource', (c) => c.text('GET'))
-
-    const res = await app.request('/resource', { method: 'POST' })
-
-    expect(res.status).toBe(404)
-    expect(res.headers.has('Allow')).toBe(false)
-    expect(await res.text()).toBe('Handled error')
-  })
-
-  it('returns 405 for HEAD when the path does not support GET', async () => {
-    const app = new Acryl()
-    app.use(methodNotAllowed({ app }))
-    app.post('/resource', (c) => c.text('POST'))
-
-    const res = await app.request('/resource', { method: 'HEAD' })
-
-    expect(res.status).toBe(405)
-    expect(res.headers.get('Allow')).toBe('POST')
-    expect(await res.text()).toBe('')
-  })
-
-  it('does not advertise explicit HEAD routes that Acryl cannot dispatch', async () => {
-    const app = new Acryl()
-    app.use(methodNotAllowed({ app }))
-    app.on('HEAD', '/resource', (c) => c.text('HEAD'))
-
-    const headRes = await app.request('/resource', { method: 'HEAD' })
-    expect(headRes.status).toBe(404)
-    expect(headRes.headers.has('Allow')).toBe(false)
-
-    const putRes = await app.request('/resource', { method: 'PUT' })
-    expect(putRes.status).toBe(404)
-    expect(putRes.headers.has('Allow')).toBe(false)
-  })
-
-  it('supports custom methods', async () => {
-    const app = new Acryl()
-    app.use(methodNotAllowed({ app }))
-    app.on('PURGE', '/cache', (c) => c.text('Purged'))
-
-    const res = await app.request('/cache', { method: 'POST' })
-
-    expect(res.status).toBe(405)
-    expect(res.headers.get('Allow')).toBe('PURGE')
-  })
-
-  it('compares request methods case-sensitively', async () => {
-    const app = new Acryl()
-    app.use(methodNotAllowed({ app }))
-    app.patch('/item', (c) => c.text('PATCH'))
-
-    const request = new Request('http://localhost/item', { method: 'PATCH' })
-    Object.defineProperty(request, 'method', { value: 'patch' })
-    const res = await app.request(request)
-
-    expect(res.status).toBe(405)
-    expect(res.headers.get('Allow')).toBe('PATCH')
-  })
-
-  it('works with routes mounted using app.route()', async () => {
-    const api = new Acryl()
-    api.get('/resource', (c) => c.text('GET'))
-    api.post('/resource', (c) => c.text('POST'))
-
-    const app = new Acryl()
-    app.use(methodNotAllowed({ app }))
-    app.route('/api', api)
-
-    const res = await app.request('/api/resource', { method: 'DELETE' })
-
-    expect(res.status).toBe(405)
-    expect(res.headers.get('Allow')).toBe('GET, HEAD, POST')
-  })
-
-  it('works when installed in an app mounted using app.route()', async () => {
-    const api = new Acryl().basePath('/v1')
-    api.use(methodNotAllowed({ app: api }))
-    api.get('/resource', (c) => c.text('GET'))
-    api.post('/resource', (c) => c.text('POST'))
-
-    const app = new Acryl()
-    app.route('/api', api)
-
-    const res = await app.request('/api/v1/resource', { method: 'DELETE' })
-
-    expect(res.status).toBe(405)
-    expect(res.headers.get('Allow')).toBe('GET, HEAD, POST')
-  })
-
-  it('ignores ALL routes when collecting allowed methods', async () => {
-    const app = new Acryl()
-    app.use(methodNotAllowed({ app }))
-    app.all('/resource', (_c, next) => next())
-
-    const res = await app.request('/resource', { method: 'POST' })
-
-    expect(res.status).toBe(404)
-    expect(res.headers.has('Allow')).toBe(false)
-  })
-
-  it('does not invoke the error handler for a 405 response', async () => {
-    const app = new Acryl()
-    const onError = vi.fn(() => new Response('error', { status: 500 }))
-    app.onError(onError)
-    app.use(methodNotAllowed({ app }))
-    app.get('/resource', (c) => c.text('GET'))
-
-    const res = await app.request('/resource', { method: 'POST' })
-
-    expect(res.status).toBe(405)
-    expect(onError).not.toHaveBeenCalled()
-  })
-
-  it('preserves downstream headers and lets outer middleware observe the final status', async () => {
-    const app = new Acryl()
-    let observedStatus: number | undefined
-
-    app.use(async (c, next) => {
-      await next()
-      observedStatus = c.res.status
-    })
-    app.use(methodNotAllowed({ app }))
-    app.use(async (c, next) => {
-      await next()
-      c.header('X-Downstream', 'true')
-    })
-    app.get('/resource', (c) => c.text('GET'))
-
-    const res = await app.request('/resource', { method: 'POST' })
-
-    expect(res.status).toBe(405)
-    expect(res.headers.get('X-Downstream')).toBe('true')
-    expect(observedStatus).toBe(405)
-  })
-
-  it('replaces stale representation headers from the not-found response', async () => {
-    const app = new Acryl()
-    app.use(methodNotAllowed({ app }))
-    app.get('/resource', (c) => c.text('GET'))
-    app.post('/resource', (c) => c.text('POST'))
-    app.notFound((c) =>
-      c.text('Missing', 404, {
-        Allow: 'BOGUS',
-        'Content-Length': '7',
-      })
-    )
-
-    const res = await app.request('/resource', { method: 'PUT' })
-
-    expect(res.status).toBe(405)
-    expect(res.headers.get('Allow')).toBe('GET, HEAD, POST')
-    expect(res.headers.has('Content-Length')).toBe(false)
-    expect(await res.text()).toBe('Method Not Allowed')
-  })
-
-  it('supports a custom method-not-allowed response', async () => {
-    const app = new Acryl()
-    app.use(
-      methodNotAllowed({
-        app,
-        onMethodNotAllowed: (c, allowedMethods) =>
-          c.json({ error: 'Method Not Allowed', allowedMethods }, 405, {
-            Allow: allowedMethods.join(', '),
-          }),
-      })
-    )
-    app.get('/resource', (c) => c.text('GET'))
-    app.post('/resource', (c) => c.text('POST'))
-
-    const res = await app.request('/resource', { method: 'PUT' })
-
-    expect(res.status).toBe(405)
-    expect(res.headers.get('Allow')).toBe('GET, HEAD, POST')
-    expect(res.headers.get('Content-Type')).toMatch(/^application\/json/)
-    expect(await res.json()).toEqual({
-      error: 'Method Not Allowed',
-      allowedMethods: ['GET', 'HEAD', 'POST'],
-    })
-  })
-
-  it('leaves a non-404 custom not-found response unchanged', async () => {
-    const app = new Acryl()
-    app.use(methodNotAllowed({ app }))
-    app.get('/resource', (c) => c.text('GET'))
-    app.notFound((c) => c.text('Missing', 410))
-
-    const res = await app.request('/resource', { method: 'POST' })
-
-    expect(res.status).toBe(410)
-    expect(res.headers.has('Allow')).toBe(false)
-    expect(await res.text()).toBe('Missing')
-  })
-})