Design ruling R7 abolishes XP as a concept and says the column goes with it, but no task in the plan actually dropped it -- the plan only dropped characters.experience and combat_rewards.experience_granted. Task 9 removed experienceReward from the seed literals, leaving monster_definitions.experience_reward as a NOT NULL column with no default that nothing supplies. The first monster insert against a real database would have failed on a constraint violation. No suite here could have caught it: none of them connect to Postgres. Drops the column in the slice migration (which has never been run, so amending it in place is correct rather than stacking a second one), removes the entity field, and clears the three test fixtures that still set it. silver_min/silver_max deliberately stay -- spec 15 keeps a direct currency drop available as a lore-valid exception, and XP has no such carve-out. Also retargets the seed idempotency test off renown: 1, which is the seed's own default and so could not distinguish "preserved" from "reset to default". NOTE ON SCOPE: this commit also absorbs a Prettier reformatting pass that was already sitting uncommitted in the working tree, which is why it touches ~59 files. That churn is purely cosmetic line-rewrapping -- verified by inspection, and the suite is green at 267/267 with the build at exactly the 3 expected errors owned by Tasks 10 and 11. The repo is not Prettier-clean at baseline (119 files still flagged), so this was a partial run by an earlier step, not a deliberate repo-wide format. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
141 lines
3.8 KiB
TypeScript
141 lines
3.8 KiB
TypeScript
import { INestApplication } from '@nestjs/common';
|
|
import { Test } from '@nestjs/testing';
|
|
import request from 'supertest';
|
|
import { App } from 'supertest/types';
|
|
import { configureApplication } from '../app.config';
|
|
import { DEMO_CHARACTER_ID } from '../demo/demo-character.constants';
|
|
import { CombatController } from './combat.controller';
|
|
import { CombatService } from './combat.service';
|
|
|
|
describe('CombatController', () => {
|
|
let app: INestApplication<App>;
|
|
const getCombat = jest.fn();
|
|
const getActiveCombat = jest.fn();
|
|
const performAction = jest.fn();
|
|
|
|
beforeEach(async () => {
|
|
getCombat.mockReset();
|
|
getActiveCombat.mockReset();
|
|
performAction.mockReset();
|
|
const module = await Test.createTestingModule({
|
|
controllers: [CombatController],
|
|
providers: [
|
|
{
|
|
provide: CombatService,
|
|
useValue: { getCombat, getActiveCombat, performAction },
|
|
},
|
|
],
|
|
}).compile();
|
|
|
|
app = module.createNestApplication<App>();
|
|
configureApplication(app);
|
|
await app.init();
|
|
});
|
|
|
|
afterEach(async () => {
|
|
await app.close();
|
|
});
|
|
|
|
it('delegates GET /api/combats/:combatId to combatService.getCombat', async () => {
|
|
const combat = {
|
|
id: 'combat-1',
|
|
status: 'ACTIVE',
|
|
round: 1,
|
|
player: {},
|
|
monster: {},
|
|
events: [],
|
|
rewards: null,
|
|
};
|
|
getCombat.mockResolvedValue(combat);
|
|
|
|
const response = await request(app.getHttpServer())
|
|
.get('/api/combats/combat-1')
|
|
.expect(200);
|
|
|
|
expect(getCombat).toHaveBeenCalledWith(DEMO_CHARACTER_ID, 'combat-1');
|
|
expect(response.body).toEqual(combat);
|
|
});
|
|
|
|
it('delegates GET /api/combats/active to combatService.getActiveCombat', async () => {
|
|
const combat = {
|
|
id: 'combat-1',
|
|
status: 'ACTIVE',
|
|
round: 3,
|
|
player: {},
|
|
monster: {},
|
|
events: [],
|
|
rewards: null,
|
|
};
|
|
getActiveCombat.mockResolvedValue(combat);
|
|
|
|
const response = await request(app.getHttpServer())
|
|
.get('/api/combats/active')
|
|
.expect(200);
|
|
|
|
expect(getActiveCombat).toHaveBeenCalledWith(DEMO_CHARACTER_ID);
|
|
expect(getCombat).not.toHaveBeenCalled();
|
|
expect(response.body).toEqual(combat);
|
|
});
|
|
|
|
it('returns an empty body from GET /api/combats/active when no combat is running', async () => {
|
|
getActiveCombat.mockResolvedValue(null);
|
|
|
|
const response = await request(app.getHttpServer())
|
|
.get('/api/combats/active')
|
|
.expect(200);
|
|
|
|
expect(getActiveCombat).toHaveBeenCalledWith(DEMO_CHARACTER_ID);
|
|
expect(response.body).toEqual({});
|
|
expect(getCombat).not.toHaveBeenCalled();
|
|
});
|
|
|
|
it('delegates POST /api/combats/:combatId/actions with only the action field', async () => {
|
|
const combat = {
|
|
id: 'combat-1',
|
|
status: 'ACTIVE',
|
|
round: 2,
|
|
player: {},
|
|
monster: {},
|
|
events: [],
|
|
rewards: null,
|
|
};
|
|
performAction.mockResolvedValue(combat);
|
|
|
|
const response = await request(app.getHttpServer())
|
|
.post('/api/combats/combat-1/actions')
|
|
.send({ action: 'ATTACK' })
|
|
.expect(201);
|
|
|
|
expect(performAction).toHaveBeenCalledWith(
|
|
DEMO_CHARACTER_ID,
|
|
'combat-1',
|
|
'ATTACK',
|
|
);
|
|
expect(response.body).toEqual(combat);
|
|
});
|
|
|
|
it('rejects an unknown action value', async () => {
|
|
await request(app.getHttpServer())
|
|
.post('/api/combats/combat-1/actions')
|
|
.send({ action: 'FLEE' })
|
|
.expect(400);
|
|
|
|
expect(performAction).not.toHaveBeenCalled();
|
|
});
|
|
|
|
it('rejects server-owned combat fields the client must never send', async () => {
|
|
await request(app.getHttpServer())
|
|
.post('/api/combats/combat-1/actions')
|
|
.send({
|
|
action: 'ATTACK',
|
|
damage: 999,
|
|
playerHp: 1,
|
|
monsterHp: 1,
|
|
round: 99,
|
|
})
|
|
.expect(400);
|
|
|
|
expect(performAction).not.toHaveBeenCalled();
|
|
});
|
|
});
|