Python.LORIS.test_query.UT_LORISQuery.test_LORIS_get() - Code Metrics - Inspection of "Major update" - CNBP/DICOMTransit - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#3)

by Yang

created 2018-09-20 19:35 UTC

UT_LORISQuery.test_LORIS_get() A

↳ Parent: Python.LORIS.test_query

Complexity

Conditions

Size

Total Lines	9
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	1
eloc	8
nop	0
dl	0
loc	9
rs	10
c	0
b	0
f	0

import logging
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
import sys
from LORIS.query import LORIS_query

from LORIS.candidates import LORIS_candidates


logging.basicConfig(stream=sys.stdout, level=logging.INFO)
logger = logging.getLogger(__name__)


import unittest

class UT_LORISQuery(unittest.TestCase):
class SomeClass:
    def some_method(self):
        """Do x and return foo."""

    @staticmethod
    def test_LORIS_login():
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        response_success, token = LORIS_query.login()

        assert response_success

        #assert len(token) == 256 #token should always be 256 char long
    @staticmethod
    def test_LORIS_get():
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        response_success, token = LORIS_query.login()
        assert response_success
        #assert len(token) == 256  # token should always be 256 char long
        response_success, json = LORIS_query.getCNBP(token, "projects")

        assert response_success
        response_success, json = LORIS_query.getCNBP(token, "candidates")
        assert response_success

    @staticmethod
    def test_checkPSCIDExist():
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        response_success, token = LORIS_query.login()
        assert response_success
        #assert len(token) == 256  # token should always be 256 char long
        response_success, exist = LORIS_candidates.checkPSCIDExist(token, "CNBP0010001")
        assert response_success
        assert not exist

    @staticmethod
    def test_checkDCCIDExist():
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
        response_success, token = LORIS_query.login()
        assert response_success
        #assert len(token) == 256  # token should always be 256 char long
        response_success, exist = LORIS_candidates.checkDCCIDExist(token, 272264)
        assert response_success
        assert exist


if __name__ == '__main__':
    UT_LORISQuery.test_checkPSCIDExist()


1			import logging
			0 ignored issues – show Coding Style introduced 2018-07-31 12:18 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			import sys
3			from LORIS.query import LORIS_query
			0 ignored issues – show introduced 2018-09-12 21:17 UTC by Report Bug Copy Issue Report Unable to import 'LORIS.query' Loading history...
4			from LORIS.candidates import LORIS_candidates
			0 ignored issues – show introduced 2018-09-12 21:17 UTC by Report Bug Copy Issue Report Unable to import 'LORIS.candidates' Loading history...
5
6			logging.basicConfig(stream=sys.stdout, level=logging.INFO)
7			logger = logging.getLogger(__name__)
			0 ignored issues – show Coding Style Naming introduced 2018-09-13 15:56 UTC by Report Bug Copy Issue Report The name `logger` does not conform to the constant naming conventions (`(([A-Z_][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...
8
9			import unittest
			0 ignored issues – show introduced 2018-09-14 17:44 UTC by Report Bug Copy Issue Report Import "import unittest" should be placed at the top of the module Loading history... introduced 2018-09-14 17:44 UTC by Report Bug Copy Issue Report standard import "import unittest" should be placed before "from LORIS.query import LORIS_query" Loading history...
10			class UT_LORISQuery(unittest.TestCase):
			0 ignored issues – show Coding Style Naming introduced 2018-09-12 21:17 UTC by Report Bug Copy Issue Report The name `UT_LORISQuery` does not conform to the class naming conventions (`[A-Z_][a-zA-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-09-12 21:17 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-09-12 21:17 UTC by Report Bug Copy Issue Report The variable `__class__` seems to be unused. Loading history...
11
12			@staticmethod
13			def test_LORIS_login():
			0 ignored issues – show Coding Style Naming introduced 2018-09-14 17:44 UTC by Report Bug Copy Issue Report The name `test_LORIS_login` does not conform to the method 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-09-14 17:44 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...
14			response_success, token = LORIS_query.login()
			0 ignored issues – show Unused Code introduced 2018-09-12 21:17 UTC by Report Bug Copy Issue Report The variable `token` seems to be unused. Loading history...
15			assert response_success
16
17			#assert len(token) == 256 #token should always be 256 char long
18			@staticmethod
19			def test_LORIS_get():
			0 ignored issues – show Coding Style Naming introduced 2018-09-14 17:44 UTC by Report Bug Copy Issue Report The name `test_LORIS_get` does not conform to the method 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-09-14 17:44 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...
20			response_success, token = LORIS_query.login()
21			assert response_success
22			#assert len(token) == 256 # token should always be 256 char long
23			response_success, json = LORIS_query.getCNBP(token, "projects")
			0 ignored issues – show Unused Code introduced 2018-09-12 21:17 UTC by Report Bug Copy Issue Report The variable `json` seems to be unused. Loading history...
24			assert response_success
25			response_success, json = LORIS_query.getCNBP(token, "candidates")
26			assert response_success
27
28			@staticmethod
29			def test_checkPSCIDExist():
			0 ignored issues – show Coding Style Naming introduced 2018-09-14 17:44 UTC by Report Bug Copy Issue Report The name `test_checkPSCIDExist` does not conform to the method 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-09-14 17:44 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...
30			response_success, token = LORIS_query.login()
31			assert response_success
32			#assert len(token) == 256 # token should always be 256 char long
33			response_success, exist = LORIS_candidates.checkPSCIDExist(token, "CNBP0010001")
34			assert response_success
35			assert not exist
36
37			@staticmethod
38			def test_checkDCCIDExist():
			0 ignored issues – show Coding Style Naming introduced 2018-09-14 17:44 UTC by Report Bug Copy Issue Report The name `test_checkDCCIDExist` does not conform to the method 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-09-14 17:44 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			response_success, token = LORIS_query.login()
40			assert response_success
41			#assert len(token) == 256 # token should always be 256 char long
42			response_success, exist = LORIS_candidates.checkDCCIDExist(token, 272264)
43			assert response_success
44			assert exist
45
46
47			if __name__ == '__main__':
48			UT_LORISQuery.test_checkPSCIDExist()
49

CNBP / DICOMTransit

Pull Request — master (#3)

UT_LORISQuery.test_LORIS_get() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like