From c6e906e3ba6837426e312b5f03d82c25d480d76e Mon Sep 17 00:00:00 2001 From: km-anthropic Date: Sun, 13 Jul 2025 09:08:53 -0700 Subject: [PATCH] test: simplify network restrictions test output --- test/network-restrictions.test.ts | 11 ++--------- 1 file changed, 2 insertions(+), 9 deletions(-) diff --git a/test/network-restrictions.test.ts b/test/network-restrictions.test.ts index d0a4d99..0accbb7 100644 --- a/test/network-restrictions.test.ts +++ b/test/network-restrictions.test.ts @@ -2,7 +2,6 @@ import { describe, it, expect } from "bun:test"; describe("Network Restrictions", () => { it("should block access to unauthorized domains", async () => { - // This test verifies that the proxy blocks unauthorized domains const unauthorizedUrls = [ "https://example.com/api/data", "https://jsonplaceholder.typicode.com/posts", @@ -15,24 +14,19 @@ describe("Network Restrictions", () => { try { const response = await fetch(url, { timeout: 5000, - // Force through proxy if set agent: undefined, }); - // If we reach here, the proxy didn't block it - test should fail expect(response.ok).toBe(false); throw new Error(`Unauthorized domain ${url} was not blocked by proxy`); } catch (error) { - // We expect an error (connection refused, timeout, etc) - // This is the desired behavior - proxy blocked the request expect(error).toBeDefined(); - console.log(`✓ Successfully blocked: ${url}`); + console.log(`Successfully blocked: ${url}`); } } }); it("should allow access to whitelisted domains", async () => { - // These should work through the proxy const allowedUrls = [ "https://api.github.com/zen", "https://registry.npmjs.org/-/ping", @@ -42,9 +36,8 @@ describe("Network Restrictions", () => { try { const response = await fetch(url, { timeout: 5000 }); expect(response.ok).toBe(true); - console.log(`✓ Successfully allowed: ${url}`); + console.log(`Successfully allowed: ${url}`); } catch (error) { - // If whitelisted domains fail, something is wrong throw new Error( `Whitelisted domain ${url} was blocked: ${error.message}`, );