tracim.views.default.default_controller.DefaultController.test_admin_page() - Code Metrics - Inspection of "disable swagger doc" - tracim/tracim_backend - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( c55bf2...b94d48 )

unknown

created 2018-04-09 12:59 UTC

DefaultController.test_admin_page() A

↳ Parent: tracim.views.default.default_controller

Complexity

Conditions

Size

Total Lines	8
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
eloc	8
dl	0
loc	8
rs	9.4285
c	0
b	0
f	0
cc	2
nop	2

# coding=utf-8
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
from tracim.views.controllers import Controller
from pyramid.config import Configurator
from pyramid.response import Response
from pyramid.exceptions import NotFound
from pyramid.httpexceptions import HTTPUnauthorized
from pyramid.httpexceptions import HTTPForbidden
from pyramid.security import forget

from tracim.lib.utils.auth import MANAGE_CONTENT_PERM

from tracim.lib.utils.auth import MANAGE_WORKSPACE_PERM

from tracim.lib.utils.auth import MANAGE_GLOBAL_PERM
from tracim.lib.utils.auth import READ_PERM

from tracim.lib.utils.auth import CONTRIBUTE_PERM
from tracim.lib.utils.auth import ADMIN_PERM
from tracim.lib.utils.auth import USER_PERM


class DefaultController(Controller):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""

    @classmethod
    def notfound_view(cls, request):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        request.response.status = 404
        return {}

    @classmethod
    def forbidden_view(cls, request):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        if request.authenticated_userid is None:
            response = HTTPUnauthorized()
            response.headers.update(forget(request))

        # user is logged in but doesn't have permissions, reject wholesale
        else:
            response = HTTPForbidden()
        return response

    @classmethod
    def test_config(cls, request):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        try:
            app_config = request.registry.settings['CFG']
            project = app_config.WEBSITE_TITLE
        except Exception as e:

            return Response(e, content_type='text/plain', status=500)
        return {'project': project}

    @classmethod
    def test_contributor_page(cls, request):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        try:
            app_config = request.registry.settings['CFG']

            project = 'contributor'
        except Exception as e:

            return Response(e, content_type='text/plain', status=500)
        return {'project': project}

    @classmethod
    def test_admin_page(cls, request):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        try:
            app_config = request.registry.settings['CFG']

            project = 'admin'
        except Exception as e:

            return Response(e, content_type='text/plain', status=500)
        return {'project': project}

    @classmethod
    def test_manager_page(cls, request):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        try:
            app_config = request.registry.settings['CFG']

            project = 'manager'
        except Exception as e:

            return Response(e, content_type='text/plain', status=500)
        return {'project': project}

    @classmethod
    def test_user_page(cls, request):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        try:
            app_config = request.registry.settings['CFG']

            project = 'user'
        except Exception as e:

            return Response(e, content_type='text/plain', status=500)
        return {'project': project}

    def bind(self, configurator: Configurator):
        configurator.add_static_view('static', 'static', cache_max_age=3600)
        configurator.add_view(
            self.notfound_view,
            renderer='tracim:templates/404.jinja2',
            context=NotFound,
        )

        configurator.add_route('test_config', '/')
        configurator.add_view(
            self.test_config,
            route_name='test_config',
            renderer='tracim:templates/mytemplate.jinja2',
        )

        configurator.add_route('test_contributor', '/test_contributor')
        configurator.add_view(
            self.test_contributor_page,
            route_name='test_contributor',
            renderer='tracim:templates/mytemplate.jinja2',
            permission=CONTRIBUTE_PERM,
        )
        configurator.add_route('test_admin', '/test_admin')
        configurator.add_view(
            self.test_admin_page,
            route_name='test_admin',
            renderer='tracim:templates/mytemplate.jinja2',
            permission=ADMIN_PERM,
        )
        configurator.add_route('test_manager', '/test_manager')
        configurator.add_view(
            self.test_user_page,
            route_name='test_manager',
            renderer='tracim:templates/mytemplate.jinja2',
            permission=MANAGE_GLOBAL_PERM,
        )
        configurator.add_route('test_user', '/test_user')
        configurator.add_view(
            self.test_user_page,
            route_name='test_user',
            renderer='tracim:templates/mytemplate.jinja2',
            permission=USER_PERM,
        )
        configurator.add_forbidden_view(self.forbidden_view)


1			# coding=utf-8
			0 ignored issues – show Coding Style introduced 2018-04-06 10:21 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...
