ProviderAuthViewTestCase

Complexity

Total Complexity

9

Size/Duplication

Total Lines	73
Duplicated Lines	45.21 %

Importance

Changes	3
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	test_post_facebook_provider_success_returns_token()	17	17	2
A	test_get_facebook_provider_fails_if_wrong_redirect_uri()	0	5	1
A	test_get_facebook_provider_fails_if_no_redirect_uri()	0	5	1
A	test_post_facebook_provider_code_validation_fails()	16	16	2
A	test_post_facebook_provider_validation_fails_if_invalid_state()	0	12	2
A	test_get_facebook_provider_provides_valid_authorization_url()	0	8	1

from django.contrib.sessions.middleware import SessionMiddleware

This module should have a docstring.

Unable to import 'django.contrib.sessions.middleware'

from django.utils import six

Unable to import 'django.utils'

from rest_framework import status

Unable to import 'rest_framework'

from djet import assertions, restframework

Unable to import 'djet'

import djoser.social.views
from social_core.exceptions import AuthException

Unable to import 'social_core.exceptions'

from ..common import create_user, mock


class ProviderAuthViewTestCase(restframework.APIViewTestCase,

This class should have a docstring.

                               assertions.StatusCodeAssertionsMixin):
    view_class = djoser.social.views.ProviderAuthView
    middleware = [SessionMiddleware]

    def test_get_facebook_provider_fails_if_no_redirect_uri(self):

The name test_get_facebook_provider_fails_if_no_redirect_uri does not conform to the method naming conventions ((([a-z][a-z0-9_]{2,30})|(_[a-z0-9_]*))$).

This method should have a docstring.

        request = self.factory.get()
        response = self.view(request, provider='facebook')

        self.assert_status_equal(response, status.HTTP_400_BAD_REQUEST)

    def test_get_facebook_provider_fails_if_wrong_redirect_uri(self):

The name test_get_facebook_provider_fails_if_wrong_redirect_uri does not conform to the method naming conventions ((([a-z][a-z0-9_]{2,30})|(_[a-z0-9_]*))$).

This method should have a docstring.

        request = self.factory.get(data={'redirect_uri': 'http://yolo.com/'})
        response = self.view(request, provider='facebook')

        self.assert_status_equal(response, status.HTTP_400_BAD_REQUEST)

    def test_get_facebook_provider_provides_valid_authorization_url(self):

The name test_get_facebook_provider_provides_valid_authorization_url does not conform to the method naming conventions ((([a-z][a-z0-9_]{2,30})|(_[a-z0-9_]*))$).

This method should have a docstring.

        request = self.factory.get(data={
            'redirect_uri': 'http://test.localhost/'
        })
        response = self.view(request, provider='facebook')

        self.assert_status_equal(response, status.HTTP_200_OK)
        self.assertIn('authorization_url', response.data)

    def test_post_facebook_provider_success_returns_token(self):

The name test_post_facebook_provider_success_returns_token does not conform to the method naming conventions ((([a-z][a-z0-9_]{2,30})|(_[a-z0-9_]*))$).

This method should have a docstring.

This code seems to be duplicated in your project.

        data = {'code': 'XYZ', 'state': 'ABC'}

        mock.patch(
            'social_core.backends.facebook.FacebookOAuth2.auth_complete',
            return_value=create_user()
        ).start()
        mock.patch(
            'social_core.backends.oauth.OAuthAuth.get_session_state',
            return_value=data['state']
        ).start()

        request = self.factory.post()
        request.GET = {k: v for k, v in six.iteritems(data)}
        response = self.view(request, provider='facebook')
        self.assert_status_equal(response, status.HTTP_201_CREATED)
        self.assertEqual(set(response.data.keys()), {'token', 'user'})

    def test_post_facebook_provider_code_validation_fails(self):

The name test_post_facebook_provider_code_validation_fails does not conform to the method naming conventions ((([a-z][a-z0-9_]{2,30})|(_[a-z0-9_]*))$).

This method should have a docstring.

This code seems to be duplicated in your project.

        data = {'code': 'XYZ', 'state': 'ABC'}

        mock.patch(
            'social_core.backends.facebook.FacebookOAuth2.auth_complete',
            side_effect=AuthException(backend=None)
        ).start()
        mock.patch(
            'social_core.backends.oauth.OAuthAuth.get_session_state',
            return_value=data['state']
        ).start()

        request = self.factory.post()
        request.GET = {k: v for k, v in six.iteritems(data)}
        response = self.view(request, provider='facebook')
        self.assert_status_equal(response, status.HTTP_400_BAD_REQUEST)

    def test_post_facebook_provider_validation_fails_if_invalid_state(self):

The name test_post_facebook_provider_validation_fails_if_invalid_state does not conform to the method naming conventions ((([a-z][a-z0-9_]{2,30})|(_[a-z0-9_]*))$).

This method should have a docstring.

        data = {'code': 'XYZ', 'state': 'ABC'}

        mock.patch(
            'social_core.backends.oauth.OAuthAuth.get_session_state',
            return_value=data['state'][::-1]
        ).start()

        request = self.factory.post()
        request.GET = {k: v for k, v in six.iteritems(data)}
        response = self.view(request, provider='facebook')
        self.assert_status_equal(response, status.HTTP_400_BAD_REQUEST)