Don't delete temp directory in fetch_via_vcs

Signed-off-by: Tushar Goel <[email protected]>

Author: TG1999

src/fetchcode/vcs/__init__.py

@@ -15,6 +15,7 @@
 # specific language governing permissions and limitations under the License.
 
 import os
+import shutil
 import tempfile
 from urllib.parse import urlparse
 
@@ -38,6 +39,14 @@ def __init__(self, dest_dir, vcs_type, domain):
         self.dest_dir = dest_dir
         self.vcs_type = vcs_type
         self.domain = domain
+    
+
+    def delete(self):
+        """
+        Delete the temporary directory.
+        """
+        if os.path.isdir(self.dest_dir):
+            shutil.rmtree(path=self.dest_dir)
 
 
 def fetch_via_vcs(url):
@@ -48,8 +57,7 @@ def fetch_via_vcs(url):
     parsed_url = urlparse(url)
     scheme = parsed_url.scheme
     domain = parsed_url.netloc
-    temp = tempfile.mkdtemp()
-    os.rmdir(temp)
+    dest_dir = os.path.join(tempfile.mkdtemp(), "checkout")
     if scheme not in vcs.all_schemes:
         raise Exception("Not a supported/known scheme.")
 
@@ -58,6 +66,6 @@ def fetch_via_vcs(url):
             vcs_type = vcs_name
 
     backend = vcs.get_backend_for_scheme(scheme)
-    backend.obtain(dest=temp, url=misc.hide_url(url))
+    backend.obtain(dest=dest_dir, url=misc.hide_url(url))
 
-    return VCSResponse(dest_dir=temp, vcs_type=vcs_type, domain=domain)
+    return VCSResponse(dest_dir=dest_dir, vcs_type=vcs_type, domain=domain)