Commit d598cbb2 by Adeel Khan Committed by adeel khan

Pass sku value in SailThru purchase call.

LEARNER-1193
parent e7ab02da
...@@ -23,7 +23,7 @@ def schedule_retry(self, config): ...@@ -23,7 +23,7 @@ def schedule_retry(self, config):
max_retries=config.get('SAILTHRU_RETRY_ATTEMPTS')) max_retries=config.get('SAILTHRU_RETRY_ATTEMPTS'))
def _build_purchase_item(course_id, course_url, cost_in_cents, mode, course_data): def _build_purchase_item(course_id, course_url, cost_in_cents, mode, course_data, sku):
"""Build and return Sailthru purchase item object""" """Build and return Sailthru purchase item object"""
# build item description # build item description
...@@ -47,6 +47,8 @@ def _build_purchase_item(course_id, course_url, cost_in_cents, mode, course_data ...@@ -47,6 +47,8 @@ def _build_purchase_item(course_id, course_url, cost_in_cents, mode, course_data
# add vars to item # add vars to item
item['vars'] = dict(course_data.get('vars', {}), mode=mode, course_run_id=course_id) item['vars'] = dict(course_data.get('vars', {}), mode=mode, course_run_id=course_id)
item['vars']['purchase_sku'] = sku
return item return item
...@@ -228,7 +230,7 @@ def can_retry_sailthru_request(error): ...@@ -228,7 +230,7 @@ def can_retry_sailthru_request(error):
@shared_task(bind=True, ignore_result=True) @shared_task(bind=True, ignore_result=True)
def update_course_enrollment(self, email, course_url, purchase_incomplete, mode, unit_cost=None, course_id=None, def update_course_enrollment(self, email, course_url, purchase_incomplete, mode, unit_cost=None, course_id=None,
currency=None, message_id=None, site_code=None): currency=None, message_id=None, site_code=None, sku=None):
"""Adds/updates Sailthru when a user adds to cart/purchases/upgrades a course """Adds/updates Sailthru when a user adds to cart/purchases/upgrades a course
Args: Args:
...@@ -289,7 +291,7 @@ def update_course_enrollment(self, email, course_url, purchase_incomplete, mode, ...@@ -289,7 +291,7 @@ def update_course_enrollment(self, email, course_url, purchase_incomplete, mode,
course_data = _get_course_content(course_id, course_url, sailthru_client, site_code, config) course_data = _get_course_content(course_id, course_url, sailthru_client, site_code, config)
# build item description # build item description
item = _build_purchase_item(course_id, course_url, cost_in_cents, mode, course_data) item = _build_purchase_item(course_id, course_url, cost_in_cents, mode, course_data, sku)
# build purchase api options list # build purchase api options list
options = {} options = {}
......
...@@ -125,6 +125,7 @@ class SailthruTests(TestCase): ...@@ -125,6 +125,7 @@ class SailthruTests(TestCase):
TEST_EMAIL, TEST_EMAIL,
[{ [{
'vars': { 'vars': {
'purchase_sku': None,
'course_run_id': self.course_id, 'course_run_id': self.course_id,
'mode': 'verified', 'mode': 'verified',
'upgrade_deadline_verified': '2020-03-12' 'upgrade_deadline_verified': '2020-03-12'
...@@ -169,6 +170,7 @@ class SailthruTests(TestCase): ...@@ -169,6 +170,7 @@ class SailthruTests(TestCase):
TEST_EMAIL, TEST_EMAIL,
[{ [{
'vars': { 'vars': {
'purchase_sku': None,
'course_run_id': self.course_id2, 'course_run_id': self.course_id2,
'mode': 'credit' 'mode': 'credit'
}, },
...@@ -212,6 +214,7 @@ class SailthruTests(TestCase): ...@@ -212,6 +214,7 @@ class SailthruTests(TestCase):
TEST_EMAIL, TEST_EMAIL,
[{ [{
'vars': { 'vars': {
'purchase_sku': None,
'course_run_id': self.course_id, 'course_run_id': self.course_id,
'mode': 'credit', 'mode': 'credit',
'upgrade_deadline_verified': '2020-03-12' 'upgrade_deadline_verified': '2020-03-12'
...@@ -252,6 +255,7 @@ class SailthruTests(TestCase): ...@@ -252,6 +255,7 @@ class SailthruTests(TestCase):
TEST_EMAIL, TEST_EMAIL,
[{ [{
'vars': { 'vars': {
'purchase_sku': None,
'course_run_id': self.course_id, 'course_run_id': self.course_id,
'mode': 'verified', 'mode': 'verified',
'upgrade_deadline_verified': '2020-03-12' 'upgrade_deadline_verified': '2020-03-12'
...@@ -295,6 +299,7 @@ class SailthruTests(TestCase): ...@@ -295,6 +299,7 @@ class SailthruTests(TestCase):
TEST_EMAIL, TEST_EMAIL,
[{ [{
'vars': { 'vars': {
'purchase_sku': None,
'course_run_id': self.course_id, 'course_run_id': self.course_id,
'mode': 'verified', 'mode': 'verified',
'upgrade_deadline_verified': '2020-03-12' 'upgrade_deadline_verified': '2020-03-12'
...@@ -334,6 +339,7 @@ class SailthruTests(TestCase): ...@@ -334,6 +339,7 @@ class SailthruTests(TestCase):
TEST_EMAIL, TEST_EMAIL,
[{ [{
'vars': { 'vars': {
'purchase_sku': None,
'course_run_id': self.course_id, 'course_run_id': self.course_id,
'mode': 'audit', 'mode': 'audit',
'upgrade_deadline_verified': '2020-03-12' 'upgrade_deadline_verified': '2020-03-12'
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment