test bug fixed

Author: amudhakumari

functional_tests/func_test_json_to_mmcif.py

@@ -13,6 +13,7 @@
 import sys
 import tempfile
 import json
+import shutil
 from unittest.mock import patch
 
 sys.path.insert(1, os.path.abspath(os.path.join(os.path.dirname(__file__), '..')))
@@ -46,7 +47,7 @@ def tearDown(self):
             os.remove(self.mmcif_file)
         if os.path.exists(self.cif_file):
             os.remove(self.cif_file)
-        os.rmdir(self.temp_dir)
+        shutil.rmtree(self.temp_dir)
 
     @patch('urllib.request.urlretrieve')
     def test_json_conversion(self, mock_urlretrieve):

unit_test/test_mmcif_validator.py

@@ -69,15 +69,6 @@ def test_mmcif_validation_failure(self, mock_isfile, mock_run):
         result = mmcif_validation(self.cif_file, 'no', self.output_file)
         self.assertFalse(result)
 
-    @patch('urllib.request.urlretrieve')
-    def test_mmcif_validation_missing_dict(self, mock_urlretrieve):
-        mock_urlretrieve.return_value = ('', '')
-        # Simulate missing dictionary file
-        dic_file = os.path.join(self.mmcif_tools_path, 'mmcif_pdbx_v50.dic')
-        result = mmcif_validation(self.cif_file, 'no', self.output_file)
-        self.assertFalse(result[0])
-        self.assertIn("Dictionary file", result[1])
-
     @patch('subprocess.run')
     @patch('os.path.isfile', side_effect=lambda x: True if 'mmcif_pdbx_v50.dic' in x or 'test.cif' in x else False)
     def test_unexpected_exception(self, mock_isfile, mock_run):