This commit is contained in:
@@ -216,10 +216,24 @@ def test_competitor_review_queue_is_canonical_unit_price_handoff():
|
||||
def test_competitor_review_filters_split_low_score_operational_buckets():
|
||||
from services.competitor_intel_repository import REVIEW_STATUS_FILTER_GROUPS
|
||||
|
||||
assert REVIEW_STATUS_FILTER_GROUPS["catalog_comparable"] == ("true_low_confidence",)
|
||||
assert REVIEW_STATUS_FILTER_GROUPS["catalog_variant_review"] == ("true_low_confidence",)
|
||||
assert REVIEW_STATUS_FILTER_GROUPS["catalog_unit_review"] == ("true_low_confidence",)
|
||||
assert REVIEW_STATUS_FILTER_GROUPS["catalog_identity_review"] == ("true_low_confidence",)
|
||||
assert REVIEW_STATUS_FILTER_GROUPS["catalog_comparable"] == (
|
||||
"true_low_confidence",
|
||||
"catalog_variant_review",
|
||||
"catalog_unit_review",
|
||||
"catalog_identity_review",
|
||||
)
|
||||
assert REVIEW_STATUS_FILTER_GROUPS["catalog_variant_review"] == (
|
||||
"true_low_confidence",
|
||||
"catalog_variant_review",
|
||||
)
|
||||
assert REVIEW_STATUS_FILTER_GROUPS["catalog_unit_review"] == (
|
||||
"true_low_confidence",
|
||||
"catalog_unit_review",
|
||||
)
|
||||
assert REVIEW_STATUS_FILTER_GROUPS["catalog_identity_review"] == (
|
||||
"true_low_confidence",
|
||||
"catalog_identity_review",
|
||||
)
|
||||
assert REVIEW_STATUS_FILTER_GROUPS["recoverable_low_score"] == ("recoverable_low_score",)
|
||||
assert REVIEW_STATUS_FILTER_GROUPS["true_low_confidence"] == ("true_low_confidence",)
|
||||
assert REVIEW_STATUS_FILTER_GROUPS["legacy_low_score"] == ("low_score", "refresh_low_score")
|
||||
|
||||
@@ -379,7 +379,23 @@ def test_competitor_feeder_blocks_identity_review_from_auto_price_write():
|
||||
match_type="comparable",
|
||||
price_basis="manual_review",
|
||||
alert_tier="identity_review",
|
||||
reasons=("variant_selection_review",),
|
||||
reasons=("variant_selection_review", "strong_product_line_match"),
|
||||
)
|
||||
unit_review = SimpleNamespace(
|
||||
hard_veto=False,
|
||||
comparison_mode="exact_identity",
|
||||
match_type="exact",
|
||||
price_basis="manual_review",
|
||||
alert_tier="identity_review",
|
||||
reasons=("commercial_condition_gap", "strong_exact_spec_match"),
|
||||
)
|
||||
catalog_identity_review = SimpleNamespace(
|
||||
hard_veto=False,
|
||||
comparison_mode="exact_identity",
|
||||
match_type="exact",
|
||||
price_basis="manual_review",
|
||||
alert_tier="identity_review",
|
||||
reasons=("strong_exact_spec_match", "shared_identity_anchor_variant_safe"),
|
||||
)
|
||||
exact_price = SimpleNamespace(
|
||||
hard_veto=False,
|
||||
@@ -391,7 +407,9 @@ def test_competitor_feeder_blocks_identity_review_from_auto_price_write():
|
||||
)
|
||||
|
||||
assert _is_auto_price_write_safe(identity_review) is False
|
||||
assert _classify_auto_write_block_attempt(0.783, identity_review) == "true_low_confidence"
|
||||
assert _classify_auto_write_block_attempt(0.783, identity_review) == "catalog_variant_review"
|
||||
assert _classify_auto_write_block_attempt(1.0, unit_review) == "catalog_unit_review"
|
||||
assert _classify_auto_write_block_attempt(1.0, catalog_identity_review) == "catalog_identity_review"
|
||||
assert _is_auto_price_write_safe(exact_price) is True
|
||||
|
||||
|
||||
|
||||
Reference in New Issue
Block a user