fix: fixed args to sync-states

hameedacpa · May 9, 2022 · c667be9 · c667be9
1 parent 7f37488
commit c667be9
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 2 deletions.
diff --git a/bench/app.py b/bench/app.py
@@ -241,7 +241,9 @@ def _get_dependencies(self):
 	def update_app_state(self):
 		from bench.bench import Bench
 		bench = Bench(self.bench.name)
-		bench.apps.sync(self.name, self.tag, self.local_resolution)
+		bench.apps.sync(app_dir=self.app_name, app_name=self.name,
+						branch=self.tag, required_list=self.local_resolution)
+
 
 
 def make_resolution_plan(app: App, bench: "Bench"):

diff --git a/bench/bench.py b/bench/bench.py
@@ -170,7 +170,13 @@ def set_states(self):
 		except FileNotFoundError:
 			self.states = {}
 
-	def update_apps_states(self, app_dir, app_name: Union[str, None] = None,  branch: Union[str, None] = None, required:List = []):
+	def update_apps_states(
+			self,
+			app_dir: str = None,
+			app_name: Union[str, None] = None,
+			branch: Union[str, None] = None,
+			required: List = [],
+	):
 		if self.apps and not os.path.exists(self.states_path):
 			# idx according to apps listed in apps.txt (backwards compatibility)
 			# Keeping frappe as the first app.
@@ -200,6 +206,9 @@ def update_apps_states(self, app_dir, app_name: Union[str, None] = None,  branch
 		for app in apps_to_remove:
 			del self.states[app]
 
+		if app_name and not app_dir:
+			app_dir = app_name
+
 		if app_name and app_name not in self.states:
 			version = get_current_version(app_name, self.bench.name)