From 4561245a71055de7cb28668f4638f175cce760e4 Mon Sep 17 00:00:00 2001 From: ouyangxq <775153912@qq.com> Date: Tue, 11 Aug 2026 15:11:50 +0800 Subject: [PATCH] test(admin): align loading state transition assertions --- .../phase3-container-loading.mjs | 22 ++++-- .../admin-test-env-modules/phase3-loading.mjs | 75 ++++++++++--------- .../phase3-producer-chain-contract.test.mjs | 16 ++++ 3 files changed, 71 insertions(+), 42 deletions(-) diff --git a/scripts/admin-test-env-modules/phase3-container-loading.mjs b/scripts/admin-test-env-modules/phase3-container-loading.mjs index bce4fc891..80c6d9922 100644 --- a/scripts/admin-test-env-modules/phase3-container-loading.mjs +++ b/scripts/admin-test-env-modules/phase3-container-loading.mjs @@ -100,7 +100,7 @@ export async function run(context) { const containerDetail = context.apiSuccess(MODULE, '回查货柜详情', containerDetailResponse, '', OPTIONS) context.assert(MODULE, '货柜详情保留候选货物路线约束', containerDetailResponse, containerDetail?.destinationWarehouseCode === (candidate.destinationWarehouseCode || null) - && containerDetail?.planStatus === 'EDITING', + && containerDetail?.planStatus === 'WAIT_LOADING', `planStatus=${containerDetail?.planStatus || 'missing'}`, OPTIONS) const planResponse = await request(context, 'POST', `${SMART}/plans`, { bookingId: containerId, remark: context.marker }) @@ -147,24 +147,24 @@ export async function run(context) { const pushResponse = await request(context, 'POST', `${SMART}/plans/${planId}/push-preload`, { expectedVersion: Number(assigned.version) }, { 'Idempotency-Key': pushKey }) const pushed = context.apiSuccess(MODULE, '推送预装并创建 PDA 任务', pushResponse, '', OPTIONS) - context.assert(MODULE, '推送后计划进入 LOADING 且版本递增', pushResponse, - pushed?.status === 'LOADING' && Number(pushed?.version) === Number(assigned.version) + 1, + context.assert(MODULE, '推送后计划保持 EDITING 且版本递增', pushResponse, + pushed?.status === 'EDITING' && Number(pushed?.version) === Number(assigned.version) + 1, `status=${pushed?.status || 'missing'}, version=${pushed?.version ?? 'missing'}`, OPTIONS) - if (pushed?.status !== 'LOADING') return + if (pushed?.status !== 'EDITING') return const pushReplayResponse = await request(context, 'POST', `${SMART}/plans/${planId}/push-preload`, { expectedVersion: Number(assigned.version) }, { 'Idempotency-Key': pushKey }) const pushReplay = context.apiSuccess(MODULE, '推送预装 Idempotency-Key 重放', pushReplayResponse, '', OPTIONS) - context.assert(MODULE, '推送重放保持同一 LOADING 计划', pushReplayResponse, - id(pushReplay?.id) === planId && pushReplay?.status === 'LOADING' + context.assert(MODULE, '推送重放保持同一 EDITING 计划', pushReplayResponse, + id(pushReplay?.id) === planId && pushReplay?.status === 'EDITING' && Number(pushReplay?.version) === Number(pushed.version), `planId=${id(pushReplay?.id) || 'missing'}, version=${pushReplay?.version ?? 'missing'}`, OPTIONS) const preloadPageResponse = await context.request('GET', `${FIRST_LEG}/loading-plans?warehouseId=${encodeURIComponent(warehouseId)}&keyword=${encodeURIComponent(containerNo)}&page=1&size=20`) const preloadPage = context.apiSuccess(MODULE, '查询本轮预装清单', preloadPageResponse, '', OPTIONS) - context.assert(MODULE, '预装清单包含本轮 LOADING 计划', preloadPageResponse, - records(preloadPage).some((item) => id(item.planId) === planId && item.status === 'LOADING'), + context.assert(MODULE, '预装清单包含本轮 EDITING 计划', preloadPageResponse, + records(preloadPage).some((item) => id(item.planId) === planId && item.status === 'EDITING'), `records=${records(preloadPage).length}`, OPTIONS) const taskResponse = await context.request('GET', @@ -193,6 +193,12 @@ export async function run(context) { && Number(claimed?.assignment?.version) === 0, `workflow=${claimed?.workflowStatus || 'missing'}, assigneeId=${id(claimed?.assignment?.assigneeId) || 'missing'}`, OPTIONS) + const claimedPlanResponse = await context.request('GET', `${SMART}/plans/${planId}`) + const claimedPlan = context.apiSuccess(MODULE, '领取后回查计划进入 LOADING', claimedPlanResponse, '', OPTIONS) + context.assert(MODULE, '领取后计划进入 LOADING 并记录版本递增', claimedPlanResponse, + claimedPlan?.status === 'LOADING' && Number(claimedPlan?.version) === Number(pushed?.version) + 1, + `status=${claimedPlan?.status || 'missing'}, version=${claimedPlan?.version ?? 'missing'}`, OPTIONS) + const batchPath = `${PDA}/tasks/${taskId}/batch` const batchItems = [{ itemId: id(pickItem.id), action: 'PICK', pickedPackageCount: 1, finalPickedPackageCount: null, releaseMode: 'MANUAL', locationIds: [], diff --git a/scripts/admin-test-env-modules/phase3-loading.mjs b/scripts/admin-test-env-modules/phase3-loading.mjs index 2b9650177..f8954fec8 100644 --- a/scripts/admin-test-env-modules/phase3-loading.mjs +++ b/scripts/admin-test-env-modules/phase3-loading.mjs @@ -169,47 +169,17 @@ export async function run(context) { { expectedVersion: Number(link.loadingPlanVersion) }, { 'Idempotency-Key': `${context.marker}-bridge-push`.slice(0, 128) }) const pushedLoadingPlan = context.apiSuccess(MODULE, '桥接后推送柜次计划并生成真实 PDA 任务', pushResponse, '', OPTIONS) - context.assert(MODULE, '关联柜次计划进入 LOADING', pushResponse, - pushedLoadingPlan?.status === 'LOADING', `status=${pushedLoadingPlan?.status || 'missing'}`, OPTIONS) - - const confirming = await request(context, 'POST', `${base}/preload/confirming?tenantId=${encodeURIComponent(tenantId)}`, - { expectedVersion: Number(link.preloadVersion) }, { 'Idempotency-Key': `${context.marker}-p3-confirming`.slice(0, 128) }) - const confirmingData = context.apiSuccess(MODULE, '提交预装清单确认', confirming, '', OPTIONS) - context.assert(MODULE, '预装清单进入 CONFIRMING 且版本递增', confirming, - confirmingData?.preloadPlan?.status === 'CONFIRMING' && Number(confirmingData?.preloadPlan?.version) === 2, - `status=${confirmingData?.preloadPlan?.status || 'missing'}, version=${confirmingData?.preloadPlan?.version ?? 'missing'}`, OPTIONS) - if (confirmingData?.preloadPlan?.status !== 'CONFIRMING') return - - const confirm = await request(context, 'POST', `${base}/preload/confirm?tenantId=${encodeURIComponent(tenantId)}`, - { expectedVersion: Number(confirmingData.preloadPlan.version) }, { 'Idempotency-Key': `${context.marker}-p3-confirm`.slice(0, 128) }) - const confirmed = context.apiSuccess(MODULE, '确认预装并进入装柜中', confirm, '', OPTIONS) - context.assert(MODULE, '预装确认推进主状态 LOADING 和计划 CONFIRMED', confirm, - confirmed?.mainStatus === 'LOADING' && confirmed?.preloadPlan?.status === 'CONFIRMED' - && Number(confirmed?.preloadPlan?.version) === 3, - `mainStatus=${confirmed?.mainStatus || 'missing'}, planStatus=${confirmed?.preloadPlan?.status || 'missing'}, version=${confirmed?.preloadPlan?.version ?? 'missing'}`, OPTIONS) - if (confirmed?.mainStatus !== 'LOADING') return + context.assert(MODULE, '推送后计划保持 EDITING 并生成真实 PDA 任务', pushResponse, + pushedLoadingPlan?.status === 'EDITING', `status=${pushedLoadingPlan?.status || 'missing'}`, OPTIONS) + if (pushedLoadingPlan?.status !== 'EDITING') return const preload = await context.request('GET', `${base}/preload?tenantId=${encodeURIComponent(tenantId)}`) - const preloadData = context.apiSuccess(MODULE, '回查已确认预装清单', preload, '', OPTIONS) + const preloadData = context.apiSuccess(MODULE, '回查预装清单', preload, '', OPTIONS) context.assert(MODULE, '预装清单归属、明细和 marker 持久化', preload, id(preloadData?.id) === planId && id(preloadData?.inboundId) === inboundId && id(preloadData?.tenantId) === tenantId && preloadData?.items?.[0]?.containerCode === `P3-${suffix}` && preloadData?.remark === context.marker, `planId=${id(preloadData?.id) || 'missing'}, items=${preloadData?.items?.length ?? 'n/a'}`, OPTIONS) - const loadingResponse = await context.request('GET', detailPath) - const loading = context.apiSuccess(MODULE, '读取装柜中入库单版本', loadingResponse, '', OPTIONS) - const loadingVersion = Number(loading?.summary?.version) - context.assert(MODULE, '入库单已进入 LOADING 且版本有效', loadingResponse, - loading?.summary?.processStatus === 'LOADING' && Number.isInteger(loadingVersion), - `status=${loading?.summary?.processStatus || 'missing'}, version=${loading?.summary?.version ?? 'missing'}`, OPTIONS) - if (!Number.isInteger(loadingVersion)) return - - const blockedBeforePicking = await request(context, 'POST', `${base}/loading/complete?tenantId=${encodeURIComponent(tenantId)}`, - { expectedVersion: loadingVersion, remark: 'must require picking' }, - { 'Idempotency-Key': `${context.marker}-p3-complete-before-picking`.slice(0, 128) }) - context.addStep(MODULE, '未完成 PDA 拣货时拒绝完成装柜', blockedBeforePicking, failure(blockedBeforePicking), - `HTTP=${blockedBeforePicking.http}, code=${codeOf(blockedBeforePicking)}`, OPTIONS) - const canonicalTaskResponse = await context.request('GET', `${pdaBase}/tasks?containerNo=${encodeURIComponent(containerNo)}&warehouseId=${encodeURIComponent(warehouseId)}`) const canonicalTask = context.apiSuccess(MODULE, '读取柜次级 PDA 唯一事实任务', canonicalTaskResponse, '', OPTIONS) @@ -229,6 +199,43 @@ export async function run(context) { claimed?.workflowStatus === 'IN_PROGRESS' && Boolean(claimed?.assignment?.assigneeId), `workflow=${claimed?.workflowStatus || 'missing'}`, OPTIONS) + const claimedPlanResponse = await request(context, 'GET', `${smartLoadingBase}/plans/${loadingPlanId}`) + const claimedPlan = context.apiSuccess(MODULE, '领取后回查柜次计划进入 LOADING', claimedPlanResponse, '', OPTIONS) + context.assert(MODULE, '领取后计划进入 LOADING', claimedPlanResponse, + claimedPlan?.status === 'LOADING', `status=${claimedPlan?.status || 'missing'}`, OPTIONS) + if (claimedPlan?.status !== 'LOADING') return + + const confirming = await request(context, 'POST', `${base}/preload/confirming?tenantId=${encodeURIComponent(tenantId)}`, + { expectedVersion: Number(link.preloadVersion) }, { 'Idempotency-Key': `${context.marker}-p3-confirming`.slice(0, 128) }) + const confirmingData = context.apiSuccess(MODULE, '提交预装清单确认', confirming, '', OPTIONS) + context.assert(MODULE, '预装清单进入 CONFIRMING 且版本递增', confirming, + confirmingData?.preloadPlan?.status === 'CONFIRMING' && Number(confirmingData?.preloadPlan?.version) === 2, + `status=${confirmingData?.preloadPlan?.status || 'missing'}, version=${confirmingData?.preloadPlan?.version ?? 'missing'}`, OPTIONS) + if (confirmingData?.preloadPlan?.status !== 'CONFIRMING') return + + const confirm = await request(context, 'POST', `${base}/preload/confirm?tenantId=${encodeURIComponent(tenantId)}`, + { expectedVersion: Number(confirmingData.preloadPlan.version) }, { 'Idempotency-Key': `${context.marker}-p3-confirm`.slice(0, 128) }) + const confirmed = context.apiSuccess(MODULE, '确认预装并进入装柜中', confirm, '', OPTIONS) + context.assert(MODULE, '预装确认推进主状态 LOADING 和计划 CONFIRMED', confirm, + confirmed?.mainStatus === 'LOADING' && confirmed?.preloadPlan?.status === 'CONFIRMED' + && Number(confirmed?.preloadPlan?.version) === 3, + `mainStatus=${confirmed?.mainStatus || 'missing'}, planStatus=${confirmed?.preloadPlan?.status || 'missing'}, version=${confirmed?.preloadPlan?.version ?? 'missing'}`, OPTIONS) + if (confirmed?.mainStatus !== 'LOADING') return + + const loadingResponse = await context.request('GET', detailPath) + const loading = context.apiSuccess(MODULE, '读取装柜中入库单版本', loadingResponse, '', OPTIONS) + const loadingVersion = Number(loading?.summary?.version) + context.assert(MODULE, '入库单已进入 LOADING 且版本有效', loadingResponse, + loading?.summary?.processStatus === 'LOADING' && Number.isInteger(loadingVersion), + `status=${loading?.summary?.processStatus || 'missing'}, version=${loading?.summary?.version ?? 'missing'}`, OPTIONS) + if (!Number.isInteger(loadingVersion)) return + + const blockedBeforePicking = await request(context, 'POST', `${base}/loading/complete?tenantId=${encodeURIComponent(tenantId)}`, + { expectedVersion: loadingVersion, remark: 'must require picking' }, + { 'Idempotency-Key': `${context.marker}-p3-complete-before-picking`.slice(0, 128) }) + context.addStep(MODULE, '未完成 PDA 拣货时拒绝完成装柜', blockedBeforePicking, failure(blockedBeforePicking), + `HTTP=${blockedBeforePicking.http}, code=${codeOf(blockedBeforePicking)}`, OPTIONS) + const batchPath = `${pdaBase}/tasks/${canonicalTaskId}/batch` const batchKey = `${context.marker}-bridge-batch`.slice(0, 128) const batchPayload = { diff --git a/scripts/admin-test-env-modules/phase3-producer-chain-contract.test.mjs b/scripts/admin-test-env-modules/phase3-producer-chain-contract.test.mjs index e2772146b..ced7a193c 100644 --- a/scripts/admin-test-env-modules/phase3-producer-chain-contract.test.mjs +++ b/scripts/admin-test-env-modules/phase3-producer-chain-contract.test.mjs @@ -1,4 +1,5 @@ import assert from 'node:assert/strict' +import fs from 'node:fs' import test from 'node:test' import { selectCustomsFlow } from './phase3-customs.mjs' @@ -50,3 +51,18 @@ test('records LOADED and clears the reason only after a real LOADED reread', () assert.equal(result.finalizeSucceeded, true) assert.equal(result.notCleanedReason, null) }) + +test('keeps push and claim state transitions in the verified order', () => { + const containerScript = fs.readFileSync(new URL('./phase3-container-loading.mjs', import.meta.url), 'utf8') + const loadingScript = fs.readFileSync(new URL('./phase3-loading.mjs', import.meta.url), 'utf8') + + assert.match(containerScript, /planStatus === 'WAIT_LOADING'/) + assert.match(containerScript, /pushed\?\.status === 'EDITING'/) + assert.match(containerScript, /claimedPlan\?\.status === 'LOADING'/) + assert.ok(containerScript.indexOf("pushed?.status === 'EDITING'") < containerScript.indexOf("claimedPlan?.status === 'LOADING'")) + + assert.match(loadingScript, /pushedLoadingPlan\?\.status === 'EDITING'/) + assert.match(loadingScript, /claimedPlan\?\.status === 'LOADING'/) + assert.ok(loadingScript.indexOf("pushedLoadingPlan?.status === 'EDITING'") < loadingScript.indexOf("claimedPlan?.status === 'LOADING'")) + assert.ok(loadingScript.indexOf("claimedPlan?.status === 'LOADING'") < loadingScript.indexOf("const confirming =")) +})