fix: get_response issue for middleware

ecommerce/extensions/analytics/tests/test_middleware.py

@@ -19,7 +19,7 @@ class TrackingMiddlewareTests(TestCase):
 
     def setUp(self):
         super(TrackingMiddlewareTests, self).setUp()
-        self.middleware = middleware.TrackingMiddleware()
+        self.middleware = middleware.TrackingMiddleware(get_response=lambda request: None)
         self.request_factory = RequestFactory()
         self.user = self.create_user()
 

ecommerce/extensions/catalogue/models.py

@@ -98,24 +98,26 @@ def is_executive_education_2u_product(self):
         ]
 
     def save(self, *args, **kwargs):
-        super(Product, self).save(*args, **kwargs)  # pylint: disable=bad-super-call
-        try:
-            if not isinstance(self.attr.note, str) and self.attr.note is not None:
+        if self.id:
+            try:
+                if not isinstance(self.attr.note, str) and self.attr.note is not None:
+                    log_message_and_raise_validation_error(
+                        'Failed to create Product. Product note value must be of type string'
+                    )
+            except AttributeError:
+                pass
+
+            try:
+                if self.attr.notify_email is not None:
+                    validate_email(self.attr.notify_email)
+            except ValidationError:
                 log_message_and_raise_validation_error(
-                    'Failed to create Product. Product note value must be of type string'
+                    'Notification email must be a valid email address.'
                 )
-        except AttributeError:
-            pass
-
-        try:
-            if self.attr.notify_email is not None:
-                validate_email(self.attr.notify_email)
-        except ValidationError:
-            log_message_and_raise_validation_error(
-                'Notification email must be a valid email address.'
-            )
-        except AttributeError:
-            pass
+            except AttributeError:
+                pass
+
+        super(Product, self).save(*args, **kwargs)  # pylint: disable=bad-super-call
 
 
 @receiver(post_init, sender=Product)

ecommerce/extensions/partner/apps.py

@@ -1,7 +1,7 @@
 
 
-from oscar.apps.partner.apps import PartnerConfig as CorePartnerConfig
+from oscar.apps.partner import apps
 
 
-class PartnerConfig(CorePartnerConfig):
+class PartnerConfig(apps.PartnerConfig):
     name = 'ecommerce.extensions.partner'

ecommerce/extensions/payment/core/tests/test_sdn.py

@@ -479,7 +479,7 @@ def test_sdn_is_down_sdn_fallback_called(
         If there is no hit, allow purchase.
         """
         request = RequestFactory().post('/payment/cybersource/submit/')
-        middleware = SessionMiddleware()
+        middleware = SessionMiddleware(get_response=lambda request: None)
         middleware.process_request(request)
         request.session.save()
         site_configuration = self.site.siteconfiguration