test_valid_confirmation_email_without_request() - Code Metrics - Inspection of "[WIP] Version 2.0" - sunscrapers/djoser - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#259)

by Piotr

created 2018-01-17 17:37 UTC

test_valid_confirmation_email_without_request() B

↳ Parent: Project

Complexity

Conditions

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes	2
Bugs	0	Features	0

Metric	Value
cc	5
c	2
b	0
f	0
dl	0
loc	10
rs	8.5454

import pytest
class SomeClass:
    def some_method(self):
        """Do x and return foo."""

from djoser import pipelines


def test_valid_activation_email_without_request(test_user, mailoutbox):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    context = {'request': None, 'user': test_user}
    assert test_user.is_active is True

    pipelines.email.activation_email(**context)
    test_user.refresh_from_db()

    assert len(mailoutbox) == 1
    assert mailoutbox[0].to == [test_user.email]
    assert test_user.is_active is False


def test_failed_activation_email_user_without_email(test_user, mailoutbox):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    test_user.email = ''
    context = {'request': None, 'user': test_user}
    pipelines.email.activation_email(**context)
    assert len(mailoutbox) == 0



def test_failed_activation_email_context_missing_user():
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    context = {'request': None}
    with pytest.raises(TypeError) as e:

        pipelines.email.activation_email(**context)

    assert "missing 1 required positional argument: 'user'" in str(e.value)


def test_failed_activation_email_context_missing_request():
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    context = {'user': None}
    with pytest.raises(TypeError) as e:

        pipelines.email.activation_email(**context)

    assert "missing 1 required positional argument: 'request'" in str(e.value)


def test_valid_confirmation_email_without_request(test_user, mailoutbox):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    context = {'request': None, 'user': test_user}
    assert test_user.is_active is True

    pipelines.email.confirmation_email(**context)
    test_user.refresh_from_db()

    assert len(mailoutbox) == 1
    assert mailoutbox[0].to == [test_user.email]
    assert test_user.is_active is True


def test_failed_confirmation_email_user_without_email(test_user, mailoutbox):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    test_user.email = ''
    context = {'request': None, 'user': test_user}
    pipelines.email.confirmation_email(**context)
    assert len(mailoutbox) == 0



def test_failed_confirmation_email_context_missing_user():
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    context = {'request': None}
    with pytest.raises(TypeError) as e:

        pipelines.email.confirmation_email(**context)

    assert "missing 1 required positional argument: 'user'" in str(e.value)


def test_failed_confirmation_email_context_missing_request():
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    context = {'user': None}
    with pytest.raises(TypeError) as e:

        pipelines.email.confirmation_email(**context)

    assert "missing 1 required positional argument: 'request'" in str(e.value)


1			import pytest
			0 ignored issues – show Coding Style introduced 2018-01-15 18:28 UTC by Report Bug Copy Issue Report This module should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history... introduced 2018-01-15 18:28 UTC by Report Bug Copy Issue Report Unable to import 'pytest' Loading history...
2
3			from djoser import pipelines
4
5
6			def test_valid_activation_email_without_request(test_user, mailoutbox):
			0 ignored issues – show Coding Style Naming introduced 2018-01-15 18:28 UTC by Report Bug Copy Issue Report The name `test_valid_activation_email_without_request` does not conform to the function naming conventions (`(([a-z][a-z0-9_]{2,30})\|(_[a-z0-9_]*))$`). This check looks for invalid names for a range of different identifiers. You can set regular expressions to which the identifiers must conform if the defaults do not match your requirements. If your project includes a Pylint configuration file, the settings contained in that file take precedence. To find out more about Pylint, please refer to their site. Loading history... Coding Style introduced 2018-01-15 18:28 UTC by Report Bug Copy Issue Report This function should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
7			context = {'request': None, 'user': test_user}
8			assert test_user.is_active is True
9
10			pipelines.email.activation_email(**context)
11			test_user.refresh_from_db()
12
13			assert len(mailoutbox) == 1
14			assert mailoutbox[0].to == [test_user.email]
15			assert test_user.is_active is False
16
17
18			def test_failed_activation_email_user_without_email(test_user, mailoutbox):
			0 ignored issues – show Coding Style Naming introduced 2018-01-15 18:28 UTC by Report Bug Copy Issue Report The name `test_failed_activation_email_user_without_email` does not conform to the function naming conventions (`(([a-z][a-z0-9_]{2,30})\|(_[a-z0-9_]*))$`). This check looks for invalid names for a range of different identifiers. You can set regular expressions to which the identifiers must conform if the defaults do not match your requirements. If your project includes a Pylint configuration file, the settings contained in that file take precedence. To find out more about Pylint, please refer to their site. Loading history... Coding Style introduced 2018-01-15 18:28 UTC by Report Bug Copy Issue Report This function should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
