Python.LORIS.test_candidates

Complexity

Total Complexity

4

Size/Duplication

Total Lines	61
Duplicated Lines	0 %

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	UT_LORISCandidates.test_check_projectID()	0	13	1
A	UT_LORISCandidates.test_create_delete_subject()	0	18	2
A	UT_LORISCandidates.test_CNBP_PSCID()	0	13	1

import logging

This module should have a docstring.

from LORIS.candidates import LORIS_candidates

Unable to import 'LORIS.candidates'

from LORIS.query import LORIS_query

Unable to import 'LORIS.query'

import unittest

standard import "import unittest" should be placed before "from LORIS.candidates import LORIS_candidates"

import sys

standard import "import sys" should be placed before "from LORIS.candidates import LORIS_candidates"

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

The name logger does not conform to the constant naming conventions ((([A-Z_][A-Z0-9_]*)|(__.*__))$).

class UT_LORISCandidates(unittest.TestCase):

The name UT_LORISCandidates does not conform to the class naming conventions ([A-Z_][a-zA-Z0-9]+$).

The variable __class__ seems to be unused.

This class should have a docstring.

    @staticmethod
    def test_create_delete_subject():
        """
        Check both the creation and deletion of the subject for LORIS.
        :return:
        """

        response_success, token = LORIS_query.login()

        if not response_success:
            raise ConnectionError

        # Example PSCI ID.
        PSCID = "CNBP9990987"

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

        success, DCCID = LORIS_candidates.createCandidateCNBP(token, PSCID)

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

        assert success
        LORIS_candidates.deleteCandidateCNBP(DCCID, PSCID)

    @staticmethod
    def test_check_projectID():

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

This method should have a docstring.

        ProjectID = "GL01"

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

        assert LORIS_candidates.check_projectID_compliance(ProjectID)

        ProjectID = "GL09"

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

        assert not LORIS_candidates.check_projectID_compliance(ProjectID)

        ProjectID = "AL01"

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

        assert not LORIS_candidates.check_projectID_compliance(ProjectID)

        ProjectID = "AB01"

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

        assert LORIS_candidates.check_projectID_compliance(ProjectID)

    @staticmethod
    def test_CNBP_PSCID():

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

This method should have a docstring.

        PSCID = "VXS" + "GL01"  + "0001"

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

        PSCID1 = "VXa" + "GL01" + "0001"

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

        PSCID2 = "VXS" + "GL02" + "0001"

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

        PSCID3 = "VXS" + "GL01" + "0001"

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

        PSCID4 = "VXS" + "GL01" + "0009"

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

        assert(LORIS_candidates.check_PSCID_compliance(PSCID))

There is an unnecessary parenthesis after assert.

        assert not LORIS_candidates.check_PSCID_compliance(PSCID1)
        assert not LORIS_candidates.check_PSCID_compliance(PSCID2)
        assert LORIS_candidates.check_PSCID_compliance(PSCID3)
        assert LORIS_candidates.check_PSCID_compliance(PSCID4)

if __name__ == "__main__":
    UT_LORISCandidates.test_CNBP_PSCID()

Final newline missing