Skip to content

Commit

Permalink
Revert
Browse files Browse the repository at this point in the history
  • Loading branch information
markdalgleish committed Oct 14, 2024
1 parent c2bb8af commit feb10d3
Showing 1 changed file with 3 additions and 19 deletions.
22 changes: 3 additions & 19 deletions integration/vite-route-config-test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -206,8 +206,6 @@ test.describe("route config", () => {
test("supports correcting a missing route config", async ({
page,
viteDev,
browserName,
context,
}) => {
let files: Files = async ({ port }) => ({
"vite.config.js": await viteConfig.basic({ port }),
Expand Down Expand Up @@ -247,15 +245,8 @@ test.describe("route config", () => {
);

await expect(async () => {
// Force new page instance for webkit.
// Otherwise browser doesn't seem to fetch new manifest probably due to caching.
if (browserName === "webkit") {
page = await context.newPage();
}
// Reload to pick up new route for current path
await page.goto(`http://localhost:${port}/`, {
waitUntil: "networkidle",
});
// Reload to pick up classic FS routes
await page.reload();
await expect(page.locator("[data-test-route]")).toHaveText("FS route");
}).toPass();

Expand All @@ -279,15 +270,8 @@ test.describe("route config", () => {
);

await expect(async () => {
// Force new page instance for webkit.
// Otherwise browser doesn't seem to fetch new manifest probably due to caching.
if (browserName === "webkit") {
page = await context.newPage();
}
// Reload to pick up new route for current path
await page.goto(`http://localhost:${port}/`, {
waitUntil: "networkidle",
});
await page.reload();
await expect(page.locator("[data-test-route]")).toHaveText(
"Test route 2"
);
Expand Down

0 comments on commit feb10d3

Please sign in to comment.