doc(comments): outdated comment in test; .IMA is Siemens dcm ext

Author: WillForan

deid/tests/test_clean.py

@@ -251,7 +251,7 @@ def test_get_clean_name(self):
         )  # defaults:  extension="dcm", prefix="clean-"
         self.assertEqual(new_name, os.path.join("abc", "clean-XYZ.dcm"))
 
-        # note IMA not removed -- unusual dicom extension not implicitly handled
+        # note IMA not removed -- Siemens dicom extension not implicitly handled
         client.dicom_file = "XYZ.IMA"
         new_name = client._get_clean_name(output_folder="abc", extension="png")
         self.assertEqual(new_name, os.path.join("abc", "clean-XYZ.IMA.png"))

deid/tests/test_cli.py

@@ -64,7 +64,8 @@ def test_deidmain_clean_pixels(self):
         # Confirm input data has value that will be scrubbed.
         indcm = utils.dcmread(self.tmpdir + "/example.dicom")
         self.assertEqual(indcm.pixel_array.shape, (456, 510, 3))
-        censor_area = indcm.pixel_array[0:250, 0:100, :]  # y,x,z
+        # index is y,x,z for censor box coordinates from deid.cfg below
+        censor_area = indcm.pixel_array[0:250, 0:100, :]
         # all voxels in region to be scrubbed are valued. lucky us
         self.assertEqual(np.count_nonzero(censor_area != 0), 75000)
 
@@ -88,7 +89,6 @@ def test_deidmain_clean_pixels(self):
         # Confirm we changed pixel data
         self.assertTrue(np.any(indcm.pixel_array != outfile.pixel_array))
         # Confirm censor area is all zeros
-        # 30126 but expect 75000
         zero_cnt = np.count_nonzero(outfile.pixel_array[0:250, 0:100, :] == 0)
         self.assertEqual(zero_cnt, 100 * 250 * 3)  # 75000