chore: Remove disabled check at the start of the test

Parkreiner · Parkreiner · commit 507dc80d30a0 · 2023-09-20T21:03:52.000Z
diff --git a/site/src/pages/TemplateSettingsPage/TemplateSchedulePage/TemplateSchedulePage.test.tsx b/site/src/pages/TemplateSettingsPage/TemplateSchedulePage/TemplateSchedulePage.test.tsx
@@ -54,15 +54,6 @@ const fillAndSubmitForm = async ({
 }: FillAndSubmitConfig) => {
   const user = userEvent.setup();
 
-  const submitButton = screen.getByRole("button", {
-    name: FooterFormLanguage.defaultSubmitLabel,
-  });
-
-  // This should be removed once we start removing disabled states from the UI;
-  // not worried about this getting forgotten, because it will trigger an error
-  // at some point once that migration starts
-  expect(submitButton).toBeDisabled();
-
   if (default_ttl_ms) {
     const defaultTtlField = await screen.findByLabelText(
       "Default autostop (hours)",
@@ -102,6 +93,10 @@ const fillAndSubmitForm = async ({
     );
   }
 
+  const submitButton = screen.getByRole("button", {
+    name: FooterFormLanguage.defaultSubmitLabel,
+  });
+
   expect(submitButton).not.toBeDisabled();
   await user.click(submitButton);
 
