fix flake8 failing tests

Author: Abhishek8394

oauth2_provider/oauth2_backends.py

@@ -2,7 +2,9 @@
 fromurllib.parseimporturlparse, urlunparse
 
 fromoauthlibimportoauth2
-fromoauthlib.commonimportquote, urlencode, urlencoded, RequestasOauthlibRequest
+fromoauthlib.commonimportRequestasOauthlibRequest
+fromoauthlib.commonimportquote, urlencode, urlencoded
+
 from .exceptionsimportFatalClientError, OAuthToolkitError
 from .settingsimportoauth2_settings
 
@@ -14,6 +16,7 @@ class OAuthLibCore(object):
  Meant for things like extracting request data and converting
  everything to formats more palatable for oauthlib's Server.
  """
+
 def__init__(self, server=None):
 """
  :params server: An instance of oauthlib.oauth2.Server class
@@ -127,9 +130,11 @@ def create_authorization_response(self, request, scopes, credentials, allow):
 returnuri, headers, body, status
 
 exceptoauth2.FatalClientErroraserror:
-raiseFatalClientError(error=error, redirect_uri=credentials["redirect_uri"])
+raiseFatalClientError(
+error=error, redirect_uri=credentials["redirect_uri"])
 exceptoauth2.OAuth2Erroraserror:
-raiseOAuthToolkitError(error=error, redirect_uri=credentials["redirect_uri"])
+raiseOAuthToolkitError(
+error=error, redirect_uri=credentials["redirect_uri"])
 
 defcreate_token_response(self, request):
 """
@@ -170,12 +175,13 @@ def verify_request(self, request, scopes):
  """
 uri, http_method, body, headers=self._extract_params(request)
 
-valid, r=self.server.verify_request(uri, http_method, body, headers, scopes=scopes)
+valid, r=self.server.verify_request(
+uri, http_method, body, headers, scopes=scopes)
 returnvalid, r
 
 defauthenticate_client(self, request):
 """Wrapper to call `authenticate_client` on `server_class` instance.
-
+
  :param request: The current django.http.HttpRequest object
  """
 uri, http_method, body, headers=self._extract_params(request)
@@ -187,6 +193,7 @@ class JSONOAuthLibCore(OAuthLibCore):
 """
  Extends the default OAuthLibCore to parse correctly application/json requests
  """
+
 defextract_body(self, request):
 """
  Extracts the JSON body from the Django request object

oauth2_provider/views/generic.py

@@ -2,11 +2,11 @@
 
 from ..settingsimportoauth2_settings
 from .mixinsimport (
-OAuthLibMixin,
-ProtectedResourceMixin, ReadWriteScopedResourceMixin, ScopedResourceMixin,
-ClientProtectedResourceMixin
+ClientProtectedResourceMixin, OAuthLibMixin, ProtectedResourceMixin,
+ReadWriteScopedResourceMixin, ScopedResourceMixin
 )
 
+
 classInitializationMixin(OAuthLibMixin):
 
 """Initializer for OauthLibMixin
@@ -16,6 +16,7 @@ class InitializationMixin(OAuthLibMixin):
 validator_class=oauth2_settings.OAUTH2_VALIDATOR_CLASS
 oauthlib_backend_class=oauth2_settings.OAUTH2_BACKEND_CLASS
 
+
 classProtectedResourceView(ProtectedResourceMixin, InitializationMixin, View):
 """
  Generic view protecting resources by providing OAuth2 authentication out of the box
@@ -38,6 +39,7 @@ class ReadWriteScopedResourceView(ReadWriteScopedResourceMixin, ProtectedResourc
  """
 pass
 
+
 classClientProtectedResourceView(ClientProtectedResourceMixin, InitializationMixin, View):
 
 """View for protecting a resource with client-credentials method.
@@ -46,6 +48,7 @@ class ClientProtectedResourceView(ClientProtectedResourceMixin, InitializationMi
 
 pass
 
+
 classClientProtectedScopedResourceView(ScopedResourceMixin, ClientProtectedResourceView):
 
 """Impose scope restrictions if client protection fallsback to access token.

oauth2_provider/views/mixins.py

