f54b0627e1
The internal one-shot route already called the SAME production runSnapshot with
the SAME default dependencies — but it passed no trigger, and runSnapshot
defaults an absent trigger to SCHEDULED. So every operator-invoked run was
recorded as though the cron had fired it. That was a lie about provenance,
present by omission, and it would have contaminated the trace of any forced
diagnostic run.
CONTROLLED_FORCED is now its own trigger. Both internal routes (`/snapshot/:sport`
and `/snapshot/all`) stamp it, along with the process generation. The scheduler
still stamps SCHEDULED, and a test asserts neither internal route can label
itself scheduled.
Trace retention moves from "scheduled only" to a named allow-list of SCHEDULED +
CONTROLLED_FORCED. INTRADAY is still refused — it runs every ~20 minutes and
would displace scheduled evidence, which is the failure the store exists to
prevent. MANUAL_API stays refused too.
THE PIPELINE IS UNTOUCHED. snapshotService, gradeSlateService, retentionService,
snapshotScheduler, oddsService and eventIdentity are all UNCHANGED. A test
asserts the route injects no dependency override — no getOdds, gradeAndCacheSlate,
retention, ledger, cacheSet/cacheGet, gameBinder, eventIdentity, mlbAdapter or
notify — so the only difference from a scheduled invocation is the label and the
absence of a scheduled hour, which a forced run genuinely does not have.
The ?limit bisect-hook invariant is preserved and tightened: the opts passed
carry exactly {trigger, processStartedAt} and never a stray limit.
Teeth, injections verified present, against a green baseline:
:sport route mislabelled SCHEDULED -> 3 fail
/all route mislabelled SCHEDULED -> 2 fail
intraday admitted to the store -> 5 fail
trigger filter removed -> 3 fail
THE FIRST TEETH RUN WAS INVALID AND IS DISCARDED: both routes live in one file,
so a single-occurrence replace hit `/snapshot/all` and left `/snapshot/:sport`
correct — the injection landed on the wrong target and the suite passed. Coverage
for `/all` was added, plus a test that the file contains exactly two
CONTROLLED_FORCED stamps and zero SCHEDULED ones, then both were re-run failing
independently.
Four stale assertions updated with the reason recorded: three pinned the
`not_scheduled` refusal string (now trigger-agnostic) and one pinned an empty
opts object on the route.
389 suites / 5,280 tests pass. web tsc exit 0. Lineage stays OFF.
Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01CQJeAG8vcDoL5zkiaJyVb8
107 lines
4.2 KiB
JavaScript
107 lines
4.2 KiB
JavaScript
// Session 45 — internal snapshot endpoints (auth-gated) + the public ticker.
|
|
|
|
const express = require('express');
|
|
const request = require('supertest');
|
|
|
|
jest.mock('../../scripts/tank01-prefetch', () => ({ main: jest.fn() }));
|
|
jest.mock('../../src/services/quotaTracker', () => ({ getAllQuotaStatuses: jest.fn() }));
|
|
jest.mock('../../src/services/snapshotService', () => ({
|
|
runSnapshot: jest.fn(async (sport) => ({ sport, status: 'ok', gradeCount: 3 })),
|
|
runAllSnapshots: jest.fn(async () => [{ sport: 'mlb', status: 'ok' }, { sport: 'nba', status: 'skipped' }]),
|
|
}));
|
|
const snapshot = require('../../src/services/snapshotService');
|
|
|
|
beforeEach(() => {
|
|
jest.clearAllMocks();
|
|
process.env.VYNDR_INTERNAL_KEY = 'test-key-123';
|
|
});
|
|
|
|
function mountInternal() {
|
|
delete require.cache[require.resolve('../../src/routes/internal')];
|
|
const internalRoutes = require('../../src/routes/internal');
|
|
const app = express();
|
|
app.use(express.json());
|
|
app.use('/api/internal', internalRoutes);
|
|
return app;
|
|
}
|
|
|
|
describe('POST /api/internal/snapshot/:sport', () => {
|
|
it('rejects without the internal key (401)', async () => {
|
|
const res = await request(mountInternal()).post('/api/internal/snapshot/mlb').send({});
|
|
expect(res.status).toBe(401);
|
|
expect(snapshot.runSnapshot).not.toHaveBeenCalled();
|
|
});
|
|
|
|
it('runs a single-sport snapshot with the key', async () => {
|
|
const res = await request(mountInternal())
|
|
.post('/api/internal/snapshot/mlb')
|
|
.set('x-internal-key', 'test-key-123')
|
|
.send({});
|
|
expect(res.status).toBe(200);
|
|
// Opts object added 2026-08-01 for the ?limit= bisect hook. With no ?limit
|
|
// there must be NO limit key — a stray one would silently cap production
|
|
// runs, which is the exact bug the hook exists to diagnose. The opts now
|
|
// also carry truthful provenance (an operator run is CONTROLLED_FORCED, not
|
|
// SCHEDULED); those two diagnostic keys are the only permitted additions.
|
|
expect(snapshot.runSnapshot).toHaveBeenCalledWith('mlb', expect.objectContaining({
|
|
trigger: 'CONTROLLED_FORCED',
|
|
}));
|
|
const passedOpts = snapshot.runSnapshot.mock.calls[0][1];
|
|
expect(Object.keys(passedOpts).sort()).toEqual(['processStartedAt', 'trigger']);
|
|
expect(passedOpts).not.toHaveProperty('limit');
|
|
expect(res.body.summary.gradeCount).toBe(3);
|
|
});
|
|
|
|
it('routes /snapshot/all to runAllSnapshots (not captured as a sport)', async () => {
|
|
const res = await request(mountInternal())
|
|
.post('/api/internal/snapshot/all')
|
|
.set('x-internal-key', 'test-key-123')
|
|
.send({});
|
|
expect(res.status).toBe(200);
|
|
expect(snapshot.runAllSnapshots).toHaveBeenCalled();
|
|
expect(snapshot.runSnapshot).not.toHaveBeenCalled();
|
|
expect(res.body.results).toHaveLength(2);
|
|
});
|
|
});
|
|
|
|
describe('GET /api/ticker', () => {
|
|
const mockCache = { value: null };
|
|
jest.mock('../../src/utils/redis', () => ({
|
|
cacheGet: async () => mockCache.value,
|
|
cacheSet: async () => true,
|
|
getRedisClient: () => ({}),
|
|
isDegraded: () => false,
|
|
}));
|
|
|
|
function mountTicker() {
|
|
delete require.cache[require.resolve('../../src/routes/ticker')];
|
|
const tickerRoutes = require('../../src/routes/ticker');
|
|
const app = express();
|
|
app.use('/api/ticker', tickerRoutes);
|
|
return app;
|
|
}
|
|
|
|
it('returns snapshot items newest-first', async () => {
|
|
mockCache.value = [{ tag: 'READ', text: 'MLB slate read · 248 props graded' }, { tag: 'A+', text: 'BOMBER Judge graded A+' }];
|
|
delete process.env.TICKER_MANUAL;
|
|
const res = await request(mountTicker()).get('/api/ticker');
|
|
expect(res.status).toBe(200);
|
|
expect(res.body.items[0].tag).toBe('READ');
|
|
});
|
|
|
|
it('merges editorial pins from TICKER_MANUAL', async () => {
|
|
mockCache.value = [{ tag: 'READ', text: 'read' }];
|
|
process.env.TICKER_MANUAL = JSON.stringify([{ tag: 'ALERT', text: 'VYNDR 2.0 is live.' }]);
|
|
const res = await request(mountTicker()).get('/api/ticker');
|
|
expect(res.body.items.find((i) => i.tag === 'ALERT')).toBeTruthy();
|
|
delete process.env.TICKER_MANUAL;
|
|
});
|
|
|
|
it('returns [] gracefully on a cold cache', async () => {
|
|
mockCache.value = null;
|
|
delete process.env.TICKER_MANUAL;
|
|
const res = await request(mountTicker()).get('/api/ticker');
|
|
expect(res.body.items).toEqual([]);
|
|
});
|
|
});
|