Merge pull request #13 from engeldevelopment/fix/agregar-discount-service

Agregando servicio Discount Service

Author: engeldevelopment

apps/comics/domain/services.py

@@ -1,8 +1,10 @@
+from injector import inject
+
 from .entities import Comic
 from .vo import ComicStatus
 
 
-class ObtainAmountToPayService:
+class DiscountService:
     def __call__(self, comic: Comic):
         DISCOUNTS = {
             ComicStatus.EXCELLENT: ExcellentComicDiscount.for_this(comic=comic).apply(),
@@ -11,9 +13,17 @@ def __call__(self, comic: Comic):
             ComicStatus.IMPAIRED: ImpairedComicDiscount.for_this(comic=comic).apply(),
             ComicStatus.DAMAGED: DamagedComicDiscount.for_this(comic=comic).apply()
         }
-
         discount = DISCOUNTS.get(comic.status)
-        return comic.price - discount
+        return discount
+
+
+class ObtainAmountToPayService:
+    @inject
+    def __init__(self, discount_service: DiscountService):
+        self.discount_service = discount_service
+
+    def __call__(self, comic: Comic):
+        return comic.price - self.discount_service(comic)
 
 
 class Discount:

apps/comics/infrastructure/di/modules.py

@@ -1,6 +1,6 @@
 from injector import Module, singleton
 
-from apps.comics.domain.services import ObtainAmountToPayService
+from apps.comics.domain.services import ObtainAmountToPayService, DiscountService
 from apps.comics.domain.repositories import ComicRepository, RentalRepository
 from apps.comics.application.finders import ComicAllFinder, LastRentalFinder, AllRentalFinder
 from apps.comics.application.services import RentComicService
@@ -14,6 +14,7 @@ def configure(self, binder):
         binder.bind(AllRentalFinder, to=AllRentalFinder)
         binder.bind(ComicAllFinder, scope=singleton)
         binder.bind(ComicRepository, to=ComicDjangoRepository)
+        binder.bind(DiscountService, to=DiscountService)
         binder.bind(LastRentalFinder, to=LastRentalFinder)
         binder.bind(ObtainAmountToPayService, to=ObtainAmountToPayService)
         binder.bind(RentComicService, to=RentComicService)

tests/apps/comics/domain/test_services.py

@@ -1,6 +1,6 @@
 from unittest import TestCase
 
-from apps.comics.domain.services import ObtainAmountToPayService
+from apps.comics.domain.services import ObtainAmountToPayService, DiscountService
 from apps.comics.domain.vo import ComicStatus
 
 
@@ -10,7 +10,7 @@
 class ObtainAmountToPayServiceTest(TestCase):
 
     def setUp(self):
-        self.obtain_amount_to_pay = ObtainAmountToPayService()
+        self.obtain_amount_to_pay = ObtainAmountToPayService(DiscountService())
         self.comic = None
 
     def assertThatAmountToPayIs(self, amount):