@etus/bhono-app 0.1.5 → 0.1.6

package/templates/base/tests/integration/vitest.config.ts

@@ -10,6 +10,7 @@ export default defineConfig({
     alias: {
       '@server': `${projectRoot}/src/server`,
       '@shared': `${projectRoot}/src/shared`,
+      '@tests': `${projectRoot}/tests`,
     },
   },
   test: {
@@ -49,17 +50,10 @@ export default defineConfig({
         // Database setup (not business logic)
         'src/server/db/client.ts',
         'src/server/db/seed.ts',
+        'src/server/db/records.ts',
         // Barrel exports (just re-exports, no logic)
-        'src/server/db/schema/index.ts',
         'src/server/services/index.ts',
         'src/server/middleware/index.ts',
-        // Drizzle schema definitions (table structures, not business logic)
-        'src/server/db/schema/users.ts',
-        'src/server/db/schema/accounts.ts',
-        'src/server/db/schema/user-accounts.ts',
-        'src/server/db/schema/audit-logs.ts',
-        'src/server/db/schema/refresh-tokens.ts',
-        'src/server/db/schema/invitations.ts',
         // API documentation (not testable)
         'src/server/routes/api.ts',
         // Dev-only endpoint (tested via E2E)
@@ -67,7 +61,6 @@ export default defineConfig({
         // Hard to test in isolation (require external services/mocking)
         'src/server/lib/providers.ts',
         'src/server/lib/transaction.ts',
-        'src/server/lib/schema-helpers.ts',
       ],
       thresholds: {
         statements: 90,

package/templates/base/{src/server/__tests__ → tests}/mocks/db.ts

@@ -1,4 +1,4 @@
-// src/server/__tests__/mocks/db.ts
+// tests/mocks/db.ts
 import { vi } from 'vitest'
 
 /**

package/templates/base/{src/server/__tests__ → tests}/mocks/index.ts

@@ -1,4 +1,4 @@
-// src/server/__tests__/mocks/index.ts
+// tests/mocks/index.ts
 
 // D1 Database mock
 export {

package/templates/base/{src/server/__tests__ → tests}/mocks/kv.ts

@@ -1,4 +1,4 @@
-// src/server/__tests__/mocks/kv.ts
+// tests/mocks/kv.ts
 import { vi } from 'vitest'
 
 /**

package/templates/base/{src/server/__tests__ → tests}/mocks/r2.ts

@@ -1,4 +1,4 @@
-// src/server/__tests__/mocks/r2.ts
+// tests/mocks/r2.ts
 import { vi } from 'vitest'
 
 /**

package/templates/base/{src/client/components/__tests__ → tests/unit/client/components}/sidebar.test.tsx

@@ -1,6 +1,6 @@
 import type { Mock } from "vitest";
 import { describe, it, expect, vi, beforeEach } from "vitest"
-import { render, screen } from "../../__tests__/test-utils"
+import { render, screen } from "@tests/helpers/client-test-utils"
 import userEvent from "@testing-library/user-event"
 import { Sidebar } from "../sidebar"
 

package/templates/base/{src/client/components/ui/__tests__ → tests/unit/client/components/ui}/avatar.test.tsx

@@ -1,5 +1,5 @@
 import { describe, it, expect, vi } from "vitest"
-import { render, screen, fireEvent, waitFor } from "../../../__tests__/test-utils"
+import { render, screen, fireEvent, waitFor } from "@tests/helpers/client-test-utils"
 import { createRef } from "react"
 import { Avatar, AvatarImage, AvatarFallback } from "../avatar"
 

package/templates/base/{src/client/__tests__ → tests/unit/client/components/ui}/button.test.tsx

@@ -1,5 +1,5 @@
 import { describe, it, expect, vi } from "vitest"
-import { render, screen } from "./test-utils"
+import { render, screen } from "@tests/helpers/client-test-utils"
 import userEvent from "@testing-library/user-event"
 import { Button } from "@/components/ui/button"
 

package/templates/base/{src/client/components/ui/__tests__ → tests/unit/client/components/ui}/card.test.tsx

@@ -1,5 +1,5 @@
 import { describe, it, expect } from "vitest"
-import { render, screen } from "../../../__tests__/test-utils"
+import { render, screen } from "@tests/helpers/client-test-utils"
 import { createRef } from "react"
 import {
   Card,

package/templates/base/{src/client/components/ui/__tests__ → tests/unit/client/components/ui}/dialog.test.tsx

@@ -1,5 +1,5 @@
 import { describe, it, expect, vi } from "vitest"
-import { render, screen, waitFor } from "../../../__tests__/test-utils"
+import { render, screen, waitFor } from "@tests/helpers/client-test-utils"
 import userEvent from "@testing-library/user-event"
 import {
   Dialog,

package/templates/base/{src/client/components/ui/__tests__ → tests/unit/client/components/ui}/input.test.tsx

@@ -1,5 +1,5 @@
 import { describe, it, expect, vi } from "vitest"
-import { render, screen } from "../../../__tests__/test-utils"
+import { render, screen } from "@tests/helpers/client-test-utils"
 import userEvent from "@testing-library/user-event"
 import { createRef } from "react"
 import { Input } from "../input"

package/templates/base/{src/client/components/ui/__tests__ → tests/unit/client/components/ui}/loading-skeleton.test.tsx

@@ -1,5 +1,5 @@
 import { describe, it, expect } from "vitest"
-import { render, screen } from "../../../__tests__/test-utils"
+import { render, screen } from "@tests/helpers/client-test-utils"
 import {
   DashboardSkeleton,
   TeamSkeleton,

package/templates/base/{src/client/components/ui/__tests__ → tests/unit/client/components/ui}/skeleton.test.tsx

@@ -1,5 +1,5 @@
 import { describe, it, expect } from "vitest"
-import { render, screen } from "../../../__tests__/test-utils"
+import { render, screen } from "@tests/helpers/client-test-utils"
 import { createRef } from "react"
 import { Skeleton } from "../skeleton"
 

package/templates/base/{src/client/components/ui/__tests__ → tests/unit/client/components/ui}/sonner.test.tsx

@@ -1,5 +1,5 @@
 import { describe, it, expect } from "vitest"
-import { render, screen } from "../../../__tests__/test-utils"
+import { render, screen } from "@tests/helpers/client-test-utils"
 import { Toaster } from "../sonner"
 
 describe("Toaster", () => {

package/templates/base/{src/client/components/ui/__tests__ → tests/unit/client/components/ui}/tabs.test.tsx

@@ -1,5 +1,5 @@
 import { describe, it, expect, vi } from "vitest"
-import { render, screen } from "../../../__tests__/test-utils"
+import { render, screen } from "@tests/helpers/client-test-utils"
 import userEvent from "@testing-library/user-event"
 import { Tabs, TabsList, TabsTrigger, TabsContent } from "../tabs"
 

package/templates/base/{src/client/routes/_authenticated/__tests__ → tests/unit/client/routes/_authenticated}/account.test.tsx

@@ -1,7 +1,7 @@
 import { describe, it, expect, vi, beforeEach } from 'vitest'
 import { screen, waitFor, fireEvent } from '@testing-library/react'
 import userEvent from '@testing-library/user-event'
-import { renderRoute } from '../../../__tests__/test-utils'
+import { renderRoute } from '@tests/helpers/client-test-utils'
 
 // Mock user for authenticated tests
 const mockUser = {

package/templates/base/{src/client/routes/_authenticated/__tests__ → tests/unit/client/routes/_authenticated}/integrations.test.tsx

@@ -1,7 +1,7 @@
 import { describe, it, expect, vi, beforeEach } from 'vitest'
 import { screen, waitFor } from '@testing-library/react'
 import userEvent from '@testing-library/user-event'
-import { renderRoute } from '../../../__tests__/test-utils'
+import { renderRoute } from '@tests/helpers/client-test-utils'
 
 // Mock user for authenticated tests
 const mockUser = {

package/templates/base/{src/client/routes/_authenticated/__tests__ → tests/unit/client/routes/_authenticated}/settings.test.tsx

@@ -1,7 +1,7 @@
 import { describe, it, expect, vi, beforeEach } from 'vitest'
 import { screen, waitFor } from '@testing-library/react'
 import userEvent from '@testing-library/user-event'
-import { renderRoute } from '../../../__tests__/test-utils'
+import { renderRoute } from '@tests/helpers/client-test-utils'
 
 // Mock user for authenticated tests
 const mockUser = {

package/templates/base/{src/client/routes/_authenticated/__tests__ → tests/unit/client/routes/_authenticated}/team.test.tsx

@@ -1,7 +1,7 @@
 import { describe, it, expect, vi, beforeEach } from 'vitest'
 import { screen, waitFor } from '@testing-library/react'
 import userEvent from '@testing-library/user-event'
-import { renderRoute } from '../../../__tests__/test-utils'
+import { renderRoute } from '@tests/helpers/client-test-utils'
 
 // Mock user for authenticated tests
 const mockUser = {

package/templates/base/{src/client/__tests__ → tests/unit/client}/routes/authenticated-layout.test.tsx

@@ -1,6 +1,6 @@
 import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest'
 import { screen, waitFor, act } from '@testing-library/react'
-import { renderRoute } from '../test-utils'
+import { renderRoute } from '@tests/helpers/client-test-utils'
 
 // Mock user for authenticated tests
 const mockUser = {

package/templates/base/{src/client/__tests__ → tests/unit/client}/routes/dashboard.test.tsx

@@ -1,6 +1,6 @@
 import { describe, it, expect, vi, beforeEach } from 'vitest'
 import { screen, waitFor } from '@testing-library/react'
-import { renderRoute } from '../test-utils'
+import { renderRoute } from '@tests/helpers/client-test-utils'
 
 // Mock user for authenticated tests
 const mockUser = {

package/templates/base/{src/client/routes/__tests__ → tests/unit/client/routes}/invite.test.tsx

@@ -1,7 +1,7 @@
 import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest'
 import { screen, waitFor } from '@testing-library/react'
 import userEvent from '@testing-library/user-event'
-import { renderRoute } from '../../__tests__/test-utils'
+import { renderRoute } from '@tests/helpers/client-test-utils'
 
 describe('Invite Token Page', () => {
   beforeEach(() => {

package/templates/base/{src/client/__tests__ → tests/unit/client}/routes/login.test.tsx

@@ -1,7 +1,7 @@
 import { describe, it, expect, vi, beforeEach } from 'vitest'
 import { screen, waitFor } from '@testing-library/react'
 import userEvent from '@testing-library/user-event'
-import { renderRoute } from '../test-utils'
+import { renderRoute } from '@tests/helpers/client-test-utils'
 
 describe('Login Page', () => {
   beforeEach(() => {

package/templates/base/{src/client/__tests__ → tests/unit/client}/routes/navigation.test.tsx

@@ -1,7 +1,7 @@
 import { describe, it, expect, vi, beforeEach } from 'vitest'
 import { screen, waitFor } from '@testing-library/react'
 import userEvent from '@testing-library/user-event'
-import { renderRoute } from '../test-utils'
+import { renderRoute } from '@tests/helpers/client-test-utils'
 
 // Mock user for authenticated tests
 const mockUser = {

package/templates/base/{src/client/__tests__ → tests/unit/client}/routes/root-layout.test.tsx

@@ -1,6 +1,6 @@
 import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest'
 import { screen, waitFor } from '@testing-library/react'
-import { renderRoute } from '../test-utils'
+import { renderRoute } from '@tests/helpers/client-test-utils'
 
 describe('Root Layout', () => {
   beforeEach(() => {

package/templates/base/{src/server/auth/__tests__ → tests/unit/server/auth}/guards.test.ts

@@ -1,7 +1,7 @@
 import { describe, it, expect } from 'vitest'
 import { Hono } from 'hono'
-import { requireRole, requirePermission } from '../guards'
-import type { HonoEnv } from '../../types'
+import { requireRole, requirePermission } from '@server/guards'
+import type { HonoEnv } from '@server/types'
 
 describe('guards', () => {
   describe('requireRole', () => {

package/templates/base/{src → tests/unit}/server/auth/permissions.test.ts

@@ -1,6 +1,6 @@
 // src/auth/permissions.test.ts
 import { describe, it, expect } from 'vitest'
-import { Permission, hasPermission, hasAnyPermission, hasAllPermissions } from './permissions'
+import { Permission, hasPermission, hasAnyPermission, hasAllPermissions } from '@server/auth/permissions'
 
 describe('permissions', () => {
   describe('hasPermission', () => {

package/templates/base/{src → tests/unit}/server/auth/roles.test.ts

@@ -9,7 +9,7 @@ import {
   getAllRoles,
   compareRoles,
   isRoleHigherThan,
-} from './roles'
+} from '@server/auth/roles'
 
 describe('roles', () => {
   describe('hasMinimumRole', () => {

package/templates/base/tests/unit/server/db/sql.test.ts

@@ -0,0 +1,68 @@
+import { describe, it, expect } from 'vitest'
+import { createMockEnv, setMockQueryResult } from '@tests/helpers/server'
+import { queryAll, queryOne, queryValue, execute } from '@server/db/sql'
+
+describe('sql helper', () => {
+  it('returns all rows with optional mapper', async () => {
+    const env = createMockEnv()
+    const mockDb = env.DB._mock
+
+    setMockQueryResult(mockDb, [1], [
+      { id: '1', name: 'Alice' },
+      { id: '2', name: 'Bob' },
+    ])
+
+    const rows = await queryAll(env.DB, 'SELECT id, name FROM users WHERE id = ?', [1])
+    expect(rows).toEqual([
+      { id: '1', name: 'Alice' },
+      { id: '2', name: 'Bob' },
+    ])
+
+    const mapped = await queryAll(env.DB, 'SELECT id FROM users WHERE id = ?', [1], (row) => ({
+      id: String(row.id),
+    }))
+    expect(mapped).toEqual([{ id: '1' }, { id: '2' }])
+  })
+
+  it('returns one row or null', async () => {
+    const env = createMockEnv()
+    const mockDb = env.DB._mock
+
+    setMockQueryResult(mockDb, [], [{ ok: 1 }])
+
+    const row = await queryOne(env.DB, 'SELECT 1 AS ok')
+    expect(row).toEqual({ ok: 1 })
+
+    const empty = await queryOne(env.DB, 'SELECT 1 AS ok', [999])
+    expect(empty).toBeNull()
+  })
+
+  it('returns a single value', async () => {
+    const env = createMockEnv()
+    const mockDb = env.DB._mock
+
+    setMockQueryResult(mockDb, [], [{ count: 42 }])
+
+    const value = await queryValue<number>(env.DB, 'SELECT count(*) AS count FROM users')
+    expect(value).toBe(42)
+  })
+
+  it('normalizes params (boolean/date/undefined)', async () => {
+    const env = createMockEnv()
+    const mockDb = env.DB._mock
+    const date = new Date('2025-01-01T00:00:00.000Z')
+
+    setMockQueryResult(mockDb, [1, date.toISOString(), null], [{ ok: 1 }])
+
+    const row = await queryOne(env.DB, 'SELECT 1 AS ok WHERE a = ? AND b = ? AND c IS ?', [true, date, undefined])
+    expect(row).toEqual({ ok: 1 })
+  })
+
+  it('executes statements and returns meta', async () => {
+    const env = createMockEnv()
+
+    const result = await execute(env.DB, 'UPDATE users SET name = ? WHERE id = ?', ['Alice', '1'])
+    expect(result).toHaveProperty('meta')
+    expect(result.meta.changes).toBe(1)
+  })
+})

package/templates/base/{src → tests/unit}/server/env.test.ts

@@ -1,5 +1,5 @@
 import { describe, it, expect } from 'vitest'
-import { getEnv, type Env } from './env'
+import { getEnv, type Env } from '@server/env'
 
 // Note: The old process.env-based tests were removed as the env module
 // now uses Cloudflare Workers bindings (passed to getEnv function)

package/templates/base/tests/unit/server/lib/audited-db.test.ts

@@ -0,0 +1,78 @@
+// src/server/lib/audited-db.test.ts
+import { describe, it, expect, vi, beforeEach } from 'vitest'
+import { auditedInsert, auditedUpdate, auditedDelete } from '@server/lib/audited-db'
+import type { ServiceContext } from '@server/types'
+import { createMockEnv, setMockQueryResult } from '@tests/helpers/server'
+
+const mockCtx: ServiceContext = {
+  accountId: 'account-123',
+  user: {
+    id: 'user-123',
+    email: 'test@example.com',
+    name: 'Test User',
+    status: 'active',
+    providerIds: [],
+    isSuperAdmin: false,
+    createdAt: '2024-01-01',
+    updatedAt: '2024-01-01',
+    deletedAt: null,
+  },
+  transactionId: 'tx-123',
+  ip: '127.0.0.1',
+  userAgent: 'test-agent',
+}
+
+describe('audited-db (sql)', () => {
+  beforeEach(() => {
+    vi.clearAllMocks()
+  })
+
+  it('auditedInsert inserts and returns rows with SQL', async () => {
+    const env = createMockEnv()
+    const mockDb = env.DB._mock
+
+    setMockQueryResult(mockDb, ['user-1', 'Alice'], [{ id: 'user-1', name: 'Alice' }])
+
+    const result = await auditedInsert(env.DB, mockCtx, 'users', { id: 'user-1', name: 'Alice' })
+
+    expect(result).toEqual([{ id: 'user-1', name: 'Alice' }])
+  })
+
+  it('auditedUpdate updates and returns rows with SQL', async () => {
+    const env = createMockEnv()
+    const mockDb = env.DB._mock
+
+    setMockQueryResult(mockDb, ['user-1'], [{ id: 'user-1', name: 'Old' }])
+    setMockQueryResult(mockDb, ['Updated', 'user-1'], [{ id: 'user-1', name: 'Updated' }])
+
+    const result = await auditedUpdate(
+      env.DB,
+      mockCtx,
+      'users',
+      { name: 'Updated' },
+      { clause: 'id = ?', params: ['user-1'] }
+    )
+
+    expect(result).toEqual([{ id: 'user-1', name: 'Updated' }])
+  })
+
+  it('auditedDelete soft deletes and logs with SQL', async () => {
+    const env = createMockEnv()
+    const mockDb = env.DB._mock
+    const fixedNow = '2025-01-01T00:00:00.000Z'
+
+    setMockQueryResult(
+      mockDb,
+      [fixedNow, mockCtx.user.id, fixedNow, mockCtx.user.id, 'user-1'],
+      [{ id: 'user-1', name: 'Alice' }]
+    )
+
+    await auditedDelete(
+      env.DB,
+      mockCtx,
+      'users',
+      { clause: 'id = ?', params: ['user-1'] },
+      { now: () => fixedNow }
+    )
+  })
+})

package/templates/base/{src → tests/unit}/server/lib/email.test.ts

@@ -1,5 +1,5 @@
 import { describe, it, expect, vi, beforeEach } from 'vitest'
-import { sendInvitationEmail } from './email'
+import { sendInvitationEmail } from '@server/lib/email'
 
 const mockFetch = vi.fn()
 vi.stubGlobal('fetch', mockFetch)

package/templates/base/{src → tests/unit}/server/lib/errors.test.ts

@@ -7,7 +7,7 @@ import {
   ConflictError,
   InternalError,
   getErrorCode,
-} from './errors'
+} from '@server/lib/errors'
 
 describe('Error Classes', () => {
   describe('getErrorCode', () => {

package/templates/base/{src → tests/unit}/server/lib/oauth.test.ts

@@ -7,7 +7,7 @@ import {
   exchangeCodeForTokens,
   getGoogleUserInfo,
   decodeIdToken,
-} from './oauth'
+} from '@server/lib/oauth'
 
 // Mock fetch globally
 const mockFetch = vi.fn()

package/templates/base/{src → tests/unit}/server/lib/pagination.test.ts

@@ -1,5 +1,5 @@
 import { describe, it, expect } from 'vitest'
-import { createPaginationMeta, PaginationQuerySchema } from './pagination'
+import { createPaginationMeta, PaginationQuerySchema } from '@server/lib/pagination'
 
 describe('pagination', () => {
   describe('createPaginationMeta', () => {

package/templates/base/{src → tests/unit}/server/lib/password.test.ts

@@ -1,5 +1,5 @@
 import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest'
-import { generateStrongPassword, isStrongPassword } from './password'
+import { generateStrongPassword, isStrongPassword } from '@server/lib/password'
 
 // Store original crypto.getRandomValues
 const originalGetRandomValues = crypto.getRandomValues.bind(crypto)

package/templates/base/{src → tests/unit}/server/lib/providers.test.ts

@@ -6,7 +6,7 @@ import {
   addProvider,
   removeProvider,
   parseProviderId,
-} from './providers'
+} from '@server/lib/providers'
 
 describe('providers', () => {
   describe('SUPPORTED_PROVIDERS', () => {

package/templates/base/{src → tests/unit}/server/lib/r2-storage.test.ts

@@ -6,8 +6,8 @@ import {
   deleteFile,
   getFileMetadata,
   listFiles,
-} from './r2-storage'
-import { createMockR2, type MockR2Bucket } from '../__tests__/mocks/r2'
+} from '@server/lib/r2-storage'
+import { createMockR2, type MockR2Bucket } from '@tests/mocks/r2'
 
 describe('r2-storage', () => {
   let mockR2: MockR2Bucket

package/templates/base/{src → tests/unit}/server/lib/session.test.ts

@@ -7,8 +7,8 @@ import {
   isAuthenticated,
   sessionMiddleware,
   type SessionData,
-} from './session'
-import { createMockKV, type MockKVNamespace } from '../__tests__/mocks/kv'
+} from '@server/lib/session'
+import { createMockKV, type MockKVNamespace } from '@tests/mocks/kv'
 
 /**
  * Create a mock Hono context for testing session functions

package/templates/base/{src → tests/unit}/server/lib/tokens.test.ts

@@ -7,7 +7,7 @@ import {
   getRefreshTokenExpiry,
   setCookieOptions,
   setOAuthStateCookieOptions,
-} from './tokens'
+} from '@server/lib/tokens'
 
 const mockEnv = {
   JWT_SECRET: 'test-secret-key',

package/templates/base/{src → tests/unit}/server/lib/transaction.test.ts

@@ -1,25 +1,19 @@
 import { describe, it, expect, vi } from 'vitest'
-import { withTransaction } from './transaction'
+import { withTransaction } from '@server/lib/transaction'
 
 describe('withTransaction', () => {
   it('returns result from successful callback', async () => {
-    const mockTx = { insert: vi.fn(), update: vi.fn() }
-    const mockDb = {
-      transaction: vi.fn(async (cb) => cb(mockTx)),
-    }
+    const mockDb = { prepare: vi.fn() }
 
     const result = await withTransaction(mockDb as any, async (tx) => {
       return { id: '123', name: 'Test' }
     })
 
     expect(result).toEqual({ id: '123', name: 'Test' })
-    expect(mockDb.transaction).toHaveBeenCalledTimes(1)
   })
 
   it('propagates errors from callback', async () => {
-    const mockDb = {
-      transaction: vi.fn(async (cb) => cb({})),
-    }
+    const mockDb = { prepare: vi.fn() }
 
     await expect(
       withTransaction(mockDb as any, async () => {
@@ -29,10 +23,7 @@ describe('withTransaction', () => {
   })
 
   it('passes transaction to callback', async () => {
-    const mockTx = { insert: vi.fn(), update: vi.fn() }
-    const mockDb = {
-      transaction: vi.fn(async (cb) => cb(mockTx)),
-    }
+    const mockDb = { prepare: vi.fn() }
 
     let receivedTx: any
     await withTransaction(mockDb as any, async (tx) => {
@@ -40,6 +31,6 @@ describe('withTransaction', () => {
       return null
     })
 
-    expect(receivedTx).toBe(mockTx)
+    expect(receivedTx).toBe(mockDb)
   })
 })