19			test_user.email = ''
20			context = {'request': None, 'user': test_user}
21			pipelines.email.activation_email(**context)
22			assert len(mailoutbox) == 0
			0 ignored issues – show Unused Code introduced 2018-01-17 13:21 UTC by Report Bug Copy Issue Report Do not use `len(SEQUENCE)` as condition value Loading history...
23
24
25			def test_failed_activation_email_context_missing_user():
			0 ignored issues – show Coding Style Naming introduced 2018-01-17 13:21 UTC by Report Bug Copy Issue Report The name `test_failed_activation_email_context_missing_user` does not conform to the function naming conventions (`(([a-z][a-z0-9_]{2,30})\|(_[a-z0-9_]*))$`). This check looks for invalid names for a range of different identifiers. You can set regular expressions to which the identifiers must conform if the defaults do not match your requirements. If your project includes a Pylint configuration file, the settings contained in that file take precedence. To find out more about Pylint, please refer to their site. Loading history... Coding Style introduced 2018-01-17 13:21 UTC by Report Bug Copy Issue Report This function should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
26			context = {'request': None}
27			with pytest.raises(TypeError) as e:
			0 ignored issues – show Coding Style Naming introduced 2018-01-17 13:21 UTC by Report Bug Copy Issue Report The name `e` does not conform to the variable naming conventions (`(([a-z][a-z0-9_]{2,30})\|(_[a-z0-9_]*))$`). This check looks for invalid names for a range of different identifiers. You can set regular expressions to which the identifiers must conform if the defaults do not match your requirements. If your project includes a Pylint configuration file, the settings contained in that file take precedence. To find out more about Pylint, please refer to their site. Loading history...
28			pipelines.email.activation_email(**context)
29
30			assert "missing 1 required positional argument: 'user'" in str(e.value)
31
32
33			def test_failed_activation_email_context_missing_request():
			0 ignored issues – show Coding Style Naming introduced 2018-01-17 13:21 UTC by Report Bug Copy Issue Report The name `test_failed_activation_e...context_missing_request` does not conform to the function naming conventions (`(([a-z][a-z0-9_]{2,30})\|(_[a-z0-9_]*))$`). This check looks for invalid names for a range of different identifiers. You can set regular expressions to which the identifiers must conform if the defaults do not match your requirements. If your project includes a Pylint configuration file, the settings contained in that file take precedence. To find out more about Pylint, please refer to their site. Loading history... Coding Style introduced 2018-01-17 13:21 UTC by Report Bug Copy Issue Report This function should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
34			context = {'user': None}
35			with pytest.raises(TypeError) as e:
			0 ignored issues – show Coding Style Naming introduced 2018-01-17 13:21 UTC by Report Bug Copy Issue Report The name `e` does not conform to the variable naming conventions (`(([a-z][a-z0-9_]{2,30})\|(_[a-z0-9_]*))$`). This check looks for invalid names for a range of different identifiers. You can set regular expressions to which the identifiers must conform if the defaults do not match your requirements. If your project includes a Pylint configuration file, the settings contained in that file take precedence. To find out more about Pylint, please refer to their site. Loading history...
36			pipelines.email.activation_email(**context)
37
38			assert "missing 1 required positional argument: 'request'" in str(e.value)
39
40
41			def test_valid_confirmation_email_without_request(test_user, mailoutbox):
			0 ignored issues – show Coding Style Naming introduced 2018-01-15 18:28 UTC by Report Bug Copy Issue Report The name `test_valid_confirmation_email_without_request` does not conform to the function naming conventions (`(([a-z][a-z0-9_]{2,30})\|(_[a-z0-9_]*))$`). This check looks for invalid names for a range of different identifiers. You can set regular expressions to which the identifiers must conform if the defaults do not match your requirements. If your project includes a Pylint configuration file, the settings contained in that file take precedence. To find out more about Pylint, please refer to their site. Loading history... Coding Style introduced 2018-01-15 18:28 UTC by Report Bug Copy Issue Report This function should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
