Remove testing against localhost

Author: zyc9012

.github/workflows/build.yml

@@ -24,7 +24,7 @@ jobs:
       - name: Run tests
         env:
           SERPAPI_TEST_KEY: ${{ secrets.SERPAPI_TEST_KEY }}
-        run: deno task test:ci
+        run: deno task test
 
       - name: Setup Node
         uses: actions/setup-node@v4

.github/workflows/release.yml

@@ -28,7 +28,7 @@ jobs:
       - name: Run tests
         env:
           SERPAPI_TEST_KEY: ${{ secrets.SERPAPI_TEST_KEY }}
-        run: deno task test:ci
+        run: deno task test
 
       - name: Setup Node
         uses: actions/setup-node@v4

CONTRIBUTING.md

@@ -66,8 +66,7 @@ and you're good to go.
 
 ### Tests that require an API key
 
-- When testing functionality that hits either `localhost` or
-  `https://serpapi.com`, and requires an API key, you need to ensure your API
+- When testing functionality that requires an API key, you need to ensure your API
   key is specified as an environment variable.
 - Create a `.env` file with the following contents, replacing `YOUR_API_KEY`
   with your API key for the appropriate environment.
@@ -91,10 +90,9 @@ it("getJson with api key from config", {
 ## Run tests
 
 ```bash
-deno task test         # Run tests that hit "localhost"
-deno task test:watch   # Run tests that hit "localhost" and in watch mode: https://deno.land/manual/getting_started/command_line_interface#watch-mode
-deno task test:cov     # Get test coverage by running tests that hit "localhost"
-deno task test:ci      # Run tests that hit "https://serpapi.com"
+deno task test         # Run tests
+deno task test:watch   # Run tests and in watch mode: https://deno.land/manual/getting_started/command_line_interface#watch-mode
+deno task test:cov     # Get test coverage by running tests
 ```
 
 ## Run examples on local source files

deno.json

@@ -1,10 +1,9 @@
 {
   "tasks": {
     "docs:gen": "npx documentation readme src/serpapi.ts --section=Functions --shallow && deno fmt",
-    "test": "ENV_TYPE=local deno test tests/ --allow-env --allow-read --allow-net --fail-fast",
+    "test": "deno test tests/ --allow-env --allow-read --allow-net",
     "test:watch": "deno task test --watch",
     "test:cov": "rm -rf cov_profile && deno task test --coverage=cov_profile && deno coverage cov_profile",
-    "test:ci": "deno test tests/ --allow-env --allow-read --allow-net",
     "npm": "deno run -A scripts/build_npm.ts"
   },
   "fmt": {

src/utils.ts

@@ -18,7 +18,7 @@ export const _internals = {
   getHostnameAndPort: getHostnameAndPort,
 };
 
-/** Facilitates stubbing in tests, e.g. localhost as the base url */
+/** Facilitates stubbing in tests */
 function getHostnameAndPort() {
   return {
     hostname: "serpapi.com",

tests/serpapi_test.ts

@@ -40,15 +40,10 @@ import {
 loadSync({ export: true });
 const SERPAPI_TEST_KEY = Deno.env.get("SERPAPI_TEST_KEY") ?? "";
 const HAS_API_KEY = SERPAPI_TEST_KEY.length > 0;
-const BASE_OPTIONS = Deno.env.get("ENV_TYPE") === "local"
-  ? {
-    hostname: "localhost",
-    port: 3000,
-  }
-  : {
-    hostname: "serpapi.com",
-    port: 443,
-  };
+const BASE_OPTIONS = {
+  hostname: "serpapi.com",
+  port: 443,
+};
 
 describe(
   "getAccount",

tests/utils_test.ts

@@ -23,15 +23,10 @@ import http from "node:http";
 import qs from "node:querystring";
 
 loadSync({ export: true });
-const BASE_OPTIONS = Deno.env.get("ENV_TYPE") === "local"
-  ? {
-    hostname: "localhost",
-    port: 3000,
-  }
-  : {
-    hostname: "serpapi.com",
-    port: 443,
-  };
+const BASE_OPTIONS = {
+  hostname: "serpapi.com",
+  port: 443,
+};
 
 describe("getSource", () => {
   it("use runtime version", async () => {