Python.LORIS.test_LORIS_Query.test_LORIS_get() - Code Metrics - Inspection of "Dicom Anonymization" - CNBP/DICOMTransit - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#2)

by Yang

created 2018-08-01 16:22 UTC

Python.LORIS.test_LORIS_Query.test_LORIS_get() A

↳ Parent: Python.LORIS.test_LORIS_Query

Complexity

Conditions

Size

Total Lines	9
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes

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

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

from LORIS.candidates import checkDCCIDExist, checkPSCIDExist


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

def test_LORIS_login():
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    logger = logging.getLogger('UT_LORIS_login')

    response_success, token = login()

    assert response_success
    #assert len(token) == 256 #token should always be 256 char long

def test_LORIS_get():
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    logger = logging.getLogger('UT_LORIS_get')

    response_success, token = login()
    assert response_success
    #assert len(token) == 256  # token should always be 256 char long
    response_success, json = getCNBP(token, "projects")

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

def test_checkPSCIDExist():
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    logger = logging.getLogger('UT_LORIS_PSCID_check')

    response_success, token = login()
    assert response_success
    #assert len(token) == 256  # token should always be 256 char long
    response_success, exist = checkPSCIDExist(token, "CNBP0010001")
    assert response_success
    assert exist

def test_checkDCCIDExist():
class SomeClass:
    def some_method(self):
        """Do x and return foo."""
    logger = logging.getLogger('UT_LORIS_DCCID_check')

    response_success, token = login()
    assert response_success
    #assert len(token) == 256  # token should always be 256 char long
    response_success, exist = checkDCCIDExist(token, 272264)
    assert response_success
    assert exist

if __name__ == '__main__':
    test_checkPSCIDExist()


1			import logging
			0 ignored issues – show Coding Style Naming introduced 2018-07-31 12:18 UTC by Report Bug Copy Issue Report The name `test_LORIS_Query` does not conform to the module naming conventions (`(([a-z_][a-z0-9_]*)\|([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-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 login, getCNBP
			0 ignored issues – show introduced 2018-07-31 12:18 UTC by Report Bug Copy Issue Report Unable to import 'LORIS.query' Loading history...
4			from LORIS.candidates import checkDCCIDExist, checkPSCIDExist
			0 ignored issues – show introduced 2018-07-31 12:18 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
8			def test_LORIS_login():
			0 ignored issues – show Coding Style Naming introduced 2018-07-31 12:18 UTC by Report Bug Copy Issue Report The name `test_LORIS_login` 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-07-31 12:18 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...
9			logger = logging.getLogger('UT_LORIS_login')
			0 ignored issues – show Unused Code introduced 2018-07-31 12:18 UTC by Report Bug Copy Issue Report The variable `logger` seems to be unused. Loading history...
10			response_success, token = login()
			0 ignored issues – show Unused Code introduced 2018-07-31 12:18 UTC by Report Bug Copy Issue Report The variable `token` seems to be unused. Loading history...
11			assert response_success
12			#assert len(token) == 256 #token should always be 256 char long
13
14			def test_LORIS_get():
			0 ignored issues – show Coding Style Naming introduced 2018-07-31 12:18 UTC by Report Bug Copy Issue Report The name `test_LORIS_get` 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-07-31 12:18 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...
15			logger = logging.getLogger('UT_LORIS_get')
			0 ignored issues – show Unused Code introduced 2018-07-31 12:18 UTC by Report Bug Copy Issue Report The variable `logger` seems to be unused. Loading history...
16			response_success, token = login()
17			assert response_success
18			#assert len(token) == 256 # token should always be 256 char long
19			response_success, json = getCNBP(token, "projects")
			0 ignored issues – show Unused Code introduced 2018-07-31 12:18 UTC by Report Bug Copy Issue Report The variable `json` seems to be unused. Loading history...
20			assert response_success
21			response_success, json = getCNBP(token, "candidates")
22			assert response_success
23
24			def test_checkPSCIDExist():
			0 ignored issues – show Coding Style Naming introduced 2018-07-31 12:18 UTC by Report Bug Copy Issue Report The name `test_checkPSCIDExist` 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-07-31 12:18 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...
25			logger = logging.getLogger('UT_LORIS_PSCID_check')
			0 ignored issues – show Unused Code introduced 2018-07-31 12:18 UTC by Report Bug Copy Issue Report The variable `logger` seems to be unused. Loading history...
26			response_success, token = login()
27			assert response_success
28			#assert len(token) == 256 # token should always be 256 char long
29			response_success, exist = checkPSCIDExist(token, "CNBP0010001")
30			assert response_success
31			assert exist
32
33			def test_checkDCCIDExist():
			0 ignored issues – show Coding Style Naming introduced 2018-07-31 12:18 UTC by Report Bug Copy Issue Report The name `test_checkDCCIDExist` 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-07-31 12:18 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			logger = logging.getLogger('UT_LORIS_DCCID_check')
			0 ignored issues – show Unused Code introduced 2018-07-31 12:18 UTC by Report Bug Copy Issue Report The variable `logger` seems to be unused. Loading history...
35			response_success, token = login()
36			assert response_success
37			#assert len(token) == 256 # token should always be 256 char long
38			response_success, exist = checkDCCIDExist(token, 272264)
39			assert response_success
40			assert exist
41
42			if __name__ == '__main__':
43			test_checkPSCIDExist()
44

CNBP / DICOMTransit

Pull Request — master (#2)

Python.LORIS.test_LORIS_Query.test_LORIS_get() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like