Commit 7b73802f by Clinton Blackburn

Update publication endpoint

The id_verification_required attribute is being handled as a proper boolean, instead of a string.

XCOM-506
parent d3c3cfe9
...@@ -207,7 +207,7 @@ class AtomicPublicationSerializer(serializers.Serializer): # pylint: disable=ab ...@@ -207,7 +207,7 @@ class AtomicPublicationSerializer(serializers.Serializer): # pylint: disable=ab
# Extract arguments required for Seat creation, deserializing as necessary. # Extract arguments required for Seat creation, deserializing as necessary.
certificate_type = attrs['certificate_type'] certificate_type = attrs['certificate_type']
id_verification_required = (attrs['id_verification_required'] == 'True') id_verification_required = attrs['id_verification_required']
price = Decimal(product['price']) price = Decimal(product['price'])
# Extract arguments which are optional for Seat creation, deserializing as necessary. # Extract arguments which are optional for Seat creation, deserializing as necessary.
......
...@@ -44,7 +44,7 @@ class AtomicPublicationTests(CourseCatalogTestMixin, UserMixin, TestCase): ...@@ -44,7 +44,7 @@ class AtomicPublicationTests(CourseCatalogTestMixin, UserMixin, TestCase):
}, },
{ {
'name': 'id_verification_required', 'name': 'id_verification_required',
'value': 'False' 'value': False
} }
] ]
}, },
...@@ -59,7 +59,7 @@ class AtomicPublicationTests(CourseCatalogTestMixin, UserMixin, TestCase): ...@@ -59,7 +59,7 @@ class AtomicPublicationTests(CourseCatalogTestMixin, UserMixin, TestCase):
}, },
{ {
'name': 'id_verification_required', 'name': 'id_verification_required',
'value': 'True' 'value': True
} }
] ]
} }
...@@ -89,7 +89,6 @@ class AtomicPublicationTests(CourseCatalogTestMixin, UserMixin, TestCase): ...@@ -89,7 +89,6 @@ class AtomicPublicationTests(CourseCatalogTestMixin, UserMixin, TestCase):
attrs = {attr['name']: attr['value'] for attr in product['attribute_values']} attrs = {attr['name']: attr['value'] for attr in product['attribute_values']}
attrs['expires'] = EXPIRES if product['expires'] else None attrs['expires'] = EXPIRES if product['expires'] else None
attrs['id_verification_required'] = (attrs['id_verification_required'] == 'True')
attrs['price'] = Decimal(product['price']) attrs['price'] = Decimal(product['price'])
course.create_or_update_seat(**attrs) course.create_or_update_seat(**attrs)
...@@ -135,7 +134,7 @@ class AtomicPublicationTests(CourseCatalogTestMixin, UserMixin, TestCase): ...@@ -135,7 +134,7 @@ class AtomicPublicationTests(CourseCatalogTestMixin, UserMixin, TestCase):
if name == 'certificate_type': if name == 'certificate_type':
certificate_type = attr['value'] certificate_type = attr['value']
elif name == 'id_verification_required': elif name == 'id_verification_required':
id_verification_required = True if attr['value'] == 'True' else False id_verification_required = attr['value']
seat_title = 'Seat in {course_name} with {certificate_type} certificate'.format( seat_title = 'Seat in {course_name} with {certificate_type} certificate'.format(
course_name=course.name, course_name=course.name,
......
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