Skip multi-fetch tests, enable passing tests

remix-run · Apr 8, 2024 · 219223d · 219223d
1 parent 2e162a0
commit 219223d
Show file tree

Hide file tree

Showing 21 changed files with 26 additions and 29 deletions.
diff --git a/integration/action-test.ts b/integration/action-test.ts
@@ -8,7 +8,7 @@ import {
 import type { Fixture, AppFixture } from "./helpers/create-fixture.js";
 import { PlaywrightFixture, selectHtml } from "./helpers/playwright-fixture.js";
 
-test.describe("actions", () => {
+test.describe.skip("actions", () => {
   let fixture: Fixture;
   let appFixture: AppFixture;
 

diff --git a/integration/catch-boundary-data-test.ts b/integration/catch-boundary-data-test.ts
@@ -28,7 +28,7 @@ let HAS_BOUNDARY_NESTED_LOADER = "/yes/loader-self-boundary" as const;
 let ROOT_DATA = "root data";
 let LAYOUT_DATA = "root data";
 
-test.describe("ErrorBoundary (thrown responses)", () => {
+test.describe.skip("ErrorBoundary (thrown responses)", () => {
   test.beforeEach(async ({ context }) => {
     await context.route(/_data/, async (route) => {
       await new Promise((resolve) => setTimeout(resolve, 50));

diff --git a/integration/catch-boundary-test.ts b/integration/catch-boundary-test.ts
@@ -8,7 +8,7 @@ import {
 import type { Fixture, AppFixture } from "./helpers/create-fixture.js";
 import { PlaywrightFixture } from "./helpers/playwright-fixture.js";
 
-test.describe("ErrorBoundary (thrown responses)", () => {
+test.describe.skip("ErrorBoundary (thrown responses)", () => {
   let fixture: Fixture;
   let appFixture: AppFixture;
 

diff --git a/integration/client-data-test.ts b/integration/client-data-test.ts
@@ -138,7 +138,7 @@ function getFiles({
   };
 }
 
-test.describe("Client Data", () => {
+test.describe.skip("Client Data", () => {
   let appFixture: AppFixture;
 
   test.afterAll(() => {

diff --git a/integration/defer-loader-test.ts b/integration/defer-loader-test.ts
@@ -11,7 +11,7 @@ import { PlaywrightFixture } from "./helpers/playwright-fixture.js";
 let fixture: Fixture;
 let appFixture: AppFixture;
 
-test.describe("deferred loaders", () => {
+test.describe.skip("deferred loaders", () => {
   test.beforeAll(async () => {
     fixture = await createFixture({
       files: {

diff --git a/integration/defer-test.ts b/integration/defer-test.ts
@@ -32,7 +32,7 @@ declare global {
   };
 }
 
-test.describe("non-aborted", () => {
+test.describe.skip("non-aborted", () => {
   let fixture: Fixture;
   let appFixture: AppFixture;
 
@@ -972,7 +972,7 @@ test.describe("non-aborted", () => {
   });
 });
 
-test.describe("aborted", () => {
+test.describe.skip("aborted", () => {
   let fixture: Fixture;
   let appFixture: AppFixture;
 

diff --git a/integration/error-boundary-test.ts b/integration/error-boundary-test.ts
@@ -9,7 +9,7 @@ import {
 import type { Fixture, AppFixture } from "./helpers/create-fixture.js";
 import { PlaywrightFixture } from "./helpers/playwright-fixture.js";
 
-test.describe("ErrorBoundary", () => {
+test.describe.skip("ErrorBoundary", () => {
   let fixture: Fixture;
   let appFixture: AppFixture;
   let _consoleError: any;
@@ -655,7 +655,7 @@ test.describe("ErrorBoundary", () => {
   });
 });
 
-test.describe("loaderData in ErrorBoundary", () => {
+test.describe.skip("loaderData in ErrorBoundary", () => {
   let fixture: Fixture;
   let appFixture: AppFixture;
   let consoleErrors: string[];
@@ -895,7 +895,7 @@ test.describe("loaderData in ErrorBoundary", () => {
   }
 });
 
-test.describe("Default ErrorBoundary", () => {
+test.describe.skip("Default ErrorBoundary", () => {
   let fixture: Fixture;
   let appFixture: AppFixture;
   let _consoleError: any;
@@ -1223,7 +1223,7 @@ test.describe("Default ErrorBoundary", () => {
   });
 });
 
-test("Allows back-button out of an error boundary after a hard reload", async ({
+test.skip("Allows back-button out of an error boundary after a hard reload", async ({
   page,
   browserName,
 }) => {

diff --git a/integration/error-boundary-v2-test.ts b/integration/error-boundary-v2-test.ts
@@ -10,7 +10,7 @@ import {
 import type { Fixture, AppFixture } from "./helpers/create-fixture.js";
 import { PlaywrightFixture } from "./helpers/playwright-fixture.js";
 
-test.describe("ErrorBoundary", () => {
+test.describe.skip("ErrorBoundary", () => {
   let fixture: Fixture;
   let appFixture: AppFixture;
   let oldConsoleError: () => void;

diff --git a/integration/error-data-request-test.ts b/integration/error-data-request-test.ts
@@ -8,7 +8,7 @@ import {
 } from "./helpers/create-fixture.js";
 import type { Fixture, AppFixture } from "./helpers/create-fixture.js";
 
-test.describe("ErrorBoundary", () => {
+test.describe.skip("ErrorBoundary", () => {
   let fixture: Fixture;
   let appFixture: AppFixture;
   let _consoleError: any;

diff --git a/integration/error-sanitization-test.ts b/integration/error-sanitization-test.ts
@@ -138,7 +138,7 @@ const routeFiles = {
   `,
 };
 
-test.describe("Error Sanitization", () => {
+test.describe.skip("Error Sanitization", () => {
   let fixture: Fixture;
   let oldConsoleError: () => void;
   let errorLogs: any[] = [];

diff --git a/integration/fetcher-layout-test.ts b/integration/fetcher-layout-test.ts
@@ -11,7 +11,7 @@ import { PlaywrightFixture } from "./helpers/playwright-fixture.js";
 let fixture: Fixture;
 let appFixture: AppFixture;
 
-test.describe("multi fetch", () => {
+test.describe.skip("multi fetch", () => {
   test.beforeAll(async () => {
     fixture = await createFixture({
       files: {

diff --git a/integration/fetcher-test.ts b/integration/fetcher-test.ts
@@ -8,7 +8,7 @@ import {
 import type { Fixture, AppFixture } from "./helpers/create-fixture.js";
 import { PlaywrightFixture } from "./helpers/playwright-fixture.js";
 
-test.describe("useFetcher", () => {
+test.describe.skip("useFetcher", () => {
   let fixture: Fixture;
   let appFixture: AppFixture;
 
@@ -427,7 +427,7 @@ test.describe("useFetcher", () => {
   });
 });
 
-test.describe("fetcher aborts and adjacent forms", () => {
+test.describe.skip("fetcher aborts and adjacent forms", () => {
   let fixture: Fixture;
   let appFixture: AppFixture;
 

diff --git a/integration/file-uploads-test.ts b/integration/file-uploads-test.ts
@@ -11,7 +11,7 @@ import {
 import type { Fixture, AppFixture } from "./helpers/create-fixture.js";
 import { PlaywrightFixture } from "./helpers/playwright-fixture.js";
 
-test.describe("file-uploads", () => {
+test.describe.skip("file-uploads", () => {
   let fixture: Fixture;
   let appFixture: AppFixture;
 

diff --git a/integration/form-data-test.ts b/integration/form-data-test.ts
@@ -5,7 +5,7 @@ import type { Fixture } from "./helpers/create-fixture.js";
 
 let fixture: Fixture;
 
-test.describe("multi fetch", () => {
+test.describe.skip("multi fetch", () => {
   test.beforeAll(async () => {
     fixture = await createFixture({
       files: {

diff --git a/integration/form-test.ts b/integration/form-test.ts
@@ -8,7 +8,7 @@ import {
 import type { Fixture, AppFixture } from "./helpers/create-fixture.js";
 import { getElement, PlaywrightFixture } from "./helpers/playwright-fixture.js";
 
-test.describe("Forms", () => {
+test.describe.skip("Forms", () => {
   let fixture: Fixture;
   let appFixture: AppFixture;
 

diff --git a/integration/loader-test.ts b/integration/loader-test.ts
@@ -8,7 +8,7 @@ import {
 import type { Fixture, AppFixture } from "./helpers/create-fixture.js";
 import { PlaywrightFixture } from "./helpers/playwright-fixture.js";
 
-test.describe("loader", () => {
+test.describe.skip("loader", () => {
   let fixture: Fixture;
 
   let ROOT_DATA = "ROOT_DATA";
@@ -69,7 +69,7 @@ test.describe("loader", () => {
   });
 });
 
-test.describe("loader in an app", () => {
+test.describe.skip("loader in an app", () => {
   let appFixture: AppFixture;
 
   let HOME_PAGE_TEXT = "hello world";

diff --git a/integration/navigation-state-test.ts b/integration/navigation-state-test.ts
@@ -22,7 +22,7 @@ const IDLE_STATE = {
   state: "idle",
 };
 
-test.describe("navigation states", () => {
+test.describe.skip("navigation states", () => {
   let fixture: Fixture;
   let appFixture: AppFixture;
 

diff --git a/integration/playwright.config.ts b/integration/playwright.config.ts
@@ -6,10 +6,8 @@ const config: PlaywrightTestConfig = {
   testMatch: ["**/*-test.ts"],
   // TODO: Temporary!  Remove from this list as we get each suite passing
   testIgnore: [
-    "**/client-data-test.ts",
     "**/error-sanitization-test.ts",
     "**/file-uploads-test.ts",
-    "**/resource-routes-test.ts",
     "**/vite-basename-test.ts",
     "**/vite-build-test.ts",
     "**/vite-cloudflare-test.ts",
@@ -18,7 +16,6 @@ const config: PlaywrightTestConfig = {
     "**/vite-dot-server-test.ts",
     "**/vite-hmr-hdr-test.ts",
     "**/vite-plugin-order-validation-test.ts",
-    "**/vite-spa-mode-test.ts",
   ],
   /* Maximum time one test can run for. */
   timeout: process.platform === "win32" ? 60_000 : 30_000,

diff --git a/integration/redirects-test.ts b/integration/redirects-test.ts
@@ -8,7 +8,7 @@ import {
 import type { Fixture, AppFixture } from "./helpers/create-fixture.js";
 import { PlaywrightFixture } from "./helpers/playwright-fixture.js";
 
-test.describe("redirects", () => {
+test.describe.skip("redirects", () => {
   let fixture: Fixture;
   let appFixture: AppFixture;
 

diff --git a/integration/revalidate-test.ts b/integration/revalidate-test.ts
@@ -8,7 +8,7 @@ import {
 import type { AppFixture } from "./helpers/create-fixture.js";
 import { PlaywrightFixture } from "./helpers/playwright-fixture.js";
 
-test.describe("Revalidation", () => {
+test.describe.skip("Revalidation", () => {
   let appFixture: AppFixture;
 
   test.beforeAll(async () => {

diff --git a/integration/set-cookie-revalidation-test.ts b/integration/set-cookie-revalidation-test.ts
@@ -13,7 +13,7 @@ let appFixture: AppFixture;
 
 let BANNER_MESSAGE = "you do not have permission to view /protected";
 
-test.describe("set-cookie revalidation", () => {
+test.describe.skip("set-cookie revalidation", () => {
   test.beforeAll(async () => {
     fixture = await createFixture({
       files: {