Remove cache before loading the app (#31)

* adding pytest-cov, fixes #6

* remove cache before loading the app, fixes #5

Author: jpchauvel

src/edit_python_pe/utils.py

@@ -2,6 +2,7 @@
 import hashlib
 import os
 import re
+import shutil
 from datetime import date, datetime
 from time import sleep
 from typing import TYPE_CHECKING
@@ -141,14 +142,14 @@ def fork_repo(token: str, original_repo: Repository) -> tuple[str, Repository]:
     forked_repo_url = forked_repo.clone_url
     repo_path = user_data_dir(appname="edit-python-pe", appauthor="python.pe")
 
-    if not os.path.exists(repo_path):
-        callbacks = pygit2.callbacks.RemoteCallbacks(
-            credentials=pygit2.UserPass(token, "x-oauth-basic")
-        )
-        sleep(3)
-        pygit2.clone_repository(
-            forked_repo_url, repo_path, callbacks=callbacks
-        )
+    if os.path.exists(repo_path):
+        shutil.rmtree(repo_path)
+
+    callbacks = pygit2.callbacks.RemoteCallbacks(
+        credentials=pygit2.UserPass(token, "x-oauth-basic")
+    )
+    sleep(3)
+    pygit2.clone_repository(forked_repo_url, repo_path, callbacks=callbacks)
     return repo_path, forked_repo
 
 

tests/test_utils.py

@@ -252,10 +252,16 @@ def test_get_repo_github_exception(self, mock_github, mock_getpass):
 class TestForkRepo(unittest.TestCase):
     @patch("edit_python_pe.utils.user_data_dir", return_value="/tmp/testrepo")
     @patch("edit_python_pe.utils.os.path.exists", return_value=False)
+    @patch("edit_python_pe.utils.shutil.rmtree")
     @patch("edit_python_pe.utils.pygit2.clone_repository")
     @patch("edit_python_pe.utils.sleep", return_value=None)
-    def test_fork_repo_clones_if_not_exists(
-        self, mock_sleep, mock_clone, mock_exists, mock_user_data_dir
+    def test_fork_repo_no_remove_if_not_exists(
+        self,
+        mock_sleep,
+        mock_clone,
+        mock_rmtree,
+        mock_exists,
+        mock_user_data_dir,
     ):
         mock_forked_repo = MagicMock()
         mock_forked_repo.clone_url = "https://github.com/fake/fork.git"
@@ -265,16 +271,24 @@ def test_fork_repo_clones_if_not_exists(
         repo_path = fork_repo(token, mock_original_repo)[0]
         mock_original_repo.create_fork.assert_called_once()
         mock_clone.assert_called_once()
+        mock_rmtree.assert_not_called()
         call_args = mock_clone.call_args
         self.assertEqual(call_args[0][0], mock_forked_repo.clone_url)
         self.assertEqual(call_args[0][1], repo_path)
         self.assertEqual(repo_path, "/tmp/testrepo")
 
     @patch("edit_python_pe.utils.user_data_dir", return_value="/tmp/testrepo")
     @patch("edit_python_pe.utils.os.path.exists", return_value=True)
+    @patch("edit_python_pe.utils.shutil.rmtree")
     @patch("edit_python_pe.utils.pygit2.clone_repository")
-    def test_fork_repo_no_clone_if_exists(
-        self, mock_clone, mock_exists, mock_user_data_dir
+    @patch("edit_python_pe.utils.sleep", return_value=None)
+    def test_fork_repo_remove_if_exists(
+        self,
+        mock_sleep,
+        mock_clone,
+        mock_rmtree,
+        mock_exists,
+        mock_user_data_dir,
     ):
         mock_forked_repo = MagicMock()
         mock_forked_repo.clone_url = "https://github.com/fake/fork.git"
@@ -283,5 +297,6 @@ def test_fork_repo_no_clone_if_exists(
         token = "fake-token"
         repo_path = fork_repo(token, mock_original_repo)[0]
         mock_original_repo.create_fork.assert_called_once()
-        mock_clone.assert_not_called()
+        mock_rmtree.assert_called_once()
+        mock_clone.assert_called_once()
         self.assertEqual(repo_path, "/tmp/testrepo")