fix: 상품 개수 제한에 따라 옵션 개수 제한이 영향받는 오류 수정

Author: MU-Software

app/shop/product/serializers/dto.py

@@ -7,11 +7,20 @@
 from rest_framework import serializers
 from shop.product.models import Option, OptionGroup, Product
 
+# product 인당 한도(product_max_quantity_per_user)는 "이 유저가 이 상품(티켓)을 몇 개 살 수 있나" 축이라,
+# 옵션/그룹 인스턴스 개수(max_quantity_per_product 축, 1 OPR 아래 다수 OPOR)를 제한하지 않는다.
+# breakdown 에는 참고용으로 남기되 leftover_stock_per_user(= binding min) 계산에서는 제외한다.
+_LEFTOVER_PER_USER_EXCLUDED_KEYS = frozenset({"product_max_quantity_per_user"})
+
 
 def _min_ignoring_none(values: Iterable[int | None]) -> int | None:
     return min(finite) if (finite := [v for v in values if v is not None]) else None
 
 
+def _leftover_stock_per_user(info: dict[str, int | None]) -> int | None:
+    return _min_ignoring_none(v for k, v in info.items() if k not in _LEFTOVER_PER_USER_EXCLUDED_KEYS)
+
+
 @dataclass(frozen=True)
 class StockContext:
     """Product 페이지에 등장하는 모든 product / option / group 의 점유 수량을 단일 stage 에서 prefetch 한 결과"""
@@ -115,7 +124,7 @@ def get_leftover_stock_info(self, option: Option) -> dict[str, int | None]:
         return self.context.get("stock_context", _EMPTY_STOCK_CONTEXT).option_leftover_info(option)
 
     def get_leftover_stock_per_user(self, option: Option) -> int | None:
-        return _min_ignoring_none(self.get_leftover_stock_info(option).values())
+        return _leftover_stock_per_user(self.get_leftover_stock_info(option))
 
 
 class OptionGroupDto(serializers.ModelSerializer):
@@ -148,7 +157,7 @@ def get_leftover_stock_info(self, group: OptionGroup) -> dict[str, int | None]:
         return self.context.get("stock_context", _EMPTY_STOCK_CONTEXT).option_group_leftover_info(group)
 
     def get_leftover_stock_per_user(self, group: OptionGroup) -> int | None:
-        return _min_ignoring_none(self.get_leftover_stock_info(group).values())
+        return _leftover_stock_per_user(self.get_leftover_stock_info(group))
 
 
 class ProductDto(serializers.ModelSerializer):

app/shop/product/test/leftover_stock_per_user_test.py

@@ -88,10 +88,11 @@ def test_breakdown_subtracts_cart_and_purchased_for_logged_in_user(
 
 
 @pytest.mark.django_db
-def test_product_level_limit_can_be_binding_constraint(
+def test_product_level_limit_does_not_cap_option_leftover(
     customer_client, customer_user, ticket_product, group_with_options
 ):
-    # Product.max_quantity_per_user 가 group / option 한도 보다 작아서 binding 이 되는 경우 — 명시적 케이스.
+    # Product.max_quantity_per_user 는 "티켓 몇 장" 축이라 옵션/그룹 인스턴스 개수(max_quantity_per_product 축)를
+    # 제한하지 않는다. breakdown 엔 잔여값을 남기되 leftover_stock_per_user 엔 반영되지 않아야 한다.
     ticket_product.max_quantity_per_user = 2
     ticket_product.save()
     group, a, _b = group_with_options
@@ -101,11 +102,16 @@ def test_product_level_limit_can_be_binding_constraint(
     a.save()
 
     data = _serialize_via_api(customer_client, ticket_product)
-    option_a = next(o for o in data["option_groups"][0]["options"] if o["id"] == str(a.id))
+    grp = data["option_groups"][0]
+    # 그룹 레벨: product 한도(2)가 breakdown 엔 남지만 leftover 는 group 한도(10)가 결정.
+    assert grp["leftover_stock_info"]["product_max_quantity_per_user"] == 2
+    assert grp["leftover_stock_per_user"] == 10
+
+    option_a = next(o for o in grp["options"] if o["id"] == str(a.id))
     assert option_a["leftover_stock_info"]["product_max_quantity_per_user"] == 2
     assert option_a["leftover_stock_info"]["option_group_max_quantity_per_user"] == 10
     assert option_a["leftover_stock_info"]["option_max_quantity_per_user"] == 10
-    assert option_a["leftover_stock_per_user"] == 2  # ticket_product 한도가 binding
+    assert option_a["leftover_stock_per_user"] == 10  # product 한도(2)는 binding 아님
 
 
 @pytest.mark.django_db