Skip to content

Commit 4cb7a90

Browse files
jaaydenhjaaydenh
committed
chore: cleanup
1 parent 5b44667 commit 4cb7a90

File tree

1 file changed

+6
-96
lines changed

1 file changed

+6
-96
lines changed

‎site/src/pages/CreateWorkspacePage/CreateWorkspacePageExperimental.test.tsx

Lines changed: 6 additions & 96 deletions
Original file line numberDiff line numberDiff line change
@@ -56,8 +56,8 @@ describe("CreateWorkspacePageExperimental", () => {
5656
mockWebSocket.addEventListener("message", (event) => {
5757
callbacks.onMessage(JSON.parse(event.data));
5858
});
59-
mockWebSocket.addEventListener("error", (event) => {
60-
callbacks.onError((event as ErrorEvent).error);
59+
mockWebSocket.addEventListener("error", () => {
60+
callbacks.onError(new Error("Connection for dynamic parameters failed."));
6161
});
6262
mockWebSocket.addEventListener("close", () => {
6363
callbacks.onClose();
@@ -111,8 +111,8 @@ describe("CreateWorkspacePageExperimental", () => {
111111
mockWebSocket.addEventListener("message", (event) => {
112112
callbacks.onMessage(JSON.parse(event.data));
113113
});
114-
mockWebSocket.addEventListener("error", (event) => {
115-
callbacks.onError((event as ErrorEvent).error);
114+
mockWebSocket.addEventListener("error", () => {
115+
callbacks.onError(new Error("Connection for dynamic parameters failed."));
116116
});
117117
mockWebSocket.addEventListener("close", () => {
118118
callbacks.onClose();
@@ -133,8 +133,6 @@ describe("CreateWorkspacePageExperimental", () => {
133133

134134
expect(screen.getByText(/instance type/i)).toBeInTheDocument();
135135

136-
expect(mockWebSocket.send).toBeDefined();
137-
138136
const instanceTypeSelect = screen.getByRole("button", {
139137
name: /instance type/i,
140138
});
@@ -165,8 +163,8 @@ describe("CreateWorkspacePageExperimental", () => {
165163
.mockImplementation((_versionId, _ownerId, callbacks) => {
166164
const [mockWebSocket, publisher] = createMockWebSocket("ws://test");
167165

168-
mockWebSocket.addEventListener("error", (event) => {
169-
callbacks.onError((event as ErrorEvent).error);
166+
mockWebSocket.addEventListener("error", () => {
167+
callbacks.onError(new Error("Connection failed"));
170168
});
171169

172170
setTimeout(() => {
@@ -271,94 +269,6 @@ describe("CreateWorkspacePageExperimental", () => {
271269
});
272270

273271
describe("Dynamic Parameter Types", () => {
274-
it("renders dropdown parameter with options", async () => {
275-
renderCreateWorkspacePageExperimental();
276-
await waitForLoaderToBeRemoved();
277-
278-
expect(screen.getByText(/instance type/i)).toBeInTheDocument();
279-
280-
const select = screen.getByRole("button", { name: /instance type/i });
281-
282-
await waitFor(async () => {
283-
await userEvent.click(select);
284-
});
285-
286-
// Options appear only in the dropdown when no value is selected
287-
expect(screen.getByText(/t3\.micro/i)).toBeInTheDocument();
288-
expect(screen.getByText(/t3\.small/i)).toBeInTheDocument();
289-
expect(screen.getByText(/t3\.medium/i)).toBeInTheDocument();
290-
});
291-
292-
it("renders number parameter with slider", async () => {
293-
renderCreateWorkspacePageExperimental();
294-
await waitForLoaderToBeRemoved();
295-
296-
await waitFor(() => {
297-
expect(screen.getByText("CPU Count")).toBeInTheDocument();
298-
});
299-
300-
await waitFor(() => {
301-
const numberInput = screen.getByDisplayValue("2");
302-
expect(numberInput).toBeInTheDocument();
303-
});
304-
});
305-
306-
it("renders boolean parameter with switch", async () => {
307-
renderCreateWorkspacePageExperimental();
308-
await waitForLoaderToBeRemoved();
309-
310-
await waitFor(() => {
311-
expect(screen.getByText("Enable Monitoring")).toBeInTheDocument();
312-
expect(
313-
screen.getByRole("switch", { name: /enable monitoring/i }),
314-
).toBeInTheDocument();
315-
});
316-
});
317-
318-
it("renders list parameter with tag input", async () => {
319-
renderCreateWorkspacePageExperimental();
320-
await waitForLoaderToBeRemoved();
321-
322-
await waitFor(() => {
323-
expect(screen.getByText("Tags")).toBeInTheDocument();
324-
expect(
325-
screen.getByRole("textbox", { name: /tags/i }),
326-
).toBeInTheDocument();
327-
});
328-
});
329-
330-
it("renders multi-select parameter", async () => {
331-
renderCreateWorkspacePageExperimental();
332-
await waitForLoaderToBeRemoved();
333-
334-
await waitFor(() => {
335-
expect(screen.getByText("IDEs")).toBeInTheDocument();
336-
});
337-
338-
const multiSelect = screen.getByTestId("multiselect-ides");
339-
expect(multiSelect).toBeInTheDocument();
340-
341-
const select = multiSelect.querySelector('[role="combobox"]');
342-
expect(select).toBeInTheDocument();
343-
344-
await waitFor(async () => {
345-
await userEvent.click(select!);
346-
});
347-
348-
expect(
349-
screen.getByRole("option", { name: /vscode/i }),
350-
).toBeInTheDocument();
351-
expect(
352-
screen.getByRole("option", { name: /cursor/i }),
353-
).toBeInTheDocument();
354-
expect(
355-
screen.getByRole("option", { name: /goland/i }),
356-
).toBeInTheDocument();
357-
expect(
358-
screen.getByRole("option", { name: /windsurf/i }),
359-
).toBeInTheDocument();
360-
});
361-
362272
it("displays parameter validation errors", async () => {
363273
jest
364274
.spyOn(API, "templateVersionDynamicParameters")

0 commit comments

Comments
 (0)
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy