corrections after review

Author: miriamjorna

Sprint-2/implement/contains.test.js

@@ -43,5 +43,5 @@ test("given an object with properties, returns false for non-existent property",
 // When passed to contains
 // Then it should return false or throw an error
 test("given invalid parameters like an array, returns false", () => {
-  expect(contains([1, 2, 3], "a")).toEqual(false);
+  expect(contains([1, 2, 3], "0")).toEqual(false);
 });

Sprint-2/implement/querystring.js

@@ -7,8 +7,10 @@ function parseQueryString(queryString) {
 
   for (const pair of keyValuePairs) {
     const index = pair.indexOf("=");
+    if (index === -1) continue; // skip pairs with no "="
     const key = pair.slice(0, index);
     const value = pair.slice(index + 1);
+    if (key === "") continue; // skip empty keys
     queryParams[key] = value;
   }
 

Sprint-2/implement/tally.js

@@ -4,13 +4,13 @@ function tally(arr) {
   }
   const result = {};
   for (const item of arr) {
-    if (result[item]) {
-      result[item]++;
-    } else {
-      result[item] = 1;
+   if (Object.hasOwn(result, item)) {
+  result[item]++;
+} else {
+  result[item] = 1;
+}
     }
   }
   return result;
-}
 
 module.exports = tally;