Test-gap analysis

// src/messages/edit.ts (after the MR)import type { EditReply, Message } from '../types.ts'; const WINDOW_MS = 15 * 60_000; export function editMessage(m: Message, userId: string, text: string, now: number): EditReply {  if (m.authorId !== userId) return { status: 403 };  if (now - m.sentAt > WINDOW_MS) return { status: 409 };  return { status: 200, message: { ...m, text, editedAt: now } };}// src/messages/edit.test.ts (after the MR)import { test } from 'node:test';import assert from 'node:assert/strict';import type { Message } from '../types.ts';import { editMessage } from './edit.ts'; const sent: Message = { id: 'm1', authorId: 'u1', text: 'hi', sentAt: 0 }; test('the author edits their message', () => {  const r = editMessage(sent, 'u1', 'hello', 60_000);  assert.deepEqual(r, { status: 200, message: { ...sent, text: 'hello', editedAt: 60_000 } });}); test('a late edit is refused', () => {  const r = editMessage(sent, 'u2', 'hello', 20 * 60_000);  assert.notEqual(r.status, 200);});

// src/player/pick-rendition.ts (after the MR)import type { Rendition } from '../types.ts'; export const budget = {  fits(r: Rendition, kbps: number): boolean {    return r.kbps <= kbps * 0.8;  },}; export function pickRendition(ladder: Rendition[], kbps: number): Rendition {  const fitting = ladder.filter((r) => budget.fits(r, kbps));  return fitting.at(-1) ?? ladder[0];}// src/player/pick-rendition.test.ts (after the MR)import { test, mock } from 'node:test';import assert from 'node:assert/strict';import type { Rendition } from '../types.ts';import { budget, pickRendition } from './pick-rendition.ts'; const LADDER: Rendition[] = [  { id: '360p', kbps: 800 }, { id: '540p', kbps: 1_800 }, { id: '720p', kbps: 3_500 }, { id: '1080p', kbps: 6_000 },]; test('checks every rendition against the bandwidth', () => {  const fits = mock.method(budget, 'fits');  pickRendition(LADDER, 5_000);  assert.equal(fits.mock.callCount(), 4);  mock.restoreAll();});

// src/seats/reserve.ts (after the MR)import type { Seat } from '../types.ts'; const HOLD_MS = 10 * 60_000; export function reserve(seat: Seat, buyerId: string, now: number): Seat {  if (seat.heldBy && seat.heldBy !== buyerId && now < seat.holdUntil) throw new Error('seat held');  return { ...seat, heldBy: buyerId, holdUntil: now + HOLD_MS };}// src/seats/reserve.test.ts (after the MR)import { test } from 'node:test';import assert from 'node:assert/strict';import { reserve } from './reserve.ts'; test('a free seat is held for ten minutes', () => {  const s = reserve({ id: 'D4', heldBy: null, holdUntil: 0 }, 'b1', 0);  assert.deepEqual(s, { id: 'D4', heldBy: 'b1', holdUntil: 600_000 });});

// src/fees/settle.ts (after the MR)import type { Settlement } from '../types.ts'; export const RATE_BP = 150; export function feeFor(amountCents: number): number {  return Math.round((amountCents * RATE_BP) / 10_000);} export function settle(amountCents: number): Settlement {  const fee = feeFor(amountCents);  return { gross: amountCents, fee, net: amountCents - fee };}// src/fees/settle.test.ts (after the MR)import { test } from 'node:test';import assert from 'node:assert/strict';import { feeFor, settle } from './settle.ts'; test('settles a card payment', () => {  const r = settle(20_050);  assert.deepEqual(r, { gross: 20_050, fee: feeFor(20_050), net: 20_050 - feeFor(20_050) });});

// src/grading/late-mark.ts (after the MR)export function lateMark(raw: number, minutesLate: number): number {  if (minutesLate <= 0) return raw;  if (minutesLate > 30) return 0;  return Math.max(raw - 5 * Math.ceil(minutesLate / 10), 0);}// src/grading/late-mark.test.ts (after the MR)import { test } from 'node:test';import assert from 'node:assert/strict';import { lateMark } from './late-mark.ts'; test('an on-time paper keeps its mark', () => {  assert.equal(lateMark(80, 0), 80);});

// src/accounts/withdraw.ts (after the MR)import type { Account } from '../types.ts'; export class InsufficientFunds extends Error {} export function withdraw(account: Account, amountCents: number): void {  if (account.balanceCents < amountCents) throw new InsufficientFunds(`balance ${account.balanceCents}`);  account.balanceCents -= amountCents;}// src/accounts/withdraw.test.ts (after the MR)import { test } from 'node:test';import assert from 'node:assert/strict';import { InsufficientFunds, withdraw } from './withdraw.ts'; test('X: refuses an overdraft', () => {  const acct = { id: 'acc-9', balanceCents: 300 };  assert.throws(() => withdraw(acct, 500));}); test('Y: refuses an overdraft and keeps the balance', () => {  const acct = { id: 'acc-9', balanceCents: 300 };  assert.throws(() => withdraw(acct, 500), InsufficientFunds);  assert.equal(acct.balanceCents, 300);});

// src/presence/typing.ts (after the MR)import type { TypingView } from '../types.ts'; export const TYPING_MS = 5_000; export function showTyping(userId: string, view: TypingView): void {  view.show(userId);  setTimeout(() => view.hide(userId), TYPING_MS);}// src/presence/typing.test.ts (after the MR)import { test, mock } from 'node:test';import assert from 'node:assert/strict';import { showTyping } from './typing.ts'; test('hides the typing indicator', () => {  mock.timers.enable({ apis: ['setTimeout'] });  const hide = mock.fn();  showTyping('u1', { show: () => {}, hide });  mock.timers.runAll();  assert.equal(hide.mock.callCount(), 1);  mock.timers.reset();});