Skip to content

Commit 033728a

Browse files
committed
refactor: move all vscode tests to e2e
1 parent 42b873f commit 033728a

File tree

2 files changed

+44
-75
lines changed

2 files changed

+44
-75
lines changed

test/e2e/routes.test.ts

Lines changed: 44 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,12 +3,13 @@ import { clean, tmpdir } from "../utils/helpers"
33
import * as path from "path"
44
import { promises as fs } from "fs"
55

6+
const routes = ["/", "/vscode", "/vscode/"]
7+
68
describe("VS Code Routes", ["--disable-workspace-trust"], {}, async () => {
79
const testName = "integrated-terminal"
810
test.beforeAll(async () => {
911
await clean(testName)
1012
})
11-
const routes = ["/", "/vscode", "/vscode/"]
1213

1314
test("should load all route variations", async ({ codeServerPage }) => {
1415
for (const route of routes) {
@@ -63,7 +64,7 @@ describe("VS Code Routes with code-workspace", ["--disable-workspace-trust", COD
6364
await clean(testName)
6465
})
6566

66-
test.only("should redirect to the passed in folder using human-readable query", async ({ codeServerPage }) => {
67+
test("should redirect to the passed in folder using human-readable query", async ({ codeServerPage }) => {
6768
const url = new URL(codeServerPage.page.url())
6869
expect(url.pathname).toBe("/")
6970
expect(url.search).toBe(`?folder=${CODE_FOLDER_DIR}`)
@@ -84,3 +85,44 @@ describe(
8485
})
8586
},
8687
)
88+
89+
describe(
90+
"VS Code Routes with no workspace or folder",
91+
["--disable-workspace-trust"],
92+
{},
93+
async () => {
94+
test("should redirect to last query folder/workspace", async ({ codeServerPage }) => {
95+
const folder = process.env.CODE_FOLDER_DIR
96+
const workspace = process.env.CODE_WORKSPACE_DIR
97+
await codeServerPage.navigate(`/?folder=${folder}&workspace=${workspace}`)
98+
99+
// If you visit again without query parameters it will re-attach them by
100+
// redirecting. It should always redirect to the same route.
101+
for (const route of routes) {
102+
await codeServerPage.navigate(route)
103+
const url = new URL(codeServerPage.page.url())
104+
expect(url.pathname).toBe(route)
105+
expect(url.search).toBe(`?folder=${folder}&workspace=${workspace}`)
106+
}
107+
})
108+
},
109+
)
110+
111+
describe(
112+
"VS Code Routes with no workspace or folder",
113+
["--disable-workspace-trust"],
114+
{},
115+
async () => {
116+
test("should not redirect if ew passed in", async ({ codeServerPage }) => {
117+
const folder = process.env.CODE_FOLDER_DIR
118+
const workspace = process.env.CODE_WORKSPACE_DIR
119+
await codeServerPage.navigate(`/?folder=${folder}&workspace=${workspace}`)
120+
121+
// Closing the folder should stop the redirecting.
122+
await codeServerPage.navigate("/?ew=true")
123+
let url = new URL(codeServerPage.page.url())
124+
expect(url.pathname).toBe("/")
125+
expect(url.search).toBe("?ew=true")
126+
})
127+
},
128+
)

test/unit/node/routes/vscode.test.ts

Lines changed: 0 additions & 73 deletions
This file was deleted.

0 commit comments

Comments
 (0)