補齊 PChome retry verifier artifact preview
Some checks failed
CD Pipeline / deploy (push) Has been cancelled
Some checks failed
CD Pipeline / deploy (push) Has been cancelled
This commit is contained in:
@@ -76,6 +76,7 @@ from services.pchome_mapping_backlog_service import (
|
||||
build_pchome_direct_mapping_candidate_exception_resolution_closeout_package,
|
||||
build_pchome_direct_mapping_retry_candidate_decision_package,
|
||||
build_pchome_direct_mapping_retry_candidate_exception_auto_resolution_package,
|
||||
build_pchome_direct_mapping_retry_candidate_exception_closeout_verifier_artifact_preview_package,
|
||||
build_pchome_direct_mapping_retry_candidate_exception_closeout_verifier_input_package,
|
||||
build_pchome_direct_mapping_retry_candidate_exception_resolution_closeout_package,
|
||||
build_pchome_growth_ai_automation_readiness,
|
||||
@@ -799,6 +800,86 @@ def test_direct_mapping_retry_candidate_exception_closeout_verifier_input_packag
|
||||
assert call_count["search"] == 3
|
||||
|
||||
|
||||
def test_direct_mapping_retry_candidate_exception_closeout_verifier_artifact_preview_package_builds_manifest():
|
||||
call_count = {"search": 0}
|
||||
|
||||
def fake_search(targets, limit_per_product, max_products, max_terms_per_product, min_score):
|
||||
call_count["search"] += 1
|
||||
if targets[0].get("source_artifact_id"):
|
||||
if call_count["search"] > 2:
|
||||
return True, "retry_clear", []
|
||||
return True, "retry_found", [
|
||||
{
|
||||
"product_id": "MOMO-RETRY-REVIEW",
|
||||
"name": "Direct mapping product 40ml 多款任選",
|
||||
"price": 499,
|
||||
"target_pchome_product_id": "PCH-2",
|
||||
"target_pchome_name": "Direct mapping product 40ml x2",
|
||||
"target_match_score": 0.74,
|
||||
"auto_compare_type": "manual_review",
|
||||
"target_hard_veto": False,
|
||||
},
|
||||
{
|
||||
"product_id": "MOMO-RETRY-REVIEW-2",
|
||||
"name": "Direct mapping product 40ml 限量組",
|
||||
"price": 520,
|
||||
"target_pchome_product_id": "PCH-2",
|
||||
"target_pchome_name": "Direct mapping product 40ml x2",
|
||||
"target_match_score": 0.91,
|
||||
"auto_compare_type": "manual_review",
|
||||
"target_hard_veto": False,
|
||||
},
|
||||
]
|
||||
return True, "found", [
|
||||
{
|
||||
"product_id": "MOMO-UNIT",
|
||||
"name": "Direct mapping product 40ml",
|
||||
"price": 499,
|
||||
"target_pchome_product_id": "PCH-2",
|
||||
"target_pchome_name": "Direct mapping product 40ml x2",
|
||||
"target_match_score": 0.91,
|
||||
"auto_compare_type": "unit_price",
|
||||
"target_hard_veto": True,
|
||||
}
|
||||
]
|
||||
|
||||
package = build_pchome_direct_mapping_retry_candidate_exception_closeout_verifier_artifact_preview_package(
|
||||
_payload(),
|
||||
batch_size=1,
|
||||
execute_search=True,
|
||||
execute_retry_search=True,
|
||||
max_terms_per_product=6,
|
||||
search_func=fake_search,
|
||||
)
|
||||
|
||||
preview = package["retry_exception_closeout_verifier_artifact_preview"]
|
||||
artifact_keys = [schema["key"] for schema in package["artifact_schemas"]]
|
||||
assert package["policy"] == (
|
||||
"read_only_pchome_growth_direct_mapping_retry_candidate_exception_closeout_verifier_artifact_preview"
|
||||
)
|
||||
assert package["result"] == "DIRECT_MAPPING_RETRY_EXCEPTION_CLOSEOUT_VERIFIER_ARTIFACT_PREVIEW_READY"
|
||||
assert package["summary"]["verifier_artifact_preview_ready_count"] == 1
|
||||
assert package["summary"]["ready_closeout_no_write_verifier_input_count"] == 2
|
||||
assert package["summary"]["blocked_closeout_no_write_verifier_input_count"] == 0
|
||||
assert package["summary"]["artifact_schema_count"] == 3
|
||||
assert package["summary"]["artifact_generation_step_count"] == 5
|
||||
assert package["summary"]["verifier_manifest_check_count"] == 15
|
||||
assert package["summary"]["writes_artifact_count"] == 0
|
||||
assert package["summary"]["writes_database_count"] == 0
|
||||
assert preview["preview_id"].startswith("pchome-retry-closeout-verifier-artifacts-")
|
||||
assert preview["ready_for_future_artifact_generation"] is True
|
||||
assert preview["ready_to_write_artifacts_now"] is False
|
||||
assert preview["ready_for_controlled_apply_now"] is False
|
||||
assert "retry_exception_closeout_verifier_input_artifact" in artifact_keys
|
||||
assert "retry_exception_identity_readback_artifact" in artifact_keys
|
||||
assert "retry_exception_controlled_apply_preflight_artifact" in artifact_keys
|
||||
assert package["artifact_generation_plan"]["writes_artifact_in_preview"] is False
|
||||
assert package["verifier_manifest"]["writes_artifact_in_preview"] is False
|
||||
assert package["safety"]["writes_artifact_in_preview"] is False
|
||||
assert package["safety"]["writes_database"] is False
|
||||
assert call_count["search"] == 2
|
||||
|
||||
|
||||
def test_ai_automation_readiness_makes_automation_visible_without_manual_primary_flow():
|
||||
readiness = build_pchome_growth_ai_automation_readiness(_payload(), batch_size=1)
|
||||
|
||||
@@ -15221,6 +15302,40 @@ def test_direct_mapping_retry_candidate_exception_closeout_verifier_input_route_
|
||||
assert payload["safety"]["writes_database"] is False
|
||||
|
||||
|
||||
def test_direct_mapping_retry_candidate_exception_closeout_verifier_artifact_preview_route_uses_cached_payload(monkeypatch):
|
||||
from flask import Flask
|
||||
from routes import ai_routes as routes
|
||||
|
||||
monkeypatch.setattr(routes, "_get_cached_pchome_growth_payload", lambda: _payload())
|
||||
|
||||
def fail_engine(database_path):
|
||||
raise AssertionError("cached retry exception closeout verifier artifact preview should not open a DB engine")
|
||||
|
||||
monkeypatch.setattr(routes, "_create_icaim_dashboard_engine", fail_engine)
|
||||
|
||||
app = Flask(__name__)
|
||||
with app.test_request_context(
|
||||
"/api/ai/pchome-growth/mapping-backlog/direct-mapping-retry-candidate-exception-closeout-verifier-artifact-preview-package?batch_size=1"
|
||||
):
|
||||
response = routes.api_pchome_growth_direct_mapping_retry_candidate_exception_closeout_verifier_artifact_preview_package.__wrapped__()
|
||||
|
||||
payload = response.get_json()
|
||||
assert payload["success"] is True
|
||||
assert payload["policy"] == (
|
||||
"read_only_pchome_growth_direct_mapping_retry_candidate_exception_closeout_verifier_artifact_preview"
|
||||
)
|
||||
assert payload["source_endpoint"] == (
|
||||
"/api/ai/pchome-growth/mapping-backlog/direct-mapping-retry-candidate-exception-closeout-verifier-input-package"
|
||||
)
|
||||
assert payload["result"] == "WAITING_FOR_RETRY_EXCEPTION_CLOSEOUT_VERIFIER_INPUTS"
|
||||
assert payload["summary"]["artifact_schema_count"] == 3
|
||||
assert payload["retry_exception_closeout_verifier_artifact_preview"]["ready_to_write_artifacts_now"] is False
|
||||
assert payload["safety"]["writes_artifact_in_preview"] is False
|
||||
assert payload["safety"]["executes_search"] is False
|
||||
assert payload["safety"]["executes_retry_search"] is False
|
||||
assert payload["safety"]["writes_database"] is False
|
||||
|
||||
|
||||
def test_ai_automation_readiness_route_defaults_to_no_search_and_uses_cached_payload(monkeypatch):
|
||||
from flask import Flask
|
||||
from routes import ai_routes as routes
|
||||
|
||||
Reference in New Issue
Block a user