remove query-string and use parseUrl directly

Author: jmisasa

package-lock.json

@@ -70,7 +70,6 @@
     "jest": "^26.6.3",
     "next": "^13.5.1",
     "prettier": "^2.2.1",
-    "query-string": "^9.0.0",
     "react": "^18.2.0",
     "react-dom": "^18.2.0",
     "react-test-renderer": "^18.2.0",

package.json

@@ -211,7 +211,9 @@ export class MemoryRouter extends BaseRouter {
  */
 function parseUrlToCompleteUrl(url: Url, currentPathname: string): UrlObjectComplete {
   const parsedUrl = typeof url === "object" ? url : parseUrl(url);
-  const query = parsedUrl.search ? objectifyQueryString(parsedUrl.search) : parsedUrl.query ?? {};
+
+  const queryFromSearch = parsedUrl.search ? objectifyQueryString(parsedUrl.search) : undefined;
+  const query = queryFromSearch ?? parsedUrl.query ?? {};
 
   return {
     pathname: normalizeTrailingSlash(parsedUrl.pathname ?? currentPathname),

src/MemoryRouter.tsx

@@ -1,6 +1,5 @@
 import type { NextRouter } from "next/router";
 import type { UrlObject } from "./MemoryRouter";
-import queryString from "querystring";
 
 export function parseUrl(url: string): UrlObject {
   const base = "https://base.com"; // base can be anything
@@ -18,8 +17,17 @@ export function parseUrl(url: string): UrlObject {
   };
 }
 export function stringifyQueryString(query: NextRouter["query"]): string {
-  return queryString.stringify(query);
+  const params = new URLSearchParams();
+  Object.keys(query).forEach((key) => {
+    const values = query[key];
+    for (const value of Array.isArray(values) ? values : [values]) {
+      params.append(key, value!);
+    }
+  });
+  return params.toString();
 }
-export function objectifyQueryString(query: string): NextRouter["query"] {
-  return queryString.parse(query);
+export function objectifyQueryString(query: string): NextRouter["query"] | undefined {
+  const parsedUrl = parseUrl(`?${query}`);
+
+  return parsedUrl.query;
 }