42			context = {'request': None, 'user': test_user}
43			assert test_user.is_active is True
44
45			pipelines.email.confirmation_email(**context)
46			test_user.refresh_from_db()
47
48			assert len(mailoutbox) == 1
49			assert mailoutbox[0].to == [test_user.email]
50			assert test_user.is_active is True
51
52
53			def test_failed_confirmation_email_user_without_email(test_user, mailoutbox):
			0 ignored issues – show Coding Style Naming introduced 2018-01-15 18:28 UTC by Report Bug Copy Issue Report The name `test_failed_confirmation_email_user_without_email` does not conform to the function naming conventions (`(([a-z][a-z0-9_]{2,30})\|(_[a-z0-9_]*))$`). This check looks for invalid names for a range of different identifiers. You can set regular expressions to which the identifiers must conform if the defaults do not match your requirements. If your project includes a Pylint configuration file, the settings contained in that file take precedence. To find out more about Pylint, please refer to their site. Loading history... Coding Style introduced 2018-01-15 18:28 UTC by Report Bug Copy Issue Report This function should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
54			test_user.email = ''
55			context = {'request': None, 'user': test_user}
56			pipelines.email.confirmation_email(**context)
57			assert len(mailoutbox) == 0
			0 ignored issues – show Unused Code introduced 2018-01-17 13:21 UTC by Report Bug Copy Issue Report Do not use `len(SEQUENCE)` as condition value Loading history...
58
59
60			def test_failed_confirmation_email_context_missing_user():
			0 ignored issues – show Coding Style Naming introduced 2018-01-17 13:21 UTC by Report Bug Copy Issue Report The name `test_failed_confirmation...il_context_missing_user` does not conform to the function naming conventions (`(([a-z][a-z0-9_]{2,30})\|(_[a-z0-9_]*))$`). This check looks for invalid names for a range of different identifiers. You can set regular expressions to which the identifiers must conform if the defaults do not match your requirements. If your project includes a Pylint configuration file, the settings contained in that file take precedence. To find out more about Pylint, please refer to their site. Loading history... Coding Style introduced 2018-01-17 13:21 UTC by Report Bug Copy Issue Report This function should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
61			context = {'request': None}
62			with pytest.raises(TypeError) as e:
			0 ignored issues – show Coding Style Naming introduced 2018-01-17 13:21 UTC by Report Bug Copy Issue Report The name `e` does not conform to the variable naming conventions (`(([a-z][a-z0-9_]{2,30})\|(_[a-z0-9_]*))$`). This check looks for invalid names for a range of different identifiers. You can set regular expressions to which the identifiers must conform if the defaults do not match your requirements. If your project includes a Pylint configuration file, the settings contained in that file take precedence. To find out more about Pylint, please refer to their site. Loading history...
63			pipelines.email.confirmation_email(**context)
64
65			assert "missing 1 required positional argument: 'user'" in str(e.value)
66
67
68			def test_failed_confirmation_email_context_missing_request():
			0 ignored issues – show Coding Style Naming introduced 2018-01-17 13:21 UTC by Report Bug Copy Issue Report The name `test_failed_confirmation...context_missing_request` does not conform to the function naming conventions (`(([a-z][a-z0-9_]{2,30})\|(_[a-z0-9_]*))$`). This check looks for invalid names for a range of different identifiers. You can set regular expressions to which the identifiers must conform if the defaults do not match your requirements. If your project includes a Pylint configuration file, the settings contained in that file take precedence. To find out more about Pylint, please refer to their site. Loading history... Coding Style introduced 2018-01-17 13:21 UTC by Report Bug Copy Issue Report This function should have a docstring. The coding style of this project requires that you add a docstring to this code element. Below, you find an example for methods: class SomeClass: def some_method(self): """Do x and return foo.""" If you would like to know more about docstrings, we recommend to read PEP-257: Docstring Conventions. Loading history...
69			context = {'user': None}
70			with pytest.raises(TypeError) as e:
			0 ignored issues – show Coding Style Naming introduced 2018-01-17 13:21 UTC by Report Bug Copy Issue Report The name `e` does not conform to the variable naming conventions (`(([a-z][a-z0-9_]{2,30})\|(_[a-z0-9_]*))$`). This check looks for invalid names for a range of different identifiers. You can set regular expressions to which the identifiers must conform if the defaults do not match your requirements. If your project includes a Pylint configuration file, the settings contained in that file take precedence. To find out more about Pylint, please refer to their site. Loading history...
71			pipelines.email.confirmation_email(**context)
72
73			assert "missing 1 required positional argument: 'request'" in str(e.value)
			0 ignored issues – show Coding Style introduced 2018-01-17 13:21 UTC by Report Bug Copy Issue Report Final newline missing Loading history...

sunscrapers / djoser

Pull Request — master (#259)

test_valid_confirmation_email_without_request() B

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like