Do less things

BrunoQuaresma · BrunoQuaresma · commit 70e80c087ec3 · 2023-03-09T23:10:11.000Z
diff --git a/site/src/pages/CreateTemplatePage/CreateTemplatePage.test.tsx b/site/src/pages/CreateTemplatePage/CreateTemplatePage.test.tsx
@@ -7,8 +7,6 @@ import {
   MockTemplateVersionVariable1,
   MockTemplateVersionVariable2,
   MockTemplateVersionVariable3,
-  MockTemplateVersionVariable4,
-  MockTemplateVersionVariable5,
   renderWithAuth,
 } from "testHelpers/renderHelpers"
 import CreateTemplatePage from "./CreateTemplatePage"
@@ -55,8 +53,6 @@ test("Create template with variables", async () => {
       MockTemplateVersionVariable1,
       MockTemplateVersionVariable2,
       MockTemplateVersionVariable3,
-      MockTemplateVersionVariable4,
-      MockTemplateVersionVariable5,
     ])
 
   // Render page, fill the name and submit
@@ -69,6 +65,7 @@ test("Create template with variables", async () => {
 
   // Wait for the variables form to be rendered and fill it
   await screen.findByText(/Variables/)
+
   // Type first variable
   await userEvent.clear(screen.getByLabelText(/var.first_variable/))
   await userEvent.type(
@@ -80,18 +77,6 @@ test("Create template with variables", async () => {
   await userEvent.type(screen.getByLabelText(/var.second_variable/), "2")
   // Select third variable on radio
   await userEvent.click(screen.getByLabelText(/True/))
-  // Type fourth variable
-  await userEvent.clear(screen.getByLabelText(/var.fourth_variable/))
-  await userEvent.type(
-    screen.getByLabelText(/var.fourth_variable/),
-    "Fourth value",
-  )
-  // Type fifth variable
-  await userEvent.clear(screen.getByLabelText(/var.fifth_variable/))
-  await userEvent.type(
-    screen.getByLabelText(/var.fifth_variable/),
-    "Fifth value",
-  )
   // Setup the mock for the second template version creation before submit the form
   jest.clearAllMocks()
   jest
@@ -101,7 +86,6 @@ test("Create template with variables", async () => {
   await userEvent.click(
     within(form).getByRole("button", { name: /create template/i }),
   )
-
   await waitFor(() => expect(API.createTemplate).toBeCalledTimes(1))
   expect(router.state.location.pathname).toEqual(
     `/templates/${MockTemplate.name}`,
@@ -116,8 +100,6 @@ test("Create template with variables", async () => {
       { name: "first_variable", value: "First value" },
       { name: "second_variable", value: "2" },
       { name: "third_variable", value: "true" },
-      { name: "fourth_variable", value: "Fourth value" },
-      { name: "fifth_variable", value: "Fifth value" },
     ],
   })
 })
