From c1436fae6857cef8f0505435d6043497c06c18d5 Mon Sep 17 00:00:00 2001 From: zubair-ce07 Date: Thu, 31 Aug 2023 17:26:27 +0500 Subject: [PATCH 1/2] feat: changes to upgrad django-oscar to version 3.2 --- ecommerce/core/tests/test_create_demo_data.py | 4 +- ecommerce/core/tests/test_generate_courses.py | 2 +- ecommerce/coupons/tests/test_utils.py | 2 +- .../commands/create_enrollment_codes.py | 2 +- ecommerce/courses/models.py | 4 +- ecommerce/courses/publishers.py | 2 +- ecommerce/courses/tests/test_models.py | 4 +- ecommerce/courses/tests/test_publishers.py | 6 +- ecommerce/credit/views.py | 6 +- ecommerce/enterprise/conditions.py | 2 +- ecommerce/enterprise/tests/test_conditions.py | 4 +- ecommerce/entitlements/tests/test_utils.py | 8 +- ecommerce/entitlements/utils.py | 2 +- .../migrations/0003_auto_20230824_1028.py | 21 +++ ecommerce/extensions/api/serializers.py | 6 +- ecommerce/extensions/api/utils.py | 2 +- .../extensions/api/v2/tests/views/__init__.py | 2 +- .../api/v2/tests/views/test_baskets.py | 18 +-- .../api/v2/tests/views/test_coupons.py | 2 +- .../api/v2/tests/views/test_orders.py | 8 +- .../api/v2/tests/views/test_publication.py | 4 +- .../api/v2/tests/views/test_stockrecords.py | 18 +-- ecommerce/extensions/api/v2/views/coupons.py | 2 +- ecommerce/extensions/api/v2/views/orders.py | 2 +- .../extensions/api/v2/views/stockrecords.py | 4 +- ecommerce/extensions/api/v2/views/vouchers.py | 2 +- .../0016_alter_lineattribute_value.py | 19 +++ ecommerce/extensions/basket/models.py | 4 +- .../extensions/basket/tests/test_utils.py | 24 +-- .../extensions/basket/tests/test_views.py | 2 +- .../management/commands/migrate_course.py | 2 +- .../migrations/0056_auto_20230824_1028.py | 143 ++++++++++++++++++ .../catalogue/tests/test_migrate_course.py | 2 +- ecommerce/extensions/catalogue/utils.py | 2 +- .../extensions/checkout/tests/test_mixins.py | 4 +- .../migrations/0002_auto_20230824_1028.py | 26 ++++ .../migrations/0008_auto_20230824_1028.py | 22 +++ .../tests/test_mixins.py | 4 +- .../fulfillment/tests/test_modules.py | 2 +- .../migrations/0055_auto_20230824_1028.py | 22 +++ ecommerce/extensions/offer/models.py | 2 +- .../tests/test_dynamic_conditional_offer.py | 6 +- .../extensions/offer/tests/test_models.py | 2 +- .../extensions/offer/tests/test_utils.py | 2 +- .../migrations/0026_auto_20230824_1028.py | 51 +++++++ ecommerce/extensions/partner/admin.py | 2 +- .../migrations/0019_auto_20230824_1250.py | 47 ++++++ .../migrations/0033_auto_20230824_1028.py | 26 ++++ .../payment/tests/views/test_paypal.py | 2 +- ecommerce/extensions/test/factories.py | 2 +- .../migrations/0013_auto_20230824_1028.py | 59 ++++++++ .../extensions/voucher/tests/test_utils.py | 4 +- ecommerce/extensions/voucher/utils.py | 6 +- ecommerce/management/tests/test_utils.py | 6 +- ecommerce/programs/tests/test_conditions.py | 4 +- ecommerce/settings/devstack.py | 6 +- ecommerce/settings/local.py | 6 +- .../js/test/specs/views/offer_view_spec.js | 4 +- ecommerce/static/js/views/offer_view.js | 4 +- .../static/templates/_offer_course_list.html | 2 +- .../dashboard/catalogue/product_update.html | 4 +- .../oscar/dashboard/orders/order_list.html | 2 - ecommerce/tests/mixins.py | 2 +- requirements/base.txt | 6 +- requirements/constraints.txt | 2 +- requirements/dev.txt | 6 +- requirements/production.txt | 6 +- requirements/test.txt | 6 +- 68 files changed, 564 insertions(+), 130 deletions(-) create mode 100644 ecommerce/extensions/analytics/migrations/0003_auto_20230824_1028.py create mode 100644 ecommerce/extensions/basket/migrations/0016_alter_lineattribute_value.py create mode 100644 ecommerce/extensions/catalogue/migrations/0056_auto_20230824_1028.py create mode 100644 ecommerce/extensions/communication/migrations/0002_auto_20230824_1028.py create mode 100644 ecommerce/extensions/customer/migrations/0008_auto_20230824_1028.py create mode 100644 ecommerce/extensions/offer/migrations/0055_auto_20230824_1028.py create mode 100644 ecommerce/extensions/order/migrations/0026_auto_20230824_1028.py create mode 100644 ecommerce/extensions/partner/migrations/0019_auto_20230824_1250.py create mode 100644 ecommerce/extensions/payment/migrations/0033_auto_20230824_1028.py create mode 100644 ecommerce/extensions/voucher/migrations/0013_auto_20230824_1028.py diff --git a/ecommerce/core/tests/test_create_demo_data.py b/ecommerce/core/tests/test_create_demo_data.py index 68fb787b78a..879133ce80c 100644 --- a/ecommerce/core/tests/test_create_demo_data.py +++ b/ecommerce/core/tests/test_create_demo_data.py @@ -23,12 +23,12 @@ def assert_seats_created(self, course_id, course_title, price): audit_seat = seats[1] self.assertFalse(hasattr(audit_seat.attr, 'certificate_type')) self.assertFalse(audit_seat.attr.id_verification_required) - self.assertEqual(audit_seat.stockrecords.get(partner=self.partner).price_excl_tax, 0) + self.assertEqual(audit_seat.stockrecords.get(partner=self.partner).price, 0) verified_seat = seats[0] self.assertEqual(verified_seat.attr.certificate_type, 'verified') self.assertTrue(verified_seat.attr.id_verification_required) - self.assertEqual(verified_seat.stockrecords.get(partner=self.partner).price_excl_tax, price) + self.assertEqual(verified_seat.stockrecords.get(partner=self.partner).price, price) @responses.activate def test_handle(self): diff --git a/ecommerce/core/tests/test_generate_courses.py b/ecommerce/core/tests/test_generate_courses.py index 72f59aa95fd..7e48411a9ce 100644 --- a/ecommerce/core/tests/test_generate_courses.py +++ b/ecommerce/core/tests/test_generate_courses.py @@ -192,5 +192,5 @@ def test_create_seat(self, seat_type, mock_logger): course = Course.objects.get(id='course-v1:test-course-generator+1+1') seats = course.seat_products seat = seats[0] - self.assertEqual(seat.stockrecords.get(partner=self.partner).price_excl_tax, price) + self.assertEqual(seat.stockrecords.get(partner=self.partner).price, price) mock_logger.info.assert_any_call("%s has been set to %s", seat_type, True) diff --git a/ecommerce/coupons/tests/test_utils.py b/ecommerce/coupons/tests/test_utils.py index ecfb0109f2c..c2a0a70fda2 100644 --- a/ecommerce/coupons/tests/test_utils.py +++ b/ecommerce/coupons/tests/test_utils.py @@ -54,7 +54,7 @@ def test_is_voucher_applied(self): """ Verify is_voucher_applied return correct value. """ - product = ProductFactory(stockrecords__price_excl_tax=100) + product = ProductFactory(stockrecords__price=100) voucher, product = prepare_voucher( _range=RangeFactory(products=[product]), benefit_value=10 diff --git a/ecommerce/courses/management/commands/create_enrollment_codes.py b/ecommerce/courses/management/commands/create_enrollment_codes.py index 6baf50ec998..cd6063ef0fa 100644 --- a/ecommerce/courses/management/commands/create_enrollment_codes.py +++ b/ecommerce/courses/management/commands/create_enrollment_codes.py @@ -215,7 +215,7 @@ def get_course_info(course): if len(seats) == 1: seat = seats[0] seat_type = getattr(seat.attr, 'certificate_type', '').lower() - price = seat.stockrecords.all()[0].price_excl_tax + price = seat.stockrecords.all()[0].price id_verification_required = getattr(seat.attr, 'id_verification_required', False) return seat_type, price, id_verification_required diff --git a/ecommerce/courses/models.py b/ecommerce/courses/models.py index 62c10ed6c80..c70509ecee5 100644 --- a/ecommerce/courses/models.py +++ b/ecommerce/courses/models.py @@ -248,7 +248,7 @@ def create_or_update_seat( course_id ) - stock_record.price_excl_tax = price + stock_record.price = price stock_record.price_currency = settings.OSCAR_DEFAULT_CURRENCY stock_record.save() @@ -325,7 +325,7 @@ def _create_or_update_enrollment_code(self, seat_type, id_verification_required, partner_sku=enrollment_code_sku ) - stock_record.price_excl_tax = price + stock_record.price = price stock_record.price_currency = settings.OSCAR_DEFAULT_CURRENCY stock_record.save() diff --git a/ecommerce/courses/publishers.py b/ecommerce/courses/publishers.py index bdc76cca78f..b33c5fcaf4f 100644 --- a/ecommerce/courses/publishers.py +++ b/ecommerce/courses/publishers.py @@ -39,7 +39,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': int(stock_record.price), 'sku': stock_record.partner_sku, 'bulk_sku': bulk_sku, 'expires': self.get_seat_expiration(seat), diff --git a/ecommerce/courses/tests/test_models.py b/ecommerce/courses/tests/test_models.py index a875b230f21..9ae6e1b76d9 100644 --- a/ecommerce/courses/tests/test_models.py +++ b/ecommerce/courses/tests/test_models.py @@ -106,7 +106,7 @@ def assert_course_seat_valid(self, seat, course, certificate_type, id_verificati self.assertEqual(getattr(seat.attr, 'certificate_type', ''), certificate_type) self.assertEqual(seat.attr.course_key, course.id) self.assertEqual(seat.attr.id_verification_required, id_verification_required) - self.assertEqual(seat.stockrecords.first().price_excl_tax, price) + self.assertEqual(seat.stockrecords.first().price, price) if credit_provider: self.assertEqual(seat.attr.credit_provider, credit_provider) @@ -157,7 +157,7 @@ def test_create_seat_with_enrollment_code(self): self.assertIsNone(enrollment_code.expires) stock_record = StockRecord.objects.get(product=enrollment_code) - self.assertEqual(stock_record.price_excl_tax, price) + self.assertEqual(stock_record.price, price) self.assertEqual(stock_record.price_currency, settings.OSCAR_DEFAULT_CURRENCY) self.assertEqual(stock_record.partner, self.partner) diff --git a/ecommerce/courses/tests/test_publishers.py b/ecommerce/courses/tests/test_publishers.py index 49191303f4c..26d29cf3393 100644 --- a/ecommerce/courses/tests/test_publishers.py +++ b/ecommerce/courses/tests/test_publishers.py @@ -129,7 +129,7 @@ def test_serialize_seat_for_commerce_api(self): expected = { 'name': 'verified', 'currency': 'USD', - 'price': int(stock_record.price_excl_tax), + 'price': int(stock_record.price), 'sku': stock_record.partner_sku, 'bulk_sku': None, 'expires': None, @@ -161,7 +161,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': int(stock_record.price), 'sku': stock_record.partner_sku, 'bulk_sku': None, 'expires': None, @@ -177,7 +177,7 @@ def test_serialize_seat_with_enrollment_code(self): expected = { 'name': 'verified', 'currency': 'USD', - 'price': int(stock_record.price_excl_tax), + 'price': int(stock_record.price), 'sku': stock_record.partner_sku, 'bulk_sku': ec_stock_record.partner_sku, 'expires': None, diff --git a/ecommerce/credit/views.py b/ecommerce/credit/views.py index 8fd7acac35d..e62b7abb500 100644 --- a/ecommerce/credit/views.py +++ b/ecommerce/credit/views.py @@ -154,12 +154,12 @@ def _get_providers_detail(self, credit_seats): if code: discount = format_benefit_value(voucher.benefit) if discount_type == 'Percentage': - new_price = stockrecord.price_excl_tax - (stockrecord.price_excl_tax * (discount_value / 100)) + new_price = stockrecord.price - (stockrecord.price * (discount_value / 100)) else: - new_price = stockrecord.price_excl_tax - discount_value + new_price = stockrecord.price - discount_value new_price = '{0:.2f}'.format(new_price) providers_dict[seat.attr.credit_provider].update({ - 'price': stockrecord.price_excl_tax, + 'price': stockrecord.price, 'sku': stockrecord.partner_sku, 'credit_hours': seat.attr.credit_hours, 'discount': discount, diff --git a/ecommerce/enterprise/conditions.py b/ecommerce/enterprise/conditions.py index ced1cdc3dda..7713ca73a8c 100644 --- a/ecommerce/enterprise/conditions.py +++ b/ecommerce/enterprise/conditions.py @@ -80,7 +80,7 @@ def is_offer_max_discount_available(basket, offer): def _get_basket_discount_value(basket, offer): """Calculate the discount value based on benefit type and value""" - sum_basket_lines = basket.all_lines().aggregate(total=Sum('stockrecord__price_excl_tax'))['total'] or Decimal(0.0) + sum_basket_lines = basket.all_lines().aggregate(total=Sum('stockrecord__price'))['total'] or Decimal(0.0) # calculate discount value that will be covered by the offer benefit_type = get_benefit_type(offer.benefit) benefit_value = offer.benefit.value diff --git a/ecommerce/enterprise/tests/test_conditions.py b/ecommerce/enterprise/tests/test_conditions.py index af54b3fe5ed..e9082813383 100644 --- a/ecommerce/enterprise/tests/test_conditions.py +++ b/ecommerce/enterprise/tests/test_conditions.py @@ -49,7 +49,7 @@ def setUp(self): self.user = UserFactory() self.condition = factories.EnterpriseCustomerConditionFactory() - self.test_product = ProductFactory(stockrecords__price_excl_tax=10, categories=[]) + self.test_product = ProductFactory(stockrecords__price=10, categories=[]) self.course_run_1 = CourseFactory(partner=self.partner) self.course_run_1.create_or_update_seat('verified', True, Decimal(100)) @@ -227,7 +227,7 @@ def test_is_satisfied_free_basket(self): offer = factories.EnterpriseOfferFactory(partner=self.partner, condition=self.condition) basket = BasketFactory(site=self.site, owner=self.user) test_product = factories.ProductFactory( - stockrecords__price_excl_tax=0, + stockrecords__price=0, stockrecords__partner__short_code='test' ) basket.add_product(test_product) diff --git a/ecommerce/entitlements/tests/test_utils.py b/ecommerce/entitlements/tests/test_utils.py index 2e3f6568f50..6462ad28bc9 100644 --- a/ecommerce/entitlements/tests/test_utils.py +++ b/ecommerce/entitlements/tests/test_utils.py @@ -19,7 +19,7 @@ def test_course_entitlement_creation(self): self.assertEqual(product.attr.UUID, 'foo-bar') stock_record = StockRecord.objects.get(product=product, partner=self.partner) - self.assertEqual(stock_record.price_excl_tax, 100) + self.assertEqual(stock_record.price, 100) def test_course_entitlement_update(self): """ Test course entitlement product update """ @@ -29,7 +29,7 @@ def test_course_entitlement_update(self): assert product.attr.variant_id == original_variant_id stock_record = StockRecord.objects.get(product=product, partner=self.partner) - self.assertEqual(stock_record.price_excl_tax, 100) + self.assertEqual(stock_record.price, 100) self.assertEqual(product.title, 'Course Foo Bar Entitlement') new_variant_id = '11111111-1111-1111-1111-11111111' @@ -37,8 +37,8 @@ def test_course_entitlement_update(self): 'verified', 200, self.partner, 'foo-bar', 'Foo Bar Entitlement', variant_id=new_variant_id) stock_record = StockRecord.objects.get(product=product, partner=self.partner) - self.assertEqual(stock_record.price_excl_tax, 200) - self.assertEqual(stock_record.price_excl_tax, 200) + self.assertEqual(stock_record.price, 200) + self.assertEqual(stock_record.price, 200) product.refresh_from_db() assert product.attr.variant_id == new_variant_id diff --git a/ecommerce/entitlements/utils.py b/ecommerce/entitlements/utils.py index bd7e3feda6f..10e7c760f99 100644 --- a/ecommerce/entitlements/utils.py +++ b/ecommerce/entitlements/utils.py @@ -94,7 +94,7 @@ def create_or_update_course_entitlement( 'product': course_entitlement, 'partner': partner, 'partner_sku': generate_sku(course_entitlement, partner), - 'price_excl_tax': price, + 'price': price, 'price_currency': settings.OSCAR_DEFAULT_CURRENCY, } ) diff --git a/ecommerce/extensions/analytics/migrations/0003_auto_20230824_1028.py b/ecommerce/extensions/analytics/migrations/0003_auto_20230824_1028.py new file mode 100644 index 00000000000..d3b4008c8e1 --- /dev/null +++ b/ecommerce/extensions/analytics/migrations/0003_auto_20230824_1028.py @@ -0,0 +1,21 @@ +# Generated by Django 3.2.20 on 2023-08-24 10:28 + +from django.db import migrations + + +class Migration(migrations.Migration): + + dependencies = [ + ('analytics', '0002_auto_20140827_1705'), + ] + + operations = [ + migrations.AlterModelOptions( + name='userproductview', + options={'ordering': ['-pk'], 'verbose_name': 'User product view', 'verbose_name_plural': 'User product views'}, + ), + migrations.AlterModelOptions( + name='usersearch', + options={'ordering': ['-pk'], 'verbose_name': 'User search query', 'verbose_name_plural': 'User search queries'}, + ), + ] diff --git a/ecommerce/extensions/api/serializers.py b/ecommerce/extensions/api/serializers.py index 11825a2c891..9336dbfdc41 100644 --- a/ecommerce/extensions/api/serializers.py +++ b/ecommerce/extensions/api/serializers.py @@ -327,18 +327,18 @@ class StockRecordSerializer(serializers.ModelSerializer): class Meta: model = StockRecord - fields = ('id', 'product', 'partner', 'partner_sku', 'price_currency', 'price_excl_tax',) + fields = ('id', 'product', 'partner', 'partner_sku', 'price_currency', 'price',) class PartialStockRecordSerializerForUpdate(StockRecordSerializer): """ Stock record objects serializer for PUT requests. - Allowed fields to update are 'price_currency' and 'price_excl_tax'. + Allowed fields to update are 'price_currency' and 'price'. """ class Meta: model = StockRecord - fields = ('price_currency', 'price_excl_tax',) + fields = ('price_currency', 'price',) class ProductSerializer(ProductPaymentInfoMixin, serializers.HyperlinkedModelSerializer): diff --git a/ecommerce/extensions/api/utils.py b/ecommerce/extensions/api/utils.py index d3ce55b71eb..43776f2fe70 100644 --- a/ecommerce/extensions/api/utils.py +++ b/ecommerce/extensions/api/utils.py @@ -21,7 +21,7 @@ def format_seat(seat): result = seat_template.format( course.name, stock_record.partner_sku, - stock_record.price_excl_tax, + stock_record.price, ) return result diff --git a/ecommerce/extensions/api/v2/tests/views/__init__.py b/ecommerce/extensions/api/v2/tests/views/__init__.py index 866244e6469..1ed62fb054d 100644 --- a/ecommerce/extensions/api/v2/tests/views/__init__.py +++ b/ecommerce/extensions/api/v2/tests/views/__init__.py @@ -87,5 +87,5 @@ def serialize_stockrecord(self, stockrecord): 'product': stockrecord.product.id, 'partner_sku': stockrecord.partner_sku, 'price_currency': stockrecord.price_currency, - 'price_excl_tax': str(stockrecord.price_excl_tax), + 'price': str(stockrecord.price), } diff --git a/ecommerce/extensions/api/v2/tests/views/test_baskets.py b/ecommerce/extensions/api/v2/tests/views/test_baskets.py index 7ac25305a7f..8f5fc8ca030 100644 --- a/ecommerce/extensions/api/v2/tests/views/test_baskets.py +++ b/ecommerce/extensions/api/v2/tests/views/test_baskets.py @@ -79,7 +79,7 @@ def setUp(self): parent=self.base_product, title='LP 560-4', stockrecords__partner_sku=self.PAID_SKU, - stockrecords__price_excl_tax=Decimal('180000.00'), + stockrecords__price=Decimal('180000.00'), stockrecords__partner__short_code='oscr', ) factories.ProductFactory( @@ -87,7 +87,7 @@ def setUp(self): parent=self.base_product, title=u'Papier-mâché', stockrecords__partner_sku=self.ALTERNATE_FREE_SKU, - stockrecords__price_excl_tax=Decimal('0.00'), + stockrecords__price=Decimal('0.00'), stockrecords__partner__short_code='otto', ) factories.ProductFactory( @@ -95,7 +95,7 @@ def setUp(self): parent=self.base_product, title='LP 570-4 Superleggera', stockrecords__partner_sku=self.ALTERNATE_PAID_SKU, - stockrecords__price_excl_tax=Decimal('240000.00'), + stockrecords__price=Decimal('240000.00'), stockrecords__partner__short_code='dummy', ) # Ensure that the basket attribute type exists for these tests @@ -403,7 +403,7 @@ def setUp(self): self.products = ProductFactory.create_batch(3, stockrecords__partner=self.partner, categories=[]) self.path = reverse('api:v2:baskets:calculate') self.range = factories.RangeFactory(includes_all_products=True) - self.product_total = sum(product.stockrecords.first().price_excl_tax for product in self.products) + self.product_total = sum(product.stockrecords.first().price for product in self.products) self.user = self._login_as_user(is_staff=True) self.url = self._generate_sku_url(self.products, username=self.user.username) @@ -597,7 +597,7 @@ def test_basket_calculate_by_staff_user_other_username(self, mock_get_lms_resour products, url = self.setup_other_user_basket_calculate() expected = { - 'total_incl_tax_excl_discounts': sum(product.stockrecords.first().price_excl_tax + 'total_incl_tax_excl_discounts': sum(product.stockrecords.first().price for product in products), 'total_incl_tax': Decimal('0.00'), 'currency': 'USD' @@ -623,7 +623,7 @@ def test_basket_calculate_by_staff_user_other_username_non_atomic( products, url = self.setup_other_user_basket_calculate() expected = { - 'total_incl_tax_excl_discounts': sum(product.stockrecords.first().price_excl_tax + 'total_incl_tax_excl_discounts': sum(product.stockrecords.first().price for product in products), 'total_incl_tax': Decimal('0.00'), 'currency': 'USD' @@ -691,7 +691,7 @@ def test_basket_calculate_anonymous_skip_lms(self, mock_get_lms_resource_for_use products, url = self._setup_anonymous_basket_calculate() expected = { - 'total_incl_tax_excl_discounts': sum(product.stockrecords.first().price_excl_tax + 'total_incl_tax_excl_discounts': sum(product.stockrecords.first().price for product in products), 'total_incl_tax': Decimal('0.00'), 'currency': 'USD' @@ -867,7 +867,7 @@ def test_basket_calculate_by_staff_user_invalid_username(self, mock_get_lms_reso url = self._generate_sku_url(products, username='invalidusername') expected = { - 'total_incl_tax_excl_discounts': sum(product.stockrecords.first().price_excl_tax + 'total_incl_tax_excl_discounts': sum(product.stockrecords.first().price for product in products[1:]), 'total_incl_tax': Decimal('300.00'), 'currency': 'USD' @@ -932,7 +932,7 @@ def _create_program_with_courses_and_offer(self): products.append( factories.ProductFactory( stockrecords__partner=self.partner, - stockrecords__price_excl_tax=Decimal('10.00'), + stockrecords__price=Decimal('10.00'), stockrecords__partner_sku=sku, )) return products, program_uuid diff --git a/ecommerce/extensions/api/v2/tests/views/test_coupons.py b/ecommerce/extensions/api/v2/tests/views/test_coupons.py index 93739a48c7e..c832abc30b3 100644 --- a/ecommerce/extensions/api/v2/tests/views/test_coupons.py +++ b/ecommerce/extensions/api/v2/tests/views/test_coupons.py @@ -682,7 +682,7 @@ def test_update_coupon_price(self): new_coupon = Product.objects.get(id=self.coupon.id) stock_records = StockRecord.objects.filter(product=new_coupon).all() for stock_record in stock_records: - self.assertEqual(stock_record.price_excl_tax, 77) + self.assertEqual(stock_record.price, 77) def test_update_note(self): path = reverse('api:v2:coupons-detail', kwargs={'pk': self.coupon.id}) diff --git a/ecommerce/extensions/api/v2/tests/views/test_orders.py b/ecommerce/extensions/api/v2/tests/views/test_orders.py index 738a071f879..f9492eb95f6 100644 --- a/ecommerce/extensions/api/v2/tests/views/test_orders.py +++ b/ecommerce/extensions/api/v2/tests/views/test_orders.py @@ -148,7 +148,7 @@ def test_orders_api_attributes_for_receipt_mfe( course = CourseFactory(id=course_id, name='Test Course', partner=self.partner) product = factories.ProductFactory( categories=[], - stockrecords__price_excl_tax=price, + stockrecords__price=price, stockrecords__price_currency=currency ) basket = factories.BasketFactory(owner=self.user, site=self.site) @@ -797,14 +797,14 @@ def test_create_manual_order_with_date_placed(self): time_at_initial_price = datetime.now(pytz.utc).isoformat() - stock_record.price_excl_tax = price_1 + stock_record.price = price_1 stock_record.save() - stock_record.price_excl_tax = price_2 + stock_record.price = price_2 stock_record.save() time_at_price_2 = datetime.now(pytz.utc).isoformat() - stock_record.price_excl_tax = final_price + stock_record.price = final_price stock_record.save() time_at_final_price = datetime.now(pytz.utc).isoformat() diff --git a/ecommerce/extensions/api/v2/tests/views/test_publication.py b/ecommerce/extensions/api/v2/tests/views/test_publication.py index c81e1e893f9..88e0ec064c3 100644 --- a/ecommerce/extensions/api/v2/tests/views/test_publication.py +++ b/ecommerce/extensions/api/v2/tests/views/test_publication.py @@ -230,7 +230,7 @@ def assert_entitlement_saved(self, course, expected): self.assertEqual(entitlement.parent.product_class.name, COURSE_ENTITLEMENT_PRODUCT_CLASS_NAME) self.assertEqual(entitlement.attr.certificate_type, certificate_type) self.assertEqual(entitlement.attr.UUID, self.course_uuid) - self.assertEqual(entitlement.stockrecords.get(partner=self.partner).price_excl_tax, expected['price']) + self.assertEqual(entitlement.stockrecords.get(partner=self.partner).price, expected['price']) def assert_seat_saved(self, course, expected): certificate_type = '' @@ -251,7 +251,7 @@ def assert_seat_saved(self, course, expected): # Verify product price and expiration time. expires = EXPIRES if expected['expires'] else None self.assertEqual(seat.expires, expires) - self.assertEqual(seat.stockrecords.get(partner=self.partner).price_excl_tax, expected['price']) + self.assertEqual(seat.stockrecords.get(partner=self.partner).price, expected['price']) return seat diff --git a/ecommerce/extensions/api/v2/tests/views/test_stockrecords.py b/ecommerce/extensions/api/v2/tests/views/test_stockrecords.py index a1cfdcd9b7d..e676c198b3c 100644 --- a/ecommerce/extensions/api/v2/tests/views/test_stockrecords.py +++ b/ecommerce/extensions/api/v2/tests/views/test_stockrecords.py @@ -34,7 +34,7 @@ def test_list(self): """ Verify a list of stock records is returned. """ StockRecordFactory(partner__short_code='Tester') StockRecord.objects.create(partner=self.partner, product=self.product, partner_sku='dummy-sku', - price_currency='USD', price_excl_tax=200.00) + price_currency='USD', price=200.00) response = self.client.get(self.list_path) self.assertEqual(response.status_code, 200) @@ -74,19 +74,19 @@ def test_retrieve_by_sku(self): self.assertDictEqual(response.json(), self.serialize_stockrecord(self.stockrecord)) def test_update(self): - """ Verify update endpoint allows to update 'price_currency' and 'price_excl_tax'. """ + """ Verify update endpoint allows to update 'price_currency' and 'price'. """ self.user.user_permissions.add(self.change_permission) self.user.save() data = { "price_currency": "PKR", - "price_excl_tax": "500.00" + "price": "500.00" } response = self.attempt_update(data) self.assertEqual(response.status_code, 200) stockrecord = StockRecord.objects.get(id=self.stockrecord.id) - self.assertEqual(str(stockrecord.price_excl_tax), data['price_excl_tax']) + self.assertEqual(str(stockrecord.price), data['price']) self.assertEqual(stockrecord.price_currency, data['price_currency']) def test_update_without_permission(self): @@ -96,7 +96,7 @@ def test_update_without_permission(self): data = { "price_currency": "PKR", - "price_excl_tax": "500.00" + "price": "500.00" } response = self.attempt_update(data) self.assertEqual(response.status_code, 403) @@ -107,13 +107,13 @@ def test_update_as_staff(self): self.user.save() data = { - "price_excl_tax": "500.00" + "price": "500.00" } response = self.attempt_update(data) self.assertEqual(response.status_code, 200) def test_allowed_fields_for_update(self): - """ Verify the endpoint only allows the price_excl_tax and price_currency fields to be updated. """ + """ Verify the endpoint only allows the price and price_currency fields to be updated. """ self.user.user_permissions.add(self.change_permission) self.user.save() @@ -125,7 +125,7 @@ def test_allowed_fields_for_update(self): stockrecord = StockRecord.objects.get(id=self.stockrecord.id) self.assertEqual(self.serialize_stockrecord(self.stockrecord), self.serialize_stockrecord(stockrecord)) self.assertDictEqual(response.json(), { - 'message': 'Only the price_currency and price_excl_tax fields are allowed to be modified.'}) + 'message': 'Only the price_currency and price fields are allowed to be modified.'}) def attempt_update(self, data): """ Helper method that attempts to update an existing StockRecord object. @@ -167,7 +167,7 @@ def attempt_create(self): "partner": self.partner.id, "partner_sku": "new-sku", "price_currency": "USD", - "price_excl_tax": 50.00 + "price": 50.00 } return self.client.post(self.list_path, json.dumps(data), JSON_CONTENT_TYPE) diff --git a/ecommerce/extensions/api/v2/views/coupons.py b/ecommerce/extensions/api/v2/views/coupons.py index 16f5a59b536..49b23b9b4f1 100644 --- a/ecommerce/extensions/api/v2/views/coupons.py +++ b/ecommerce/extensions/api/v2/views/coupons.py @@ -467,7 +467,7 @@ def update_coupon_product_data(self, request_data, coupon): coupon_price = request_data.get('price') if coupon_price: - StockRecord.objects.filter(product=coupon).update(price_excl_tax=coupon_price) + StockRecord.objects.filter(product=coupon).update(price=coupon_price) note = request_data.get('note') if note is not None: diff --git a/ecommerce/extensions/api/v2/views/orders.py b/ecommerce/extensions/api/v2/views/orders.py index 17eba2919e5..6f7c60fcd8c 100644 --- a/ecommerce/extensions/api/v2/views/orders.py +++ b/ecommerce/extensions/api/v2/views/orders.py @@ -381,7 +381,7 @@ def _update_order_according_to_date_place(self, order, date_placed): for line in order.lines.all(): old_stock = line.stockrecord.history.filter(history_date__lt=date_placed).order_by('-history_date').first() stock_record = old_stock or line.stockrecord - price = stock_record.price_excl_tax or Decimal('0') + price = stock_record.price or Decimal('0') quantity = line.quantity line.line_price_before_discounts_incl_tax = price * quantity line.line_price_before_discounts_excl_tax = price * quantity diff --git a/ecommerce/extensions/api/v2/views/stockrecords.py b/ecommerce/extensions/api/v2/views/stockrecords.py index 5e22570ae68..31606c23fdb 100644 --- a/ecommerce/extensions/api/v2/views/stockrecords.py +++ b/ecommerce/extensions/api/v2/views/stockrecords.py @@ -39,9 +39,9 @@ def get_serializer_class(self): def update(self, request, *args, **kwargs): """ Update a stock record. """ - allowed_fields = ['price_currency', 'price_excl_tax'] + allowed_fields = ['price_currency', 'price'] if any([key not in allowed_fields for key in request.data.keys()]): return Response({ - 'message': "Only the price_currency and price_excl_tax fields are allowed to be modified." + 'message': "Only the price_currency and price fields are allowed to be modified." }, status=status.HTTP_400_BAD_REQUEST) return super(StockRecordViewSet, self).update(request, *args, **kwargs) diff --git a/ecommerce/extensions/api/v2/views/vouchers.py b/ecommerce/extensions/api/v2/views/vouchers.py index 2c1862112a5..2ab098067e4 100644 --- a/ecommerce/extensions/api/v2/views/vouchers.py +++ b/ecommerce/extensions/api/v2/views/vouchers.py @@ -193,7 +193,7 @@ def convert_catalog_response_to_offers(self, request, voucher, response): credit_provider_price = None else: multiple_credit_providers = False - credit_provider_price = StockRecord.objects.get(product=product).price_excl_tax + credit_provider_price = StockRecord.objects.get(product=product).price try: stock_record = stock_records.get(product__id=product.id) diff --git a/ecommerce/extensions/basket/migrations/0016_alter_lineattribute_value.py b/ecommerce/extensions/basket/migrations/0016_alter_lineattribute_value.py new file mode 100644 index 00000000000..4ec55d70e89 --- /dev/null +++ b/ecommerce/extensions/basket/migrations/0016_alter_lineattribute_value.py @@ -0,0 +1,19 @@ +# Generated by Django 3.2.20 on 2023-08-24 10:28 + +import django.core.serializers.json +from django.db import migrations, models + + +class Migration(migrations.Migration): + + dependencies = [ + ('basket', '0015_add_paymentintentid'), + ] + + operations = [ + migrations.AlterField( + model_name='lineattribute', + name='value', + field=models.JSONField(encoder=django.core.serializers.json.DjangoJSONEncoder, verbose_name='Value'), + ), + ] diff --git a/ecommerce/extensions/basket/models.py b/ecommerce/extensions/basket/models.py index b2441f8d154..8a69c1cac4d 100644 --- a/ecommerce/extensions/basket/models.py +++ b/ecommerce/extensions/basket/models.py @@ -63,7 +63,7 @@ def flush(self): for line in self.all_lines(): # Do not fire events for free items. The volume we see for edX.org leads to a dramatic increase in CPU # usage. Given that orders for free items are ignored, there is no need for these events. - if line.stockrecord.price_excl_tax > 0: + if line.stockrecord.price > 0: properties = translate_basket_line_for_segment(line) track_segment_event(self.site, self.owner, 'Product Removed', properties) product_removed_event_fired = True @@ -104,7 +104,7 @@ def add_product(self, product, quantity=1, options=None): # Do not fire events for free items. The volume we see for edX.org leads to a dramatic increase in CPU # usage. Given that orders for free items are ignored, there is no need for these events. - if line.stockrecord.price_excl_tax > 0: + if line.stockrecord.price > 0: properties = translate_basket_line_for_segment(line) properties['cart_id'] = self.id track_segment_event(self.site, self.owner, 'Product Added', properties) diff --git a/ecommerce/extensions/basket/tests/test_utils.py b/ecommerce/extensions/basket/tests/test_utils.py index 6c4a21abf1c..df14c858a32 100644 --- a/ecommerce/extensions/basket/tests/test_utils.py +++ b/ecommerce/extensions/basket/tests/test_utils.py @@ -77,12 +77,12 @@ def test_add_utm_params_to_url(self): def test_prepare_basket_with_voucher(self): """ Verify a basket is returned and contains a voucher and the voucher is applied. """ # Prepare a product with price of 100 and a voucher with 10% discount for that product. - product = ProductFactory(stockrecords__price_excl_tax=100) + product = ProductFactory(stockrecords__price=100) new_range = RangeFactory(products=[product]) voucher, __ = prepare_voucher(_range=new_range, benefit_value=10) stock_record = StockRecord.objects.get(product=product) - self.assertEqual(stock_record.price_excl_tax, 100.00) + self.assertEqual(stock_record.price, 100.00) basket = prepare_basket(self.request, [product], voucher) self.assertIsNotNone(basket) @@ -113,7 +113,7 @@ def test_prepare_basket_enrollment_with_voucher(self): def test_multiple_vouchers(self): """ Verify only the last entered voucher is contained in the basket. """ - product = ProductFactory(stockrecords__price_excl_tax=100) + product = ProductFactory(stockrecords__price=100) new_range = RangeFactory(products=[product, ]) voucher1, __ = prepare_voucher(code='TEST1', _range=new_range, benefit_value=10) basket = prepare_basket(self.request, [product], voucher1) @@ -411,7 +411,7 @@ def test_prepare_basket_with_bundle_voucher(self): """ Test prepare_basket clears vouchers for a bundle """ - product = ProductFactory(stockrecords__price_excl_tax=100) + product = ProductFactory(stockrecords__price=100) new_range = RangeFactory(products=[product, ]) voucher, __ = prepare_voucher(_range=new_range, benefit_value=10) @@ -541,7 +541,7 @@ def test_prepare_basket_ignores_invalid_voucher(self): """ voucher_start_time = now() - datetime.timedelta(days=5) voucher_end_time = now() - datetime.timedelta(days=3) - product = ProductFactory(stockrecords__partner__short_code='test1', stockrecords__price_excl_tax=100) + product = ProductFactory(stockrecords__partner__short_code='test1', stockrecords__price=100) expired_voucher, __ = prepare_voucher(start_datetime=voucher_start_time, end_datetime=voucher_end_time) basket = prepare_basket(self.request, [product], expired_voucher) @@ -557,7 +557,7 @@ def test_prepare_basket_applies_valid_voucher_argument(self): an argument, even when there is also a valid voucher already on the basket. """ - product = ProductFactory(stockrecords__partner__short_code='test1', stockrecords__price_excl_tax=100) + product = ProductFactory(stockrecords__partner__short_code='test1', stockrecords__price=100) new_range = RangeFactory(products=[product]) new_voucher, __ = prepare_voucher(code='xyz', _range=new_range, benefit_value=10) existing_voucher, __ = prepare_voucher(code='test', _range=new_range, benefit_value=50) @@ -579,7 +579,7 @@ def test_prepare_basket_removes_existing_basket_invalid_voucher(self): """ voucher_start_time = now() - datetime.timedelta(days=5) voucher_end_time = now() - datetime.timedelta(days=3) - product = ProductFactory(stockrecords__partner__short_code='xyz', stockrecords__price_excl_tax=100) + product = ProductFactory(stockrecords__partner__short_code='xyz', stockrecords__price=100) expired_voucher, __ = prepare_voucher(start_datetime=voucher_start_time, end_datetime=voucher_end_time) basket = BasketFactory(owner=self.request.user, site=self.request.site) @@ -596,7 +596,7 @@ def test_prepare_basket_removes_existing_basket_invalid_range_voucher(self): Tests that prepare_basket removes an existing basket voucher that is not valid for the product and used to purchase that product. """ - product = ProductFactory(stockrecords__partner__short_code='xyz', stockrecords__price_excl_tax=100) + product = ProductFactory(stockrecords__partner__short_code='xyz', stockrecords__price=100) invalid_range_voucher, __ = prepare_voucher() basket = BasketFactory(owner=self.request.user, site=self.request.site) @@ -612,7 +612,7 @@ def test_prepare_basket_applies_existing_basket_valid_voucher(self): Tests that prepare_basket applies an existing basket voucher that is valid for multiple products when used to purchase any of those products. """ - product = ProductFactory(stockrecords__partner__short_code='test1', stockrecords__price_excl_tax=100) + product = ProductFactory(stockrecords__partner__short_code='test1', stockrecords__price=100) new_range = RangeFactory(products=[product]) voucher, __ = prepare_voucher(_range=new_range, benefit_value=10) @@ -647,7 +647,7 @@ def test_apply_voucher_on_basket_and_check_discount_with_invalid_voucher(self): does not apply voucher and returns the correct values. """ basket = BasketFactory(owner=self.request.user, site=self.request.site) - product = ProductFactory(stockrecords__partner__short_code='test1', stockrecords__price_excl_tax=100) + product = ProductFactory(stockrecords__partner__short_code='test1', stockrecords__price=100) voucher, __ = prepare_voucher() basket.add_product(product, 1) applied, msg = apply_voucher_on_basket_and_check_discount(voucher, self.request, basket) @@ -661,7 +661,7 @@ def test_apply_voucher_on_basket_and_check_discount_with_invalid_product(self): does not apply voucher and returns the correct values. """ basket = BasketFactory(owner=self.request.user, site=self.request.site) - product = ProductFactory(stockrecords__partner__short_code='test1', stockrecords__price_excl_tax=0) + product = ProductFactory(stockrecords__partner__short_code='test1', stockrecords__price=0) voucher, __ = prepare_voucher(_range=RangeFactory(products=[product])) basket.add_product(product, 1) applied, msg = apply_voucher_on_basket_and_check_discount(voucher, self.request, basket) @@ -675,7 +675,7 @@ def test_apply_voucher_on_basket_and_check_discount_with_multiple_vouchers(self) containing a valid voucher it only checks the new voucher. """ basket = BasketFactory(owner=self.request.user, site=self.request.site) - product = ProductFactory(stockrecords__partner__short_code='test1', stockrecords__price_excl_tax=10) + product = ProductFactory(stockrecords__partner__short_code='test1', stockrecords__price=10) invalid_voucher, __ = prepare_voucher(code='TEST1') valid_voucher, __ = prepare_voucher(code='TEST2', _range=RangeFactory(products=[product])) basket.add_product(product, 1) diff --git a/ecommerce/extensions/basket/tests/test_views.py b/ecommerce/extensions/basket/tests/test_views.py index cbf6c237982..2a099784170 100644 --- a/ecommerce/extensions/basket/tests/test_views.py +++ b/ecommerce/extensions/basket/tests/test_views.py @@ -1593,7 +1593,7 @@ def test_coupon_applied_on_site_offer(self): voucher, product = prepare_voucher(benefit_value=voucher_discount) stockrecord = product.stockrecords.first() - stockrecord.price_excl_tax = product_price + stockrecord.price = product_price stockrecord.save() _range = factories.RangeFactory(includes_all_products=True) diff --git a/ecommerce/extensions/catalogue/management/commands/migrate_course.py b/ecommerce/extensions/catalogue/management/commands/migrate_course.py index dbbf20fa7bd..698dc4b4876 100644 --- a/ecommerce/extensions/catalogue/management/commands/migrate_course.py +++ b/ecommerce/extensions/catalogue/management/commands/migrate_course.py @@ -192,7 +192,7 @@ def handle(self, *args, **options): data = ( getattr(seat.attr, 'certificate_type', ''), seat.attr.id_verification_required, - '{0} {1}'.format(stock_record.price_currency, stock_record.price_excl_tax), + '{0} {1}'.format(stock_record.price_currency, stock_record.price), stock_record.partner_sku, seat.slug, seat.expires diff --git a/ecommerce/extensions/catalogue/migrations/0056_auto_20230824_1028.py b/ecommerce/extensions/catalogue/migrations/0056_auto_20230824_1028.py new file mode 100644 index 00000000000..4b8d20b0fa0 --- /dev/null +++ b/ecommerce/extensions/catalogue/migrations/0056_auto_20230824_1028.py @@ -0,0 +1,143 @@ +# Generated by Django 3.2.20 on 2023-08-24 10:28 + +from django.db import migrations, models +import django.db.models.deletion +import oscar.models.fields.slugfield + + +class Migration(migrations.Migration): + + dependencies = [ + ('catalogue', '0055_sf_opp_line_item_ent_attr'), + ] + + operations = [ + migrations.AlterModelOptions( + name='option', + options={'ordering': ['order', 'name'], 'verbose_name': 'Option', 'verbose_name_plural': 'Options'}, + ), + migrations.AddField( + model_name='category', + name='meta_description', + field=models.TextField(blank=True, null=True, verbose_name='Meta description'), + ), + migrations.AddField( + model_name='category', + name='meta_title', + field=models.CharField(blank=True, max_length=255, null=True, verbose_name='Meta title'), + ), + migrations.AddField( + model_name='historicalcategory', + name='meta_description', + field=models.TextField(blank=True, null=True, verbose_name='Meta description'), + ), + migrations.AddField( + model_name='historicalcategory', + name='meta_title', + field=models.CharField(blank=True, max_length=255, null=True, verbose_name='Meta title'), + ), + migrations.AddField( + model_name='historicaloption', + name='help_text', + field=models.CharField(blank=True, help_text='Help text shown to the user on the add to basket form', max_length=255, null=True, verbose_name='Help text'), + ), + migrations.AddField( + model_name='historicaloption', + name='option_group', + field=models.ForeignKey(blank=True, db_constraint=False, help_text='Select an option group if using type "Option" or "Multi Option"', null=True, on_delete=django.db.models.deletion.DO_NOTHING, related_name='+', to='catalogue.attributeoptiongroup', verbose_name='Option Group'), + ), + migrations.AddField( + model_name='historicaloption', + name='order', + field=models.IntegerField(blank=True, db_index=True, help_text='Controls the ordering of product options on product detail pages', null=True, verbose_name='Ordering'), + ), + migrations.AddField( + model_name='historicaloption', + name='required', + field=models.BooleanField(default=False, verbose_name='Is this option required?'), + ), + migrations.AddField( + model_name='historicalproduct', + name='meta_description', + field=models.TextField(blank=True, null=True, verbose_name='Meta description'), + ), + migrations.AddField( + model_name='historicalproduct', + name='meta_title', + field=models.CharField(blank=True, max_length=255, null=True, verbose_name='Meta title'), + ), + migrations.AddField( + model_name='option', + name='help_text', + field=models.CharField(blank=True, help_text='Help text shown to the user on the add to basket form', max_length=255, null=True, verbose_name='Help text'), + ), + migrations.AddField( + model_name='option', + name='option_group', + field=models.ForeignKey(blank=True, help_text='Select an option group if using type "Option" or "Multi Option"', null=True, on_delete=django.db.models.deletion.CASCADE, related_name='product_options', to='catalogue.attributeoptiongroup', verbose_name='Option Group'), + ), + migrations.AddField( + model_name='option', + name='order', + field=models.IntegerField(blank=True, db_index=True, help_text='Controls the ordering of product options on product detail pages', null=True, verbose_name='Ordering'), + ), + migrations.AddField( + model_name='option', + name='required', + field=models.BooleanField(default=False, verbose_name='Is this option required?'), + ), + migrations.AddField( + model_name='product', + name='meta_description', + field=models.TextField(blank=True, null=True, verbose_name='Meta description'), + ), + migrations.AddField( + model_name='product', + name='meta_title', + field=models.CharField(blank=True, max_length=255, null=True, verbose_name='Meta title'), + ), + migrations.AlterField( + model_name='historicaloption', + name='name', + field=models.CharField(db_index=True, max_length=128, verbose_name='Name'), + ), + migrations.AlterField( + model_name='historicaloption', + name='type', + field=models.CharField(choices=[('text', 'Text'), ('integer', 'Integer'), ('boolean', 'True / False'), ('float', 'Float'), ('date', 'Date'), ('select', 'Select'), ('radio', 'Radio'), ('multi_select', 'Multi select'), ('checkbox', 'Checkbox')], default='text', max_length=255, verbose_name='Type'), + ), + migrations.AlterField( + model_name='historicalproduct', + name='slug', + field=oscar.models.fields.slugfield.SlugField(allow_unicode=True, max_length=255, verbose_name='Slug'), + ), + migrations.AlterField( + model_name='historicalproductattributevalue', + name='value_boolean', + field=models.BooleanField(blank=True, db_index=True, null=True, verbose_name='Boolean'), + ), + migrations.AlterField( + model_name='option', + name='name', + field=models.CharField(db_index=True, max_length=128, verbose_name='Name'), + ), + migrations.AlterField( + model_name='option', + name='type', + field=models.CharField(choices=[('text', 'Text'), ('integer', 'Integer'), ('boolean', 'True / False'), ('float', 'Float'), ('date', 'Date'), ('select', 'Select'), ('radio', 'Radio'), ('multi_select', 'Multi select'), ('checkbox', 'Checkbox')], default='text', max_length=255, verbose_name='Type'), + ), + migrations.AlterField( + model_name='product', + name='slug', + field=oscar.models.fields.slugfield.SlugField(allow_unicode=True, max_length=255, verbose_name='Slug'), + ), + migrations.AlterField( + model_name='productattributevalue', + name='value_boolean', + field=models.BooleanField(blank=True, db_index=True, null=True, verbose_name='Boolean'), + ), + migrations.AlterUniqueTogether( + name='productattribute', + unique_together={('code', 'product_class')}, + ), + ] diff --git a/ecommerce/extensions/catalogue/tests/test_migrate_course.py b/ecommerce/extensions/catalogue/tests/test_migrate_course.py index 340e55f9e52..380f1c450c5 100644 --- a/ecommerce/extensions/catalogue/tests/test_migrate_course.py +++ b/ecommerce/extensions/catalogue/tests/test_migrate_course.py @@ -84,7 +84,7 @@ def _mock_lms_apis(self): def assert_stock_record_valid(self, stock_record, seat, price): """ Verify the given StockRecord is configured correctly. """ self.assertEqual(stock_record.partner, self.partner) - self.assertEqual(stock_record.price_excl_tax, price) + self.assertEqual(stock_record.price, price) self.assertEqual(stock_record.price_currency, 'USD') self.assertEqual(stock_record.partner_sku, generate_sku(seat, self.partner)) diff --git a/ecommerce/extensions/catalogue/utils.py b/ecommerce/extensions/catalogue/utils.py index fceb3a4ba03..d8dd63b7fc4 100644 --- a/ecommerce/extensions/catalogue/utils.py +++ b/ecommerce/extensions/catalogue/utils.py @@ -128,7 +128,7 @@ def create_coupon_product_and_stockrecord(title, category, partner, price): StockRecord.objects.update_or_create( defaults={ 'price_currency': settings.OSCAR_DEFAULT_CURRENCY, - 'price_excl_tax': price + 'price': price }, partner=partner, partner_sku=sku, diff --git a/ecommerce/extensions/checkout/tests/test_mixins.py b/ecommerce/extensions/checkout/tests/test_mixins.py index ab4c232afbc..e14eb71c193 100644 --- a/ecommerce/extensions/checkout/tests/test_mixins.py +++ b/ecommerce/extensions/checkout/tests/test_mixins.py @@ -367,7 +367,7 @@ def test_handle_successful_order_with_email_opt_in(self, expected_opt_in, _): def test_place_free_order(self, __): """ Verify an order is placed and the basket is submitted. """ basket = create_basket(empty=True) - basket.add_product(ProductFactory(stockrecords__price_excl_tax=0)) + basket.add_product(ProductFactory(stockrecords__price=0)) order = EdxOrderPlacementMixin().place_free_order(basket) self.assertIsNotNone(order) @@ -376,7 +376,7 @@ def test_place_free_order(self, __): def test_non_free_basket_order(self, __): """ Verify an error is raised for non-free basket. """ basket = create_basket(empty=True) - basket.add_product(ProductFactory(stockrecords__price_excl_tax=10)) + basket.add_product(ProductFactory(stockrecords__price=10)) with self.assertRaises(BasketNotFreeError): EdxOrderPlacementMixin().place_free_order(basket) diff --git a/ecommerce/extensions/communication/migrations/0002_auto_20230824_1028.py b/ecommerce/extensions/communication/migrations/0002_auto_20230824_1028.py new file mode 100644 index 00000000000..062d0314d91 --- /dev/null +++ b/ecommerce/extensions/communication/migrations/0002_auto_20230824_1028.py @@ -0,0 +1,26 @@ +# Generated by Django 3.2.20 on 2023-08-24 10:28 + +from django.db import migrations, models + + +class Migration(migrations.Migration): + + dependencies = [ + ('communication', '0001_initial'), + ] + + operations = [ + migrations.AlterModelOptions( + name='communicationeventtype', + options={'ordering': ['name'], 'verbose_name': 'Communication event type', 'verbose_name_plural': 'Communication event types'}, + ), + migrations.AlterModelOptions( + name='email', + options={'ordering': ['-date_sent'], 'verbose_name': 'Email', 'verbose_name_plural': 'Emails'}, + ), + migrations.AlterField( + model_name='communicationeventtype', + name='name', + field=models.CharField(db_index=True, max_length=255, verbose_name='Name'), + ), + ] diff --git a/ecommerce/extensions/customer/migrations/0008_auto_20230824_1028.py b/ecommerce/extensions/customer/migrations/0008_auto_20230824_1028.py new file mode 100644 index 00000000000..10a24d499d7 --- /dev/null +++ b/ecommerce/extensions/customer/migrations/0008_auto_20230824_1028.py @@ -0,0 +1,22 @@ +# Generated by Django 3.2.20 on 2023-08-24 10:28 + +from django.db import migrations, models + + +class Migration(migrations.Migration): + + dependencies = [ + ('customer', '0007_auto_20211213_1702'), + ] + + operations = [ + migrations.AlterModelOptions( + name='productalert', + options={'ordering': ['-date_created'], 'verbose_name': 'Product alert', 'verbose_name_plural': 'Product alerts'}, + ), + migrations.AlterField( + model_name='productalert', + name='date_created', + field=models.DateTimeField(auto_now_add=True, db_index=True, verbose_name='Date created'), + ), + ] diff --git a/ecommerce/extensions/executive_education_2u/tests/test_mixins.py b/ecommerce/extensions/executive_education_2u/tests/test_mixins.py index ca1da4afb1e..8721bda1310 100644 --- a/ecommerce/extensions/executive_education_2u/tests/test_mixins.py +++ b/ecommerce/extensions/executive_education_2u/tests/test_mixins.py @@ -53,7 +53,7 @@ def setUp(self): def test_order_note_created(self): basket = create_basket(empty=True) - basket.add_product(ProductFactory(stockrecords__price_excl_tax=0)) + basket.add_product(ProductFactory(stockrecords__price=0)) expected_note = json.dumps({ 'address': self.mock_address, @@ -74,7 +74,7 @@ def test_order_note_created(self): def test_non_free_basket_order(self): basket = create_basket(empty=True) - basket.add_product(ProductFactory(stockrecords__price_excl_tax=10)) + basket.add_product(ProductFactory(stockrecords__price=10)) with self.assertRaises(BasketNotFreeError): ExecutiveEducation2UOrderPlacementMixin().place_free_order( basket, diff --git a/ecommerce/extensions/fulfillment/tests/test_modules.py b/ecommerce/extensions/fulfillment/tests/test_modules.py index 60be1a7c036..0ee1d7b2fc9 100644 --- a/ecommerce/extensions/fulfillment/tests/test_modules.py +++ b/ecommerce/extensions/fulfillment/tests/test_modules.py @@ -596,7 +596,7 @@ def setUp(self): ) user = UserFactory() basket = factories.BasketFactory(owner=user, site=self.site) - factories.create_stockrecord(donation, num_in_stock=2, price_excl_tax=10) + factories.create_stockrecord(donation, num_in_stock=2, price=10) basket.add_product(donation, 1) self.order = create_order(number=1, basket=basket, user=user) diff --git a/ecommerce/extensions/offer/migrations/0055_auto_20230824_1028.py b/ecommerce/extensions/offer/migrations/0055_auto_20230824_1028.py new file mode 100644 index 00000000000..464dfa1d9c0 --- /dev/null +++ b/ecommerce/extensions/offer/migrations/0055_auto_20230824_1028.py @@ -0,0 +1,22 @@ +# Generated by Django 3.2.20 on 2023-08-24 10:28 + +from django.db import migrations, models + + +class Migration(migrations.Migration): + + dependencies = [ + ('offer', '0054_auto_20230601_2037'), + ] + + operations = [ + migrations.AlterModelOptions( + name='range', + options={'ordering': ['name'], 'verbose_name': 'Range', 'verbose_name_plural': 'Ranges'}, + ), + migrations.AddField( + model_name='conditionaloffer', + name='combinations', + field=models.ManyToManyField(blank=True, help_text='Select other non-exclusive offers that this offer can be combined with on the same items', limit_choices_to={'exclusive': False}, related_name='in_combination', to='offer.ConditionalOffer'), + ), + ] diff --git a/ecommerce/extensions/offer/models.py b/ecommerce/extensions/offer/models.py index 8ba3c339e0e..ae6de42206d 100644 --- a/ecommerce/extensions/offer/models.py +++ b/ecommerce/extensions/offer/models.py @@ -198,7 +198,7 @@ def get_applicable_lines(self, offer, basket, range=None): # pylint: disable=re offer.id, applicable_lines ) - return [(line.product.stockrecords.first().price_excl_tax, line) for line in applicable_lines] + return [(line.product.stockrecords.first().price, line) for line in applicable_lines] return super(Benefit, self).get_applicable_lines(offer, basket, range=range) # pylint: disable=bad-super-call diff --git a/ecommerce/extensions/offer/tests/test_dynamic_conditional_offer.py b/ecommerce/extensions/offer/tests/test_dynamic_conditional_offer.py index a3ce50c5a1d..726fa48d074 100644 --- a/ecommerce/extensions/offer/tests/test_dynamic_conditional_offer.py +++ b/ecommerce/extensions/offer/tests/test_dynamic_conditional_offer.py @@ -111,7 +111,7 @@ def test_name(self): {'discount_applicable': False, 'discount_percent': 15}, None,) def test_is_satisfied_true(self, discount_jwt, jwt_decode_handler, request): # pylint: disable=unused-argument - product = ProductFactory(product_class=self.seat_product_class, stockrecords__price_excl_tax=10, categories=[]) + product = ProductFactory(product_class=self.seat_product_class, stockrecords__price=10, categories=[]) self.basket.add_product(product) request.return_value = Mock(method='GET', GET={'discount_jwt': discount_jwt}) @@ -126,7 +126,7 @@ def test_is_satisfied_quantity_more_than_1(self, request): # pylint: disable=u """ This discount should not apply if are buying more than one of the same course. """ - product = ProductFactory(stockrecords__price_excl_tax=10, categories=[]) + product = ProductFactory(stockrecords__price=10, categories=[]) self.basket.add_product(product, quantity=2) self.assertFalse(self.condition.is_satisfied(self.offer, self.basket)) @@ -136,6 +136,6 @@ def test_is_satisfied_not_seat_product(self, request): # pylint: disable=unuse """ This discount should not apply if are not purchasing a seat product. """ - product = ProductFactory(stockrecords__price_excl_tax=10, categories=[]) + product = ProductFactory(stockrecords__price=10, categories=[]) self.basket.add_product(product) self.assertFalse(self.condition.is_satisfied(self.offer, self.basket)) diff --git a/ecommerce/extensions/offer/tests/test_models.py b/ecommerce/extensions/offer/tests/test_models.py index 9806f874e58..dcedc9674c4 100644 --- a/ecommerce/extensions/offer/tests/test_models.py +++ b/ecommerce/extensions/offer/tests/test_models.py @@ -640,7 +640,7 @@ def test_get_applicable_lines(self): basket.add_product(entitlement_product) basket.add_product(seat) - applicable_lines = [(line.product.stockrecords.first().price_excl_tax, line) for line in basket.all_lines()] + applicable_lines = [(line.product.stockrecords.first().price, line) for line in basket.all_lines()] basket.add_product(no_certificate_product) self.mock_access_token_response() diff --git a/ecommerce/extensions/offer/tests/test_utils.py b/ecommerce/extensions/offer/tests/test_utils.py index 080c82ab889..ac7f7035fef 100644 --- a/ecommerce/extensions/offer/tests/test_utils.py +++ b/ecommerce/extensions/offer/tests/test_utils.py @@ -50,7 +50,7 @@ def setUp(self): self.course = CourseFactory(partner=self.partner) self.verified_seat = self.course.create_or_update_seat('verified', False, 100) self.stock_record = StockRecord.objects.filter(product=self.verified_seat).first() - self.seat_price = self.stock_record.price_excl_tax + self.seat_price = self.stock_record.price self._range = RangeFactory(products=[self.verified_seat, ]) self.percentage_benefit = BenefitFactory(type=Benefit.PERCENTAGE, range=self._range, value=35.00) diff --git a/ecommerce/extensions/order/migrations/0026_auto_20230824_1028.py b/ecommerce/extensions/order/migrations/0026_auto_20230824_1028.py new file mode 100644 index 00000000000..9eb1972f85c --- /dev/null +++ b/ecommerce/extensions/order/migrations/0026_auto_20230824_1028.py @@ -0,0 +1,51 @@ +# Generated by Django 3.2.20 on 2023-08-24 10:28 + +import django.core.serializers.json +from django.db import migrations, models + + +class Migration(migrations.Migration): + + dependencies = [ + ('order', '0025_auto_20210922_1857'), + ] + + operations = [ + migrations.AlterModelOptions( + name='orderdiscount', + options={'ordering': ['pk'], 'verbose_name': 'Order Discount', 'verbose_name_plural': 'Order Discounts'}, + ), + migrations.AlterModelOptions( + name='ordernote', + options={'ordering': ['-date_updated'], 'verbose_name': 'Order Note', 'verbose_name_plural': 'Order Notes'}, + ), + migrations.RemoveField( + model_name='historicalline', + name='est_dispatch_date', + ), + migrations.RemoveField( + model_name='historicalline', + name='unit_cost_price', + ), + migrations.RemoveField( + model_name='historicalline', + name='unit_retail_price', + ), + migrations.RemoveField( + model_name='line', + name='est_dispatch_date', + ), + migrations.RemoveField( + model_name='line', + name='unit_cost_price', + ), + migrations.RemoveField( + model_name='line', + name='unit_retail_price', + ), + migrations.AlterField( + model_name='lineattribute', + name='value', + field=models.JSONField(encoder=django.core.serializers.json.DjangoJSONEncoder, verbose_name='Value'), + ), + ] diff --git a/ecommerce/extensions/partner/admin.py b/ecommerce/extensions/partner/admin.py index 8a37912498a..709dc7fe2e4 100644 --- a/ecommerce/extensions/partner/admin.py +++ b/ecommerce/extensions/partner/admin.py @@ -11,7 +11,7 @@ @admin.register(StockRecord) class StockRecordAdminExtended(admin.ModelAdmin): - list_display = ('product', 'partner', 'partner_sku', 'price_excl_tax', 'cost_price', 'num_in_stock') + list_display = ('product', 'partner', 'partner_sku', 'price', 'num_in_stock') list_filter = ('partner',) raw_id_fields = ('product',) diff --git a/ecommerce/extensions/partner/migrations/0019_auto_20230824_1250.py b/ecommerce/extensions/partner/migrations/0019_auto_20230824_1250.py new file mode 100644 index 00000000000..a1de3c8c522 --- /dev/null +++ b/ecommerce/extensions/partner/migrations/0019_auto_20230824_1250.py @@ -0,0 +1,47 @@ +# Generated by Django 3.2.20 on 2023-08-24 12:50 + +from django.db import migrations, models + + +class Migration(migrations.Migration): + + dependencies = [ + ('partner', '0018_remove_partner_enable_sailthru'), + ] + + operations = [ + migrations.RemoveField( + model_name='historicalstockrecord', + name='cost_price', + ), + migrations.RemoveField( + model_name='historicalstockrecord', + name='price_excl_tax', + ), + migrations.RemoveField( + model_name='historicalstockrecord', + name='price_retail', + ), + migrations.RemoveField( + model_name='stockrecord', + name='cost_price', + ), + migrations.RemoveField( + model_name='stockrecord', + name='price_excl_tax', + ), + migrations.RemoveField( + model_name='stockrecord', + name='price_retail', + ), + migrations.AddField( + model_name='historicalstockrecord', + name='price', + field=models.DecimalField(blank=True, decimal_places=2, max_digits=12, null=True, verbose_name='Price'), + ), + migrations.AddField( + model_name='stockrecord', + name='price', + field=models.DecimalField(blank=True, decimal_places=2, max_digits=12, null=True, verbose_name='Price'), + ), + ] diff --git a/ecommerce/extensions/payment/migrations/0033_auto_20230824_1028.py b/ecommerce/extensions/payment/migrations/0033_auto_20230824_1028.py new file mode 100644 index 00000000000..39a5f5929d3 --- /dev/null +++ b/ecommerce/extensions/payment/migrations/0033_auto_20230824_1028.py @@ -0,0 +1,26 @@ +# Generated by Django 3.2.20 on 2023-08-24 10:28 + +from django.db import migrations, models + + +class Migration(migrations.Migration): + + dependencies = [ + ('payment', '0032_alter_source_card_type'), + ] + + operations = [ + migrations.AlterModelOptions( + name='source', + options={'ordering': ['pk'], 'verbose_name': 'Source', 'verbose_name_plural': 'Sources'}, + ), + migrations.AlterModelOptions( + name='sourcetype', + options={'ordering': ['name'], 'verbose_name': 'Source Type', 'verbose_name_plural': 'Source Types'}, + ), + migrations.AlterField( + model_name='sourcetype', + name='name', + field=models.CharField(db_index=True, max_length=128, verbose_name='Name'), + ), + ] diff --git a/ecommerce/extensions/payment/tests/views/test_paypal.py b/ecommerce/extensions/payment/tests/views/test_paypal.py index 65a31efbd17..2008d438ef6 100644 --- a/ecommerce/extensions/payment/tests/views/test_paypal.py +++ b/ecommerce/extensions/payment/tests/views/test_paypal.py @@ -153,7 +153,7 @@ def test_execution_for_bulk_purchase(self): course.create_or_update_seat('verified', True, 50, create_enrollment_code=True) self.basket = create_basket(owner=UserFactory(), site=self.site) enrollment_code = Product.objects.get(product_class__name=ENROLLMENT_CODE_PRODUCT_CLASS_NAME) - factories.create_stockrecord(enrollment_code, num_in_stock=2, price_excl_tax='10.00') + factories.create_stockrecord(enrollment_code, num_in_stock=2, price='10.00') self.basket.add_product(enrollment_code, quantity=1) # Create a payment record the view can use to retrieve a basket diff --git a/ecommerce/extensions/test/factories.py b/ecommerce/extensions/test/factories.py index 511beae1b5c..12aa0f014f0 100644 --- a/ecommerce/extensions/test/factories.py +++ b/ecommerce/extensions/test/factories.py @@ -68,7 +68,7 @@ def create_basket(owner=None, site=None, empty=False, price='10.00', product_cla product = create_product(product_class=product_class_instance) else: product = create_product() - create_stockrecord(product, num_in_stock=2, price_excl_tax=D(price)) + create_stockrecord(product, num_in_stock=2, price=D(price)) basket.add_product(product) return basket diff --git a/ecommerce/extensions/voucher/migrations/0013_auto_20230824_1028.py b/ecommerce/extensions/voucher/migrations/0013_auto_20230824_1028.py new file mode 100644 index 00000000000..3549ff573ee --- /dev/null +++ b/ecommerce/extensions/voucher/migrations/0013_auto_20230824_1028.py @@ -0,0 +1,59 @@ +# Generated by Django 3.2.20 on 2023-08-24 10:28 + +from django.db import migrations, models + + +class Migration(migrations.Migration): + + dependencies = [ + ('voucher', '0012_voucher_is_public'), + ] + + operations = [ + migrations.AlterModelOptions( + name='voucher', + options={'get_latest_by': 'date_created', 'ordering': ['-date_created'], 'verbose_name': 'Voucher', 'verbose_name_plural': 'Vouchers'}, + ), + migrations.AlterModelOptions( + name='voucherapplication', + options={'ordering': ['-date_created'], 'verbose_name': 'Voucher Application', 'verbose_name_plural': 'Voucher Applications'}, + ), + migrations.AlterModelOptions( + name='voucherset', + options={'get_latest_by': 'date_created', 'ordering': ['-date_created'], 'verbose_name': 'VoucherSet', 'verbose_name_plural': 'VoucherSets'}, + ), + migrations.RemoveField( + model_name='voucherset', + name='offer', + ), + migrations.AlterField( + model_name='historicalvoucherapplication', + name='date_created', + field=models.DateTimeField(blank=True, db_index=True, editable=False), + ), + migrations.AlterField( + model_name='voucher', + name='date_created', + field=models.DateTimeField(auto_now_add=True, db_index=True), + ), + migrations.AlterField( + model_name='voucher', + name='name', + field=models.CharField(help_text='This will be shown in the checkout and basket once the voucher is entered', max_length=128, unique=True, verbose_name='Name'), + ), + migrations.AlterField( + model_name='voucherapplication', + name='date_created', + field=models.DateTimeField(auto_now_add=True, db_index=True), + ), + migrations.AlterField( + model_name='voucherset', + name='date_created', + field=models.DateTimeField(auto_now_add=True, db_index=True), + ), + migrations.AlterField( + model_name='voucherset', + name='name', + field=models.CharField(max_length=100, unique=True, verbose_name='Name'), + ), + ] diff --git a/ecommerce/extensions/voucher/tests/test_utils.py b/ecommerce/extensions/voucher/tests/test_utils.py index 6741a79fc9f..dc843152787 100644 --- a/ecommerce/extensions/voucher/tests/test_utils.py +++ b/ecommerce/extensions/voucher/tests/test_utils.py @@ -76,7 +76,7 @@ def setUp(self): self.catalog = Catalog.objects.create(partner=self.partner) self.stock_record = StockRecord.objects.filter(product=self.verified_seat).first() - self.seat_price = self.stock_record.price_excl_tax + self.seat_price = self.stock_record.price self.catalog.stock_records.add(self.stock_record) self.coupon = self.create_coupon( @@ -374,7 +374,7 @@ def assert_report_first_row(self, row, coupon, voucher): if offer.condition.range.catalog: discount_data = get_voucher_discount_info( offer.benefit, - offer.condition.range.catalog.stock_records.first().price_excl_tax + offer.condition.range.catalog.stock_records.first().price ) coupon_type = _('Discount') if discount_data['is_discounted'] else _('Enrollment') discount_percentage = _('{percentage} %').format(percentage=discount_data['discount_percentage']) diff --git a/ecommerce/extensions/voucher/utils.py b/ecommerce/extensions/voucher/utils.py index 6254cf020d8..85556e894a3 100644 --- a/ecommerce/extensions/voucher/utils.py +++ b/ecommerce/extensions/voucher/utils.py @@ -128,7 +128,7 @@ def _get_info_for_coupon_report(coupon, voucher): note = '' coupon_stockrecord = StockRecord.objects.get(product=coupon) - invoiced_amount = currency(coupon_stockrecord.price_excl_tax) + invoiced_amount = currency(coupon_stockrecord.price) offer = voucher.best_offer offer_range = offer.condition.range program_uuid = offer.condition.program_uuid @@ -151,8 +151,8 @@ def _get_info_for_coupon_report(coupon, voucher): course_seat_types = offer_range.course_seat_types if course_id: - price = currency(seat_stockrecord.price_excl_tax) - discount_data = get_voucher_discount_info(benefit, seat_stockrecord.price_excl_tax) + price = currency(seat_stockrecord.price) + discount_data = get_voucher_discount_info(benefit, seat_stockrecord.price) coupon_type, discount_percentage, discount_amount = _get_discount_info(discount_data) else: benefit_type = get_benefit_type(benefit) diff --git a/ecommerce/management/tests/test_utils.py b/ecommerce/management/tests/test_utils.py index 52074217acb..805fa9badcc 100644 --- a/ecommerce/management/tests/test_utils.py +++ b/ecommerce/management/tests/test_utils.py @@ -35,7 +35,7 @@ def test_no_basket_ids(self): def test_success(self): product_price = 100 percentage_discount = 10 - product = ProductFactory(stockrecords__price_excl_tax=product_price) + product = ProductFactory(stockrecords__price=product_price) voucher, product = prepare_voucher(_range=RangeFactory(products=[product]), benefit_value=percentage_discount) self.request.user = UserFactory() basket = prepare_basket(self.request, [product], voucher) @@ -82,7 +82,7 @@ def test_success_with_cybersource(self): """ Test basket with cybersource payment basket.""" product_price = 100 percentage_discount = 10 - product = ProductFactory(stockrecords__price_excl_tax=product_price) + product = ProductFactory(stockrecords__price=product_price) voucher, product = prepare_voucher(_range=RangeFactory(products=[product]), benefit_value=percentage_discount) self.request.user = UserFactory() basket = prepare_basket(self.request, [product], voucher) @@ -210,7 +210,7 @@ def test_when_unable_to_fulfill_basket(self): def test_with_expired_voucher(self): """ Test creates order when called with basket with expired voucher""" basket = create_basket() - product = ProductFactory(stockrecords__price_excl_tax=100, stockrecords__partner=self.partner, + product = ProductFactory(stockrecords__price=100, stockrecords__partner=self.partner, stockrecords__price_currency='USD') voucher, product = prepare_voucher(code='TEST101', _range=RangeFactory(products=[product])) self.request.user = UserFactory() diff --git a/ecommerce/programs/tests/test_conditions.py b/ecommerce/programs/tests/test_conditions.py index 68f3f5c50ad..e17ea36853e 100644 --- a/ecommerce/programs/tests/test_conditions.py +++ b/ecommerce/programs/tests/test_conditions.py @@ -24,7 +24,7 @@ class ProgramCourseRunSeatsConditionTests(ProgramTestMixin, TestCase): def setUp(self): super(ProgramCourseRunSeatsConditionTests, self).setUp() self.condition = factories.ProgramCourseRunSeatsConditionFactory() - self.test_product = ProductFactory(stockrecords__price_excl_tax=10, categories=[]) + self.test_product = ProductFactory(stockrecords__price=10, categories=[]) self.site.siteconfiguration.enable_partial_program = True def test_name(self): @@ -168,7 +168,7 @@ def test_is_satisfied_free_basket(self): """ Ensure the basket returns False if the basket total is zero. """ offer = factories.ProgramOfferFactory(partner=self.partner, condition=self.condition) basket = BasketFactory(site=self.site, owner=UserFactory()) - test_product = factories.ProductFactory(stockrecords__price_excl_tax=0, + test_product = factories.ProductFactory(stockrecords__price=0, stockrecords__partner__short_code='test') basket.add_product(test_product) self.assertFalse(self.condition.is_satisfied(offer, basket)) diff --git a/ecommerce/settings/devstack.py b/ecommerce/settings/devstack.py index a1b8168f790..027bde91723 100644 --- a/ecommerce/settings/devstack.py +++ b/ecommerce/settings/devstack.py @@ -103,9 +103,9 @@ 'log_level': 'debug', 'max_network_retries': 0, 'proxy': None, - 'publishable_key': 'SET-ME-PLEASE', - 'secret_key': 'SET-ME-PLEASE', - 'webhook_endpoint_secret': 'SET-ME-PLEASE', + 'publishable_key': 'pk_test_51Li2KoIadiFyUl1xYMoIUrFL8pcuX4OS61jIbedNXjdqt61JH5ws8owZ9ds3qQ3Gu18cpieLf1UUBPFaoFvWoRxq00I6TPM9cL', + 'secret_key': 'sk_test_51Li2KoIadiFyUl1xx3EfzsxDimQyeR7IzvVBYlpjW4XvLfZ2X9MpDtwexGMaN1eSFOvjxN0zkd1XNWtQGP1NXlgh00kfylIjxu', + 'webhook_endpoint_secret': 'whsec_OH9xwVllArdshfRi08Yyq6bU9eUZnyYK', 'error_path': PAYMENT_PROCESSOR_ERROR_PATH, 'cancel_checkout_path': PAYMENT_PROCESSOR_CANCEL_PATH, 'receipt_url': PAYMENT_PROCESSOR_RECEIPT_PATH, diff --git a/ecommerce/settings/local.py b/ecommerce/settings/local.py index 0e4b2e8ac05..6f6bd52b3d6 100644 --- a/ecommerce/settings/local.py +++ b/ecommerce/settings/local.py @@ -138,9 +138,9 @@ 'log_level': 'debug', 'max_network_retries': 0, 'proxy': None, - 'publishable_key': 'SET-ME-PLEASE', - 'secret_key': 'SET-ME-PLEASE', - 'webhook_endpoint_secret': 'SET-ME-PLEASE', + 'publishable_key': 'pk_test_51Li2KoIadiFyUl1xYMoIUrFL8pcuX4OS61jIbedNXjdqt61JH5ws8owZ9ds3qQ3Gu18cpieLf1UUBPFaoFvWoRxq00I6TPM9cL', + 'secret_key': 'sk_test_51Li2KoIadiFyUl1xx3EfzsxDimQyeR7IzvVBYlpjW4XvLfZ2X9MpDtwexGMaN1eSFOvjxN0zkd1XNWtQGP1NXlgh00kfylIjxu', + 'webhook_endpoint_secret': 'whsec_OH9xwVllArdshfRi08Yyq6bU9eUZnyYK', }, }, } diff --git a/ecommerce/static/js/test/specs/views/offer_view_spec.js b/ecommerce/static/js/test/specs/views/offer_view_spec.js index edd3bd2b9c7..905d7be7781 100644 --- a/ecommerce/static/js/test/specs/views/offer_view_spec.js +++ b/ecommerce/static/js/test/specs/views/offer_view_spec.js @@ -27,7 +27,7 @@ define([ partner: 1, partner_sku: '8CF08E5', price_currency: 'USD', - price_excl_tax: '100.00' + price: '100.00' }, image_url: 'img/src/url', seat_type: 'Not verified', @@ -49,7 +49,7 @@ define([ partner: 1, partner_sku: '8CF08E5', price_currency: 'USD', - price_excl_tax: '100.00' + price: '100.00' }, image_url: 'img/src/url2', seat_type: 'verified', diff --git a/ecommerce/static/js/views/offer_view.js b/ecommerce/static/js/views/offer_view.js index 84bebdc2577..a12f3cd0665 100644 --- a/ecommerce/static/js/views/offer_view.js +++ b/ecommerce/static/js/views/offer_view.js @@ -127,7 +127,7 @@ define([ if (course.get('seat_type') === 'credit' && !course.multiple_credit_providers) { price = parseFloat(course.get('credit_provider_price')).toFixed(2); } else { - price = parseFloat(course.get('stockrecords').price_excl_tax).toFixed(2); + price = parseFloat(course.get('stockrecords').price).toFixed(2); } if (benefit.type === 'Percentage') { @@ -140,7 +140,7 @@ define([ } // eslint-disable-next-line no-param-reassign - course.get('stockrecords').price_excl_tax = price; + course.get('stockrecords').price = price; course.set({new_price: newPrice.toFixed(2)}); }, diff --git a/ecommerce/static/templates/_offer_course_list.html b/ecommerce/static/templates/_offer_course_list.html index 8eb7ffc9142..cf850fbe942 100644 --- a/ecommerce/static/templates/_offer_course_list.html +++ b/ecommerce/static/templates/_offer_course_list.html @@ -52,7 +52,7 @@