i have fixed contains and tally file

Author: Dagim-Daniel

Sprint-2/implement/contains.js

@@ -1,5 +1,9 @@
 function contains(keyInput, valueInput) {
-  if (Array.isArray(keyInput) == true) {
+  if (
+    keyInput == null ||
+    typeof keyInput !== "object" ||
+    Array.isArray(keyInput)
+  ) {
     return false;
   } else {
     for (const key in keyInput) {
@@ -10,5 +14,5 @@ function contains(keyInput, valueInput) {
   }
   return false;
 }
-
+// make values like null, undefined ,1234, true,"Abc" to be consider invalid and return false same as array
 module.exports = contains;

Sprint-2/implement/contains.test.js

@@ -15,3 +15,14 @@ test("Give a contains with object and property and if the property is non-existe
 test("Give invalid parameter in this case array when passed to contains it returns false", () => {
   expect(contains(["a", 2, "b", 3, 2], 2)).toEqual(false);
 });
+
+test("Give invalid parameter in this case empty  when passed to contains it returns false", () => {
+  expect(contains(null)).toEqual(false);
+});
+test("returns false for undefined", () => {
+  expect(contains(undefined, "key")).toEqual(false);
+});
+
+test("returns false for numbers", () => {
+  expect(contains(1234, "key")).toEqual(false);
+});

Sprint-2/implement/tally.js

@@ -4,10 +4,9 @@ function tally(items) {
   }
   const result = {};
   for (const item of items) {
-    result[item] = (result[item] || 0) + 1;
+    const count = Object.hasOwn(result, item) ? result[item] : 0;
+    result[item] = count + 1;
   }
-
   return result;
 }
-
 module.exports = tally;