Skip to content

Commit 6d162c6

Browse files
committed
Fix e2e
1 parent 44a59c8 commit 6d162c6

File tree

3 files changed

+5
-11
lines changed

3 files changed

+5
-11
lines changed

site/e2e/tests/groups/createGroup.spec.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ test("create group", async ({ page, baseURL }) => {
2727
await page.getByLabel("Name", { exact: true }).fill(groupValues.name);
2828
await page.getByLabel("Display Name").fill(groupValues.displayName);
2929
await page.getByLabel("Avatar URL").fill(groupValues.avatarURL);
30-
await page.getByRole("button", { name: "Create group" }).click();
30+
await page.getByRole("button", { name: /save/i }).click();
3131

3232
await expect(page).toHaveTitle(`${groupValues.displayName} - Coder`);
3333
await expect(page.getByText(groupValues.displayName)).toBeVisible();

site/src/pages/CreateTemplatePage/CreateTemplateForm.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -358,7 +358,7 @@ export const CreateTemplateForm: FC<CreateTemplateFormProps> = (props) => {
358358
</Button>
359359
<Button type="submit" disabled={isSubmitting}>
360360
{isSubmitting && <Spinner />}
361-
{jobError ? "Retry" : "Create template"}
361+
{jobError ? "Retry" : "Save"}
362362
</Button>
363363
{logs && (
364364
<button

site/src/pages/CreateTemplatePage/CreateTemplatePage.test.tsx

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -61,9 +61,7 @@ test("Create template from starter template", async () => {
6161
MockTemplateVersionVariable3,
6262
]);
6363
await userEvent.type(screen.getByLabelText(/Name/), "my-template");
64-
await userEvent.click(
65-
within(form).getByRole("button", { name: /create template/i }),
66-
);
64+
await userEvent.click(within(form).getByRole("button", { name: /save/i }));
6765

6866
// Wait for the drawer error to be rendered
6967
await screen.findByRole("heading", { name: /missing variables/i });
@@ -92,9 +90,7 @@ test("Create template from starter template", async () => {
9290
.mockResolvedValue(MockTemplateVersion);
9391
jest.spyOn(API, "getTemplateVersion").mockResolvedValue(MockTemplateVersion);
9492
jest.spyOn(API, "createTemplate").mockResolvedValue(MockTemplate);
95-
await userEvent.click(
96-
within(form).getByRole("button", { name: /create template/i }),
97-
);
93+
await userEvent.click(within(form).getByRole("button", { name: /save/i }));
9894
await waitFor(() => expect(API.createTemplate).toBeCalledTimes(1));
9995
expect(router.state.location.pathname).toEqual(
10096
`/templates/${MockTemplate.name}/files`,
@@ -142,9 +138,7 @@ test("Create template from duplicating a template", async () => {
142138
.mockResolvedValue(MockTemplateVersion);
143139
jest.spyOn(API, "getTemplateVersion").mockResolvedValue(MockTemplateVersion);
144140
jest.spyOn(API, "createTemplate").mockResolvedValue(MockTemplate);
145-
await userEvent.click(
146-
screen.getByRole("button", { name: /create template/i }),
147-
);
141+
await userEvent.click(screen.getByRole("button", { name: /save/i }));
148142
await waitFor(() => {
149143
expect(router.state.location.pathname).toEqual(
150144
`/templates/${MockTemplate.name}/files`,

0 commit comments

Comments
 (0)