fix: merge src and tests (#283)

<!-- This is an auto-generated comment: release notes by coderabbit.ai
-->
## Summary by CodeRabbit

* **Tests**
  * Enforced 100% statement coverage and added coverage reporters.
* Enabled automatic mock cleanup and restoration for more reliable
tests.
* Consolidated and relocated many tests to a narrower discovery scope;
added new middleware tests and removed legacy duplicates.

* **Chores**
* Narrowed TypeScript test compilation to exclude runtime test files and
streamlined test-related config for faster, more isolated runs.
<!-- end of auto-generated comment: release notes by coderabbit.ai -->

Author: csikb

tests/config.defaults.test.ts src/config.defaults.test.tstests/config.defaults.test.ts renamed to src/config.defaults.test.ts

@@ -1,6 +1,6 @@
 import dotenv from 'dotenv'
 import { describe, expect, it, vi } from 'vitest'
-import type { Config } from '../src/config.js'
+import type { Config } from './config.js'
 
 vi.mock('dotenv', () => ({
   default: {

tests/config.error.test.ts src/config.error.test.tstests/config.error.test.ts renamed to src/config.error.test.ts

@@ -1,6 +1,6 @@
 import dotenv from 'dotenv'
 import { describe, expect, it, vi } from 'vitest'
-import type { Config } from '../src/config.js'
+import type { Config } from './config.js'
 
 vi.mock('dotenv', () => ({
   default: {

tests/config.overwrite.test.ts src/config.overwrite.test.tstests/config.overwrite.test.ts renamed to src/config.overwrite.test.ts

@@ -1,7 +1,7 @@
 import { describe, expect, it, vi } from 'vitest'
-import postgres from '../../src/database/postgres.js'
+import postgres from './postgres.js'
 
-vi.mock('../../src/database/postgres.js', () => ({
+vi.mock('./postgres.js', () => ({
   default: vi.fn(),
 }))
 
@@ -11,7 +11,7 @@ describe('index', () => {
   it('should export databases', async () => {
     expect.assertions(2)
 
-    const actual = await import('../../src/database/index.js')
+    const actual = await import('./index.js')
 
     const expected = { postgres: mockPostgres }
     expect.soft(actual).toMatchObject(expected)

tests/database/index.test.ts src/database/index.test.tstests/database/index.test.ts renamed to src/database/index.test.ts

@@ -1,6 +1,6 @@
 import pg from 'pg'
 import { describe, expect, it, vi } from 'vitest'
-import config from '../../src/config.js'
+import config from '../config.js'
 
 vi.mock('pg', () => {
   return {
@@ -9,7 +9,7 @@ vi.mock('pg', () => {
     },
   }
 })
-vi.mock('../../src/config', () => ({
+vi.mock('../config', () => ({
   default: {
     database: { connectionString: 'connectionString' },
   },
@@ -22,7 +22,7 @@ describe('postgres', () => {
   it('should be tested', async () => {
     expect.assertions(2)
 
-    const { default: actual } = await import('../../src/database/postgres.js')
+    const { default: actual } = await import('./postgres.js')
 
     expect.soft(actual).toStrictEqual(new pg.Pool(mockConfig.database))
     expect.soft(mockPg.Pool).toHaveBeenCalledWith(mockConfig.database)

tests/database/postgres.test.ts src/database/postgres.test.tstests/database/postgres.test.ts renamed to src/database/postgres.test.ts

@@ -1,13 +1,13 @@
 import koa from 'koa'
-import { describe, expect, it, vi } from 'vitest'
-import config from '../src/config.js'
+import { beforeEach, describe, expect, it, vi } from 'vitest'
+import config from './config.js'
 import {
   bodyParser,
   compress,
   helmet,
   postGraphile,
-} from '../src/middleware/index.js'
-import { healthRouter } from '../src/router/index.js'
+} from './middleware/index.js'
+import { healthRouter } from './router/index.js'
 
 vi.mock('koa', () => ({
   default: vi.fn(
@@ -19,30 +19,33 @@ vi.mock('koa', () => ({
     },
   ),
 }))
-vi.mock('../src/config.js', () => ({
+vi.mock('./config.js', () => ({
   default: {
     port: 1234,
   },
 }))
-vi.mock('../src/middleware/index.js', () => ({
+vi.mock('./middleware/index.js', () => ({
   bodyParser: vi.fn().mockName('bodyParser'),
   compress: vi.fn().mockName('compress'),
   helmet: vi.fn().mockName('helmet'),
   postGraphile: vi.fn().mockName('postGraphile'),
 }))
-vi.mock('../src/router/index.js', () => ({
+vi.mock('./router/index.js', () => ({
   healthRouter: {
     routes: vi.fn().mockName('healthRouter.routes'),
     allowedMethods: vi.fn().mockName('healthRouter.allowedMethods'),
   },
 }))
-vi.spyOn(globalThis.console, 'log').mockImplementation(() => {})
 
 describe('index', () => {
+  beforeEach(() => {
+    vi.spyOn(globalThis.console, 'log').mockImplementation(() => {})
+  })
+
   it('should be tested', async () => {
     expect.assertions(12)
 
-    await import('../src/index.js')
+    await import('./index.js')
 
     expect.soft(vi.mocked(koa)).toHaveBeenCalledTimes(1)
     const mockKoaInstance = vi.mocked(koa).mock.results[0]?.value

tests/index.test.ts src/index.test.tstests/index.test.ts renamed to src/index.test.ts

@@ -8,9 +8,7 @@ describe('bodyparser', () => {
   it('should export bodyparser', async () => {
     expect.assertions(1)
 
-    const { default: actual } = await import(
-      '../../src/middleware/bodyparser.js'
-    )
+    const { default: actual } = await import('./bodyparser.js')
 
     expect(actual).toBe(mockBodyParser())
   })

tests/middleware/bodyparser.test.ts src/middleware/bodyparser.test.tstests/middleware/bodyparser.test.ts renamed to src/middleware/bodyparser.test.ts

@@ -0,0 +1,16 @@
+import compress from 'koa-compress'
+import { describe, expect, it, vi } from 'vitest'
+
+vi.mock('koa-compress')
+
+describe('compress', () => {
+  it('should export compress', async () => {
+    expect.assertions(3)
+
+    const { default: actual } = await import('./compress.js')
+
+    expect.soft(compress).toHaveBeenCalledOnce()
+    expect.soft(compress).toHaveBeenCalledWith({ threshold: 0 })
+    expect.soft(actual).toBe(compress({ threshold: 0 }))
+  })
+})

src/middleware/compress.test.ts

@@ -0,0 +1,15 @@
+import helmet from 'koa-helmet'
+import { describe, expect, it, vi } from 'vitest'
+
+vi.mock('koa-helmet')
+
+describe('helmet', () => {
+  it('should export helmet', async () => {
+    expect.assertions(2)
+
+    const { default: actual } = await import('./helmet.js')
+
+    expect(helmet.default).toHaveBeenCalledOnce()
+    expect(actual).toStrictEqual(helmet.default())
+  })
+})

src/middleware/helmet.test.ts

@@ -0,0 +1,35 @@
+import { describe, expect, it, vi } from 'vitest'
+import bodyParser from './bodyparser.js'
+import compress from './compress.js'
+import helmet from './helmet.js'
+import postGraphile from './postgraphile.js'
+
+vi.mock('./bodyparser', () => ({
+  default: vi.fn().mockName('bodyparser'),
+}))
+vi.mock('./compress', () => ({
+  default: vi.fn().mockName('compress'),
+}))
+vi.mock('./helmet', () => ({
+  default: vi.fn().mockName('helmet'),
+}))
+vi.mock('./postgraphile', () => ({
+  default: vi.fn().mockName('postgraphile'),
+}))
+
+describe('index', () => {
+  it('should export middlewares', async () => {
+    expect.assertions(2)
+
+    const actual = await import('./index.js')
+
+    const expected = {
+      bodyParser,
+      compress,
+      helmet,
+      postGraphile,
+    }
+    expect.soft(actual).toMatchObject(expected)
+    expect.soft(Object.keys(actual)).toEqual(Object.keys(expected))
+  })
+})