Sending sales messages on status change

Close #4880

Author: renzo

pythonpro/discord/bot.py

@@ -11,5 +11,8 @@ class _DevProDiscordBotClient(DiscordBotClient):
     def send_to_sales_channel(self, msg: str) -> dict:
         return self.create_message(settings.DISCORD_GUILD_SALES_CHANNEL_ID, msg)
 
+    def send_to_checkout_channel(self, msg: str) -> dict:
+        return self.create_message(settings.DISCORD_GUILD_SALES_CHANNEL_ID, msg)
+
 
 devpro_discord_bot_client = _DevProDiscordBotClient(settings.DISCORD_APP_BOT_TOKEN)

pythonpro/discord/tasks.py

@@ -58,7 +58,6 @@ def clean_discord_user(discord_user_id):
 ---------------------------------------------------------
 """
 
-
 _WARN_USER_TEMPLATE = """Olá {user_name},
 
 Sua assinatura anual da DevPro está prestes a expirar, e queremos oferecer uma oportunidade imperdível para que você continue aproveitando todos os benefícios de ser nosso assinante. 
@@ -119,3 +118,23 @@ def warn_subscription_expiration(user_id: int, expiration_date: str):
         )
         logger.info(f'Subscription warn sent to user with id: {user.id}')
     return None
+
+
+_CHECKOUT_STATUS_CHANGED_TEMPLATE = """#####################################
+O status agora é {status}
+Transaction id = https://beta.dashboard.pagar.me/#/transactions/{transaction_id}
+"""
+
+
+@shared_task(
+    rate_limit=1,
+    max_retries=5,
+    retry_backoff=True,
+    retry_backoff_max=700,
+    retry_jitter=True
+)
+def send_payment_status_change(status, transaction_id):
+    return devpro_discord_bot_client.send_to_checkout_channel(_CHECKOUT_STATUS_CHANGED_TEMPLATE.format(
+        status=status,
+        transaction_id=transaction_id
+    ))

pythonpro/domain/checkout_domain.py

@@ -3,6 +3,7 @@
 from django_pagarme import facade as django_pagarme_facade
 
 from pythonpro.core import facade as core_facade
+from pythonpro.discord.tasks import send_payment_status_change
 from pythonpro.domain import user_domain, subscription_domain
 from pythonpro.domain.hotzapp_domain import verify_purchase, send_purchase_notification
 from pythonpro.email_marketing import facade as email_marketing_facade
@@ -57,6 +58,7 @@ def payment_handler_task(payment_id):
         pass  # no need to handle payment with no Item
     else:
         status = payment.status()
+        send_payment_status_change.delay(status, payment.transaction_id)
         if status == django_pagarme_facade.PAID:
             user = payment.user
             subscription_domain.create_subscription_and_activate_services(payment)

pythonpro/domain/tests/test_checkout/test_boleto_generation.py

@@ -60,6 +60,11 @@ def send_purchase_notification_mock(mocker):
     return mocker.patch('pythonpro.domain.checkout_domain.send_purchase_notification.delay')
 
 
+@pytest.fixture(autouse=True)
+def send_payment_status_change_mock(mocker):
+    return mocker.patch('pythonpro.domain.checkout_domain.send_payment_status_change.delay')
+
+
 @pytest.fixture
 def resp(client, pagarme_responses, create_or_update_lead_mock, payment_handler_task_mock, tag_as_mock,
          active_product_item, sync_on_discourse_mock, send_purchase_notification_mock):

pythonpro/domain/tests/test_checkout/test_credit_card_payment.py

@@ -95,6 +95,11 @@ def send_purchase_notification_mock(mocker):
     return mocker.patch('pythonpro.domain.checkout_domain.send_purchase_notification.delay')
 
 
+@pytest.fixture(autouse=True)
+def send_payment_status_change_mock(mocker):
+    return mocker.patch('pythonpro.domain.checkout_domain.send_payment_status_change.delay')
+
+
 # tests for user not logged
 
 @pytest.fixture

pythonpro/domain/tests/test_checkout/test_payment_handler.py

@@ -24,6 +24,11 @@ def send_purchase_notification_mock(mocker):
     return mocker.patch('pythonpro.domain.checkout_domain.send_purchase_notification.delay')
 
 
+@pytest.fixture(autouse=True)
+def send_payment_status_change_mock(mocker):
+    return mocker.patch('pythonpro.domain.checkout_domain.send_payment_status_change.delay')
+
+
 def test_pagarme_payment_paid_boleto(db, tag_as_mock, remove_tags_mock, logged_user,
                                      send_purchase_notification_mock, mock_subscription_creation):
     payment = baker.make(PagarmePayment, payment_method=facade.BOLETO, user=logged_user)
@@ -113,7 +118,8 @@ def mock_subscription_creation(mocker):
 
 
 def test_pagarme_payment_paid_credit_card(mock_subscription_creation, db, tag_as_mock, remove_tags_mock,
-                                          logged_user, send_purchase_notification_mock):
+                                          logged_user, send_purchase_notification_mock,
+                                          send_payment_status_change_mock):
     payment = baker.make(PagarmePayment, payment_method=facade.CREDIT_CARD, user=logged_user)
     baker.make(PagarmeNotification, status=facade.PAID, payment=payment)
     config = baker.make(PagarmeItemConfig, payments=[payment])
@@ -124,7 +130,8 @@ def test_pagarme_payment_paid_credit_card(mock_subscription_creation, db, tag_as
     remove_tags_mock.assert_called_once_with(
         logged_user.email, logged_user.id, f'{config.slug}-refused'
     )
-    send_purchase_notification_mock.asser_called_once_with(payment.id)
+    send_purchase_notification_mock.assert_called_once_with(payment.id)
+    send_payment_status_change_mock.assert_called_once_with(facade.PAID, payment.transaction_id)
 
 
 def test_pagarme_payment_waiting_payment_boleto(mock_subscription_creation, db, tag_as_mock, remove_tags_mock,
@@ -135,7 +142,7 @@ def test_pagarme_payment_waiting_payment_boleto(mock_subscription_creation, db,
     checkout_domain.payment_handler_task(payment.id)
     assert remove_tags_mock.called is False
     tag_as_mock.assert_called_once_with(logged_user.email, logged_user.id, f'{config.slug}-boleto')
-    send_purchase_notification_mock.asser_called_once_with(payment.id)
+    send_purchase_notification_mock.assert_called_once_with(payment.id)
 
 
 @pytest.mark.parametrize(
@@ -212,4 +219,4 @@ def test_pagarme_payment_refused(db, tag_as_mock, remove_tags_mock, logged_user,
     checkout_domain.payment_handler_task(payment.id)
     assert remove_tags_mock.called is False
     tag_as_mock.assert_called_once_with(logged_user.email, logged_user.id, f'{config.slug}-refused')
-    send_purchase_notification_mock.asser_called_once_with(payment.id)
+    send_purchase_notification_mock.assert_called_once_with(payment.id)