Add support for Python 3.8, remove traces of Python 2 (chibisov#282)

* Add support for Python 3.8

* new-style classes

* new-style super

* Remove source encoding

.travis.yml

@@ -7,6 +7,7 @@ python:
   - 3.5
   - 3.6
   - 3.7
+  - 3.8
 
 install:
   - pip install tox tox-travis

docs/backdoc.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 #!/usr/bin/env python
 """
 Backdoc is a tool for backbone-like documentation generation. 
@@ -9,8 +8,6 @@
 import sys
 import argparse
 
-# -*- coding: utf-8 -*-
-#!/usr/bin/env python
 # Copyright (c) 2012 Trent Mick.
 # Copyright (c) 2007-2008 ActiveState Corp.
 # License: MIT (http://www.opensource.org/licenses/mit-license.php)
@@ -185,7 +182,7 @@ def markdown(text, html4tags=False, tab_width=DEFAULT_TAB_WIDTH,
                     link_patterns=link_patterns,
                     use_file_vars=use_file_vars).convert(text)
 
-class Markdown(object):
+class Markdown:
     # The dict of "extras" to enable in processing -- a mapping of
     # extra name to argument for the extra. Most extras do not have an
     # argument, in which case the value is None.
@@ -2095,7 +2092,7 @@ def _dedent(text, tabsize=8, skip_first_line=False):
     return ''.join(lines)
 
 
-class _memoized(object):
+class _memoized:
    """Decorator that caches a function's return value each time it is called.
    If called later with the same arguments, the cached value is returned, and
    not re-evaluated.
@@ -2660,7 +2657,7 @@ def force_text(text):
         return text.decode('utf-8')
 
 
-class BackDoc(object):
+class BackDoc:
     def __init__(self, markdown_converter, template_html, stdin, stdout):
         self.markdown_converter = markdown_converter
         self.template_html = force_text(template_html)

rest_framework_extensions/cache/mixins.py

@@ -2,7 +2,7 @@
 from rest_framework_extensions.settings import extensions_api_settings
 
 
-class BaseCacheResponseMixin(object):
+class BaseCacheResponseMixin:
     # todo: test me. Create generic test like
     # test_cache_reponse(view_instance, method, should_rebuild_after_method_evaluation)
     object_cache_key_func = extensions_api_settings.DEFAULT_OBJECT_CACHE_KEY_FUNC

rest_framework_extensions/etag/decorators.py

@@ -167,15 +167,15 @@ def __init__(self, etag_func=None, rebuild_after_method_evaluation=False, precon
                                                          'the key is the HTTP verb, and the value is a list of '
                                                          'HTTP headers that must all be present for that request.')
 
-        super(APIETAGProcessor, self).__init__(etag_func=etag_func,
+        super().__init__(etag_func=etag_func,
                                                rebuild_after_method_evaluation=rebuild_after_method_evaluation)
 
     def get_etags_and_matchers(self, request):
         """Get the etags from the header and perform a validation against the required preconditions."""
         # evaluate the preconditions, raises 428 if condition is not met
         self.evaluate_preconditions(request)
         # alright, headers are present, extract the values and match the conditions
-        return super(APIETAGProcessor, self).get_etags_and_matchers(request)
+        return super().get_etags_and_matchers(request)
 
     def evaluate_preconditions(self, request):
         """Evaluate whether the precondition for the request is met."""

rest_framework_extensions/etag/mixins.py

@@ -12,7 +12,7 @@ class BaseETAGMixin:
 class ListETAGMixin(BaseETAGMixin):
     @etag(etag_func='list_etag_func')
     def list(self, request, *args, **kwargs):
-        return super(ListETAGMixin, self).list(request, *args, **kwargs)
+        return super().list(request, *args, **kwargs)
 
 
 class RetrieveETAGMixin(BaseETAGMixin):

rest_framework_extensions/fields.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 from rest_framework.relations import HyperlinkedRelatedField
 
 

rest_framework_extensions/key_constructor/bits.py

@@ -7,13 +7,13 @@
 from rest_framework_extensions import compat
 
 
-class AllArgsMixin(object):
+class AllArgsMixin:
 
     def __init__(self, params='*'):
-        super(AllArgsMixin, self).__init__(params)
+        super().__init__(params)
 
 
-class KeyBitBase(object):
+class KeyBitBase:
     def __init__(self, params=None):
         self.params = params
 
@@ -186,7 +186,7 @@ def get_data(self, **kwargs):
                 if param:
                     kwargs['params'].append(param)
 
-        return super(PaginationKeyBit, self).get_data(**kwargs)
+        return super().get_data(**kwargs)
 
 
 class SqlQueryKeyBitBase(KeyBitBase):

rest_framework_extensions/mixins.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 # Try to import six from Django, fallback to included `six`.
 
 

rest_framework_extensions/permissions.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 from rest_framework.permissions import DjangoObjectPermissions
 
 

rest_framework_extensions/routers.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 from rest_framework.routers import DefaultRouter, SimpleRouter
 from rest_framework_extensions.utils import compose_parent_pk_kwarg_name
 

rest_framework_extensions/test.py

@@ -103,7 +103,7 @@ def options(self, path, data=None, format=None, content_type=None, **extra):
         return self.generic('OPTIONS', path, data, content_type, **extra)
 
     def request(self, **kwargs):
-        request = super(APIRequestFactory, self).request(**kwargs)
+        request = super().request(**kwargs)
         request._dont_enforce_csrf_checks = not self.enforce_csrf_checks
         return request
 
@@ -128,7 +128,7 @@ def get_response(self, request):
 
 class APIClient(APIRequestFactory, DjangoClient):
     def __init__(self, enforce_csrf_checks=False, **defaults):
-        super(APIClient, self).__init__(**defaults)
+        super().__init__(**defaults)
         self.handler = ForceAuthClientHandler(enforce_csrf_checks)
         self._credentials = {}
 

tests_app/tests/functional/_examples/etags/remove_etag_gzip_postfix/middleware.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 try:
     from django.utils.deprecation import MiddlewareMixin
 except ImportError:

tests_app/tests/functional/_examples/etags/remove_etag_gzip_postfix/tests.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 from django.test import TestCase, override_settings
 
 

tests_app/tests/functional/_examples/etags/remove_etag_gzip_postfix/urls.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 from django.conf.urls import url
 
 from .views import MyView

tests_app/tests/functional/_examples/etags/remove_etag_gzip_postfix/views.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 from django.views import View
 from django.http import HttpResponse
 

tests_app/tests/functional/cache/decorators/urls.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 from django.conf.urls import url
 
 from .views import HelloView

tests_app/tests/functional/key_constructor/bits/urls.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 from rest_framework import routers
 
 from .views import UserModelViewSet

tests_app/tests/functional/mixins/detail_serializer_mixin/urls.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 from rest_framework import routers
 
 from .views import (

tests_app/tests/functional/mixins/detail_serializer_mixin/views.py

@@ -39,4 +39,4 @@ class CommentWithDetailSerializerAndNoArgsForGetQuerySetViewSet(DetailSerializer
     queryset_detail = Comment.objects.filter(id=1)
 
     def get_queryset(self):
-        return super(CommentWithDetailSerializerAndNoArgsForGetQuerySetViewSet, self).get_queryset()
+        return super().get_queryset()

tests_app/tests/functional/routers/nested_router_mixin/views.py

@@ -80,14 +80,14 @@ class CommentViewSet(NestedViewSetMixin, ModelViewSet):
 
 class TaskCommentViewSet(CommentViewSet):
     def get_queryset(self):
-        return super(TaskCommentViewSet, self).get_queryset().filter(
+        return super().get_queryset().filter(
             content_type=ContentType.objects.get_for_model(TaskModel)
         )
 
 
 class BookCommentViewSet(CommentViewSet):
     def get_queryset(self):
-        return super(BookCommentViewSet, self).get_queryset().filter(
+        return super().get_queryset().filter(
             content_type=ContentType.objects.get_for_model(BookModel)
         )
 

tests_app/tests/unit/_etag/decorators/tests.py

@@ -221,7 +221,7 @@ def run_for_methods(self, methods, condition_failed_status):
 
 class ETAGProcessorTestBehavior_if_none_match(ETAGProcessorTestBehaviorMixin, TestCase):
     def setUp(self):
-        super(ETAGProcessorTestBehavior_if_none_match, self).setUp()
+        super().setUp()
         self.header_name = 'if-none-match'
         self.experiments = [
             {
@@ -275,7 +275,7 @@ def test_for_unsafe_methods(self):
 
 class ETAGProcessorTestBehavior_if_match(ETAGProcessorTestBehaviorMixin, TestCase):
     def setUp(self):
-        super(ETAGProcessorTestBehavior_if_match, self).setUp()
+        super().setUp()
         self.header_name = 'if-match'
         self.experiments = [
             {
@@ -342,7 +342,7 @@ def test_should_raise_assertion_error_if_etag_func_not_specified_decorator(self)
             class View(views.APIView):
                 @api_etag()
                 def get(self, request, *args, **kwargs):
-                    return super(View, self).get(request, *args, **kwargs)
+                    return super().get(request, *args, **kwargs)
 
     def test_should_raise_assertion_error_if_precondition_map_not_a_dict(self):
         with self.assertRaises(AssertionError):
@@ -353,7 +353,7 @@ def test_should_raise_assertion_error_if_precondition_map_not_a_dict_decorator(s
             class View(views.APIView):
                 @api_etag(dummy_api_etag_func, precondition_map=['header-name'])
                 def get(self, request, *args, **kwargs):
-                    return super(View, self).get(request, *args, **kwargs)
+                    return super().get(request, *args, **kwargs)
 
     def test_should_add_object_etag_value(self):
         class TestView(views.APIView):
@@ -608,7 +608,7 @@ def run_for_methods(self, methods, condition_failed_status, experiments=None):
 
 class APIETAGProcessorTestBehavior_if_match(APIETAGProcessorTestBehaviorMixin, TestCase):
     def setUp(self):
-        super(APIETAGProcessorTestBehavior_if_match, self).setUp()
+        super().setUp()
         self.header_name = 'if-match'
         self.experiments = [
             {
@@ -662,7 +662,7 @@ def test_for_all_methods(self):
 
 class APIETAGProcessorTestBehavior_if_none_match(APIETAGProcessorTestBehaviorMixin, TestCase):
     def setUp(self):
-        super(APIETAGProcessorTestBehavior_if_none_match, self).setUp()
+        super().setUp()
         self.header_name = 'if-none-match'
         self.experiments = [
             {

tests_app/tests/unit/cache/decorators/tests.py

@@ -14,7 +14,7 @@
 
 class CacheResponseTest(TestCase):
     def setUp(self):
-        super(CacheResponseTest, self).setUp()
+        super().setUp()
         self.request = factory.get('')
         self.cache = caches[extensions_api_settings.DEFAULT_USE_CACHE]
 
@@ -242,7 +242,7 @@ class TestView(views.APIView):
 
             def __init__(self, status, *args, **kwargs):
                 self.status = status
-                super(TestView, self).__init__(*args, **kwargs)
+                super().__init__(*args, **kwargs)
 
             @cache_response_decorator
             def get(self, request, *args, **kwargs):
@@ -262,7 +262,7 @@ class TestView(views.APIView):
 
             def __init__(self, status, *args, **kwargs):
                 self.status = status
-                super(TestView, self).__init__(*args, **kwargs)
+                super().__init__(*args, **kwargs)
 
             @cache_response_decorator
             def get(self, request, *args, **kwargs):

tests_app/tests/unit/key_constructor/constructor/tests.py

@@ -1,4 +1,3 @@
-# -*- coding: utf-8 -*-
 from copy import deepcopy
 import hashlib
 import json

tox.ini

@@ -1,7 +1,8 @@
 [tox]
 envlist = py{35,36,37}-django{111}-drf39,
-          py{35,36,37}-django{21,22}-drf{39,310,311}
-          py{36,37}-django{30}-drf{310,311}
+          py{35,36,37}-django{21}-drf{39,310,311}
+          py{35,36,37,38}-django{22}-drf{39,310,311}
+          py{36,37,38}-django{30}-drf{310,311}
 
 [testenv]
 deps=
@@ -16,7 +17,7 @@ deps=
     django111: Django>=1.11,<2.0
     django21: Django>=2.1,<2.2
     django22: Django>=2.2,<3.0
-    django30: Django>=3.0a1,<3.1
+    django30: Django>=3.0,<3.1
 setenv =
     PYTHONPATH = {toxinidir}:{toxinidir}/tests_app
 commands =