fixing the validation when array as a property value

Author: Jacknguyen4438

Sprint-2/implement/contains.js

@@ -1,19 +1,29 @@
 function contains(object, key) {
+  if (
+    object === null ||
+    object === undefined ||
+    typeof object !== "object" ||
+    Array.isArray(object)
+  ) {
+    return false;
+  }
+
   if (Object.keys(object).length === 0) {
     return false;
   }
 
-  for (const checkItem in object) {
-    let validation = object[checkItem];
-    if (Array.isArray(validation)) {
+  for (const item in object) {
+    if (Array.isArray(object[item])) {
       return false;
     }
   }
+
   for (const item in object) {
     if (item === key) {
       return true;
     }
   }
+
   return false;
 }
 

Sprint-2/implement/contains.test.js

@@ -54,4 +54,10 @@ test("should return false if the input object do not have non existing key value
 test("should return false if the input object do not have non existing key value", () => {
   let mixObject = { a: 2, b: "hello", c: [] };
   expect(contains(mixObject, "c")).toBe(false);
+
+
+expect(contains(["A", "B"], "1" )).toBe(false);
+expect(contains(null, "1" )).toBe(false);
+expect(contains(undefined, "1" )).toBe(false);
+expect(contains("ABC", "1" )).toBe(false);
 });