Change LMSPublisher serializer to not convert price to integer, inste…

…ad to str, letting LMS handle the Decimal conversion
openedx-unsupported · Feb 9, 2021 · 57ed0f1 · 57ed0f1
1 parent 2151b73
commit 57ed0f1
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/ecommerce/courses/publishers.py b/ecommerce/courses/publishers.py
@@ -38,7 +38,7 @@ def serialize_seat_for_commerce_api(self, seat):
         return {
             'name': mode_for_product(seat),
             'currency': stock_record.price_currency,
-            'price': int(stock_record.price_excl_tax),
+            'price': str(stock_record.price_excl_tax),
             'sku': stock_record.partner_sku,
             'bulk_sku': bulk_sku,
             'expires': self.get_seat_expiration(seat),

diff --git a/ecommerce/courses/tests/test_publishers.py b/ecommerce/courses/tests/test_publishers.py
@@ -130,7 +130,7 @@ def test_serialize_seat_for_commerce_api(self):
         expected = {
             'name': 'verified',
             'currency': 'USD',
-            'price': int(stock_record.price_excl_tax),
+            'price': str(stock_record.price_excl_tax),
             'sku': stock_record.partner_sku,
             'bulk_sku': None,
             'expires': None,
@@ -162,7 +162,7 @@ def test_serialize_seat_for_commerce_api_with_professional(self, is_verified, ex
         expected = {
             'name': expected_mode,
             'currency': 'USD',
-            'price': int(stock_record.price_excl_tax),
+            'price': str(stock_record.price_excl_tax),
             'sku': stock_record.partner_sku,
             'bulk_sku': None,
             'expires': None,
@@ -178,7 +178,7 @@ def test_serialize_seat_with_enrollment_code(self):
         expected = {
             'name': 'verified',
             'currency': 'USD',
-            'price': int(stock_record.price_excl_tax),
+            'price': str(stock_record.price_excl_tax),
             'sku': stock_record.partner_sku,
             'bulk_sku': ec_stock_record.partner_sku,
             'expires': None,