Merge pull request #147 from ServiceNow/scratch/remove-call-get-form

remove call to _get_form as it causes issues in workspace forms

Author: marquezo

src/browsergym/workarena/tasks/form_workspace.py

@@ -63,6 +63,20 @@ def setup_goal(self, page: Page) -> tuple[str, dict]:
 
         return goal, info
 
+    def start(self, page: Page) -> None:
+        # Overriding start because calling self._get_form(page) in the parent class gives an error
+        logging.debug("Navigating to task start page")
+
+        # Authenticate
+        url_login(
+            instance=self.instance,
+            page=page,
+        )
+
+        # Navigate to the task's url
+        page.goto(self.start_url)
+        self._wait_for_ready(page)
+
     def validate(
         self, page: Page, chat_messages: list[str]
     ) -> Tuple[float, bool, str, dict]:
@@ -237,20 +251,6 @@ def __init__(self, seed: int = None,
         self.start_url = self.instance.snow_url
         self.created_sys_id = None
 
-    def start(self, page: Page) -> None:
-        # Overriding start because calling self._get_form(page) in the parent class gives an error
-        logging.debug("Navigating to task start page")
-
-        # Authenticate
-        url_login(
-            instance=self.instance,
-            page=page,
-        )
-
-        # Navigate to the task's url
-        page.goto(self.start_url)
-        self._wait_for_ready(page)
-
     def get_init_scripts(self) -> List[str]:
        # Override this to avoid changing functionality of submit button
         return []