Added tests and implemented solution for the implement section - max.js and max.test.js

Author: Konvaly

Sprint-1/implement/max.js

@@ -1,4 +1,16 @@
 function findMax(elements) {
+  if (elements.length === 0) {
+    return -Infinity;
+  } else if (elements.length === 1 && typeof elements[0] === "number") {
+    return elements[0];
+  }
+  const maxEl = Math.max(
+    ...elements.filter((el) => typeof el === "number" && !isNaN(el))
+  );
+
+  if (elements.length > 1) {
+    return maxEl;
+  }
 }
 
 module.exports = findMax;

Sprint-1/implement/max.test.js

@@ -12,32 +12,55 @@ We have set things up already so that this file can see your function from the o
 
 const findMax = require("./max.js");
 
-// Given an empty array
-// When passed to the max function
-// Then it should return -Infinity
-// Delete this test.todo and replace it with a test.
-test.todo("given an empty array, returns -Infinity");
-
-// Given an array with one number
-// When passed to the max function
-// Then it should return that number
-
-// Given an array with both positive and negative numbers
-// When passed to the max function
-// Then it should return the largest number overall
-
-// Given an array with just negative numbers
-// When passed to the max function
-// Then it should return the closest one to zero
-
-// Given an array with decimal numbers
-// When passed to the max function
-// Then it should return the largest decimal number
-
-// Given an array with non-number values
-// When passed to the max function
-// Then it should return the max and ignore non-numeric values
-
-// Given an array with only non-number values
-// When passed to the max function
-// Then it should return the least surprising value given how it behaves for all other inputs
+describe("findMax", () => {
+  // Given an empty array
+  // When passed to the max function
+  // Then it should return -Infinity
+  test("given an empty array, returns -Infinity", () => {
+    expect(findMax([])).toBe(-Infinity);
+  });
+
+  // Given an array with one number
+  // When passed to the max function
+  // Then it should return that number
+  test("given an array with one number, returns that number", () => {
+    expect(findMax([15])).toBe(15);
+  });
+
+  // Given an array with both positive and negative numbers
+  // When passed to the max function
+  // Then it should return the largest number overall
+  test("given an array with both positive and negative numbers, returns the largest number", () => {
+    expect(findMax([-112, 7, 3, 20, -2])).toBe(20);
+  });
+
+  // Given an array with just negative numbers
+  // When passed to the max function
+  // Then it should return the closest one to zero
+  test("given an array with just negative numbers, returns the largest number - the closest one to zero", () => {
+    expect(findMax([-653, -80, -7, -4, -12])).toBe(-4);
+  });
+
+  // Given an array with decimal numbers
+  // When passed to the max function
+  // Then it should return the largest decimal number
+  test("given an array with decimal numbers, returns the largest decimal number", () => {
+    expect(findMax([27.3, 654.1, 45.4, 325.9])).toBe(654.1);
+  });
+
+  // Given an array with non-number values
+  // When passed to the max function
+  // Then it should return the max and ignore non-numeric values
+  test("given an array with non-number values, returns the max and ignores non-numeric values", () => {
+    expect(
+      findMax([7, {}, "Greetings!", 34, null, 15.3, undefined, 30, 3878, NaN])
+    ).toBe(3878);
+  });
+
+  // Given an array with only non-number values
+  // When passed to the max function
+  // Then it should return the least surprising value given how it behaves for all other inputs
+  test("given an array with only non-number values, returns -Infinity", () => {
+    expect(findMax([{}, "Hello!", null, undefined, NaN])).toBe(-Infinity);
+  });
+});