@@ -177,7 +177,7 @@ def error_response(self, error, **kwargs):
 defauthenticate_client(self, request):
 """Returns a boolean representing if client is authenticated with client credentials
  method. Returns `True` if authenticated.
-
+
  :param request: The current django.http.HttpRequest object
  """
 core=self.get_oauthlib_core()
@@ -209,6 +209,7 @@ class ProtectedResourceMixin(OAuthLibMixin):
  Helper mixin that implements OAuth2 protection on request dispatch,
  specially useful for Django Generic Views
  """
+
 defdispatch(self, request, *args, **kwargs):
 # let preflight OPTIONS requests pass
 ifrequest.method.upper() =="OPTIONS":
@@ -232,12 +233,14 @@ class ReadWriteScopedResourceMixin(ScopedResourceMixin, OAuthLibMixin):
 
 def__new__(cls, *args, **kwargs):
 provided_scopes=get_scopes_backend().get_all_scopes()
-read_write_scopes= [oauth2_settings.READ_SCOPE, oauth2_settings.WRITE_SCOPE]
+read_write_scopes= [oauth2_settings.READ_SCOPE,
+oauth2_settings.WRITE_SCOPE]
 
 ifnotset(read_write_scopes).issubset(set(provided_scopes)):
 raiseImproperlyConfigured(
 "ReadWriteScopedResourceMixin requires following scopes{}"
-' to be in OAUTH2_PROVIDER["SCOPES"] list in settings'.format(read_write_scopes)
+' to be in OAUTH2_PROVIDER["SCOPES"] list in settings'.format(
+read_write_scopes)
  )
 
 returnsuper().__new__(cls, *args, **kwargs)
@@ -256,11 +259,12 @@ def get_scopes(self, *args, **kwargs):
 # this returns a copy so that self.required_scopes is not modified
 returnscopes+ [self.read_write_scope]
 
+
 classClientProtectedResourceMixin(OAuthLibMixin):
 
 """Mixin for protecting resources with client authentication as mentioned in rfc:`3.2.1`
- This involves authenticating with any of: HTTP Basic Auth, Client Credentials and Access token in that order.
- Breaks off after first validation.
+ This involves authenticating with any of: HTTP Basic Auth, Client Credentials and
+Access token in that order. Breaks off after first validation.
  """
 
 defdispatch(self, request, *args, **kwargs):

tests/test_introspection_view.py

@@ -11,6 +11,7 @@
 
 from .utilsimportget_basic_auth_header
 
+
 Application=get_application_model()
 AccessToken=get_access_token_model()
 UserModel=get_user_model()
@@ -20,9 +21,12 @@ class TestTokenIntrospectionViews(TestCase):
 """
  Tests for Authorized Token Introspection Views
  """
+
 defsetUp(self):
-self.resource_server_user=UserModel.objects.create_user("resource_server", "[email protected]")
-self.test_user=UserModel.objects.create_user("bar_user", "[email protected]")
+self.resource_server_user=UserModel.objects.create_user(
+"resource_server", "[email protected]")
+self.test_user=UserModel.objects.create_user(
+"bar_user", "[email protected]")
 
 self.application=Application.objects.create(
 name="Test Application",
@@ -261,7 +265,8 @@ def test_view_post_notexisting_token(self):
 deftest_view_post_valid_client_creds_basic_auth(self):
 """Test HTTP basic auth working
  """
-auth_headers=get_basic_auth_header(self.application.client_id, self.application.client_secret)
+auth_headers=get_basic_auth_header(
+self.application.client_id, self.application.client_secret)
 response=self.client.post(
 reverse("oauth2_provider:introspect"),
 {"token": self.valid_token.token},
@@ -280,7 +285,8 @@ def test_view_post_valid_client_creds_basic_auth(self):
 deftest_view_post_invalid_client_creds_basic_auth(self):
 """Must fail for invalid client credentials
  """
-auth_headers=get_basic_auth_header(self.application.client_id, self.application.client_secret+"_so_wrong")
+auth_headers=get_basic_auth_header(
+self.application.client_id, self.application.client_secret+"_so_wrong")
 response=self.client.post(
 reverse("oauth2_provider:introspect"),
 {"token": self.valid_token.token},