TokenCreateViewTest

Complexity

Total Complexity

7

Size/Duplication

Total Lines	81
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	test_post_should_not_login_if_invalid_credentials()	0	17	1
A	setUp()	0	2	1
A	signal_receiver()	0	2	1
A	test_post_should_login_user()	0	14	1
A	test_post_should_not_login_if_empty_request()	0	10	1
B	test_post_should_not_login_if_user_is_not_active()	0	26	2

from django.contrib.auth import user_logged_in, user_login_failed

This module should have a docstring.

Unable to import 'django.contrib.auth'

from djet import assertions, restframework

Unable to import 'djet'

from rest_framework import status

Unable to import 'rest_framework'

import django

Unable to import 'django'

Imports from package django are not grouped

import djoser.constants
import djoser.utils
import djoser.views

from .common import create_user


class TokenCreateViewTest(restframework.APIViewTestCase,

This class should have a docstring.

                          assertions.StatusCodeAssertionsMixin,
                          assertions.InstanceAssertionsMixin):
    view_class = djoser.views.TokenCreateView

    def setUp(self):

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

This method should have a docstring.

        self.signal_sent = False

    def signal_receiver(self, *args, **kwargs):

This method should have a docstring.

The argument args seems to be unused.

The argument kwargs seems to be unused.

        self.signal_sent = True

    def test_post_should_login_user(self):

This method should have a docstring.

        user = create_user()
        data = {
            'username': user.username,
            'password': user.raw_password,
        }
        user_logged_in.connect(self.signal_receiver)
        request = self.factory.post(data=data)

        response = self.view(request)

        self.assert_status_equal(response, status.HTTP_200_OK)
        self.assertEqual(response.data['auth_token'], user.auth_token.key)
        self.assertTrue(self.signal_sent)

    def test_post_should_not_login_if_user_is_not_active(self):

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

        """
        In Django >= 1.10 authenticate() returns None if
        user is inactive, while in Django < 1.10 authenticate()
        succeeds if user is inactive.
        """
        user = create_user()
        data = {
            'username': user.username,
            'password': user.raw_password,
        }
        user.is_active = False
        user.save()
        user_logged_in.connect(self.signal_receiver)
        request = self.factory.post(data=data)

        response = self.view(request)

        if django.VERSION >= (1, 10):
            expected_errors = [djoser.constants.INVALID_CREDENTIALS_ERROR]
        else:
            expected_errors = [djoser.constants.INACTIVE_ACCOUNT_ERROR]

        self.assert_status_equal(response, status.HTTP_400_BAD_REQUEST)
        self.assertEqual(response.data['non_field_errors'], expected_errors)
        self.assertFalse(self.signal_sent)

    def test_post_should_not_login_if_invalid_credentials(self):

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

This method should have a docstring.

        user = create_user()
        data = {
            'username': user.username,
            'password': 'wrong',
        }
        user_login_failed.connect(self.signal_receiver)
        request = self.factory.post(data=data)

        response = self.view(request)

        self.assert_status_equal(response, status.HTTP_400_BAD_REQUEST)
        self.assertEqual(
            response.data['non_field_errors'],
            [djoser.constants.INVALID_CREDENTIALS_ERROR]
        )
        self.assertTrue(self.signal_sent)

    def test_post_should_not_login_if_empty_request(self):

The name test_post_should_not_login_if_empty_request 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 = {}
        request = self.factory.post(data=data)

        response = self.view(request)

        self.assert_status_equal(response, status.HTTP_400_BAD_REQUEST)
        self.assertEqual(
            response.data['non_field_errors'],
            [djoser.constants.INVALID_CREDENTIALS_ERROR]
        )