feat: gate PDA loading evidence by ready task

tests/picking-evidence-stage.test.mjs

@@ -49,8 +49,10 @@ test('picking and evidence are separate operator-selected stages', () => {
 test('evidence identity is bound to task, inbound, loading plan and container booking', () => {
 	assert.match(page, /captureEvidenceIdentity\(\)[\s\S]*?taskId:[\s\S]*?inboundId:[\s\S]*?loadingPlanId:[\s\S]*?containerBookingId:/)
 	assert.match(page, /isSameEvidenceIdentity\(left, right\)[\s\S]*?taskId[\s\S]*?inboundId[\s\S]*?loadingPlanId[\s\S]*?containerBookingId/)
-	assert.match(page, /this\.currentBox\.inboundId = firstItem\.inboundId/)
-	assert.doesNotMatch(page, /firstItem\.inboundId \|\| this\.currentBox\.inboundId/)
+	assert.match(page, /this\.currentBox\.inboundId = task\.inboundId/)
+	assert.match(page, /String\(detailTaskId \|\| ''\) !== String\(requestedTaskId\)/)
+	assert.match(page, /!task\.inboundId \|\| !task\.loadingPlanId \|\| !task\.containerBookingId/)
+	assert.match(page, /evidenceItem[\s\S]*?String\(task\.inboundId\)/)
 	assert.match(page, /this\.evidenceIdentity = this\.captureEvidenceIdentity\(\)/)
 })
 
@@ -114,6 +116,15 @@ test('returning from the camera does not refresh and invalidate the active evide
 	assert.match(chooser, /fail: \(error\) => \{[\s\S]*?this\.cameraPicking = false/)
 })
 
+test('returning from an image preview does not refresh the picking task', () => {
+	assert.match(page, /evidencePreviewOpening: false, evidencePreviewReturning: false/)
+	assert.match(page, /onShow\(\) \{[\s\S]*?if \(this\.evidencePreviewReturning\)[\s\S]*?preview\.resume[\s\S]*?return[\s\S]*?this\.syncRemoteTask\(\)/)
+	assert.match(page, /onHide\(\) \{[\s\S]*?if \(this\.evidencePreviewOpening\)[\s\S]*?this\.evidencePreviewReturning = true/)
+	const preview = page.match(/previewEvidencePhoto\(photo, type\) \{[\s\S]*?\n\t\t\t\},/)?.[0] || ''
+	assert.match(preview, /this\.evidencePreviewOpening = true/)
+	assert.match(preview, /uni\.previewImage\(\{ current, urls, fail:/)
+})
+
 test('evidence save, void and loading completion ignore stale cabinet responses', () => {
 	for (const [method, awaitedCall] of [
 		['saveEvidence', 'upsertEvidence'],
@@ -132,6 +143,16 @@ test('evidence save, void and loading completion ignore stale cabinet responses'
 
 test('completed server tasks restore evidence stage without being claimed again', () => {
 	assert.match(page, /const remoteTask = normalizePickingTask\(task\)/)
-	assert.match(page, /if \(!isPickingCompleteTask\(remoteTask\)\) \{[\s\S]*?claimContainerPickingTask/)
+	assert.match(page, /if \(remoteTask\.workflowStatus === 'UNASSIGNED'\) \{[\s\S]*?claimContainerPickingTask/)
+	assert.match(page, /else if \(!isPickingCompleteTask\(remoteTask\)\)/)
 	assert.match(page, /if \(this\.loadingStageReady\) await this\.loadLoadingEvidence\(\)/)
 })
+
+test('picking mutations stay closed until the canonical task has been claimed', () => {
+	assert.match(page, /pickingOperationReady\(\)[\s\S]*?workflowStatus === 'IN_PROGRESS'[\s\S]*?remoteTaskId[\s\S]*?inboundId[\s\S]*?loadingPlanId[\s\S]*?containerBookingId/)
+	assert.match(page, /requirePickingOperation\(\)[\s\S]*?任务尚未领取完成/)
+	for (const method of ['processScannedValue', 'pickItem', 'openSkip', 'openPartial', 'submitPicking']) {
+		const body = page.match(new RegExp(`${method}\\([^)]*\\) \\{[\\s\\S]*?\\n\\t\\t\\t\\},`))?.[0] || ''
+		assert.match(body, /if \(!this\.requirePickingOperation\(\)\) return/, `${method} should wait for claim`)
+	}
+})