[Fix] Resolve lint errors (#328)

* fix 🔧 resolve lint errors

* fix 🔧 resolve pylint errors

* fix 🔧 resolve tests failing

.gitignore

@@ -17,4 +17,5 @@ version-django-oscar-commerce-connect.txt
 sandbox/media/
 sandbox/static/
 docs/build/
-lint.installed
+lint.installed
+.vscode

oscarapi/fixtures/offer.xml

@@ -27,6 +27,7 @@
         <field type="PositiveIntegerField" name="num_applications">0</field>
         <field type="PositiveIntegerField" name="num_orders">0</field>
         <field type="CharField" name="redirect_url"/>
+        <field type="BooleanField" name="exclusive">False</field>
         <field type="DateTimeField" name="date_created">2015-11-16T11:08:15.653721+00:00</field>
     </object>
     <object pk="1" model="offer.benefit">

oscarapi/fixtures/voucher.xml

@@ -19,6 +19,7 @@
     <field type="PositiveIntegerField" name="num_applications">0</field>
     <field type="PositiveIntegerField" name="num_orders">0</field>
     <field type="CharField" name="redirect_url"></field>
+    <field type="BooleanField" name="exclusive">False</field>
     <field type="DateTimeField" name="date_created">2015-11-27T15:38:48.134807+00:00</field>
   </object>
   <object pk="2" model="offer.benefit">

oscarapi/middleware.py

@@ -5,7 +5,7 @@
 from django.contrib.sessions.middleware import SessionMiddleware
 from django.core.exceptions import PermissionDenied
 from django.http.response import HttpResponse
-from django.utils.translation import ugettext as _
+from django.utils.translation import gettext as _
 
 from oscar.core.loading import get_class
 

oscarapi/settings.py

@@ -5,6 +5,7 @@
 `OSCARAPI_`. So :const:`oscarapi.settings.EXPOSE_USER_DETAILS` becomes:
 `OSCARAPI_EXPOSE_USER_DETAILS`
 """
+
 from django.contrib.auth import get_user_model
 from oscarapi.utils.settings import overridable
 from oscarapi import version

oscarapi/signals.py

@@ -1,10 +1,4 @@
-import django
 from django.dispatch import Signal
 
 
-oscarapi_post_checkout_args = ["order", "user", "request", "response"]
-
-if django.VERSION >= (3, 0):
-    oscarapi_post_checkout = Signal()
-else:
-    oscarapi_post_checkout = Signal(providing_args=oscarapi_post_checkout_args)
+oscarapi_post_checkout = Signal()

oscarapi/tests/unit/testmiddleware.py

@@ -105,7 +105,7 @@ def test_process_request(self):
         request = self.rf.get(
             basket_url, HTTP_SESSION_ID="SID:ANON:example.com:987171879"
         )
-        response = HeaderSessionMiddleware().process_request(request)
+        response = HeaderSessionMiddleware(lambda: None).process_request(request)
         self.assertEqual(response.status_code, 403)
         self.assertEqual(
             response.content,

oscarapi/tests/unit/testproduct.py

@@ -1173,8 +1173,9 @@ def test_add_local_image(self):
                 "description": "Henk",
                 "images": [
                     {
-                        "original": "https://testserver/%s/image.jpg"
-                        % settings.MEDIA_ROOT,
+                        "original": (
+                            "https://testserver/%s/image.jpg" % settings.MEDIA_ROOT
+                        ),
                         "caption": "HA! IK HEET HARRIE",
                     }
                 ],
@@ -1210,7 +1211,9 @@ def test_add_broken_image(self, urlopen):
                 "description": "Henk",
                 "images": [
                     {
-                        "original": "https://example.com/image-that-does-not-exist-at-all.png",
+                        "original": (
+                            "https://example.com/image-that-does-not-exist-at-all.png"
+                        ),
                         "caption": "HA! IK HEET HARRIE",
                     }
                 ],

oscarapi/utils/exists.py

@@ -2,6 +2,7 @@
 This module contains functions that can be used to identify an existing piece
 of data in the database based on it's unique attributes
 """
+
 from django.db import models
 
 from oscar.core.loading import get_model