pr changes made

Author: Alex Jamshidi

Sprint-2/implement/contains.test.js

@@ -49,9 +49,8 @@ describe("contains", () => {
   // Then it should return false or throw an error
   it("given invalid parameter (an array) returns false or throws an error", () => {
     expect(contains([], [])).toEqual(false);
-    expect(contains(["a", 1], "a")).toEqual(false);
+    expect(contains(["a", 1], 1)).toEqual(false);
     expect(contains({ a: 1, b: 2 }, ["a"])).toEqual(false);
-    expect(contains(["a", 1], ["a"])).toEqual(false);
   });
 
   it("given null returns false", () => {

Sprint-2/implement/querystring.js

@@ -4,12 +4,8 @@ function parseQueryString(queryString) {
     return queryParams;
   }
 
-  // Adds percentage-encoded characters
-  queryString = decodeURIComponent(queryString);
-
   // Replaces + with space
   queryString = queryString.replaceAll("+", " ");
-
   const keyValuePairs = queryString.split("&");
 
   for (const pair of keyValuePairs) {
@@ -18,7 +14,13 @@ function parseQueryString(queryString) {
       if (!pair.includes("=")) {
         index = pair.length;
       }
-      const [key, value] = [pair.slice(0, index), pair.slice(index + 1)];
+      const rawKey = pair.slice(0, index);
+      const rawValue = pair.slice(index + 1);
+
+      // Remove percentage-encoded characters
+      const key = decodeURIComponent(rawKey);
+      const value = decodeURIComponent(rawValue);
+
       queryParams[key] = value;
     }
   }