chore: add a test for ignored option and switch to vitest

Author: v1rtl

.github/workflows/ci.yml

@@ -10,12 +10,11 @@ jobs:
           bun-version: latest
       - run: bun install
       - run: bun test:coverage
-      - run: bun test:report
       - name: Coveralls
         uses: coverallsapp/github-action@master
         with:
           github-token: ${{ secrets.GITHUB_TOKEN }}
-          path-to-lcov: ./lcov.info
+          path-to-lcov: ./coverage/lcov.info
   build:
     runs-on: ubuntu-latest
     steps:

bun.lock

@@ -34,9 +34,8 @@
   },
   "scripts": {
     "build": "tsc",
-    "test": "tsx tests/index.test.ts",
-    "test:coverage": "c8 tsx --test tests/index.test.ts",
-    "test:report": "c8 report --reporter=text-lcov > lcov.info",
+    "test": "vitest run --dir tests",
+    "test:coverage": "vitest run --dir tests --coverage",
     "check": "biome check",
     "check:fix": "biome check --write",
     "prepare": "husky"
@@ -47,11 +46,11 @@
     "@commitlint/config-conventional": "^19.7.1",
     "@tinyhttp/app": "^2.5.2",
     "@types/node": "^20.17.22",
-    "c8": "^10.1.3",
-    "expect": "^29.7.0",
+    "@vitest/coverage-v8": "^3.0.7",
+    "@vitest/spy": "^3.0.7",
     "husky": "^9.1.7",
     "supertest-fetch": "^2.0.0",
-    "tsx": "^4.19.3",
-    "typescript": "^5.8.2"
+    "typescript": "^5.8.2",
+    "vitest": "^3.0.7"
   }
 }

package.json

@@ -73,7 +73,6 @@ export const logger = (options: LoggerOptions = {}) => {
   return (req: Request, res: Response, next?: () => void) => {
     res.on('finish', () => {
       const args: (string | number)[] = []
-
       if (methods.includes(req.method) && !ignore.some((url) => req.url.startsWith(url))) {
         const s = res.statusCode.toString()
         let stringToLog = ''

src/index.ts

@@ -2,11 +2,11 @@ import * as assert from 'node:assert/strict'
 import { readFileSync } from 'node:fs'
 import { access, rm } from 'node:fs/promises'
 import type { AddressInfo } from 'node:net'
-import { describe, it, mock } from 'node:test'
 import { App } from '@tinyhttp/app'
+import * as mock from '@vitest/spy'
 import { bold, cyan, magenta, red } from 'colorette'
-import { expect } from 'expect'
 import { makeFetch } from 'supertest-fetch'
+import { describe, expect, it, vi } from 'vitest'
 import { LogLevel, logger } from '../src/index.ts'
 
 async function checkFileExists(file: string) {
@@ -99,35 +99,37 @@ it('should ignore route paths listed in the "ignore" option', async () => {
 
   const logs: string[] = []
 
-  const callback = mock.fn((path: string) => {
+  const callback = vi.fn((path: string) => {
     logs.push(path)
   })
 
-  app.use(
-    logger({
-      output: {
-        color: false,
-        level: undefined,
-        callback
-      },
-      ignore: ['/ignore']
-    })
-  )
+  app.use(logger({ output: { color: false, level: undefined, callback }, ignore: ['/ignore'] }))
 
   const server = app.listen()
 
-  const address = (server.address() as AddressInfo).port
-
-  const baseUrl = `http://localhost:${address}`
-
-  await fetch(new URL('/ignor', baseUrl))
-  await fetch(new URL('/ignore', baseUrl))
-  await fetch(new URL('/ignore/path', baseUrl))
-  await fetch(new URL('/ignore?query=string', baseUrl))
-  await fetch(new URL('/path/ignore', baseUrl))
+  const fetch = makeFetch(server)
 
-  expect(callback.mock.callCount()).toEqual(2)
-  expect(logs).toEqual(['GET 404 Not Found /ignor', 'GET 404 Not Found /path/ignore'])
+  fetch('/ignor')
+    .expect(404)
+    .then(() => {
+      fetch('/ignore')
+        .expect(404)
+        .then(() => {
+          fetch('/path/ignore')
+            .expect(404)
+            .then(() => {
+              fetch('/ignore?query=string')
+                .expect(404)
+                .then(() => {
+                  expect(callback).toHaveBeenCalledTimes(2)
+                  expect(logs).toEqual(['GET 404 Not Found /ignor', 'GET 404 Not Found /path/ignore'])
+
+                  vi.clearAllMocks()
+                  server.close()
+                })
+            })
+        })
+    })
 })
 
 describe('Log file tests', () => {

tests/index.test.ts

@@ -0,0 +1,11 @@
+import { defineConfig } from 'vitest/config'
+
+export default defineConfig({
+  test: {
+    coverage: {
+      provider: 'v8',
+      include: ['src/**/*.ts'],
+      reporter: ['lcov', 'text']
+    }
+  }
+})