fix: get-app on existing apps

get-app to replace existing folder would fail due to bad url generation. Changes * Archive old repo instead of overwritting * Resetting flags in App instance
Zawala · Jul 27, 2022 · e03f597 · e03f597
1 parent beac865
commit e03f597
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/bench/app.py b/bench/app.py
@@ -209,6 +209,9 @@ def remove(self, no_backup: bool = False):
 			shutil.move(active_app_path, archived_app_path)
 			log(f"App moved from {active_app_path} to {archived_app_path}")
 
+		self.from_apps = False
+		self.on_disk = False
+
 	@step(title="Installing App {repo}", success="App {repo} Installed")
 	def install(
 		self,
@@ -419,7 +422,7 @@ def get_app(
 			"Do you want to continue and overwrite it?"
 		)
 	):
-		shutil.rmtree(cloned_path)
+		app.remove()
 		to_clone = True
 
 	if to_clone: