From a59b47873f13d14aa6c587a38ab5af892e1f2eaa Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Gerg=C5=91=20M=C3=B3ricz?= Date: Fri, 29 Aug 2025 10:43:47 +0200 Subject: [PATCH] chore: remove tests that don't make sense --- .../api/src/__tests__/snips/v1/scrape.test.ts | 21 ------------------- .../api/src/__tests__/snips/v2/scrape.test.ts | 21 ------------------- 2 files changed, 42 deletions(-) diff --git a/apps/api/src/__tests__/snips/v1/scrape.test.ts b/apps/api/src/__tests__/snips/v1/scrape.test.ts index c49581e1f..0b4d590c9 100644 --- a/apps/api/src/__tests__/snips/v1/scrape.test.ts +++ b/apps/api/src/__tests__/snips/v1/scrape.test.ts @@ -31,27 +31,6 @@ describe("Scrape tests", () => { }, scrapeTimeout); describe("waitFor validation", () => { - it.concurrent("allows waitFor when it's less than half of timeout", async () => { - const response = await scrape({ - url: "http://firecrawl.dev", - waitFor: 5000, - timeout: 15000, - }, identity); - - expect(response.markdown).toContain("Firecrawl"); - }, scrapeTimeout); - - // TODO: does not make sense. reevaluate - mogery - // it.concurrent("allows waitFor when it's exactly half of timeout", async () => { - // const response = await scrape({ - // url: "http://firecrawl.dev", - // waitFor: 7500, - // timeout: 15000, - // }, identity); - - // expect(response.markdown).toContain("Firecrawl"); - // }, scrapeTimeout); - it.concurrent("rejects waitFor when it exceeds half of timeout", async () => { const raw = await scrapeRaw({ url: "http://firecrawl.dev", diff --git a/apps/api/src/__tests__/snips/v2/scrape.test.ts b/apps/api/src/__tests__/snips/v2/scrape.test.ts index 9de6f38e6..b811b0cc3 100644 --- a/apps/api/src/__tests__/snips/v2/scrape.test.ts +++ b/apps/api/src/__tests__/snips/v2/scrape.test.ts @@ -31,27 +31,6 @@ describe("Scrape tests", () => { }, scrapeTimeout); describe("waitFor validation", () => { - it.concurrent("allows waitFor when it's less than half of timeout", async () => { - const response = await scrape({ - url: "http://firecrawl.dev", - waitFor: 5000, - timeout: 15000, - }, identity); - - expect(response.markdown).toContain("Firecrawl"); - }, scrapeTimeout); - - // TODO: does not make sense. reevaluate - mogery - // it.concurrent("allows waitFor when it's exactly half of timeout", async () => { - // const response = await scrape({ - // url: "http://firecrawl.dev", - // waitFor: 7500, - // timeout: 15000, - // }, identity); - - // expect(response.markdown).toContain("Firecrawl"); - // }, scrapeTimeout); - it.concurrent("rejects waitFor when it exceeds half of timeout", async () => { const raw = await scrapeRaw({ url: "http://firecrawl.dev",