2			from tracim.views.controllers import Controller
3			from pyramid.config import Configurator
4			from pyramid.response import Response
5			from pyramid.exceptions import NotFound
6			from pyramid.httpexceptions import HTTPUnauthorized
7			from pyramid.httpexceptions import HTTPForbidden
8			from pyramid.security import forget
9
10			from tracim.lib.utils.auth import MANAGE_CONTENT_PERM
			0 ignored issues – show Unused Code introduced 2018-04-09 13:01 UTC by Report Bug Copy Issue Report Unused MANAGE_CONTENT_PERM imported from tracim.lib.utils.auth Loading history... introduced 2018-04-09 13:01 UTC by Report Bug Copy Issue Report Imports from package tracim are not grouped Loading history...
11			from tracim.lib.utils.auth import MANAGE_WORKSPACE_PERM
			0 ignored issues – show Unused Code introduced 2018-04-09 13:01 UTC by Report Bug Copy Issue Report Unused MANAGE_WORKSPACE_PERM imported from tracim.lib.utils.auth Loading history...
12			from tracim.lib.utils.auth import MANAGE_GLOBAL_PERM
13			from tracim.lib.utils.auth import READ_PERM
			0 ignored issues – show Unused Code introduced 2018-04-09 13:01 UTC by Report Bug Copy Issue Report Unused READ_PERM imported from tracim.lib.utils.auth Loading history...
14			from tracim.lib.utils.auth import CONTRIBUTE_PERM
15			from tracim.lib.utils.auth import ADMIN_PERM
16			from tracim.lib.utils.auth import USER_PERM
17
18
19			class DefaultController(Controller):
			0 ignored issues – show Coding Style introduced 2018-04-06 10:21 UTC by Report Bug Copy Issue Report This class 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... Unused Code introduced 2018-04-06 10:21 UTC by Report Bug Copy Issue Report The variable `__class__` seems to be unused. Loading history...
20
21			@classmethod
22			def notfound_view(cls, request):
			0 ignored issues – show Coding Style introduced 2018-04-06 10:21 UTC by Report Bug Copy Issue Report This method 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...
23			request.response.status = 404
24			return {}
25
26			@classmethod
27			def forbidden_view(cls, request):
			0 ignored issues – show Coding Style introduced 2018-04-06 10:21 UTC by Report Bug Copy Issue Report This method 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...
28			if request.authenticated_userid is None:
29			response = HTTPUnauthorized()
30			response.headers.update(forget(request))
31
32			# user is logged in but doesn't have permissions, reject wholesale
33			else:
34			response = HTTPForbidden()
35			return response
36
37			@classmethod
38			def test_config(cls, request):
			0 ignored issues – show Coding Style introduced 2018-04-06 10:21 UTC by Report Bug Copy Issue Report This method 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...
39			try:
40			app_config = request.registry.settings['CFG']
41			project = app_config.WEBSITE_TITLE
42			except Exception as e:
			0 ignored issues – show Best Practice introduced 2018-03-30 20:04 UTC by Report Bug Copy Issue Report Catching very general exceptions such as `Exception` is usually not recommended. Generally, you would want to handle very specific errors in the exception handler. This ensure that you do not hide other types of errors which should be fixed. So, unless you specifically plan to handle any error, consider adding a more specific exception. Loading history... Coding Style Naming introduced 2018-03-30 20:04 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...
43			return Response(e, content_type='text/plain', status=500)
44			return {'project': project}
45
46			@classmethod
47			def test_contributor_page(cls, request):
			0 ignored issues – show Coding Style introduced 2018-04-09 13:01 UTC by Report Bug Copy Issue Report This method 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...
48			try:
49			app_config = request.registry.settings['CFG']
			0 ignored issues – show Unused Code introduced 2018-04-06 10:21 UTC by Report Bug Copy Issue Report The variable `app_config` seems to be unused. Loading history...
50			project = 'contributor'
51			except Exception as e:
			0 ignored issues – show Best Practice introduced 2018-04-06 10:21 UTC by Report Bug Copy Issue Report Catching very general exceptions such as `Exception` is usually not recommended. Generally, you would want to handle very specific errors in the exception handler. This ensure that you do not hide other types of errors which should be fixed. So, unless you specifically plan to handle any error, consider adding a more specific exception. Loading history... Coding Style Naming introduced 2018-04-06 10: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...
52			return Response(e, content_type='text/plain', status=500)
53			return {'project': project}
54
55			@classmethod
56			def test_admin_page(cls, request):
			0 ignored issues – show Coding Style introduced 2018-04-06 10:21 UTC by Report Bug Copy Issue Report This method 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...
