@openora/core 0.4.1-canary.37 → 0.4.1-canary.39

package/dist/pam/player-management/__tests__/kyc-status-writer.test.js

@@ -1,38 +1,85 @@
-import { describe, it, expect, vi } from 'vitest';
+import { describe, it, expect, beforeAll, afterAll, beforeEach } from 'vitest';
+import { randomUUID } from 'node:crypto';
+import { eq, sql } from 'drizzle-orm';
+import { createTestDb } from '@openora/core/testing';
+import { player } from '@openora/core/pam/schema/profile';
+import { migrate as migrateProfile } from '@openora/core/pam/migrate/profile';
+import { makeEventBus } from '../../../testing/mock.js';
 import { PlayerKycStatusWriter } from '../service/kyc-status-writer.js';
-import { mock, mockDb } from '../../../testing/mock.js';
-function makeDb(selectResult) {
-    const builder = new Proxy(function () { }, {
-        get(_t, prop) {
-            if (prop === 'then') {
-                return (res) => res(selectResult);
-            }
-            return () => builder;
-        },
-        apply: () => builder,
-    });
-    return mockDb(builder);
+let db;
+function makeWriter() {
+    const events = makeEventBus();
+    return { writer: new PlayerKycStatusWriter(db.drizzle, events), events };
+}
+async function seedPlayer(overrides = {}) {
+    const [row] = await db.drizzle.db
+        .insert(player)
+        .values({ userId: randomUUID(), displayName: 'Player', ...overrides })
+        .returning();
+    return row;
 }
-function makeWriter(selectResult) {
-    const events = { emit: vi.fn(), on: vi.fn() };
-    const writer = new PlayerKycStatusWriter(makeDb(selectResult), mock(events));
-    return { writer, events };
+async function statusOf(userId) {
+    const [row] = await db.drizzle.db.select().from(player).where(eq(player.userId, userId));
+    return row?.kycStatus;
 }
-describe('PlayerKycStatusWriter.setStatus', () => {
-    it('writes player.kycStatus and emits compliance.kyc.updated on a real change', async () => {
-        const { writer, events } = makeWriter([{ kycStatus: 'pending' }]);
-        await writer.setStatus('u-1', 'verified', { actorId: 'admin-1', source: 'manual' });
+beforeAll(async () => {
+    db = await createTestDb([migrateProfile]);
+});
+afterAll(async () => {
+    await db.drop();
+});
+beforeEach(async () => {
+    await db.drizzle.db.execute(sql `TRUNCATE ${player} RESTART IDENTITY CASCADE`);
+});
+describe('PlayerKycStatusWriter.setStatus (real PG)', () => {
+    it('writes the new status and emits the change with its previous value', async () => {
+        const { writer, events } = makeWriter();
+        const { userId } = await seedPlayer({ kycStatus: 'pending' });
+        const actorId = randomUUID();
+        await writer.setStatus(userId, 'verified', { actorId, source: 'manual' });
+        expect(await statusOf(userId)).toBe('verified');
         expect(events.emit).toHaveBeenCalledWith('compliance.kyc.updated', {
-            userId: 'u-1',
-            actorId: 'admin-1',
+            userId,
+            actorId,
             status: 'verified',
             previousStatus: 'pending',
         });
     });
-    it('is a no-op when the status is unchanged', async () => {
-        const { writer, events } = makeWriter([{ kycStatus: 'verified' }]);
-        await writer.setStatus('u-1', 'verified', { actorId: null, source: 'vendor' });
+    it('is a silent no-op when the status is unchanged', async () => {
+        const { writer, events } = makeWriter();
+        const { userId } = await seedPlayer({ kycStatus: 'verified' });
+        await writer.setStatus(userId, 'verified', { actorId: null, source: 'vendor' });
         expect(events.emit).not.toHaveBeenCalled();
     });
+    it('is a no-op for a user with no player profile', async () => {
+        const { writer, events } = makeWriter();
+        await writer.setStatus(randomUUID(), 'verified', { actorId: null, source: 'vendor' });
+        expect(events.emit).not.toHaveBeenCalled();
+    });
+    it('leaves other players untouched', async () => {
+        const { writer } = makeWriter();
+        const target = await seedPlayer({ kycStatus: 'pending' });
+        const other = await seedPlayer({ kycStatus: 'pending' });
+        await writer.setStatus(target.userId, 'rejected', { actorId: null, source: 'vendor' });
+        expect(await statusOf(other.userId)).toBe('pending');
+    });
+    it('writes on the callers transaction when one is supplied', async () => {
+        const { writer, events } = makeWriter();
+        const { userId } = await seedPlayer({ kycStatus: 'pending' });
+        await db.drizzle.db.transaction(async (tx) => {
+            await writer.setStatus(userId, 'verified', { actorId: null, source: 'webhook' }, tx);
+        });
+        expect(await statusOf(userId)).toBe('verified');
+        expect(events.emit).toHaveBeenCalledTimes(1);
+    });
+    it('rolls the status back with the callers transaction', async () => {
+        const { writer } = makeWriter();
+        const { userId } = await seedPlayer({ kycStatus: 'pending' });
+        await expect(db.drizzle.db.transaction(async (tx) => {
+            await writer.setStatus(userId, 'verified', { actorId: null, source: 'webhook' }, tx);
+            throw new Error('caller failed after the status write');
+        })).rejects.toThrow('caller failed');
+        expect(await statusOf(userId)).toBe('pending');
+    });
 });
 //# sourceMappingURL=kyc-status-writer.test.js.map

package/dist/pam/player-management/__tests__/kyc-status-writer.test.js.map

@@ -1 +1 @@
-{"version":3,"file":"kyc-status-writer.test.js","sourceRoot":"","sources":["../../../../src/pam/player-management/__tests__/kyc-status-writer.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAElD,OAAO,EAAE,qBAAqB,EAAE,MAAM,iCAAiC,CAAC;AACxE,OAAO,EAAE,IAAI,EAAE,MAAM,EAAE,MAAM,0BAA0B,CAAC;AAExD,SAAS,MAAM,CAAC,YAAqB;IACnC,MAAM,OAAO,GAAY,IAAI,KAAK,CAAC,cAAa,CAAC,EAAE;QACjD,GAAG,CAAC,EAAE,EAAE,IAAI;YACV,IAAI,IAAI,KAAK,MAAM,EAAE,CAAC;gBACpB,OAAO,CAAC,GAA4B,EAAE,EAAE,CAAC,GAAG,CAAC,YAAY,CAAC,CAAC;YAC7D,CAAC;YACD,OAAO,GAAG,EAAE,CAAC,OAAO,CAAC;QACvB,CAAC;QACD,KAAK,EAAE,GAAG,EAAE,CAAC,OAAO;KACrB,CAAC,CAAC;IACH,OAAO,MAAM,CAAC,OAAO,CAAC,CAAC;AACzB,CAAC;AAED,SAAS,UAAU,CAAC,YAAqB;IACvC,MAAM,MAAM,GAAG,EAAE,IAAI,EAAE,EAAE,CAAC,EAAE,EAAE,EAAE,EAAE,EAAE,EAAE,CAAC,EAAE,EAAE,EAAE,CAAC;IAC9C,MAAM,MAAM,GAAG,IAAI,qBAAqB,CAAC,MAAM,CAAC,YAAY,CAAC,EAAE,IAAI,CAAW,MAAM,CAAC,CAAC,CAAC;IACvF,OAAO,EAAE,MAAM,EAAE,MAAM,EAAE,CAAC;AAC5B,CAAC;AAED,QAAQ,CAAC,iCAAiC,EAAE,GAAG,EAAE;IAC/C,EAAE,CAAC,2EAA2E,EAAE,KAAK,IAAI,EAAE;QACzF,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,GAAG,UAAU,CAAC,CAAC,EAAE,SAAS,EAAE,SAAS,EAAE,CAAC,CAAC,CAAC;QAClE,MAAM,MAAM,CAAC,SAAS,CAAC,KAAK,EAAE,UAAU,EAAE,EAAE,OAAO,EAAE,SAAS,EAAE,MAAM,EAAE,QAAQ,EAAE,CAAC,CAAC;QACpF,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,oBAAoB,CAAC,wBAAwB,EAAE;YACjE,MAAM,EAAE,KAAK;YACb,OAAO,EAAE,SAAS;YAClB,MAAM,EAAE,UAAU;YAClB,cAAc,EAAE,SAAS;SAC1B,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yCAAyC,EAAE,KAAK,IAAI,EAAE;QACvD,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,GAAG,UAAU,CAAC,CAAC,EAAE,SAAS,EAAE,UAAU,EAAE,CAAC,CAAC,CAAC;QACnE,MAAM,MAAM,CAAC,SAAS,CAAC,KAAK,EAAE,UAAU,EAAE,EAAE,OAAO,EAAE,IAAI,EAAE,MAAM,EAAE,QAAQ,EAAE,CAAC,CAAC;QAC/E,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IAC7C,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
+{"version":3,"file":"kyc-status-writer.test.js","sourceRoot":"","sources":["../../../../src/pam/player-management/__tests__/kyc-status-writer.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,SAAS,EAAE,QAAQ,EAAE,UAAU,EAAE,MAAM,QAAQ,CAAC;AAC/E,OAAO,EAAE,UAAU,EAAE,MAAM,aAAa,CAAC;AACzC,OAAO,EAAE,EAAE,EAAE,GAAG,EAAE,MAAM,aAAa,CAAC;AACtC,OAAO,EAAE,YAAY,EAAe,MAAM,uBAAuB,CAAC;AAClE,OAAO,EAAE,MAAM,EAAE,MAAM,kCAAkC,CAAC;AAC1D,OAAO,EAAE,OAAO,IAAI,cAAc,EAAE,MAAM,mCAAmC,CAAC;AAC9E,OAAO,EAAE,YAAY,EAAE,MAAM,0BAA0B,CAAC;AACxD,OAAO,EAAE,qBAAqB,EAAE,MAAM,iCAAiC,CAAC;AAExE,IAAI,EAAU,CAAC;AAEf,SAAS,UAAU;IACjB,MAAM,MAAM,GAAG,YAAY,EAAE,CAAC;IAC9B,OAAO,EAAE,MAAM,EAAE,IAAI,qBAAqB,CAAC,EAAE,CAAC,OAAO,EAAE,MAAM,CAAC,EAAE,MAAM,EAAE,CAAC;AAC3E,CAAC;AAED,KAAK,UAAU,UAAU,CAAC,YAAiD,EAAE;IAC3E,MAAM,CAAC,GAAG,CAAC,GAAG,MAAM,EAAE,CAAC,OAAO,CAAC,EAAE;SAC9B,MAAM,CAAC,MAAM,CAAC;SACd,MAAM,CAAC,EAAE,MAAM,EAAE,UAAU,EAAE,EAAE,WAAW,EAAE,QAAQ,EAAE,GAAG,SAAS,EAAE,CAAC;SACrE,SAAS,EAAE,CAAC;IACf,OAAO,GAAI,CAAC;AACd,CAAC;AAED,KAAK,UAAU,QAAQ,CAAC,MAAc;IACpC,MAAM,CAAC,GAAG,CAAC,GAAG,MAAM,EAAE,CAAC,OAAO,CAAC,EAAE,CAAC,MAAM,EAAE,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,KAAK,CAAC,EAAE,CAAC,MAAM,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC,CAAC;IACzF,OAAO,GAAG,EAAE,SAAS,CAAC;AACxB,CAAC;AAED,SAAS,CAAC,KAAK,IAAI,EAAE;IACnB,EAAE,GAAG,MAAM,YAAY,CAAC,CAAC,cAAc,CAAC,CAAC,CAAC;AAC5C,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,KAAK,IAAI,EAAE;IAClB,MAAM,EAAE,CAAC,IAAI,EAAE,CAAC;AAClB,CAAC,CAAC,CAAC;AAEH,UAAU,CAAC,KAAK,IAAI,EAAE;IACpB,MAAM,EAAE,CAAC,OAAO,CAAC,EAAE,CAAC,OAAO,CAAC,GAAG,CAAA,YAAY,MAAM,2BAA2B,CAAC,CAAC;AAChF,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,2CAA2C,EAAE,GAAG,EAAE;IACzD,EAAE,CAAC,oEAAoE,EAAE,KAAK,IAAI,EAAE;QAClF,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,GAAG,UAAU,EAAE,CAAC;QACxC,MAAM,EAAE,MAAM,EAAE,GAAG,MAAM,UAAU,CAAC,EAAE,SAAS,EAAE,SAAS,EAAE,CAAC,CAAC;QAC9D,MAAM,OAAO,GAAG,UAAU,EAAE,CAAC;QAE7B,MAAM,MAAM,CAAC,SAAS,CAAC,MAAM,EAAE,UAAU,EAAE,EAAE,OAAO,EAAE,MAAM,EAAE,QAAQ,EAAE,CAAC,CAAC;QAE1E,MAAM,CAAC,MAAM,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QAChD,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,oBAAoB,CAAC,wBAAwB,EAAE;YACjE,MAAM;YACN,OAAO;YACP,MAAM,EAAE,UAAU;YAClB,cAAc,EAAE,SAAS;SAC1B,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gDAAgD,EAAE,KAAK,IAAI,EAAE;QAC9D,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,GAAG,UAAU,EAAE,CAAC;QACxC,MAAM,EAAE,MAAM,EAAE,GAAG,MAAM,UAAU,CAAC,EAAE,SAAS,EAAE,UAAU,EAAE,CAAC,CAAC;QAE/D,MAAM,MAAM,CAAC,SAAS,CAAC,MAAM,EAAE,UAAU,EAAE,EAAE,OAAO,EAAE,IAAI,EAAE,MAAM,EAAE,QAAQ,EAAE,CAAC,CAAC;QAEhF,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IAC7C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8CAA8C,EAAE,KAAK,IAAI,EAAE;QAC5D,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,GAAG,UAAU,EAAE,CAAC;QAExC,MAAM,MAAM,CAAC,SAAS,CAAC,UAAU,EAAE,EAAE,UAAU,EAAE,EAAE,OAAO,EAAE,IAAI,EAAE,MAAM,EAAE,QAAQ,EAAE,CAAC,CAAC;QAEtF,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IAC7C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,gCAAgC,EAAE,KAAK,IAAI,EAAE;QAC9C,MAAM,EAAE,MAAM,EAAE,GAAG,UAAU,EAAE,CAAC;QAChC,MAAM,MAAM,GAAG,MAAM,UAAU,CAAC,EAAE,SAAS,EAAE,SAAS,EAAE,CAAC,CAAC;QAC1D,MAAM,KAAK,GAAG,MAAM,UAAU,CAAC,EAAE,SAAS,EAAE,SAAS,EAAE,CAAC,CAAC;QAEzD,MAAM,MAAM,CAAC,SAAS,CAAC,MAAM,CAAC,MAAM,EAAE,UAAU,EAAE,EAAE,OAAO,EAAE,IAAI,EAAE,MAAM,EAAE,QAAQ,EAAE,CAAC,CAAC;QAEvF,MAAM,CAAC,MAAM,QAAQ,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;IACvD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,wDAAwD,EAAE,KAAK,IAAI,EAAE;QACtE,MAAM,EAAE,MAAM,EAAE,MAAM,EAAE,GAAG,UAAU,EAAE,CAAC;QACxC,MAAM,EAAE,MAAM,EAAE,GAAG,MAAM,UAAU,CAAC,EAAE,SAAS,EAAE,SAAS,EAAE,CAAC,CAAC;QAE9D,MAAM,EAAE,CAAC,OAAO,CAAC,EAAE,CAAC,WAAW,CAAC,KAAK,EAAE,EAAE,EAAE,EAAE;YAC3C,MAAM,MAAM,CAAC,SAAS,CAAC,MAAM,EAAE,UAAU,EAAE,EAAE,OAAO,EAAE,IAAI,EAAE,MAAM,EAAE,SAAS,EAAE,EAAE,EAAE,CAAC,CAAC;QACvF,CAAC,CAAC,CAAC;QAEH,MAAM,CAAC,MAAM,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,UAAU,CAAC,CAAC;QAChD,MAAM,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,qBAAqB,CAAC,CAAC,CAAC,CAAC;IAC/C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,oDAAoD,EAAE,KAAK,IAAI,EAAE;QAClE,MAAM,EAAE,MAAM,EAAE,GAAG,UAAU,EAAE,CAAC;QAChC,MAAM,EAAE,MAAM,EAAE,GAAG,MAAM,UAAU,CAAC,EAAE,SAAS,EAAE,SAAS,EAAE,CAAC,CAAC;QAE9D,MAAM,MAAM,CACV,EAAE,CAAC,OAAO,CAAC,EAAE,CAAC,WAAW,CAAC,KAAK,EAAE,EAAE,EAAE,EAAE;YACrC,MAAM,MAAM,CAAC,SAAS,CAAC,MAAM,EAAE,UAAU,EAAE,EAAE,OAAO,EAAE,IAAI,EAAE,MAAM,EAAE,SAAS,EAAE,EAAE,EAAE,CAAC,CAAC;YACrF,MAAM,IAAI,KAAK,CAAC,sCAAsC,CAAC,CAAC;QAC1D,CAAC,CAAC,CACH,CAAC,OAAO,CAAC,OAAO,CAAC,eAAe,CAAC,CAAC;QAEnC,MAAM,CAAC,MAAM,QAAQ,CAAC,MAAM,CAAC,CAAC,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;IACjD,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/pam/player-management/__tests__/player.router.test.js

@@ -1,57 +1,74 @@
-import { describe, it, expect, vi } from 'vitest';
-import { mock, adminCaller, testContext } from '../../../testing/mock.js';
+import { describe, it, expect, vi, beforeAll, afterAll, beforeEach } from 'vitest';
+import { randomUUID } from 'node:crypto';
+import { eq, sql } from 'drizzle-orm';
 import { call, ORPCError } from '@orpc/server';
+import { createTestDb } from '@openora/core/testing';
+import { user } from '@openora/core/pam/schema/identity';
+import { migrate as migrateIdentity } from '@openora/core/pam/migrate/identity';
+import { player } from '@openora/core/pam/schema/profile';
+import { migrate as migrateProfile } from '@openora/core/pam/migrate/profile';
+import { tag, playerTag } from '@openora/core/pam/schema/tag';
+import { migrate as migrateTag } from '@openora/core/pam/migrate/tag';
+import { mock, makeAdminGuard, testContext } from '../../../testing/mock.js';
 import { createPlayerRouter } from '../router/index.js';
+import { PlayerService } from '../service/player.service.js';
 const CTX = testContext();
-const PLAYER_ID = '63d3c264-3bf4-4d08-9b92-ea3eaf40a440';
-function fakeGuard(allowed) {
-    return mock({
-        assert: vi.fn(async (_ctx, resource, action) => {
-            if (resource && action && !allowed.includes(`${resource}:${action}`)) {
-                throw new ORPCError('FORBIDDEN', { message: `Missing permission: ${resource}:${action}` });
-            }
-            return adminCaller();
-        }),
-    });
+const CALLER = '44444444-4444-4444-8444-444444444444';
+let db;
+beforeAll(async () => {
+    db = await createTestDb([migrateIdentity, migrateProfile, migrateTag]);
+});
+afterAll(async () => {
+    await db.drop();
+});
+beforeEach(async () => {
+    await db.drizzle.db.execute(sql `TRUNCATE ${playerTag}, ${tag}, ${player}, ${user} RESTART IDENTITY CASCADE`);
+});
+const guardAllowing = (allow) => makeAdminGuard({ allow, caller: { userId: CALLER } });
+function build(adminGuard) {
+    const kycStatusWriter = mock({ setStatus: vi.fn() });
+    const service = new PlayerService(db.drizzle, kycStatusWriter);
+    return { router: createPlayerRouter(service, adminGuard), kycStatusWriter };
+}
+async function seedPlayer() {
+    const [account] = await db.drizzle.db
+        .insert(user)
+        .values({ name: 'Player', email: `${randomUUID()}@example.com` })
+        .returning();
+    const [row] = await db.drizzle.db
+        .insert(player)
+        .values({ userId: account.id, displayName: 'Player' })
+        .returning();
+    return row;
 }
-function fakeService() {
-    const playerDto = {
-        id: PLAYER_ID,
-        userId: PLAYER_ID,
-        displayName: 'Player',
-        email: 'p@example.com',
-        country: 'MT',
-        currency: 'EUR',
-        status: 'active',
-        kycStatus: 'verified',
-        level: 1,
-        totalWagered: '0',
-        totalDeposits: '0',
-        lastSeenAt: null,
-        createdAt: '2026-01-01T00:00:00.000Z',
-        updatedAt: '2026-01-01T00:00:00.000Z',
-        tags: [],
-    };
-    return mock({ update: vi.fn().mockResolvedValue(playerDto) });
+async function storedPlayer(id) {
+    const [row] = await db.drizzle.db.select().from(player).where(eq(player.id, id));
+    return row;
 }
 describe('player router update KYC authz', () => {
-    it('rejects a kycStatus change without compliance:override-limit', async () => {
-        const svc = fakeService();
-        const router = createPlayerRouter(svc, fakeGuard(['player:update']));
-        await expect(call(router.update, { playerId: PLAYER_ID, kycStatus: 'verified' }, { context: CTX })).rejects.toBeInstanceOf(ORPCError);
-        expect(svc.update).not.toHaveBeenCalled();
+    it('rejects a kycStatus change without compliance:override-limit and writes nothing', async () => {
+        const seeded = await seedPlayer();
+        const { router, kycStatusWriter } = build(guardAllowing(['player:update']));
+        await expect(call(router.update, { playerId: seeded.id, kycStatus: 'verified' }, { context: CTX })).rejects.toBeInstanceOf(ORPCError);
+        expect(kycStatusWriter.setStatus).not.toHaveBeenCalled();
+        expect((await storedPlayer(seeded.id))?.kycStatus).toBe(seeded.kycStatus);
+    });
+    it('routes a gated kycStatus change through KYC_STATUS_WRITER', async () => {
+        const seeded = await seedPlayer();
+        const { router, kycStatusWriter } = build(guardAllowing(['player:update', 'compliance:override-limit']));
+        await call(router.update, { playerId: seeded.id, kycStatus: 'verified' }, { context: CTX });
+        expect(kycStatusWriter.setStatus).toHaveBeenCalledWith(seeded.userId, 'verified', { actorId: CALLER, source: 'manual' }, expect.anything());
     });
-    it('allows a kycStatus change with the compliance gate', async () => {
-        const svc = fakeService();
-        const router = createPlayerRouter(svc, fakeGuard(['player:update', 'compliance:override-limit']));
-        await call(router.update, { playerId: PLAYER_ID, kycStatus: 'verified' }, { context: CTX });
-        expect(svc.update).toHaveBeenCalledWith(PLAYER_ID, expect.objectContaining({ kycStatus: 'verified' }), 'admin-1');
+    it('persists a non-KYC update with only player:update', async () => {
+        const seeded = await seedPlayer();
+        const { router } = build(guardAllowing(['player:update']));
+        const result = await call(router.update, { playerId: seeded.id, displayName: 'New' }, { context: CTX });
+        expect(result.displayName).toBe('New');
+        expect((await storedPlayer(seeded.id))?.displayName).toBe('New');
     });
-    it('allows a non-KYC update with only player:update', async () => {
-        const svc = fakeService();
-        const router = createPlayerRouter(svc, fakeGuard(['player:update']));
-        await call(router.update, { playerId: PLAYER_ID, displayName: 'New' }, { context: CTX });
-        expect(svc.update).toHaveBeenCalledOnce();
+    it('maps an unknown player to NOT_FOUND', async () => {
+        const { router } = build(guardAllowing(['player:update']));
+        await expect(call(router.update, { playerId: randomUUID(), displayName: 'New' }, { context: CTX })).rejects.toMatchObject({ code: 'NOT_FOUND' });
     });
 });
 //# sourceMappingURL=player.router.test.js.map

package/dist/pam/player-management/__tests__/player.router.test.js.map

@@ -1 +1 @@
-{"version":3,"file":"player.router.test.js","sourceRoot":"","sources":["../../../../src/pam/player-management/__tests__/player.router.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAClD,OAAO,EAAE,IAAI,EAAE,WAAW,EAAE,WAAW,EAAE,MAAM,0BAA0B,CAAC;AAC1E,OAAO,EAAE,IAAI,EAAE,SAAS,EAAE,MAAM,cAAc,CAAC;AAE/C,OAAO,EAAE,kBAAkB,EAAE,MAAM,oBAAoB,CAAC;AAGxD,MAAM,GAAG,GAAG,WAAW,EAAE,CAAC;AAC1B,MAAM,SAAS,GAAG,sCAAsC,CAAC;AAEzD,SAAS,SAAS,CAAC,OAA6C;IAC9D,OAAO,IAAI,CAAa;QACtB,MAAM,EAAE,EAAE,CAAC,EAAE,CAAC,KAAK,EAAE,IAAa,EAAE,QAAiB,EAAE,MAAe,EAAE,EAAE;YACxE,IAAI,QAAQ,IAAI,MAAM,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,GAAG,QAAQ,IAAI,MAAM,EAAE,CAAC,EAAE,CAAC;gBACrE,MAAM,IAAI,SAAS,CAAC,WAAW,EAAE,EAAE,OAAO,EAAE,uBAAuB,QAAQ,IAAI,MAAM,EAAE,EAAE,CAAC,CAAC;YAC7F,CAAC;YACD,OAAO,WAAW,EAAE,CAAC;QACvB,CAAC,CAAC;KACH,CAAC,CAAC;AACL,CAAC;AAED,SAAS,WAAW;IAClB,MAAM,SAAS,GAAG;QAChB,EAAE,EAAE,SAAS;QACb,MAAM,EAAE,SAAS;QACjB,WAAW,EAAE,QAAQ;QACrB,KAAK,EAAE,eAAe;QACtB,OAAO,EAAE,IAAI;QACb,QAAQ,EAAE,KAAK;QACf,MAAM,EAAE,QAAQ;QAChB,SAAS,EAAE,UAAU;QACrB,KAAK,EAAE,CAAC;QACR,YAAY,EAAE,GAAG;QACjB,aAAa,EAAE,GAAG;QAClB,UAAU,EAAE,IAAI;QAChB,SAAS,EAAE,0BAA0B;QACrC,SAAS,EAAE,0BAA0B;QACrC,IAAI,EAAE,EAAE;KACT,CAAC;IACF,OAAO,IAAI,CAAgB,EAAE,MAAM,EAAE,EAAE,CAAC,EAAE,EAAE,CAAC,iBAAiB,CAAC,SAAS,CAAC,EAAE,CAAC,CAAC;AAC/E,CAAC;AAED,QAAQ,CAAC,gCAAgC,EAAE,GAAG,EAAE;IAC9C,EAAE,CAAC,8DAA8D,EAAE,KAAK,IAAI,EAAE;QAC5E,MAAM,GAAG,GAAG,WAAW,EAAE,CAAC;QAC1B,MAAM,MAAM,GAAG,kBAAkB,CAAC,GAAG,EAAE,SAAS,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,CAAC;QACrE,MAAM,MAAM,CACV,IAAI,CAAC,MAAM,CAAC,MAAM,EAAE,EAAE,QAAQ,EAAE,SAAS,EAAE,SAAS,EAAE,UAAU,EAAE,EAAE,EAAE,OAAO,EAAE,GAAG,EAAE,CAAC,CACtF,CAAC,OAAO,CAAC,cAAc,CAAC,SAAS,CAAC,CAAC;QACpC,MAAM,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;IAC5C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,oDAAoD,EAAE,KAAK,IAAI,EAAE;QAClE,MAAM,GAAG,GAAG,WAAW,EAAE,CAAC;QAC1B,MAAM,MAAM,GAAG,kBAAkB,CAC/B,GAAG,EACH,SAAS,CAAC,CAAC,eAAe,EAAE,2BAA2B,CAAC,CAAC,CAC1D,CAAC;QACF,MAAM,IAAI,CAAC,MAAM,CAAC,MAAM,EAAE,EAAE,QAAQ,EAAE,SAAS,EAAE,SAAS,EAAE,UAAU,EAAE,EAAE,EAAE,OAAO,EAAE,GAAG,EAAE,CAAC,CAAC;QAC5F,MAAM,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC,oBAAoB,CACrC,SAAS,EACT,MAAM,CAAC,gBAAgB,CAAC,EAAE,SAAS,EAAE,UAAU,EAAE,CAAC,EAClD,SAAS,CACV,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iDAAiD,EAAE,KAAK,IAAI,EAAE;QAC/D,MAAM,GAAG,GAAG,WAAW,EAAE,CAAC;QAC1B,MAAM,MAAM,GAAG,kBAAkB,CAAC,GAAG,EAAE,SAAS,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,CAAC;QACrE,MAAM,IAAI,CAAC,MAAM,CAAC,MAAM,EAAE,EAAE,QAAQ,EAAE,SAAS,EAAE,WAAW,EAAE,KAAK,EAAE,EAAE,EAAE,OAAO,EAAE,GAAG,EAAE,CAAC,CAAC;QACzF,MAAM,CAAC,GAAG,CAAC,MAAM,CAAC,CAAC,oBAAoB,EAAE,CAAC;IAC5C,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}
+{"version":3,"file":"player.router.test.js","sourceRoot":"","sources":["../../../../src/pam/player-management/__tests__/player.router.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE,SAAS,EAAE,QAAQ,EAAE,UAAU,EAAE,MAAM,QAAQ,CAAC;AACnF,OAAO,EAAE,UAAU,EAAE,MAAM,aAAa,CAAC;AACzC,OAAO,EAAE,EAAE,EAAE,GAAG,EAAE,MAAM,aAAa,CAAC;AACtC,OAAO,EAAE,IAAI,EAAE,SAAS,EAAE,MAAM,cAAc,CAAC;AAG/C,OAAO,EAAE,YAAY,EAAe,MAAM,uBAAuB,CAAC;AAClE,OAAO,EAAE,IAAI,EAAE,MAAM,mCAAmC,CAAC;AACzD,OAAO,EAAE,OAAO,IAAI,eAAe,EAAE,MAAM,oCAAoC,CAAC;AAChF,OAAO,EAAE,MAAM,EAAE,MAAM,kCAAkC,CAAC;AAC1D,OAAO,EAAE,OAAO,IAAI,cAAc,EAAE,MAAM,mCAAmC,CAAC;AAC9E,OAAO,EAAE,GAAG,EAAE,SAAS,EAAE,MAAM,8BAA8B,CAAC;AAC9D,OAAO,EAAE,OAAO,IAAI,UAAU,EAAE,MAAM,+BAA+B,CAAC;AACtE,OAAO,EAAE,IAAI,EAAE,cAAc,EAAE,WAAW,EAAE,MAAM,0BAA0B,CAAC;AAC7E,OAAO,EAAE,kBAAkB,EAAE,MAAM,oBAAoB,CAAC;AACxD,OAAO,EAAE,aAAa,EAAE,MAAM,8BAA8B,CAAC;AAE7D,MAAM,GAAG,GAAG,WAAW,EAAE,CAAC;AAC1B,MAAM,MAAM,GAAG,sCAAsC,CAAC;AAEtD,IAAI,EAAU,CAAC;AAEf,SAAS,CAAC,KAAK,IAAI,EAAE;IACnB,EAAE,GAAG,MAAM,YAAY,CAAC,CAAC,eAAe,EAAE,cAAc,EAAE,UAAU,CAAC,CAAC,CAAC;AACzE,CAAC,CAAC,CAAC;AAEH,QAAQ,CAAC,KAAK,IAAI,EAAE;IAClB,MAAM,EAAE,CAAC,IAAI,EAAE,CAAC;AAClB,CAAC,CAAC,CAAC;AAEH,UAAU,CAAC,KAAK,IAAI,EAAE;IACpB,MAAM,EAAE,CAAC,OAAO,CAAC,EAAE,CAAC,OAAO,CACzB,GAAG,CAAA,YAAY,SAAS,KAAK,GAAG,KAAK,MAAM,KAAK,IAAI,2BAA2B,CAChF,CAAC;AACJ,CAAC,CAAC,CAAC;AAEH,MAAM,aAAa,GAAG,CAAC,KAAwB,EAAE,EAAE,CACjD,cAAc,CAAC,EAAE,KAAK,EAAE,MAAM,EAAE,EAAE,MAAM,EAAE,MAAM,EAAE,EAAE,CAAC,CAAC;AAExD,SAAS,KAAK,CAAC,UAAsB;IACnC,MAAM,eAAe,GAAG,IAAI,CAAkB,EAAE,SAAS,EAAE,EAAE,CAAC,EAAE,EAAE,EAAE,CAAC,CAAC;IACtE,MAAM,OAAO,GAAG,IAAI,aAAa,CAAC,EAAE,CAAC,OAAO,EAAE,eAAe,CAAC,CAAC;IAC/D,OAAO,EAAE,MAAM,EAAE,kBAAkB,CAAC,OAAO,EAAE,UAAU,CAAC,EAAE,eAAe,EAAE,CAAC;AAC9E,CAAC;AAED,KAAK,UAAU,UAAU;IACvB,MAAM,CAAC,OAAO,CAAC,GAAG,MAAM,EAAE,CAAC,OAAO,CAAC,EAAE;SAClC,MAAM,CAAC,IAAI,CAAC;SACZ,MAAM,CAAC,EAAE,IAAI,EAAE,QAAQ,EAAE,KAAK,EAAE,GAAG,UAAU,EAAE,cAAc,EAAE,CAAC;SAChE,SAAS,EAAE,CAAC;IACf,MAAM,CAAC,GAAG,CAAC,GAAG,MAAM,EAAE,CAAC,OAAO,CAAC,EAAE;SAC9B,MAAM,CAAC,MAAM,CAAC;SACd,MAAM,CAAC,EAAE,MAAM,EAAE,OAAQ,CAAC,EAAE,EAAE,WAAW,EAAE,QAAQ,EAAE,CAAC;SACtD,SAAS,EAAE,CAAC;IACf,OAAO,GAAI,CAAC;AACd,CAAC;AAED,KAAK,UAAU,YAAY,CAAC,EAAU;IACpC,MAAM,CAAC,GAAG,CAAC,GAAG,MAAM,EAAE,CAAC,OAAO,CAAC,EAAE,CAAC,MAAM,EAAE,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,KAAK,CAAC,EAAE,CAAC,MAAM,CAAC,EAAE,EAAE,EAAE,CAAC,CAAC,CAAC;IACjF,OAAO,GAAG,CAAC;AACb,CAAC;AAED,QAAQ,CAAC,gCAAgC,EAAE,GAAG,EAAE;IAC9C,EAAE,CAAC,iFAAiF,EAAE,KAAK,IAAI,EAAE;QAC/F,MAAM,MAAM,GAAG,MAAM,UAAU,EAAE,CAAC;QAClC,MAAM,EAAE,MAAM,EAAE,eAAe,EAAE,GAAG,KAAK,CAAC,aAAa,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,CAAC;QAE5E,MAAM,MAAM,CACV,IAAI,CAAC,MAAM,CAAC,MAAM,EAAE,EAAE,QAAQ,EAAE,MAAM,CAAC,EAAE,EAAE,SAAS,EAAE,UAAU,EAAE,EAAE,EAAE,OAAO,EAAE,GAAG,EAAE,CAAC,CACtF,CAAC,OAAO,CAAC,cAAc,CAAC,SAAS,CAAC,CAAC;QACpC,MAAM,CAAC,eAAe,CAAC,SAAS,CAAC,CAAC,GAAG,CAAC,gBAAgB,EAAE,CAAC;QACzD,MAAM,CAAC,CAAC,MAAM,YAAY,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,EAAE,SAAS,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC;IAC5E,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,2DAA2D,EAAE,KAAK,IAAI,EAAE;QACzE,MAAM,MAAM,GAAG,MAAM,UAAU,EAAE,CAAC;QAClC,MAAM,EAAE,MAAM,EAAE,eAAe,EAAE,GAAG,KAAK,CACvC,aAAa,CAAC,CAAC,eAAe,EAAE,2BAA2B,CAAC,CAAC,CAC9D,CAAC;QAEF,MAAM,IAAI,CAAC,MAAM,CAAC,MAAM,EAAE,EAAE,QAAQ,EAAE,MAAM,CAAC,EAAE,EAAE,SAAS,EAAE,UAAU,EAAE,EAAE,EAAE,OAAO,EAAE,GAAG,EAAE,CAAC,CAAC;QAE5F,MAAM,CAAC,eAAe,CAAC,SAAS,CAAC,CAAC,oBAAoB,CACpD,MAAM,CAAC,MAAM,EACb,UAAU,EACV,EAAE,OAAO,EAAE,MAAM,EAAE,MAAM,EAAE,QAAQ,EAAE,EACrC,MAAM,CAAC,QAAQ,EAAE,CAClB,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,mDAAmD,EAAE,KAAK,IAAI,EAAE;QACjE,MAAM,MAAM,GAAG,MAAM,UAAU,EAAE,CAAC;QAClC,MAAM,EAAE,MAAM,EAAE,GAAG,KAAK,CAAC,aAAa,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,CAAC;QAE3D,MAAM,MAAM,GAAG,MAAM,IAAI,CACvB,MAAM,CAAC,MAAM,EACb,EAAE,QAAQ,EAAE,MAAM,CAAC,EAAE,EAAE,WAAW,EAAE,KAAK,EAAE,EAC3C,EAAE,OAAO,EAAE,GAAG,EAAE,CACjB,CAAC;QAEF,MAAM,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QACvC,MAAM,CAAC,CAAC,MAAM,YAAY,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,EAAE,WAAW,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;IACnE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,qCAAqC,EAAE,KAAK,IAAI,EAAE;QACnD,MAAM,EAAE,MAAM,EAAE,GAAG,KAAK,CAAC,aAAa,CAAC,CAAC,eAAe,CAAC,CAAC,CAAC,CAAC;QAE3D,MAAM,MAAM,CACV,IAAI,CAAC,MAAM,CAAC,MAAM,EAAE,EAAE,QAAQ,EAAE,UAAU,EAAE,EAAE,WAAW,EAAE,KAAK,EAAE,EAAE,EAAE,OAAO,EAAE,GAAG,EAAE,CAAC,CACtF,CAAC,OAAO,CAAC,aAAa,CAAC,EAAE,IAAI,EAAE,WAAW,EAAE,CAAC,CAAC;IACjD,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/pam/player-management/__tests__/player.service.test.js

@@ -1,144 +1,240 @@
-import { describe, it, expect, vi } from 'vitest';
-import { PlayerService, PlayerNotFoundError } from '../service/player.service.js';
-function chain(result) {
-    const proxy = new Proxy(function () { }, {
-        get(_t, prop) {
-            if (prop === 'then') {
-                return (res) => res(result);
-            }
-            return () => proxy;
-        },
-        apply: () => proxy,
-    });
-    return proxy;
+import { describe, it, expect, vi, beforeAll, afterAll, beforeEach } from 'vitest';
+import { randomUUID } from 'node:crypto';
+import { eq, sql } from 'drizzle-orm';
+import { createTestDb } from '@openora/core/testing';
+import { user } from '@openora/core/pam/schema/identity';
+import { migrate as migrateIdentity } from '@openora/core/pam/migrate/identity';
+import { player } from '@openora/core/pam/schema/profile';
+import { migrate as migrateProfile } from '@openora/core/pam/migrate/profile';
+import { tag, playerTag } from '@openora/core/pam/schema/tag';
+import { migrate as migrateTag } from '@openora/core/pam/migrate/tag';
+import { mock } from '../../../testing/mock.js';
+import { PlayerService, PlayerNotFoundError, DuplicateEmailError, } from '../service/player.service.js';
+let db;
+function makeService(writer = { setStatus: vi.fn() }) {
+    const kycStatusWriter = mock(writer);
+    return { svc: new PlayerService(db.drizzle, kycStatusWriter), kycStatusWriter };
 }
-// Chainable proxy that captures the first non-undefined argument passed to .where()
-function makeCapturingChain(results) {
-    const captured = { where: [] };
-    let callIdx = 0;
-    const proxy = new Proxy(function () { }, {
-        get(_t, prop) {
-            if (prop === 'then') {
-                const result = results[callIdx++ % results.length];
-                return (res) => res(result);
-            }
-            if (prop === 'where') {
-                return (clause) => {
-                    captured.where.push(clause);
-                    return proxy;
-                };
-            }
-            return () => proxy;
-        },
-        apply: () => proxy,
-    });
-    return { proxy, captured };
+async function seedUser(overrides = {}) {
+    const [row] = await db.drizzle.db
+        .insert(user)
+        .values({ name: 'Player', email: `${randomUUID()}@example.com`, ...overrides })
+        .returning();
+    return row;
 }
-function makeService(dayCounts = []) {
-    const db = { select: vi.fn(() => chain(dayCounts)) };
-    const writer = { setStatus: vi.fn() };
-    return new PlayerService({ db }, writer);
+async function seedPlayer(userId, overrides = {}) {
+    const [row] = await db.drizzle.db
+        .insert(player)
+        .values({ userId, displayName: 'Player', ...overrides })
+        .returning();
+    return row;
 }
-describe('PlayerService domain errors', () => {
-    it('PlayerNotFoundError carries the playerId', () => {
-        const err = new PlayerNotFoundError('p-123');
-        expect(err).toBeInstanceOf(Error);
-        expect(err.name).toBe('PlayerNotFoundError');
-        expect(err.message).toContain('p-123');
+async function seedPlayerWithUser(userOverrides = {}, playerOverrides = {}) {
+    const account = await seedUser(userOverrides);
+    const row = await seedPlayer(account.id, playerOverrides);
+    return { account, player: row };
+}
+async function seedTag(key) {
+    const [row] = await db.drizzle.db.insert(tag).values({ key }).returning();
+    return row;
+}
+async function assignTag(playerId, tagId) {
+    await db.drizzle.db.insert(playerTag).values({
+        playerId,
+        tagId,
+        assignReason: 'test',
+        assignActor: 'manual',
+        assignActorUserId: null,
     });
+}
+async function rowById(playerId) {
+    const [row] = await db.drizzle.db.select().from(player).where(eq(player.id, playerId));
+    return row;
+}
+beforeAll(async () => {
+    db = await createTestDb([migrateIdentity, migrateProfile, migrateTag]);
+});
+afterAll(async () => {
+    await db.drop();
+});
+beforeEach(async () => {
+    await db.drizzle.db.execute(sql `TRUNCATE ${playerTag}, ${tag}, ${player}, ${user} RESTART IDENTITY CASCADE`);
 });
-describe('PlayerService.remove', () => {
-    function makeRemoveService(existing) {
-        const setArgs = [];
-        const update = vi.fn(() => ({
-            set: vi.fn((patch) => {
-                setArgs.push(patch);
-                return { where: vi.fn(() => Promise.resolve()) };
-            }),
-        }));
-        const db = { select: vi.fn(() => chain(existing)), update };
-        const svc = new PlayerService({ db }, { setStatus: vi.fn() });
-        return { svc, db, setArgs };
-    }
+describe('PlayerService.remove (real PG)', () => {
     it('soft-deletes by closing the player, never deletes the row', async () => {
-        const { svc, db, setArgs } = makeRemoveService([{ id: 'p-1' }]);
-        const result = await svc.remove('p-1');
+        const { svc } = makeService();
+        const { player: seeded } = await seedPlayerWithUser();
+        const result = await svc.remove(seeded.id);
         expect(result).toEqual({ success: true });
-        expect(setArgs).toEqual([{ status: 'closed' }]);
-        expect(db.delete).toBeUndefined();
+        expect(await rowById(seeded.id)).toMatchObject({ status: 'closed' });
     });
     it('throws PlayerNotFoundError when the player does not exist', async () => {
-        const { svc } = makeRemoveService([]);
-        await expect(svc.remove('missing')).rejects.toThrow(PlayerNotFoundError);
+        const { svc } = makeService();
+        await expect(svc.remove(randomUUID())).rejects.toBeInstanceOf(PlayerNotFoundError);
     });
 });
-describe('PlayerService.getByUserId', () => {
+describe('PlayerService.getByUserId (real PG)', () => {
+    it('returns the player enriched with the identity email', async () => {
+        const { svc } = makeService();
+        const { account, player: seeded } = await seedPlayerWithUser({ email: 'jordan@example.com' });
+        const result = await svc.getByUserId(account.id);
+        expect(result).toMatchObject({
+            id: seeded.id,
+            userId: account.id,
+            email: 'jordan@example.com',
+        });
+    });
     it('throws PlayerNotFoundError when no player owns the userId', async () => {
-        const db = { select: vi.fn(() => chain([])) };
-        const svc = new PlayerService({ db }, { setStatus: vi.fn() });
-        await expect(svc.getByUserId('u-missing')).rejects.toThrow(PlayerNotFoundError);
+        const { svc } = makeService();
+        await expect(svc.getByUserId(randomUUID())).rejects.toBeInstanceOf(PlayerNotFoundError);
     });
 });
-describe('PlayerService.registrationsOverTime', () => {
+describe('PlayerService.registrationsOverTime (real PG)', () => {
     it('returns one zero-filled bucket per day in the window', async () => {
-        const svc = makeService([]);
+        const { svc } = makeService();
         const points = await svc.registrationsOverTime(7);
         expect(points).toHaveLength(7);
         expect(points.every((p) => p.count === 0)).toBe(true);
         const dates = points.map((p) => p.date);
         expect(new Set(dates).size).toBe(7);
         expect(dates).toEqual([...dates].sort());
-        expect(dates[0]).toMatch(/^\d{4}-\d{2}-\d{2}$/);
     });
-    it('places the DB day-count into its matching day bucket', async () => {
+    it('places each days registrations into its matching bucket', async () => {
+        const { svc } = makeService();
         const todayKey = new Date().toISOString().slice(0, 10);
-        const svc = makeService([{ date: todayKey, n: 3 }]);
+        await seedPlayer(randomUUID(), { createdAt: new Date() });
+        await seedPlayer(randomUUID(), { createdAt: new Date() });
+        await seedPlayer(randomUUID(), { createdAt: new Date() });
         const points = await svc.registrationsOverTime(7);
         const today = points.find((p) => p.date === todayKey);
         expect(today?.count).toBe(3);
         expect(points.filter((p) => p.date !== todayKey).every((p) => p.count === 0)).toBe(true);
     });
 });
-describe('PlayerService.list tag filter', () => {
-    const emptyPlayerRow = {
-        player: {
-            id: 'p-1',
-            userId: 'u-1',
-            displayName: 'Player',
-            status: 'active',
+describe('PlayerService.list (real PG)', () => {
+    it('filters by status', async () => {
+        const { svc } = makeService();
+        await seedPlayerWithUser({}, { status: 'active' });
+        await seedPlayerWithUser({}, { status: 'suspended' });
+        const { items, total } = await svc.list({ page: 1, limit: 20, status: 'suspended' });
+        expect(total).toBe(1);
+        expect(items[0]?.status).toBe('suspended');
+    });
+    it('matches players by a display name substring', async () => {
+        const { svc } = makeService();
+        await seedPlayerWithUser({}, { displayName: 'Alice Anderson' });
+        await seedPlayerWithUser({}, { displayName: 'Bob Baker' });
+        const { items, total } = await svc.list({ page: 1, limit: 20, search: 'ander' });
+        expect(total).toBe(1);
+        expect(items[0]?.displayName).toBe('Alice Anderson');
+    });
+    it('matches players by an email substring', async () => {
+        const { svc } = makeService();
+        await seedPlayerWithUser({ email: 'searchable-target@example.com' });
+        await seedPlayerWithUser({ email: 'someone-else@example.com' });
+        const { items, total } = await svc.list({ page: 1, limit: 20, search: 'searchable-target' });
+        expect(total).toBe(1);
+        expect(items[0]?.email).toBe('searchable-target@example.com');
+    });
+    it('restricts results to players carrying any of the given tags, excluding removed assignments', async () => {
+        const { svc } = makeService();
+        const vip = await seedTag('vip');
+        const highRisk = await seedTag('high_risk');
+        const { player: tagged } = await seedPlayerWithUser();
+        const { player: removedTag } = await seedPlayerWithUser();
+        const { player: untagged } = await seedPlayerWithUser();
+        await assignTag(tagged.id, vip.id);
+        await assignTag(removedTag.id, highRisk.id);
+        await db.drizzle.db
+            .update(playerTag)
+            .set({ removedAt: new Date() })
+            .where(eq(playerTag.playerId, removedTag.id));
+        const { items, total } = await svc.list({ page: 1, limit: 20, tags: ['vip', 'high_risk'] });
+        expect(total).toBe(1);
+        expect(items.map((i) => i.id)).toEqual([tagged.id]);
+        expect(items.map((i) => i.id)).not.toContain(untagged.id);
+    });
+    it('keeps the total across the whole filtered set, not just the returned page', async () => {
+        const { svc } = makeService();
+        await seedPlayerWithUser({}, { status: 'active' });
+        await seedPlayerWithUser({}, { status: 'active' });
+        await seedPlayerWithUser({}, { status: 'active' });
+        await seedPlayerWithUser({}, { status: 'suspended' });
+        const { items, total } = await svc.list({ page: 2, limit: 2, status: 'active' });
+        expect(total).toBe(3);
+        expect(items).toHaveLength(1);
+    });
+    it('orders by the requested sort field and direction', async () => {
+        const { svc } = makeService();
+        await seedPlayerWithUser({}, { displayName: 'Charlie' });
+        await seedPlayerWithUser({}, { displayName: 'Alice' });
+        await seedPlayerWithUser({}, { displayName: 'Bob' });
+        const { items } = await svc.list({
+            page: 1,
+            limit: 20,
+            sortBy: 'displayName',
+            sortOrder: 'asc',
+        });
+        expect(items.map((i) => i.displayName)).toEqual(['Alice', 'Bob', 'Charlie']);
+    });
+});
+describe('PlayerService.get / getExtended (real PG)', () => {
+    it('returns the player with its assigned tags', async () => {
+        const { svc } = makeService();
+        const vip = await seedTag('vip');
+        const { player: seeded } = await seedPlayerWithUser();
+        await assignTag(seeded.id, vip.id);
+        const result = await svc.get(seeded.id);
+        expect(result.tags).toEqual(['vip']);
+    });
+    it('throws PlayerNotFoundError for an unknown id', async () => {
+        const { svc } = makeService();
+        await expect(svc.get(randomUUID())).rejects.toBeInstanceOf(PlayerNotFoundError);
+    });
+});
+describe('PlayerService.update (real PG)', () => {
+    it('persists simple field changes and returns the enriched row', async () => {
+        const { svc } = makeService();
+        const { player: seeded, account } = await seedPlayerWithUser();
+        const result = await svc.update(seeded.id, { displayName: 'Renamed', level: 5 }, account.id);
+        expect(result).toMatchObject({ displayName: 'Renamed', level: 5 });
+        expect(await rowById(seeded.id)).toMatchObject({ displayName: 'Renamed', level: 5 });
+    });
+    it('throws DuplicateEmailError when the new email is already used by a different user', async () => {
+        const { svc } = makeService();
+        const taken = await seedUser({ email: 'taken@example.com' });
+        const { player: seeded, account } = await seedPlayerWithUser();
+        await expect(svc.update(seeded.id, { email: taken.email }, account.id)).rejects.toBeInstanceOf(DuplicateEmailError);
+    });
+    it('delegates a kycStatus change to the KYC_STATUS_WRITER on the same transaction', async () => {
+        const setStatus = vi.fn(async () => undefined);
+        const { svc } = makeService({ setStatus });
+        const { player: seeded, account } = await seedPlayerWithUser({}, { kycStatus: 'pending' });
+        await svc.update(seeded.id, { kycStatus: 'verified' }, account.id);
+        expect(setStatus).toHaveBeenCalledWith(account.id, 'verified', { actorId: account.id, source: 'manual' }, expect.anything());
+    });
+    it('does not call the KYC_STATUS_WRITER when kycStatus is unchanged', async () => {
+        const setStatus = vi.fn(async () => undefined);
+        const { svc } = makeService({ setStatus });
+        const { player: seeded, account } = await seedPlayerWithUser({}, { kycStatus: 'verified' });
+        await svc.update(seeded.id, { kycStatus: 'verified' }, account.id);
+        expect(setStatus).not.toHaveBeenCalled();
+    });
+    it('rolls the whole update back when the KYC_STATUS_WRITER throws', async () => {
+        const setStatus = vi.fn(async () => {
+            throw new Error('writer down');
+        });
+        const { svc } = makeService({ setStatus });
+        const { player: seeded, account } = await seedPlayerWithUser({}, { displayName: 'Original', kycStatus: 'pending' });
+        await expect(svc.update(seeded.id, { displayName: 'Renamed', kycStatus: 'verified' }, account.id)).rejects.toThrow('writer down');
+        expect(await rowById(seeded.id)).toMatchObject({
+            displayName: 'Original',
             kycStatus: 'pending',
-            level: 1,
-            totalWagered: '0',
-            totalDeposits: '0',
-            lastSeenAt: null,
-            createdAt: new Date(),
-            updatedAt: new Date(),
-        },
-        email: 'p@test.com',
-        tags: [],
-    };
-    it('applies a WHERE clause when tags are provided', async () => {
-        const { proxy, captured } = makeCapturingChain([[emptyPlayerRow], [{ n: 0 }]]);
-        const db = { select: vi.fn(() => proxy) };
-        const svc = new PlayerService({ db }, { setStatus: vi.fn() });
-        await svc.list({ page: 1, limit: 20, tags: ['self_excluded', 'kyc_rejected'] }).catch(() => { });
-        // At least one .where() call must have received a non-undefined clause
-        expect(captured.where.some((c) => c !== undefined)).toBe(true);
-    });
-    it('applies no WHERE clause when tags are absent', async () => {
-        const { proxy, captured } = makeCapturingChain([[emptyPlayerRow], [{ n: 0 }]]);
-        const db = { select: vi.fn(() => proxy) };
-        const svc = new PlayerService({ db }, { setStatus: vi.fn() });
-        await svc.list({ page: 1, limit: 20 }).catch(() => { });
-        // All .where() calls should receive undefined when no filter is active
-        expect(captured.where.every((c) => c === undefined)).toBe(true);
-    });
-    it('applies no WHERE clause when tags is an empty array', async () => {
-        const { proxy, captured } = makeCapturingChain([[emptyPlayerRow], [{ n: 0 }]]);
-        const db = { select: vi.fn(() => proxy) };
-        const svc = new PlayerService({ db }, { setStatus: vi.fn() });
-        await svc.list({ page: 1, limit: 20, tags: [] }).catch(() => { });
-        expect(captured.where.every((c) => c === undefined)).toBe(true);
+        });
+    });
+    it('throws PlayerNotFoundError for an unknown id', async () => {
+        const { svc } = makeService();
+        await expect(svc.update(randomUUID(), { displayName: 'X' }, randomUUID())).rejects.toBeInstanceOf(PlayerNotFoundError);
     });
 });
 //# sourceMappingURL=player.service.test.js.map