FIX: forking should do nothing if already exists

Author: ytausch

conda_forge_tick/git_utils.py

@@ -961,7 +961,8 @@ def push_to_repository(
     @lock_git_operation()
     def fork(self, owner: str, repo_name: str):
         if repo_name in self._repos:
-            raise ValueError(f"Fork of {repo_name} already exists.")
+            logger.debug(f"Fork of {repo_name} already exists. Doing nothing.")
+            return
 
         logger.debug(
             f"Dry Run: Creating fork of {owner}/{repo_name} for user {self._USER}."

tests/test_git_utils.py

@@ -1635,11 +1635,8 @@ def test_dry_run_backend_fork(caplog):
         in caplog.text
     )
 
-    with pytest.raises(ValueError, match="Fork of REPO already exists"):
-        backend.fork("UPSTREAM_OWNER", "REPO")
-
-    with pytest.raises(ValueError, match="Fork of REPO already exists"):
-        backend.fork("OTHER_OWNER", "REPO")
+    # this should not raise an error
+    backend.fork("UPSTREAM_OWNER", "REPO")
 
 
 def test_dry_run_backend_sync_default_branch(caplog):