learning-output-style 플러그인 2회차

요청 1: 함수 두 개, 규칙을 다 정해 준 요청

요청문: src/stock.mjs에 두 함수를 추가해 줘. expiringSoon(items, today)는 유통기한이 오늘부터 3일 뒤까지(오늘 포함)인 재료 이름 목록을, expired(items, today)는 유통기한이 이미 지난 재료 이름 목록을 돌려줘. today는 'YYYY-MM-DD' 문자열이야. 순서는 들어온 순서 그대로. test/stock.test.mjs에 테스트도 추가하고 돌려 봐.

클로드의 마지막 답변 (그대로)

두 함수를 추가했고, 테스트를 돌려 보니 기존 테스트와 새 테스트가 모두 통과했습니다. **추가한 함수** (`src/stock.mjs`) - `expiringSoon(items, today)`: 유통기한이 오늘부터 3일 뒤까지(오늘 포함)인 재료 이름을 돌려줍니다. - `expired(items, today)`: 유통기한이 이미 지난 재료 이름을 돌려줍니다. - 두 함수 모두 기존 `daysBetween`으로 날짜 차이를 계산하고, `filter → map` 구조라서 들어온 순서가 그대로 유지됩니다. **추가한 테스트** (`test/stock.test.mjs`) - **오늘 기준 경계:** 어제(만료), 오늘(임박), 3일 뒤(임박), 4일 뒤(해당 없음) - **달력 경계:** 월·연도가 바뀌는 경우(12/31 → 1/2, 1/1 기준 12/31) - **빈 배열:** 두 함수 모두 빈 목록을 돌려주는지 - **순서 유지:** 일부러 날짜 순서를 섞어 넣고, 입력 순서대로 나오는지 `★ Insight ─────────────────────────────────────` - 경계 조건은 범위 안쪽 끝(3일)과 바로 바깥(4일)을 짝으로 테스트해야 `<=`를 `<`로 잘못 쓰는 실수(off-by-one)를 잡을 수 있습니다. - 날짜 계산을 `daysBetween` 한 곳에서만 하므로, 나중에 시간대 처리를 바꿔도 함수 하나만 고치면 세 함수에 모두 반영됩니다. `─────────────────────────────────────────────────`

바뀐 코드 (git diff)

diff --git a/src/stock.mjs b/src/stock.mjs index f81b897..77aaa20 100644 --- a/src/stock.mjs +++ b/src/stock.mjs @@ -19,3 +19,16 @@ export function orderList(items) { .filter((it) => it.qty < it.minQty) .map((it) => ({ name: it.name, need: it.minQty * 2 - it.qty, unit: it.unit })); } + +export function expiringSoon(items, today) { + return items + .filter((it) => { + const d = daysBetween(today, it.expires); + return d >= 0 && d <= 3; + }) + .map((it) => it.name); +} + +export function expired(items, today) { + return items.filter((it) => daysBetween(today, it.expires) < 0).map((it) => it.name); +} diff --git a/test/stock.test.mjs b/test/stock.test.mjs index 71de632..f595052 100644 --- a/test/stock.test.mjs +++ b/test/stock.test.mjs @@ -1,5 +1,5 @@ import assert from 'node:assert/strict'; -import { lowStock, orderList, daysBetween } from '../src/stock.mjs'; +import { lowStock, orderList, daysBetween, expiringSoon, expired } from '../src/stock.mjs'; const items = [ { name: '우유', qty: 3, unit: 'L', expires: '2026-10-20', minQty: 5 }, { name: '원두', qty: 4, unit: 'kg', expires: '2027-01-10', minQty: 2 }, @@ -8,3 +8,20 @@ assert.deepEqual(lowStock(items), ['우유']); assert.deepEqual(orderList(items), [{ name: '우유', need: 7, unit: 'L' }]); assert.equal(daysBetween('2026-10-18', '2026-10-20'), 2); console.log('기본 테스트 통과'); + +const dated = [ + { name: '생크림', qty: 1, unit: 'L', expires: '2026-10-17', minQty: 1 }, // 어제 → 만료 + { name: '우유', qty: 3, unit: 'L', expires: '2026-10-18', minQty: 5 }, // 오늘 → 임박 + { name: '시럽', qty: 2, unit: '병', expires: '2026-10-25', minQty: 1 }, // 7일 뒤 → 해당 없음 + { name: '치즈', qty: 1, unit: 'kg', expires: '2026-10-21', minQty: 1 }, // 3일 뒤 → 임박(경계) + { name: '버터', qty: 1, unit: 'kg', expires: '2026-10-22', minQty: 1 }, // 4일 뒤 → 해당 없음(경계) + { name: '요거트', qty: 1, unit: 'L', expires: '2026-09-30', minQty: 1 }, // 지난달 → 만료 +]; +assert.deepEqual(expiringSoon(dated, '2026-10-18'), ['우유', '치즈']); +assert.deepEqual(expired(dated, '2026-10-18'), ['생크림', '요거트']); +// 월·연도 경계 +assert.deepEqual(expiringSoon([{ name: 'A', expires: '2027-01-02' }], '2026-12-31'), ['A']); +assert.deepEqual(expired([{ name: 'B', expires: '2026-12-31' }], '2027-01-01'), ['B']); +assert.deepEqual(expiringSoon([], '2026-10-18'), []); +assert.deepEqual(expired([], '2026-10-18'), []); +console.log('유통기한 테스트 통과');