Revert "Add .only to document request tests"

This reverts commit 18e800a.
remix-run · Nov 28, 2022 · 21f28d1 · 21f28d1
1 parent 2ad7b04
commit 21f28d1
Show file tree

Hide file tree

Showing 13 changed files with 48 additions and 48 deletions.
diff --git a/integration/action-test.ts b/integration/action-test.ts
@@ -87,13 +87,13 @@ test.describe("actions", () => {
     expect(logs).toHaveLength(0);
   });
 
-  test.only("is not called on document GET requests", async () => {
+  test("is not called on document GET requests", async () => {
     let res = await fixture.requestDocument("/urlencoded");
     let html = selectHtml(await res.text(), "#text");
     expect(html).toMatch(WAITING_VALUE);
   });
 
-  test.only("is called on document POST requests", async () => {
+  test("is called on document POST requests", async () => {
     let FIELD_VALUE = "cheeseburger";
 
     let params = new URLSearchParams();
@@ -115,7 +115,7 @@ test.describe("actions", () => {
     await page.waitForSelector(`#text:has-text("${SUBMITTED_VALUE}")`);
   });
 
-  test.only("redirects a thrown response on document requests", async () => {
+  test("redirects a thrown response on document requests", async () => {
     let params = new URLSearchParams();
     let res = await fixture.postDocument(`/${THROWS_REDIRECT}`, params);
     expect(res.status).toBe(302);

diff --git a/integration/catch-boundary-data-test.ts b/integration/catch-boundary-data-test.ts
@@ -158,7 +158,7 @@ test.beforeAll(async () => {
 
 test.afterAll(async () => appFixture.close());
 
-test.only("renders root boundary with data available", async () => {
+test("renders root boundary with data available", async () => {
   let res = await fixture.requestDocument(NO_BOUNDARY_LOADER);
   expect(res.status).toBe(401);
   let html = await res.text();
@@ -176,7 +176,7 @@ test("renders root boundary with data available on transition", async ({
   await page.waitForSelector(`#root-boundary-data:has-text("${ROOT_DATA}")`);
 });
 
-test.only("renders layout boundary with data available", async () => {
+test("renders layout boundary with data available", async () => {
   let res = await fixture.requestDocument(HAS_BOUNDARY_LAYOUT_NESTED_LOADER);
   expect(res.status).toBe(401);
   let html = await res.text();
@@ -200,7 +200,7 @@ test("renders layout boundary with data available on transition", async ({
   );
 });
 
-test.only("renders self boundary with layout data available", async () => {
+test("renders self boundary with layout data available", async () => {
   let res = await fixture.requestDocument(HAS_BOUNDARY_NESTED_LOADER);
   expect(res.status).toBe(401);
   let html = await res.text();

diff --git a/integration/catch-boundary-test.ts b/integration/catch-boundary-test.ts
@@ -185,7 +185,7 @@ test.describe("CatchBoundary", () => {
 
   test.afterAll(() => appFixture.close());
 
-  test.only("non-matching urls on document requests", async () => {
+  test("non-matching urls on document requests", async () => {
     let res = await fixture.requestDocument(NOT_FOUND_HREF);
     expect(res.status).toBe(404);
     expect(await res.text()).toMatch(ROOT_BOUNDARY_TEXT);
@@ -198,7 +198,7 @@ test.describe("CatchBoundary", () => {
     await page.waitForSelector("#root-boundary");
   });
 
-  test.only("own boundary, action, document request", async () => {
+  test("own boundary, action, document request", async () => {
     let params = new URLSearchParams();
     let res = await fixture.postDocument(HAS_BOUNDARY_ACTION, params);
     expect(res.status).toBe(401);
@@ -223,7 +223,7 @@ test.describe("CatchBoundary", () => {
     await page.waitForSelector("#action-boundary");
   });
 
-  test.only("bubbles to parent in action document requests", async () => {
+  test("bubbles to parent in action document requests", async () => {
     let params = new URLSearchParams();
     let res = await fixture.postDocument(NO_BOUNDARY_ACTION, params);
     expect(res.status).toBe(401);
@@ -248,7 +248,7 @@ test.describe("CatchBoundary", () => {
     await page.waitForSelector("#root-boundary");
   });
 
-  test.only("own boundary, loader, document request", async () => {
+  test("own boundary, loader, document request", async () => {
     let res = await fixture.requestDocument(HAS_BOUNDARY_LOADER);
     expect(res.status).toBe(401);
     expect(await res.text()).toMatch(OWN_BOUNDARY_TEXT);
@@ -261,7 +261,7 @@ test.describe("CatchBoundary", () => {
     await page.waitForSelector("#boundary-loader");
   });
 
-  test.only("bubbles to parent in loader document requests", async () => {
+  test("bubbles to parent in loader document requests", async () => {
     let res = await fixture.requestDocument(NO_BOUNDARY_LOADER);
     expect(res.status).toBe(401);
     expect(await res.text()).toMatch(ROOT_BOUNDARY_TEXT);

diff --git a/integration/error-boundary-test.ts b/integration/error-boundary-test.ts
@@ -286,13 +286,13 @@ test.describe("ErrorBoundary", () => {
     await appFixture.close();
   });
 
-  test.only("invalid request methods", async () => {
+  test("invalid request methods", async () => {
     let res = await fixture.requestDocument("/", { method: "OPTIONS" });
     expect(res.status).toBe(405);
     expect(await res.text()).toMatch(ROOT_BOUNDARY_TEXT);
   });
 
-  test.only("own boundary, action, document request", async () => {
+  test("own boundary, action, document request", async () => {
     let params = new URLSearchParams();
     let res = await fixture.postDocument(HAS_BOUNDARY_ACTION, params);
     expect(res.status).toBe(500);
@@ -319,7 +319,7 @@ test.describe("ErrorBoundary", () => {
     expect(await app.getHtml("main")).toMatch(OWN_BOUNDARY_TEXT);
   });
 
-  test.only("bubbles to parent in action document requests", async () => {
+  test("bubbles to parent in action document requests", async () => {
     let params = new URLSearchParams();
     let res = await fixture.postDocument(NO_BOUNDARY_ACTION, params);
     expect(res.status).toBe(500);
@@ -344,7 +344,7 @@ test.describe("ErrorBoundary", () => {
     expect(await app.getHtml("main")).toMatch(ROOT_BOUNDARY_TEXT);
   });
 
-  test.only("own boundary, loader, document request", async () => {
+  test("own boundary, loader, document request", async () => {
     let res = await fixture.requestDocument(HAS_BOUNDARY_LOADER);
     expect(res.status).toBe(500);
     expect(await res.text()).toMatch(OWN_BOUNDARY_TEXT);
@@ -357,7 +357,7 @@ test.describe("ErrorBoundary", () => {
     expect(await app.getHtml("main")).toMatch(OWN_BOUNDARY_TEXT);
   });
 
-  test.only("bubbles to parent in loader document requests", async () => {
+  test("bubbles to parent in loader document requests", async () => {
     let res = await fixture.requestDocument(NO_BOUNDARY_LOADER);
     expect(res.status).toBe(500);
     expect(await res.text()).toMatch(ROOT_BOUNDARY_TEXT);
@@ -372,7 +372,7 @@ test.describe("ErrorBoundary", () => {
     expect(await app.getHtml("main")).toMatch(ROOT_BOUNDARY_TEXT);
   });
 
-  test.only("ssr rendering errors with no boundary", async () => {
+  test("ssr rendering errors with no boundary", async () => {
     let res = await fixture.requestDocument(NO_BOUNDARY_RENDER);
     expect(res.status).toBe(500);
     expect(await res.text()).toMatch(ROOT_BOUNDARY_TEXT);
@@ -387,7 +387,7 @@ test.describe("ErrorBoundary", () => {
     expect(await app.getHtml("main")).toMatch(ROOT_BOUNDARY_TEXT);
   });
 
-  test.only("ssr rendering errors with boundary", async () => {
+  test("ssr rendering errors with boundary", async () => {
     let res = await fixture.requestDocument(HAS_BOUNDARY_RENDER);
     expect(res.status).toBe(500);
     expect(await res.text()).toMatch(OWN_BOUNDARY_TEXT);
@@ -432,7 +432,7 @@ test.describe("ErrorBoundary", () => {
     await page.waitForSelector("#root-boundary");
   });
 
-  test.only("renders root boundary in document POST without action requests", async () => {
+  test("renders root boundary in document POST without action requests", async () => {
     let res = await fixture.requestDocument(NO_BOUNDARY_NO_LOADER_OR_ACTION, {
       method: "post",
     });
@@ -449,7 +449,7 @@ test.describe("ErrorBoundary", () => {
     await page.waitForSelector("#root-boundary");
   });
 
-  test.only("renders own boundary in document POST without action requests", async () => {
+  test("renders own boundary in document POST without action requests", async () => {
     let res = await fixture.requestDocument(HAS_BOUNDARY_NO_LOADER_OR_ACTION, {
       method: "post",
     });
@@ -595,7 +595,7 @@ test.describe("ErrorBoundary", () => {
       expect(await app.getHtml("h1")).toMatch(INTERNAL_ERROR_BOUNDARY_HEADING);
     });
 
-    test.only("bubbles to internal boundary if loader doesn't return (document requests)", async () => {
+    test("bubbles to internal boundary if loader doesn't return (document requests)", async () => {
       let res = await fixture.requestDocument(NO_ROOT_BOUNDARY_LOADER_RETURN);
       expect(res.status).toBe(500);
       expect(await res.text()).toMatch(INTERNAL_ERROR_BOUNDARY_HEADING);
@@ -611,7 +611,7 @@ test.describe("ErrorBoundary", () => {
       expect(await app.getHtml("h1")).toMatch(INTERNAL_ERROR_BOUNDARY_HEADING);
     });
 
-    test.only("bubbles to internal boundary if action doesn't return (document requests)", async () => {
+    test("bubbles to internal boundary if action doesn't return (document requests)", async () => {
       let res = await fixture.requestDocument(NO_ROOT_BOUNDARY_ACTION_RETURN, {
         method: "post",
       });

diff --git a/integration/form-data-test.ts b/integration/form-data-test.ts
@@ -24,15 +24,15 @@ test.beforeAll(async () => {
   });
 });
 
-test.only("invalid content-type does not crash server", async () => {
+test("invalid content-type does not crash server", async () => {
   let response = await fixture.requestDocument("/", {
     method: "post",
     headers: { "content-type": "application/json" },
   });
   expect(await response.text()).toMatch("no pizza");
 });
 
-test.only("invalid urlencoded body does not crash server", async () => {
+test("invalid urlencoded body does not crash server", async () => {
   let response = await fixture.requestDocument("/", {
     method: "post",
     headers: { "content-type": "application/x-www-form-urlencoded" },
@@ -41,7 +41,7 @@ test.only("invalid urlencoded body does not crash server", async () => {
   expect(await response.text()).toMatch("pizza");
 });
 
-test.only("invalid multipart content-type does not crash server", async () => {
+test("invalid multipart content-type does not crash server", async () => {
   let response = await fixture.requestDocument("/", {
     method: "post",
     headers: { "content-type": "multipart/form-data" },
@@ -50,7 +50,7 @@ test.only("invalid multipart content-type does not crash server", async () => {
   expect(await response.text()).toMatch("pizza");
 });
 
-test.only("invalid multipart body does not crash server", async () => {
+test("invalid multipart body does not crash server", async () => {
   let response = await fixture.requestDocument("/", {
     method: "post",
     headers: { "content-type": "multipart/form-data; boundary=abc" },

diff --git a/integration/form-test.ts b/integration/form-test.ts
@@ -458,7 +458,7 @@ test.describe("Forms", () => {
     await appFixture.close();
   });
 
-  test.describe.only("without JavaScript", () => {
+  test.describe("without JavaScript", () => {
     test.use({ javaScriptEnabled: false });
 
     runFormTests();

diff --git a/integration/headers-test.ts b/integration/headers-test.ts
@@ -81,20 +81,20 @@ test.describe("headers export", () => {
     });
   });
 
-  test.only("can use `action` headers", async () => {
+  test("can use `action` headers", async () => {
     let response = await appFixture.postDocument(
       "/action",
       new URLSearchParams()
     );
     expect(response.headers.get(ACTION_HKEY)).toBe(ACTION_HVALUE);
   });
 
-  test.only("can use the loader headers when all routes have loaders", async () => {
+  test("can use the loader headers when all routes have loaders", async () => {
     let response = await appFixture.requestDocument("/");
     expect(response.headers.get(ROOT_HEADER_KEY)).toBe(ROOT_HEADER_VALUE);
   });
 
-  test.only("can use the loader headers when parents don't have loaders", async () => {
+  test("can use the loader headers when parents don't have loaders", async () => {
     let HEADER_KEY = "X-Test";
     let HEADER_VALUE = "SUCCESS";
 

diff --git a/integration/path-mapping-test.ts b/integration/path-mapping-test.ts
@@ -100,18 +100,18 @@ test.beforeAll(async () => {
   });
 });
 
-test.only("import internal library via alias other than ~", async () => {
+test("import internal library via alias other than ~", async () => {
   // test for https://github.com/remix-run/remix/issues/2298
   let response = await fixture.requestDocument("/");
   expect(await response.text()).toMatch("this is a pizza");
 });
 
-test.only("import internal library via ~ alias", async () => {
+test("import internal library via ~ alias", async () => {
   let response = await fixture.requestDocument("/tilde-alias");
   expect(await response.text()).toMatch("this is a pizza");
 });
 
-test.only("works for mdx files", async () => {
+test("works for mdx files", async () => {
   let response = await fixture.requestDocument("/mdx");
   expect(await response.text()).toMatch("this is a pizza");
 });
diff --git a/integration/prefetch-test.ts b/integration/prefetch-test.ts
@@ -85,7 +85,7 @@ test.describe("prefetch=none", () => {
 
   test.afterAll(() => appFixture.close());
 
-  test.only("does not render prefetch tags during SSR", async ({ page }) => {
+  test("does not render prefetch tags during SSR", async ({ page }) => {
     let res = await fixture.requestDocument("/");
     expect(res.status).toBe(200);
     expect(await page.locator("#nav link").count()).toBe(0);
@@ -111,7 +111,7 @@ test.describe("prefetch=render", () => {
     await appFixture.close();
   });
 
-  test.only("does not render prefetch tags during SSR", async ({ page }) => {
+  test("does not render prefetch tags during SSR", async ({ page }) => {
     let res = await fixture.requestDocument("/");
     expect(res.status).toBe(200);
     expect(await page.locator("#nav link").count()).toBe(0);
@@ -153,7 +153,7 @@ test.describe("prefetch=intent (hover)", () => {
     await appFixture.close();
   });
 
-  test.only("does not render prefetch tags during SSR", async ({ page }) => {
+  test("does not render prefetch tags during SSR", async ({ page }) => {
     let res = await fixture.requestDocument("/");
     expect(res.status).toBe(200);
     expect(await page.locator("#nav link").count()).toBe(0);
@@ -224,7 +224,7 @@ test.describe("prefetch=intent (focus)", () => {
     await appFixture.close();
   });
 
-  test.only("does not render prefetch tags during SSR", async ({ page }) => {
+  test("does not render prefetch tags during SSR", async ({ page }) => {
     let res = await fixture.requestDocument("/");
     expect(res.status).toBe(200);
     expect(await page.locator("#nav link").count()).toBe(0);

diff --git a/integration/rendering-test.ts b/integration/rendering-test.ts
@@ -46,7 +46,7 @@ test.describe("rendering", () => {
 
   test.afterAll(async () => appFixture.close());
 
-  test.only("server renders matching routes", async () => {
+  test("server renders matching routes", async () => {
     let res = await fixture.requestDocument("/");
     expect(res.status).toBe(200);
     expect(selectHtml(await res.text(), "#content")).toBe(`<div id="content">

diff --git a/integration/resource-routes-test.ts b/integration/resource-routes-test.ts
@@ -106,7 +106,7 @@ test.describe("loader in an app", async () => {
     runTests();
   });
 
-  test.describe.only("without JavaScript", () => {
+  test.describe("without JavaScript", () => {
     test.use({ javaScriptEnabled: false });
     runTests();
   });

diff --git a/integration/splat-routes-test.ts b/integration/splat-routes-test.ts
@@ -87,42 +87,42 @@ test.describe("rendering", () => {
     });
   });
 
-  test.only("flat exact match", async () => {
+  test("flat exact match", async () => {
     let res = await fixture.requestDocument("/flat");
     expect(await res.text()).toMatch(FLAT_$);
   });
 
-  test.only("flat deep match", async () => {
+  test("flat deep match", async () => {
     let res = await fixture.requestDocument("/flat/swig");
     expect(await res.text()).toMatch(FLAT_$);
   });
 
-  test.only("prioritizes index over root splat", async () => {
+  test("prioritizes index over root splat", async () => {
     let res = await fixture.requestDocument("/");
     expect(await res.text()).toMatch(ROOT_INDEX);
   });
 
-  test.only("matches root splat", async () => {
+  test("matches root splat", async () => {
     let res = await fixture.requestDocument("/twisted/sugar");
     expect(await res.text()).toMatch(ROOT_$);
   });
 
-  test.only("prioritizes index over splat for parent route match", async () => {
+  test("prioritizes index over splat for parent route match", async () => {
     let res = await fixture.requestDocument("/nested");
     expect(await res.text()).toMatch(NESTED_INDEX);
   });
 
-  test.only("nested child", async () => {
+  test("nested child", async () => {
     let res = await fixture.requestDocument("/nested/sodalicious");
     expect(await res.text()).toMatch(NESTED_$);
   });
 
-  test.only("parentless exact match", async () => {
+  test("parentless exact match", async () => {
     let res = await fixture.requestDocument("/parentless");
     expect(await res.text()).toMatch(PARENTLESS_$);
   });
 
-  test.only("parentless deep match", async () => {
+  test("parentless deep match", async () => {
     let res = await fixture.requestDocument("/parentless/chip");
     expect(await res.text()).toMatch(PARENTLESS_$);
   });

diff --git a/integration/upload-test.ts b/integration/upload-test.ts
@@ -237,7 +237,7 @@ test("can catch MaxPartSizeExceededError when file is too big with createMemoryU
   expect(await app.getHtml("#size")).toMatch(">13<");
 });
 
-test.describe.only("without javascript", () => {
+test.describe("without javascript", () => {
   test.use({ javaScriptEnabled: false });
 
   test("can upload a file with createFileUploadHandler", async ({ page }) => {