DrSmoothl
2b3d1c67c2
Reimplements PR #86 independently (author abandoned it after change-requests), with the review feedback applied — the original tests asserted behavior the code never had. #84 — add @snowluma/common unit tests: - hex: toHex/toHexUpper/hexPreview boundary + fromHex round-trip & odd-length throw - log-summary: summarizeParams (top-level primitive is String()'d WITHOUT quotes; fields quoted; nested collapsed; total cap) and renderParamsVerbose (redaction, long-string marker is the Unicode ellipsis '…<N B>' not ASCII '...', circular guard) - request-context: nextRequestId monotonic; runWithRequestId sync/async + nested scopes - root scripts test:common/test:all/coverage; coverage/ gitignored #85 — Utf8Validator: the second surrogate check was unreachable (within the 0x80–0xBF continuation-byte guard, 'b & 0x20' is true iff 'b >= 0xA0'); merge to one check. No behavior change. common suite green (26 tests).
35 lines
1.1 KiB
TypeScript
35 lines
1.1 KiB
TypeScript
import { describe, expect, it } from 'vitest';
|
|
import { currentRequestId, nextRequestId, runWithRequestId } from '../src/request-context';
|
|
|
|
describe('request-context', () => {
|
|
it('nextRequestId increments monotonically and stays a positive integer', () => {
|
|
const a = nextRequestId();
|
|
const b = nextRequestId();
|
|
expect(a).toBeGreaterThan(0);
|
|
expect(b).toBe(a + 1);
|
|
});
|
|
|
|
it('runWithRequestId binds the id across sync + async, with nested scopes restoring', async () => {
|
|
expect(currentRequestId()).toBeUndefined();
|
|
|
|
runWithRequestId(42, () => {
|
|
expect(currentRequestId()).toBe(42);
|
|
runWithRequestId(99, () => {
|
|
expect(currentRequestId()).toBe(99);
|
|
});
|
|
expect(currentRequestId()).toBe(42); // outer scope restored after the nested one
|
|
});
|
|
|
|
expect(currentRequestId()).toBeUndefined(); // cleared outside any scope
|
|
|
|
await runWithRequestId(7, async () => {
|
|
await Promise.resolve();
|
|
expect(currentRequestId()).toBe(7); // survives awaits in the same async chain
|
|
});
|
|
});
|
|
|
|
it('returns the callback result', () => {
|
|
expect(runWithRequestId(1, () => 'value')).toBe('value');
|
|
});
|
|
});
|