vliaudatandClaude Opus 5
5f7aa951d3
Two things the shop asked for, and one it will notice. **A message on an exception.** A closure or a late opening can now carry a reason, and it reaches the panel. It shows while the door is shut and disappears the moment the shop opens — a notice explaining a late opening is worse than useless once the door is open. It outranks a free message on purpose: "closed this afternoon, back tomorrow" is what someone standing outside needs, and "new collection in store" can wait. Say so if you would rather it were the other way. The first version keyed this off the day's `isOpen`, which means "this day has opening hours" — so a day opening at 14:00 counted as open all morning, exactly when the reason is needed. A test caught it; the rule now reads the state at this minute. **Reusable phrases.** The same handful of notices get written over and over. /admin/modeles keeps them, translated once, and offers them wherever a message is composed — exceptions, closure periods, the banner. Picking one costs no translation at all: the saved English is reused directly, where the service takes the better part of ten seconds. Exception notes and holiday labels are translated too, which they were not before. After the save rather than during it, so a slow service never costs the shop its dates. **Previewing the future.** The dashboard can render the panel at any moment within about a year: "what will the window say while I'm away?" is worth answering before someone is standing in front of a locked door. The whole pipeline was already a function of "now", so this costs passing a different instant. An unparseable or absurd value falls back to the present rather than confidently rendering nonsense. Checked against a three-week holiday, which surfaced something worth knowing: the automatic "opens on…" line stays empty, because the resolver's search is bounded to fourteen days. During a long closure the message is the only thing that tells customers when the shop is back — which is a good reason for this feature to exist. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_012cSY9pVhZmJUKNN7wf1Myd
48 lines
1.8 KiB
TypeScript
48 lines
1.8 KiB
TypeScript
import { describe, expect, it } from 'vitest';
|
|
|
|
import { parsePreviewTime, PREVIEW_RANGE_DAYS } from './preview-time';
|
|
|
|
const NOW = new Date('2026-09-23T08:00:00Z');
|
|
|
|
describe('parsePreviewTime', () => {
|
|
it('defaults to now when nothing is asked for', () => {
|
|
expect(parsePreviewTime(null, NOW)).toEqual({ at: NOW, isNow: true });
|
|
});
|
|
|
|
it('accepts a moment in the future', () => {
|
|
const result = parsePreviewTime('2026-09-24T08:00:00Z', NOW);
|
|
expect(result.at.toISOString()).toBe('2026-09-24T08:00:00.000Z');
|
|
expect(result.isNow).toBe(false);
|
|
});
|
|
|
|
it('accepts a local datetime as a browser sends it', () => {
|
|
// <input type="datetime-local"> has no timezone; it is read as local time.
|
|
expect(parsePreviewTime('2026-12-24T10:00', NOW).at.getFullYear()).toBe(2026);
|
|
});
|
|
|
|
it('accepts a moment in the past', () => {
|
|
expect(parsePreviewTime('2026-09-20T08:00:00Z', NOW).isNow).toBe(false);
|
|
});
|
|
|
|
it('falls back to now on an unparseable value', () => {
|
|
// Better a correct screen than a confident rendering of nonsense.
|
|
expect(parsePreviewTime('demain matin', NOW)).toEqual({ at: NOW, isNow: true });
|
|
expect(parsePreviewTime('', NOW)).toEqual({ at: NOW, isNow: true });
|
|
});
|
|
|
|
it('refuses a moment beyond the planning horizon', () => {
|
|
const far = new Date(NOW.getTime() + (PREVIEW_RANGE_DAYS + 1) * 86_400_000);
|
|
expect(parsePreviewTime(far.toISOString(), NOW).isNow).toBe(true);
|
|
});
|
|
|
|
it('accepts a moment just inside the horizon', () => {
|
|
const edge = new Date(NOW.getTime() + (PREVIEW_RANGE_DAYS - 1) * 86_400_000);
|
|
expect(parsePreviewTime(edge.toISOString(), NOW).isNow).toBe(false);
|
|
});
|
|
|
|
it('treats a moment within the minute as now', () => {
|
|
const almost = new Date(NOW.getTime() + 30_000);
|
|
expect(parsePreviewTime(almost.toISOString(), NOW).isNow).toBe(true);
|
|
});
|
|
});
|