From ae10b5f89cc747dc81799c8710f95a1bd1038537 Mon Sep 17 00:00:00 2001 From: Blaine Bublitz Date: Mon, 10 Jun 2024 10:49:30 -0400 Subject: [PATCH] remove commented tests --- arcjet/test/index.node.test.ts | 47 ---------------------------------- 1 file changed, 47 deletions(-) diff --git a/arcjet/test/index.node.test.ts b/arcjet/test/index.node.test.ts index bd3a75fb7..6f2fe4c52 100644 --- a/arcjet/test/index.node.test.ts +++ b/arcjet/test/index.node.test.ts @@ -166,53 +166,6 @@ class ArcjetInvalidDecision extends ArcjetDecision { const log = new Logger({ level: "info" }); -// describe("defaultBaseUrl", () => { -// test("uses process.env.ARCJET_BASE_URL if set and allowed", () => { -// jest.replaceProperty(process, "env", { -// NODE_ENV: "production", -// ARCJET_BASE_URL: "https://decide.arcjet.orb.local:4082", -// }); -// expect(defaultBaseUrl()).toEqual("https://decide.arcjet.orb.local:4082"); -// }); - -// test("does not use process.env.ARCJET_BASE_URL if not allowed", () => { -// jest.replaceProperty(process, "env", { -// NODE_ENV: "production", -// ARCJET_BASE_URL: "http://localhost:1234", -// }); -// expect(defaultBaseUrl()).toEqual("https://decide.arcjet.com"); -// }); - -// test("does not use process.env.ARCJET_BASE_URL if empty string", () => { -// jest.replaceProperty(process, "env", { -// NODE_ENV: "production", -// ARCJET_BASE_URL: "", -// }); -// expect(defaultBaseUrl()).toEqual("https://decide.arcjet.com"); -// }); - -// test("uses production url if process.env.ARCJET_BASE_URL not set", () => { -// expect(defaultBaseUrl()).toEqual("https://decide.arcjet.com"); -// }); - -// // TODO(#90): Remove these tests once production conditional is removed -// test("uses process.env.ARCJET_BASE_URL if set (in development)", () => { -// jest.replaceProperty(process, "env", { -// NODE_ENV: "development", -// ARCJET_BASE_URL: "http://localhost:1234", -// }); -// expect(defaultBaseUrl()).toEqual("http://localhost:1234"); -// }); - -// test("does not use process.env.ARCJET_BASE_URL if empty string (in development)", () => { -// jest.replaceProperty(process, "env", { -// NODE_ENV: "development", -// ARCJET_BASE_URL: "", -// }); -// expect(defaultBaseUrl()).toEqual("https://decide.arcjet.com"); -// }); -// }); - describe("createRemoteClient", () => { const defaultRemoteClientOptions = { baseUrl: "",