Commit 89296271 by Vedran Karacic

Disallow showing the note about verified certificate in baskets with credit seats.

parent cd2a7cd0
...@@ -42,6 +42,7 @@ Basket = get_model('basket', 'Basket') ...@@ -42,6 +42,7 @@ Basket = get_model('basket', 'Basket')
Benefit = get_model('offer', 'Benefit') Benefit = get_model('offer', 'Benefit')
Catalog = get_model('catalogue', 'Catalog') Catalog = get_model('catalogue', 'Catalog')
Product = get_model('catalogue', 'Product') Product = get_model('catalogue', 'Product')
ProductAttribute = get_model('catalogue', 'ProductAttribute')
Selector = get_class('partner.strategy', 'Selector') Selector = get_class('partner.strategy', 'Selector')
StockRecord = get_model('partner', 'StockRecord') StockRecord = get_model('partner', 'StockRecord')
Voucher = get_model('voucher', 'Voucher') Voucher = get_model('voucher', 'Voucher')
...@@ -428,6 +429,28 @@ class BasketSummaryViewTests(CourseCatalogTestMixin, CourseCatalogMockMixin, Lms ...@@ -428,6 +429,28 @@ class BasketSummaryViewTests(CourseCatalogTestMixin, CourseCatalogMockMixin, Lms
self.assertEqual(line_data.get('image_url'), '') self.assertEqual(line_data.get('image_url'), '')
self.assertEqual(line_data.get('course_short_description'), None) self.assertEqual(line_data.get('course_short_description'), None)
@ddt.data(
('verified', True),
('credit', False)
)
@ddt.unpack
def test_verification_message(self, cert_type, ver_req):
""" Verify the variable for verification requirement is False for credit seats. """
seat = self.create_seat(self.course, cert_type=cert_type)
self.create_basket_and_add_product(seat)
response = self.client.get(self.path)
self.assertEqual(response.status_code, 200)
self.assertEqual(response.context['is_verification_required'], ver_req)
def test_verification_attribute_missing(self):
""" Verify the variable for verification requirement is False when the attribute is missing. """
seat = self.create_seat(self.course)
ProductAttribute.objects.filter(name='id_verification_required').delete()
self.create_basket_and_add_product(seat)
response = self.client.get(self.path)
self.assertEqual(response.status_code, 200)
self.assertEqual(response.context['is_verification_required'], False)
class VoucherAddMessagesViewTests(TestCase): class VoucherAddMessagesViewTests(TestCase):
""" VoucherAddMessagesView view tests. """ """ VoucherAddMessagesView view tests. """
......
...@@ -151,7 +151,8 @@ class BasketSummaryView(BasketView): ...@@ -151,7 +151,8 @@ class BasketSummaryView(BasketView):
# Check product attributes to determine if ID verification is required for this basket # Check product attributes to determine if ID verification is required for this basket
try: try:
is_verification_required = is_verification_required or line.product.attr.id_verification_required is_verification_required = line.product.attr.id_verification_required \
and line.product.attr.certificate_type != 'credit'
except AttributeError: except AttributeError:
pass pass
......
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