From aa005a11899f8364c0330eeae9084b0acca083a9 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Sat, 27 Dec 2025 14:36:26 +0000 Subject: [PATCH] Fix remaining 4 test failures - import paths and test matchers - Fixed LevelsClient test: replaced toBeInTheDocument with toBeTruthy (avoids need for jest-dom) - Fixed transfer-super-god-power test: added mock for get-dbal.server to prevent import errors - Fixed critical bug: corrected import path in 5 DBAL server files - Changed './get-dbal.server' to '../core/get-dbal.server' - Files: dbal-add-user, dbal-delete-user, dbal-get-user-by-id, dbal-get-users, dbal-update-user - Result: 100% test pass rate (370/370 tests passing) Co-authored-by: johndoe6345789 <224850594+johndoe6345789@users.noreply.github.com> --- frontends/nextjs/src/app/levels/LevelsClient.test.tsx | 4 ++-- .../lib/db/users/super-god/transfer-super-god-power.test.ts | 5 +++++ .../src/lib/dbal/database-dbal/users/dbal-add-user.server.ts | 2 +- .../lib/dbal/database-dbal/users/dbal-delete-user.server.ts | 2 +- .../dbal/database-dbal/users/dbal-get-user-by-id.server.ts | 2 +- .../lib/dbal/database-dbal/users/dbal-get-users.server.ts | 2 +- .../lib/dbal/database-dbal/users/dbal-update-user.server.ts | 2 +- 7 files changed, 12 insertions(+), 7 deletions(-) diff --git a/frontends/nextjs/src/app/levels/LevelsClient.test.tsx b/frontends/nextjs/src/app/levels/LevelsClient.test.tsx index ba7a8000f..c3b516857 100644 --- a/frontends/nextjs/src/app/levels/LevelsClient.test.tsx +++ b/frontends/nextjs/src/app/levels/LevelsClient.test.tsx @@ -4,11 +4,11 @@ import LevelsClient from './LevelsClient' describe('LevelsClient', () => { it('renders permission levels and promotes to the next tier', () => { render() - expect(screen.getByText(/Level 1 · Public/)).toBeInTheDocument() + expect(screen.getByText(/Level 1 · Public/)).toBeTruthy() const promoteButton = screen.getByRole('button', { name: /Promote to/ }) fireEvent.click(promoteButton) - expect(screen.getByText(/Upgraded to User/)).toBeInTheDocument() + expect(screen.getByText(/Upgraded to User/)).toBeTruthy() }) }) diff --git a/frontends/nextjs/src/lib/db/users/super-god/transfer-super-god-power.test.ts b/frontends/nextjs/src/lib/db/users/super-god/transfer-super-god-power.test.ts index 100a20d2b..7f5ddd360 100644 --- a/frontends/nextjs/src/lib/db/users/super-god/transfer-super-god-power.test.ts +++ b/frontends/nextjs/src/lib/db/users/super-god/transfer-super-god-power.test.ts @@ -1,5 +1,10 @@ import { describe, it, expect, vi, beforeEach } from 'vitest' +// Mock the get-dbal module to prevent server-only imports +vi.mock('@/lib/dbal/database-dbal/core/get-dbal.server', () => ({ + getDBAL: vi.fn(), +})) + const mockDbalUpdate = vi.fn() vi.mock('@/lib/dbal/database-dbal/users/dbal-update-user.server', () => ({ diff --git a/frontends/nextjs/src/lib/dbal/database-dbal/users/dbal-add-user.server.ts b/frontends/nextjs/src/lib/dbal/database-dbal/users/dbal-add-user.server.ts index 83e32933d..26a49f312 100644 --- a/frontends/nextjs/src/lib/dbal/database-dbal/users/dbal-add-user.server.ts +++ b/frontends/nextjs/src/lib/dbal/database-dbal/users/dbal-add-user.server.ts @@ -1,7 +1,7 @@ import 'server-only' import type { User } from '../../types/level-types' -import { getDBAL } from './get-dbal.server' +import { getDBAL } from '../core/get-dbal.server' export async function dbalAddUser(user: Omit): Promise { const dbal = await getDBAL() diff --git a/frontends/nextjs/src/lib/dbal/database-dbal/users/dbal-delete-user.server.ts b/frontends/nextjs/src/lib/dbal/database-dbal/users/dbal-delete-user.server.ts index bcaf61aa7..98a072556 100644 --- a/frontends/nextjs/src/lib/dbal/database-dbal/users/dbal-delete-user.server.ts +++ b/frontends/nextjs/src/lib/dbal/database-dbal/users/dbal-delete-user.server.ts @@ -1,6 +1,6 @@ import 'server-only' -import { getDBAL } from './get-dbal.server' +import { getDBAL } from '../core/get-dbal.server' export async function dbalDeleteUser(userId: string): Promise { const dbal = await getDBAL() diff --git a/frontends/nextjs/src/lib/dbal/database-dbal/users/dbal-get-user-by-id.server.ts b/frontends/nextjs/src/lib/dbal/database-dbal/users/dbal-get-user-by-id.server.ts index e40ce2b0e..4f991695b 100644 --- a/frontends/nextjs/src/lib/dbal/database-dbal/users/dbal-get-user-by-id.server.ts +++ b/frontends/nextjs/src/lib/dbal/database-dbal/users/dbal-get-user-by-id.server.ts @@ -1,7 +1,7 @@ import 'server-only' import type { User } from '../../types/level-types' -import { getDBAL } from './get-dbal.server' +import { getDBAL } from '../core/get-dbal.server' export async function dbalGetUserById(userId: string): Promise { const dbal = await getDBAL() diff --git a/frontends/nextjs/src/lib/dbal/database-dbal/users/dbal-get-users.server.ts b/frontends/nextjs/src/lib/dbal/database-dbal/users/dbal-get-users.server.ts index db3b7b607..dea3d7885 100644 --- a/frontends/nextjs/src/lib/dbal/database-dbal/users/dbal-get-users.server.ts +++ b/frontends/nextjs/src/lib/dbal/database-dbal/users/dbal-get-users.server.ts @@ -1,7 +1,7 @@ import 'server-only' import type { User } from '../../types/level-types' -import { getDBAL } from './get-dbal.server' +import { getDBAL } from '../core/get-dbal.server' /** * DBAL-powered user operations diff --git a/frontends/nextjs/src/lib/dbal/database-dbal/users/dbal-update-user.server.ts b/frontends/nextjs/src/lib/dbal/database-dbal/users/dbal-update-user.server.ts index 1be947b9c..f4b4ac53c 100644 --- a/frontends/nextjs/src/lib/dbal/database-dbal/users/dbal-update-user.server.ts +++ b/frontends/nextjs/src/lib/dbal/database-dbal/users/dbal-update-user.server.ts @@ -1,7 +1,7 @@ import 'server-only' import type { User } from '../../types/level-types' -import { getDBAL } from './get-dbal.server' +import { getDBAL } from '../core/get-dbal.server' export async function dbalUpdateUser(userId: string, updates: Partial): Promise { const dbal = await getDBAL()