test(issue-update): 覆蓋 Milestone 解析、日期格式與估算的就地更新

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
2026-09-17 06:06:06 +00:00
parent 45bfd99d80
commit 3f97fe0901
1 changed files with 191 additions and 0 deletions
@@ -0,0 +1,191 @@
+/**
+ * 掛 Milestone、寫截止日、記人天估算。
+ * 三件事都經同一個 PATCH 送出，因為它們常常要一起改，分開發等於多兩次往返。
+ */
+import test from 'node:test';
+import assert from 'node:assert/strict';
+import { runScript } from './helpers/run-script.js';
+import { healthyRoutes, stubEnv as envFor, withStubGitea } from './helpers/stub-gitea.js';
+
+const REPO = 'plugins/tea-sdlc';
+const INDEX = 12;
+const BODY = '## 這個工作包在做什麼\n\n一句話。\n\n## 關聯\n\n需求議題：#1\n';
+
+function routes(overrides = {}, { body = BODY } = {}) {
+  return healthyRoutes(REPO, {
+    [`GET /api/v1/repos/${REPO}/issues/${INDEX}`]: {
+      status: 200,
+      body: { number: INDEX, title: '領取工作包並備妥分支', body, html_url: 'https://example.com/12' },
+    },
+    [`GET /api/v1/repos/${REPO}/milestones`]: {
+      status: 200,
+      body: [
+        { id: 3, title: '第一階段' },
+        { id: 4, title: '第二階段' },
+      ],
+    },
+    [`PATCH /api/v1/repos/${REPO}/issues/${INDEX}`]: (req) => ({
+      status: 200,
+      body: { number: INDEX, ...req.body },
+    }),
+    ...overrides,
+  });
+}
+
+const withStub = (t, overrides = {}, options) => withStubGitea(t, routes(overrides, options));
+
+const run = (args, stub) =>
+  runScript('issue-update.js', ['--repo', REPO, '--index', String(INDEX), ...args], {
+    env: envFor(stub),
+  });
+
+const patchOf = (stub) => stub.requests.find((r) => r.method === 'PATCH' && !r.path.endsWith('/0'));
+
+// ── Milestone ─────────────────────────────────────────────────────
+
+test('以名稱指定 Milestone，送出時換成 id', async (t) => {
+  const stub = await withStub(t);
+
+  const { code, json } = await run(['--milestone', '第二階段'], stub);
+
+  assert.equal(code, 0, JSON.stringify(json));
+  assert.equal(patchOf(stub).body.milestone, 4);
+});
+
+test('Milestone 不存在時中止，並列出可選的', async (t) => {
+  const stub = await withStub(t);
+
+  const { code, json } = await run(['--milestone', '第三階段'], stub);
+
+  assert.equal(code, 1);
+  assert.equal(json.error.code, 'UNKNOWN_MILESTONE');
+  assert.match(json.error.message, /第三階段/);
+  assert.match(json.error.message, /第一階段/, '要列出可選的');
+  assert.equal(stub.requests.some((r) => r.method === 'PATCH' && !r.path.endsWith('/0')), false);
+});
+
+test('不建立 Milestone：沒有任何請求寫到 milestones 端點', async (t) => {
+  const stub = await withStub(t);
+
+  await run(['--milestone', '第一階段'], stub);
+
+  const writes = stub.requests.filter((r) => r.path.endsWith('/milestones') && r.method !== 'GET');
+  assert.deepEqual(writes, []);
+});
+
+// ── 截止日 ─────────────────────────────────────────────────────────
+
+test('截止日以 RFC3339 送出', async (t) => {
+  const stub = await withStub(t);
+
+  await run(['--due-date', '2026-09-24'], stub);
+
+  assert.equal(patchOf(stub).body.due_date, '2026-09-24T00:00:00Z');
+});
+
+test('截止日格式不對時擋在打 Gitea 之前', async (t) => {
+  const stub = await withStub(t);
+
+  const { json } = await run(['--due-date', '2026/09/24'], stub);
+
+  assert.equal(json.error.code, 'BAD_DUE_DATE');
+  assert.equal(stub.requests.length, 0);
+});
+
+// ── 人天估算 ───────────────────────────────────────────────────────
+
+test('人天估算寫成議題 body 裡人類可讀的一行', async (t) => {
+  const stub = await withStub(t);
+
+  await run(['--estimate-days', '3'], stub);
+
+  assert.match(patchOf(stub).body.body, /^估算人天：3$/m);
+});
+
+test('估算寫進關聯段落，不是隨便黏在檔尾', async (t) => {
+  const stub = await withStub(t);
+
+  await run(['--estimate-days', '3'], stub);
+
+  const updated = patchOf(stub).body.body;
+  const relation = updated.slice(updated.indexOf('## 關聯'));
+  assert.match(relation, /需求議題：#1/);
+  assert.match(relation, /估算人天：3/);
+});
+
+test('重跑改估算時就地更新那一行，不會累積成兩行', async (t) => {
+  const stub = await withStub(t, {}, {
+    body: '## 關聯\n\n需求議題：#1\n估算人天：3\n',
+  });
+
+  await run(['--estimate-days', '5'], stub);
+
+  const updated = patchOf(stub).body.body;
+  assert.equal((updated.match(/估算人天：/g) ?? []).length, 1);
+  assert.match(updated, /估算人天：5/);
+});
+
+test('估算沒有變時不重寫 body', async (t) => {
+  const stub = await withStub(t, {}, {
+    body: '## 關聯\n\n需求議題：#1\n估算人天：3\n',
+  });
+
+  await run(['--estimate-days', '3'], stub);
+
+  assert.equal('body' in patchOf(stub).body, false, '沒變就不該把 body 塞進 PATCH');
+});
+
+test('人天必須是正數', async (t) => {
+  const stub = await withStub(t);
+
+  const { json } = await run(['--estimate-days', '0'], stub);
+
+  assert.equal(json.error.code, 'BAD_ESTIMATE');
+  assert.equal(stub.requests.length, 0);
+});
+
+test('議題沒有關聯段落時，估算補在 body 結尾而不是靜靜消失', async (t) => {
+  const stub = await withStub(t, {}, { body: '## 描述\n\n沒有關聯段落的議題。\n' });
+
+  await run(['--estimate-days', '2'], stub);
+
+  assert.match(patchOf(stub).body.body, /估算人天：2/);
+});
+
+// ── 一起送 ─────────────────────────────────────────────────────────
+
+test('三件事一次 PATCH 送出', async (t) => {
+  const stub = await withStub(t);
+
+  await run(['--milestone', '第一階段', '--due-date', '2026-09-24', '--estimate-days', '2'], stub);
+
+  const patches = stub.requests.filter((r) => r.method === 'PATCH' && !r.path.endsWith('/0'));
+  assert.equal(patches.length, 1);
+  assert.equal(patches[0].body.milestone, 3);
+  assert.equal(patches[0].body.due_date, '2026-09-24T00:00:00Z');
+  assert.match(patches[0].body.body, /估算人天：2/);
+});
+
+test('什麼都沒指定時中止，不發空的 PATCH', async (t) => {
+  const stub = await withStub(t);
+
+  const { code, json } = await run([], stub);
+
+  assert.equal(code, 1);
+  assert.equal(json.error.code, 'NOTHING_TO_UPDATE');
+  assert.equal(stub.requests.length, 0);
+});
+
+// ── --dry-run ─────────────────────────────────────────────────────
+
+test('--dry-run 列出將送出的 PATCH，且不寫入', async (t) => {
+  const stub = await withStub(t);
+
+  const { code, json } = await run(['--milestone', '第一階段', '--due-date', '2026-09-24', '--dry-run'], stub);
+
+  assert.equal(code, 0);
+  assert.equal(json.data.dryRun, true);
+  assert.equal(json.data.requests.length, 1);
+  assert.equal(json.data.requests[0].body.milestone, 3, '預覽要看得出 Milestone 已解析成 id');
+  assert.equal(stub.requests.some((r) => r.method !== 'GET'), false);
+});