57			try:
58			app_config = request.registry.settings['CFG']
			0 ignored issues – show Unused Code introduced 2018-04-06 10:21 UTC by Report Bug Copy Issue Report The variable `app_config` seems to be unused. Loading history...
59			project = 'admin'
60			except Exception as e:
			0 ignored issues – show Best Practice introduced 2018-04-06 10:21 UTC by Report Bug Copy Issue Report Catching very general exceptions such as `Exception` is usually not recommended. Generally, you would want to handle very specific errors in the exception handler. This ensure that you do not hide other types of errors which should be fixed. So, unless you specifically plan to handle any error, consider adding a more specific exception. Loading history... Coding Style Naming introduced 2018-04-06 10: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...
61			return Response(e, content_type='text/plain', status=500)
62			return {'project': project}
63
64			@classmethod
65			def test_manager_page(cls, request):
			0 ignored issues – show Coding Style introduced 2018-04-06 10:21 UTC by Report Bug Copy Issue Report This method 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...
66			try:
67			app_config = request.registry.settings['CFG']
			0 ignored issues – show Unused Code introduced 2018-04-06 10:21 UTC by Report Bug Copy Issue Report The variable `app_config` seems to be unused. Loading history...
68			project = 'manager'
69			except Exception as e:
			0 ignored issues – show Best Practice introduced 2018-04-06 10:21 UTC by Report Bug Copy Issue Report Catching very general exceptions such as `Exception` is usually not recommended. Generally, you would want to handle very specific errors in the exception handler. This ensure that you do not hide other types of errors which should be fixed. So, unless you specifically plan to handle any error, consider adding a more specific exception. Loading history... Coding Style Naming introduced 2018-04-06 10: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...
70			return Response(e, content_type='text/plain', status=500)
71			return {'project': project}
72
73			@classmethod
74			def test_user_page(cls, request):
			0 ignored issues – show Coding Style introduced 2018-04-06 10:21 UTC by Report Bug Copy Issue Report This method 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...
75			try:
76			app_config = request.registry.settings['CFG']
			0 ignored issues – show Unused Code introduced 2018-04-09 13:01 UTC by Report Bug Copy Issue Report The variable `app_config` seems to be unused. Loading history...
77			project = 'user'
78			except Exception as e:
			0 ignored issues – show Best Practice introduced 2018-04-09 13:01 UTC by Report Bug Copy Issue Report Catching very general exceptions such as `Exception` is usually not recommended. Generally, you would want to handle very specific errors in the exception handler. This ensure that you do not hide other types of errors which should be fixed. So, unless you specifically plan to handle any error, consider adding a more specific exception. Loading history... Coding Style Naming introduced 2018-04-09 13:01 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...
79			return Response(e, content_type='text/plain', status=500)
80			return {'project': project}
81
82			def bind(self, configurator: Configurator):
83			configurator.add_static_view('static', 'static', cache_max_age=3600)
84			configurator.add_view(
85			self.notfound_view,
86			renderer='tracim:templates/404.jinja2',
87			context=NotFound,
88			)
89
90			configurator.add_route('test_config', '/')
91			configurator.add_view(
92			self.test_config,
93			route_name='test_config',
94			renderer='tracim:templates/mytemplate.jinja2',
95			)
96
97			configurator.add_route('test_contributor', '/test_contributor')
98			configurator.add_view(
99			self.test_contributor_page,
100			route_name='test_contributor',
101			renderer='tracim:templates/mytemplate.jinja2',
102			permission=CONTRIBUTE_PERM,
103			)
104			configurator.add_route('test_admin', '/test_admin')
105			configurator.add_view(
106			self.test_admin_page,
107			route_name='test_admin',
108			renderer='tracim:templates/mytemplate.jinja2',
109			permission=ADMIN_PERM,
110			)
111			configurator.add_route('test_manager', '/test_manager')
112			configurator.add_view(
113			self.test_user_page,
114			route_name='test_manager',
115			renderer='tracim:templates/mytemplate.jinja2',
116			permission=MANAGE_GLOBAL_PERM,
117			)
118			configurator.add_route('test_user', '/test_user')
119			configurator.add_view(
120			self.test_user_page,
121			route_name='test_user',
122			renderer='tracim:templates/mytemplate.jinja2',
123			permission=USER_PERM,
124			)
125			configurator.add_forbidden_view(self.forbidden_view)
126

tracim / tracim_backend

Push — master ( c55bf2...b94d48 )

DefaultController.test_admin_page() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like