Fixed code quality issues

- Added '.deepsource.toml' file for continuous analysis on bug risk

- Remove `return` from `__init__()` method

- Remove duplicate dictionary key(s)

- Use `max` built-in to get the maximum of two values

- Remove redundant `None` default

- Remove unnecessary comprehension

Signed-off-by: ankitdobhal <[email protected]>

Author: powerexploit

.deepsource.toml

@@ -0,0 +1,14 @@
+version = 1
+
+test_patterns = ["tests/**"]
+
+[[analyzers]]
+name = "python"
+enabled = true
+
+  [analyzers.meta]
+  runtime_version = "3.x.x"
+
+[[analyzers]]
+name = "javascript"
+enabled = true

rest_framework/pagination.py

@@ -488,8 +488,7 @@ def get_html_context(self):
                 _divide_with_ceil(self.offset, self.limit)
             )
 
-            if final < 1:
-                final = 1
+            final = max(final, 1)
         else:
             current = 1
             final = 1

rest_framework/relations.py

@@ -104,11 +104,11 @@ def __init__(self, **kwargs):
             self.html_cutoff_text or _(api_settings.HTML_SELECT_CUTOFF_TEXT)
         )
         if not method_overridden('get_queryset', RelatedField, self):
-            assert self.queryset is not None or kwargs.get('read_only', None), (
+            assert self.queryset is not None or kwargs.get('read_only'), (
                 'Relational field must provide a `queryset` argument, '
                 'override `get_queryset`, or set read_only=`True`.'
             )
-        assert not (self.queryset is not None and kwargs.get('read_only', None)), (
+        assert not (self.queryset is not None and kwargs.get('read_only')), (
             'Relational fields should not provide a `queryset` argument, '
             'when setting read_only=`True`.'
         )
@@ -339,7 +339,7 @@ def get_url(self, obj, view_name, request, format):
         return self.reverse(view_name, kwargs=kwargs, request=request, format=format)
 
     def to_internal_value(self, data):
-        request = self.context.get('request', None)
+        request = self.context.get('request')
         try:
             http_prefix = data.startswith(('http:', 'https:'))
         except AttributeError:
@@ -382,7 +382,7 @@ def to_representation(self, value):
         )
 
         request = self.context['request']
-        format = self.context.get('format', None)
+        format = self.context.get('format')
 
         # By default use whatever format is given for the current context
         # unless the target is a different type to the source.

rest_framework/request.py

@@ -316,7 +316,7 @@ def _supports_form_parsing(self):
             'application/x-www-form-urlencoded',
             'multipart/form-data'
         )
-        return any([parser.media_type in form_media for parser in self.parsers])
+        return any(parser.media_type in form_media for parser in self.parsers)
 
     def _parse(self):
         """

rest_framework/routers.py

@@ -290,7 +290,7 @@ def get(self, request, *args, **kwargs):
                     args=args,
                     kwargs=kwargs,
                     request=request,
-                    format=kwargs.get('format', None)
+                    format=kwargs.get('format')
                 )
             except NoReverseMatch:
                 # Don't bail out if eg. no list routes exist, only detail routes.

rest_framework/templatetags/rest_framework.py

@@ -206,7 +206,7 @@ def format_value(value):
     if value is None or isinstance(value, bool):
         return mark_safe('<code>%s</code>' % {True: 'true', False: 'false', None: 'null'}[value])
     elif isinstance(value, list):
-        if any([isinstance(item, (list, dict)) for item in value]):
+        if any(isinstance(item, (list, dict)) for item in value):
             template = loader.get_template('rest_framework/admin/list_value.html')
         else:
             template = loader.get_template('rest_framework/admin/simple_list_value.html')
@@ -285,7 +285,7 @@ def schema_links(section, sec_key=None):
 def add_nested_class(value):
     if isinstance(value, dict):
         return 'class=nested'
-    if isinstance(value, list) and any([isinstance(item, (list, dict)) for item in value]):
+    if isinstance(value, list) and any(isinstance(item, (list, dict)) for item in value):
         return 'class=nested'
     return ''
 

rest_framework/test.py

@@ -124,7 +124,7 @@ class CoreAPIClient(coreapi.Client):
         def __init__(self, *args, **kwargs):
             self._session = RequestsClient()
             kwargs['transports'] = [coreapi.transports.HTTPTransport(session=self.session)]
-            return super().__init__(*args, **kwargs)
+            super().__init__(*args, **kwargs)
 
         @property
         def session(self):

tests/test_fields.py

@@ -640,8 +640,7 @@ class TestBooleanField(FieldValues):
         '0': False,
         1: True,
         0: False,
-        True: True,
-        False: False,
+
     }
     invalid_inputs = {
         'foo': ['Must be a valid boolean.'],
@@ -654,8 +653,6 @@ class TestBooleanField(FieldValues):
         '0': False,
         1: True,
         0: False,
-        True: True,
-        False: False,
         'other': True
     }
     field = serializers.BooleanField()
@@ -988,8 +985,6 @@ class TestIntegerField(FieldValues):
         '0': 0,
         1: 1,
         0: 0,
-        1.0: 1,
-        0.0: 0,
         '1.0': 1
     }
     invalid_inputs = {
@@ -1002,8 +997,7 @@ class TestIntegerField(FieldValues):
         '0': 0,
         1: 1,
         0: 0,
-        1.0: 1,
-        0.0: 0
+
     }
     field = serializers.IntegerField()
 
@@ -1037,8 +1031,7 @@ class TestFloatField(FieldValues):
         '0': 0.0,
         1: 1.0,
         0: 0.0,
-        1.0: 1.0,
-        0.0: 0.0,
+
     }
     invalid_inputs = {
         'abc': ["A valid number is required."]
@@ -1048,8 +1041,7 @@ class TestFloatField(FieldValues):
         '0': 0.0,
         1: 1.0,
         0: 0.0,
-        1.0: 1.0,
-        0.0: 0.0,
+
     }
     field = serializers.FloatField()
 
@@ -1063,8 +1055,7 @@ class TestMinMaxFloatField(FieldValues):
         '3': 3,
         1: 1,
         3: 3,
-        1.0: 1.0,
-        3.0: 3.0,
+
     }
     invalid_inputs = {
         0.9: ['Ensure this value is greater than or equal to 1.'],