Python.DICOM.test_anonymize

Complexity

Total Complexity

3

Size/Duplication

Total Lines	26
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	UT_DICOMAnonymization.test_folder()	0	17	3

import unittest

This module should have a docstring.

from tempfile import TemporaryDirectory
from DICOM.anonymize import DICOM_anonymize

Unable to import 'DICOM.anonymize'

from shutil import copyfile

standard import "from shutil import copyfile" should be placed before "from DICOM.anonymize import DICOM_anonymize"

import os

standard import "import os" should be placed before "from DICOM.anonymize import DICOM_anonymize"

class UT_DICOMAnonymization(unittest.TestCase):

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

This class should have a docstring.

The variable __class__ seems to be unused.

    @staticmethod
    def test_folder():

This method should have a docstring.

        from pydicom.data import get_testdata_files

        # Make Temporary Folder
        files = get_testdata_files("[mM][rR][iI]")

        # Copy files to Temporary Folder
        with TemporaryDirectory() as temp_folder:

            # Copy all files
            for file in files:
                path, fileName = os.path.split(file)

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

The variable path seems to be unused.

                copyfile(file, os.path.join(temp_folder, fileName))

            # Anonymize that folder.
            DICOM_anonymize.folder(temp_folder, "NEW_CONTESTANT")

Trailing newlines