Fix tests

Author: zyc9012

src/utils.ts

@@ -84,7 +84,13 @@ export function buildUrl(
   path: string,
   parameters: qs.ParsedUrlQueryInput,
 ): string {
-  return `${_internals.getBaseUrl()}${path}?${qs.stringify(parameters)}`;
+  const clonedParams = { ...parameters };
+  for (const k in clonedParams) {
+    if (clonedParams[k] === undefined) {
+      delete clonedParams[k];
+    }
+  }
+  return `${_internals.getBaseUrl()}${path}?${qs.stringify(clonedParams)}`;
 }
 
 export function execute(

tests/engines/google_test.ts

@@ -98,6 +98,8 @@ describe("google", {
         api_key: "test_override_api_key",
         q: "coffee",
       });
+    } catch {
+      // pass
     } finally {
       executeSpy.restore();
     }
@@ -177,6 +179,8 @@ describe("google", {
         api_key: "test_override_api_key",
         q: "coffee",
       });
+    } catch {
+      // pass
     } finally {
       executeSpy.restore();
     }

tests/serpapi_test.ts

@@ -90,6 +90,7 @@ describe("getAccount", {
       "extra_credits",
       "last_hour_searches",
       "plan_id",
+      "plan_monthly_price",
       "plan_name",
       "plan_searches_left",
       "searches_per_month",
@@ -113,6 +114,7 @@ describe("getAccount", {
       "extra_credits",
       "last_hour_searches",
       "plan_id",
+      "plan_monthly_price",
       "plan_name",
       "plan_searches_left",
       "searches_per_month",
@@ -135,6 +137,7 @@ describe("getAccount", {
       "extra_credits",
       "last_hour_searches",
       "plan_id",
+      "plan_monthly_price",
       "plan_name",
       "plan_searches_left",
       "searches_per_month",

tests/utils_test.ts

@@ -211,7 +211,7 @@ describe("buildUrl", () => {
   it("with undefined parameters", async () => {
     assertEquals(
       await buildUrl("/search", { q: "coffee", gl: undefined, hl: null }),
-      `${BASE_URL}/search?q=coffee&hl=null`,
+      `${BASE_URL}/search?q=coffee&hl=`,
     );
   });
 });