From 7d8f40217b35f51aaa9e271c5627a222a2a27cf0 Mon Sep 17 00:00:00 2001
From: ChinweP <exponentialresults1973@gmail.com>
Date: Mon, 27 Jul 2026 16:02:59 +0100
Subject: [PATCH 01/17] Fix calculateMedian implementation so it passes all
 tests

---
 Sprint-1/fix/median.js | 28 +++++++++++++++++++++++++---
 1 file changed, 25 insertions(+), 3 deletions(-)

diff --git a/Sprint-1/fix/median.js b/Sprint-1/fix/median.js
index b22590bc6..db759d5ba 100644
--- a/Sprint-1/fix/median.js
+++ b/Sprint-1/fix/median.js
@@ -6,9 +6,31 @@
 // or 'list' has mixed values (the function is expected to sort only numbers).
 
 function calculateMedian(list) {
-  const middleIndex = Math.floor(list.length / 2);
-  const median = list.splice(middleIndex, 1)[0];
-  return median;
+  // Validate input: must be an array
+  if (!Array.isArray(list)) {
+    return null;
+  }
+
+  // Filter out non-numeric values
+  const numbers = list.filter((item) => typeof item === "number");
+
+  // If no numeric values, return null
+  if (numbers.length === 0) {
+    return null;
+  }
+
+  // Sort numbers without mutating original list
+  const sorted = [...numbers].sort((a, b) => a - b);
+
+  const middleIndex = Math.floor(sorted.length / 2);
+
+  // Odd length → return middle number
+  if (sorted.length % 2 !== 0) {
+    return sorted[middleIndex];
+  }
+
+  // Even length → average of two middle numbers
+  return (sorted[middleIndex - 1] + sorted[middleIndex]) / 2;
 }
 
 module.exports = calculateMedian;

From 46499633bcc3b0d67808e9fd9efb7b8910ec8787 Mon Sep 17 00:00:00 2001
From: ChinweP <exponentialresults1973@gmail.com>
Date: Mon, 27 Jul 2026 16:33:57 +0100
Subject: [PATCH 02/17] Add dedupe implementation that passes all current tests

---
 Sprint-1/implement/dedupe.js | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/Sprint-1/implement/dedupe.js b/Sprint-1/implement/dedupe.js
index 781e8718a..209465014 100644
--- a/Sprint-1/implement/dedupe.js
+++ b/Sprint-1/implement/dedupe.js
@@ -1 +1,5 @@
-function dedupe() {}
+function dedupe(arr) {
+  return [...new Set(arr)];
+}
+
+module.exports = dedupe;
\ No newline at end of file

From bb247fd85a200cb06047f8fe029766a8c81c3e7d Mon Sep 17 00:00:00 2001
From: ChinweP <exponentialresults1973@gmail.com>
Date: Mon, 27 Jul 2026 17:05:14 +0100
Subject: [PATCH 03/17] Implement findMax and add full test coverage for all
 input scenarios

---
 Sprint-1/implement/max.js      |  5 +++++
 Sprint-1/implement/max.test.js | 23 ++++++++++++++++++++++-
 2 files changed, 27 insertions(+), 1 deletion(-)

diff --git a/Sprint-1/implement/max.js b/Sprint-1/implement/max.js
index 6dd76378e..ceacf7c5a 100644
--- a/Sprint-1/implement/max.js
+++ b/Sprint-1/implement/max.js
@@ -1,4 +1,9 @@
 function findMax(elements) {
+  const numbers = elements.filter((item) => typeof item === "number");
+  if (numbers.length === 0) {
+    return -Infinity;
+  }
+  return Math.max(...numbers);
 }
 
 module.exports = findMax;
diff --git a/Sprint-1/implement/max.test.js b/Sprint-1/implement/max.test.js
index 82f18fd88..ebfabf0fa 100644
--- a/Sprint-1/implement/max.test.js
+++ b/Sprint-1/implement/max.test.js
@@ -16,28 +16,49 @@ const findMax = require("./max.js");
 // 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");
+//test.todo("given an empty array, returns -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([42])).toBe(42);
+});
 
 // 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 positive and negative numbers, returns the largest", () => {
+  expect(findMax([-10, 5, 3, -2])).toBe(5);
+});
 
 // 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 only negative numbers, returns the closest to zero", () => {
+  expect(findMax([-10, -3, -20])).toBe(-3);
+});
 
 // 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", () => {
+  expect(findMax([1.2, 3.5, 2.8])).toBe(3.5);
+});
 
 // 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, ignores them and returns the max", () => {
+  expect(findMax(['hey', 10, 'hi', 60, 10])).toBe(60);
+});
 
 // 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(['a', 'b', null, undefined])).toBe(-Infinity);
+});
\ No newline at end of file

From 816b1041eb978c9b1316554f62dbe666734a286b Mon Sep 17 00:00:00 2001
From: ChinweP <exponentialresults1973@gmail.com>
Date: Mon, 27 Jul 2026 18:51:37 +0100
Subject: [PATCH 04/17] Implement sum function and add full test coverage

---
 Sprint-1/implement/sum.js      |  5 +++++
 Sprint-1/implement/sum.test.js | 20 +++++++++++++++++++-
 2 files changed, 24 insertions(+), 1 deletion(-)

diff --git a/Sprint-1/implement/sum.js b/Sprint-1/implement/sum.js
index 9062aafe3..7d4ec4186 100644
--- a/Sprint-1/implement/sum.js
+++ b/Sprint-1/implement/sum.js
@@ -1,4 +1,9 @@
 function sum(elements) {
+  const numbers = elements.filter((item) => typeof item === "number");
+  if (numbers.length === 0) {
+    return 0;
+  }
+  return numbers.reduce((total, num) => total + num, 0);
 }
 
 module.exports = sum;
diff --git a/Sprint-1/implement/sum.test.js b/Sprint-1/implement/sum.test.js
index dd0a090ca..32bab4ade 100644
--- a/Sprint-1/implement/sum.test.js
+++ b/Sprint-1/implement/sum.test.js
@@ -13,24 +13,42 @@ const sum = require("./sum.js");
 // Given an empty array
 // When passed to the sum function
 // Then it should return 0
-test.todo("given an empty array, returns 0")
+// test.todo("given an empty array, returns 0")
+test("given an empty array, returns 0", () => {
+  expect(sum([])).toBe(0);
+});
 
 // Given an array with just one number
 // When passed to the sum function
 // Then it should return that number
+test("given an array with one number, returns that number", () => {
+  expect(sum([42])).toBe(42);
+});
 
 // Given an array containing negative numbers
 // When passed to the sum function
 // Then it should still return the correct total sum
+test("given an array with negative numbers, returns the correct total", () => {
+  expect(sum([-5, 10, -3])).toBe(2);
+});
 
 // Given an array with decimal/float numbers
 // When passed to the sum function
 // Then it should return the correct total sum
+test("given an array with decimal numbers, returns the correct total", () => {
+  expect(sum([1.5, 2.5, 3.1])).toBe(7.1);
+});
 
 // Given an array containing non-number values
 // When passed to the sum function
 // Then it should ignore the non-numerical values and return the sum of the numerical elements
+test("given an array with non-number values, ignores them and returns the sum of numbers", () => {
+  expect(sum(["hey", 10, "hi", 60, 10])).toBe(80);
+});
 
 // Given an array with only non-number values
 // When passed to the sum 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 0", () => {
+  expect(sum(["a", null, undefined, "b"])).toBe(0);
+});

From 64a2b806f781c961d1a8bf7e8dacff4390d5884d Mon Sep 17 00:00:00 2001
From: ChinweP <exponentialresults1973@gmail.com>
Date: Mon, 27 Jul 2026 19:18:19 +0100
Subject: [PATCH 05/17] Refactor includes to use a for-of loop

---
 Sprint-1/refactor/includes.js | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/Sprint-1/refactor/includes.js b/Sprint-1/refactor/includes.js
index 29dad81f0..8c9ae2e66 100644
--- a/Sprint-1/refactor/includes.js
+++ b/Sprint-1/refactor/includes.js
@@ -1,8 +1,7 @@
 // Refactor the implementation of includes to use a for...of loop
 
 function includes(list, target) {
-  for (let index = 0; index < list.length; index++) {
-    const element = list[index];
+  for (const element of list) {
     if (element === target) {
       return true;
     }

From b057112fe57c5e516f554cacdd3e0906502ff17b Mon Sep 17 00:00:00 2001
From: ChinweP <exponentialresults1973@gmail.com>
Date: Mon, 27 Jul 2026 19:50:58 +0100
Subject: [PATCH 06/17] Fix address lookup by using correct object property

---
 Sprint-2/debug/address.js | 15 ++++++++++++++-
 1 file changed, 14 insertions(+), 1 deletion(-)

diff --git a/Sprint-2/debug/address.js b/Sprint-2/debug/address.js
index 940a6af83..a3f63cd92 100644
--- a/Sprint-2/debug/address.js
+++ b/Sprint-2/debug/address.js
@@ -4,6 +4,19 @@
 // but it isn't working...
 // Fix anything that isn't working
 
+/*const address = {
+  houseNumber: 42,
+  street: "Imaginary Road",
+  city: "Manchester",
+  country: "England",
+  postcode: "XYZ 123",
+};
+
+console.log(`My house number is ${address[0]}`); */
+
+// The line "address[0]" is wrong. Because address is an object not an array, so "address[0]" is undefined. The output will be "My house number is undefined".
+
+// Corrected code
 const address = {
   houseNumber: 42,
   street: "Imaginary Road",
@@ -12,4 +25,4 @@ const address = {
   postcode: "XYZ 123",
 };
 
-console.log(`My house number is ${address[0]}`);
+console.log(`My house number is ${address.houseNumber}`);
\ No newline at end of file

From c76ea742ae20bf6ead59dc5c4ba2cedd2451b68c Mon Sep 17 00:00:00 2001
From: ChinweP <exponentialresults1973@gmail.com>
Date: Mon, 27 Jul 2026 20:32:00 +0100
Subject: [PATCH 07/17] Fix object iteration by using Object.values() instead
 of for-of

---
 Sprint-2/debug/author.js | 20 +++++++++++++++++++-
 1 file changed, 19 insertions(+), 1 deletion(-)

diff --git a/Sprint-2/debug/author.js b/Sprint-2/debug/author.js
index 8c2125977..37a05131d 100644
--- a/Sprint-2/debug/author.js
+++ b/Sprint-2/debug/author.js
@@ -3,7 +3,7 @@
 // This program attempts to log out all the property values in the object.
 // But it isn't working. Explain why first and then fix the problem
 
-const author = {
+/* const author = {
   firstName: "Zadie",
   lastName: "Smith",
   occupation: "writer",
@@ -14,3 +14,21 @@ const author = {
 for (const value of author) {
   console.log(value);
 }
+*/
+
+// Prediction and Explanation
+/* The line "for (const value of author) {console.log(value);}" will not work because for-of is used only for iterable things, e.g: arrays, strings, maps, sets.
+But author is a plain object (created with {}), and plain objects are not iterable. So Javascript throws: TypeError: author is not iterable because objects don't have a natural order to loop through. */
+
+// Corrected code
+const author = {
+  firstName: "Zadie",
+  lastName: "Smith",
+  occupation: "writer",
+  age: 40,
+  alive: true,
+};
+
+for (const value of Object.values(author)) {
+  console.log(value);
+} 

From 17433111761614b4254423460fb59b88c4c3ed58 Mon Sep 17 00:00:00 2001
From: ChinweP <exponentialresults1973@gmail.com>
Date: Mon, 27 Jul 2026 23:49:25 +0100
Subject: [PATCH 08/17] Add correct ingredient logging by looping through
 recipe.ingredients

---
 Sprint-2/debug/recipe.js | 25 ++++++++++++++++++++++++-
 1 file changed, 24 insertions(+), 1 deletion(-)

diff --git a/Sprint-2/debug/recipe.js b/Sprint-2/debug/recipe.js
index 6cbdd22cd..ab5c9d0cd 100644
--- a/Sprint-2/debug/recipe.js
+++ b/Sprint-2/debug/recipe.js
@@ -4,7 +4,7 @@
 // Each ingredient should be logged on a new line
 // How can you fix it?
 
-const recipe = {
+/*const recipe = {
   title: "bruschetta",
   serves: 2,
   ingredients: ["olive oil", "tomatoes", "salt", "pepper"],
@@ -13,3 +13,26 @@ const recipe = {
 console.log(`${recipe.title} serves ${recipe.serves}
   ingredients:
 ${recipe}`);
+*/
+
+// Prediction and Explanation
+/* 
+The line inside the template string: ${recipe} does not print the ingredients. 
+Instead, JavaScript converts the entire recipe object into a string, which becomes "[object Object]". This happens 
+because plain objects {} are not automatically converted into readable text. The program should print each ingredient on a new line, but the current code 
+never loops through recipe.ingredients, so nothing is printed correctly.
+*/
+
+// Corrected code
+const recipe = {
+  title: "bruschetta",
+  serves: 2,
+  ingredients: ["olive oil", "tomatoes", "salt", "pepper"],
+};
+
+console.log(`${recipe.title} serves ${recipe.serves}
+ingredients:`);
+
+for (const ingredient of recipe.ingredients) {
+  console.log(ingredient);
+}

From 5ee54090fb639ed5e126c60fd1729872d9ba5c43 Mon Sep 17 00:00:00 2001
From: ChinweP <exponentialresults1973@gmail.com>
Date: Tue, 28 Jul 2026 01:30:22 +0100
Subject: [PATCH 09/17] Add jest tests covering all contains() acceptance
 criteria

---
 Sprint-2/implement/contains.js      |  9 ++++++++-
 Sprint-2/implement/contains.test.js | 29 ++++++++++++++++++++++++++++-
 2 files changed, 36 insertions(+), 2 deletions(-)

diff --git a/Sprint-2/implement/contains.js b/Sprint-2/implement/contains.js
index cd779308a..05f18e140 100644
--- a/Sprint-2/implement/contains.js
+++ b/Sprint-2/implement/contains.js
@@ -1,3 +1,10 @@
-function contains() {}
+function contains(obj, prop) {
+  if (typeof obj !== "object" || obj === null || Array.isArray(obj)) {
+    return false;
+  }
+
+  return obj.hasOwnProperty(prop);
+}
 
 module.exports = contains;
+
diff --git a/Sprint-2/implement/contains.test.js b/Sprint-2/implement/contains.test.js
index 326bdb1f2..9a232aa56 100644
--- a/Sprint-2/implement/contains.test.js
+++ b/Sprint-2/implement/contains.test.js
@@ -16,20 +16,47 @@ as the object doesn't contains a key of 'c'
 // Given a contains function
 // When passed an object and a property name
 // Then it should return true if the object contains the property, false otherwise
+test("returns true when object contains the property", () => {
+  const obj = { a: 1, b: 2 };
+  expect(contains(obj, "a")).toBe(true);
+});
+
+test("returns false when object does not contain the property", () => {
+  const obj = { a: 1, b: 2 };
+  expect(contains(obj, "c")).toBe(false);
+});
+
 
 // Given an empty object
 // When passed to contains
 // Then it should return false
-test.todo("contains on empty object returns false");
+// test.todo("contains on empty object returns false");
+test("contains on empty object returns false", () => {
+  expect(contains({}, "a")).toBe(false);
+});
 
 // Given an object with properties
 // When passed to contains with an existing property name
 // Then it should return true
+test("returns true when object contains the property", () => {
+  const obj = { a: 1, b: 2 };
+  expect(contains(obj, "a")).toBe(true);
+});
 
 // Given an object with properties
 // When passed to contains with a non-existent property name
 // Then it should return false
+test("returns false when object does not contain the property", () => {
+  const obj = { a: 1, b: 2 };
+  expect(contains(obj, "c")).toBe(false);
+});
 
 // Given invalid parameters like an array
 // When passed to contains
 // Then it should return false or throw an error
+test("returns false for invalid parameters like an array", () => {
+  expect(contains([], "a")).toBe(false);
+  expect(contains(null, "a")).toBe(false);
+  expect(contains(123, "a")).toBe(false);
+  expect(contains("hello", "a")).toBe(false);
+});

From 03c6296dec0dae24098fd15bf046e023b835d1c0 Mon Sep 17 00:00:00 2001
From: ChinweP <exponentialresults1973@gmail.com>
Date: Wed, 29 Jul 2026 19:45:14 +0100
Subject: [PATCH 10/17] Fix invert function and add tests for key-value
 swapping

---
 Sprint-2/interpret/invert.js      | 26 +++++++++++++++++++++-----
 Sprint-2/interpret/invert.test.js | 17 +++++++++++++++++
 2 files changed, 38 insertions(+), 5 deletions(-)
 create mode 100644 Sprint-2/interpret/invert.test.js

diff --git a/Sprint-2/interpret/invert.js b/Sprint-2/interpret/invert.js
index bb353fb1f..48466c281 100644
--- a/Sprint-2/interpret/invert.js
+++ b/Sprint-2/interpret/invert.js
@@ -5,7 +5,7 @@
 // Then it should swap the keys and values in the object
 
 // E.g. invert({x : 10, y : 20}), target output: {"10": "x", "20": "y"}
-
+/*
 function invert(obj) {
   const invertedObj = {};
 
@@ -15,15 +15,31 @@ function invert(obj) {
 
   return invertedObj;
 }
+  */
 
-// a) What is the current return value when invert is called with { a : 1 }
+// a) What is the current return value when invert is called with { a : 1 } - The current return value is { key: 1}
 
-// b) What is the current return value when invert is called with { a: 1, b: 2 }
+// b) What is the current return value when invert is called with { a: 1, b: 2 } - The current return value is { key: 2}
 
-// c) What is the target return value when invert is called with {a : 1, b: 2}
+// c) What is the target return value when invert is called with {a : 1, b: 2} - The target return value is { "1": "a", "2": "b" }
 
 // c) What does Object.entries return? Why is it needed in this program?
+/* Object.entries(obj) returns an array of [key, value] pairs. Example: Object.entries({ a:1, b:2 }) returns [["a", 1], ["b", 2]].
+It is needed because the loop "for (const [key, value] of Object.entries(obj))" let's each pair destructure easily.
+*/
 
-// d) Explain why the current return value is different from the target output
+// d) Explain why the current return value is different from the target output - It is because this line "invertedObj.key = value;" uses literal string "key instead of the variable key" which is a wrong property name,
+// instead of this "invertedObj[value] = key;"
 
 // e) Fix the implementation of invert (and write tests to prove it's fixed!)
+function invert(obj) {
+  const invertedObj = {};
+
+  for (const [key, value] of Object.entries(obj)) {
+    invertedObj[value] = key;
+  }
+
+  return invertedObj;
+}
+
+module.exports = invert;
diff --git a/Sprint-2/interpret/invert.test.js b/Sprint-2/interpret/invert.test.js
new file mode 100644
index 000000000..f8643a570
--- /dev/null
+++ b/Sprint-2/interpret/invert.test.js
@@ -0,0 +1,17 @@
+const invert = require("./invert.js");
+
+test("inverts a single key-value pair", () => {
+  expect(invert({ a: 1 })).toEqual({ 1: "a" });
+});
+
+test("inverts multiple key-value pairs", () => {
+  expect(invert({ a: 1, b: 2 })).toEqual({ 1: "a", 2: "b" });
+});
+
+test("handles empty objects", () => {
+  expect(invert({})).toEqual({});
+});
+
+test("handles string values", () => {
+  expect(invert({ x: "hello" })).toEqual({ hello: "x" });
+});

From 3c5b7b97627e473b9262d1a6e45bcd9a9b2fb241 Mon Sep 17 00:00:00 2001
From: ChinweP <exponentialresults1973@gmail.com>
Date: Wed, 29 Jul 2026 23:13:30 +0100
Subject: [PATCH 11/17] Complete alarm clock countdown and flashing background

---
 Sprint-3/alarmclock/alarmclock.js | 36 ++++++++++++++++++++++++++++++-
 Sprint-3/alarmclock/index.html    |  2 +-
 Sprint-3/alarmclock/style.css     | 10 +++++++++
 3 files changed, 46 insertions(+), 2 deletions(-)

diff --git a/Sprint-3/alarmclock/alarmclock.js b/Sprint-3/alarmclock/alarmclock.js
index 6ca81cd3b..ebb09aa0c 100644
--- a/Sprint-3/alarmclock/alarmclock.js
+++ b/Sprint-3/alarmclock/alarmclock.js
@@ -1,4 +1,38 @@
-function setAlarm() {}
+function setAlarm() {
+  const input = document.getElementById("alarmSet");
+  const heading = document.getElementById("timeRemaining");
+
+  let remainingSeconds = Number(input.value);
+
+  function formatTime(totalSeconds) {
+    const minutes = Math.floor(totalSeconds / 60);
+    const seconds = totalSeconds % 60;
+
+    const mm = String(minutes).padStart(2, "0");
+    const ss = String(seconds).padStart(2, "0");
+
+    return `${mm}:${ss}`;
+  }
+
+  heading.innerText = `Time Remaining: ${formatTime(remainingSeconds)}`;
+
+  const intervalId = setInterval(() => {
+    remainingSeconds -= 1;
+
+    if (remainingSeconds > 0) {
+      heading.innerText = `Time Remaining: ${formatTime(remainingSeconds)}`;
+    } else {
+      heading.innerText = "Time Remaining: 00:00";
+      clearInterval(intervalId);
+      playAlarm();
+      document.body.classList.add("flash");
+    }
+  }, 1000);
+}
+
+function stopFlashingBackground() {
+  document.body.classList.remove("flash");
+}
 
 // DO NOT EDIT BELOW HERE
 
diff --git a/Sprint-3/alarmclock/index.html b/Sprint-3/alarmclock/index.html
index 48e2e80d9..ff2d3b453 100644
--- a/Sprint-3/alarmclock/index.html
+++ b/Sprint-3/alarmclock/index.html
@@ -4,7 +4,7 @@
     <meta charset="utf-8" />
     <meta name="viewport" content="width=device-width, initial-scale=1.0" />
     <link rel="stylesheet" href="style.css" />
-    <title>Title here</title>
+    <title>Alarm clock app</title>
   </head>
   <body>
     <div class="centre">
diff --git a/Sprint-3/alarmclock/style.css b/Sprint-3/alarmclock/style.css
index 0c72de38b..f2ccb38e6 100644
--- a/Sprint-3/alarmclock/style.css
+++ b/Sprint-3/alarmclock/style.css
@@ -13,3 +13,13 @@
 h1 {
   text-align: center;
 }
+
+.flash {
+  animation: flash-bg 0.5s infinite alternate;
+}
+
+@keyframes flash-bg {
+  from { background-color: white; }
+  to { background-color: red; }
+}
+

From 9cf496bf6d50f7edbce93059e3795e8ee6d9d62b Mon Sep 17 00:00:00 2001
From: ChinweP <exponentialresults1973@gmail.com>
Date: Thu, 30 Jul 2026 01:26:12 +0100
Subject: [PATCH 12/17] Use jest 28 for Windows compatibility

---
 Sprint-3/package.json | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/Sprint-3/package.json b/Sprint-3/package.json
index 711a5390f..d7728b8bb 100644
--- a/Sprint-3/package.json
+++ b/Sprint-3/package.json
@@ -26,7 +26,7 @@
     "@testing-library/dom": "^10.4.0",
     "@testing-library/jest-dom": "^6.6.3",
     "@testing-library/user-event": "^14.6.1",
-    "jest": "^30.0.4",
+    "jest": "^28.1.3",
     "jest-environment-jsdom": "^30.0.4"
   }
 }

From 8e5fb57e1756c3764d07e3b4e0a85dd141932fe8 Mon Sep 17 00:00:00 2001
From: ChinweP <exponentialresults1973@gmail.com>
Date: Sat, 1 Aug 2026 14:18:51 +0100
Subject: [PATCH 13/17] Fix revert files from Sprint 1

---
 Sprint-1/fix/median.js         | 28 +++-------------------------
 Sprint-1/implement/dedupe.js   |  6 +-----
 Sprint-1/implement/max.js      |  5 -----
 Sprint-1/implement/max.test.js | 23 +----------------------
 Sprint-1/implement/sum.js      |  5 -----
 Sprint-1/implement/sum.test.js | 20 +-------------------
 Sprint-1/refactor/includes.js  |  3 ++-
 7 files changed, 8 insertions(+), 82 deletions(-)

diff --git a/Sprint-1/fix/median.js b/Sprint-1/fix/median.js
index db759d5ba..b22590bc6 100644
--- a/Sprint-1/fix/median.js
+++ b/Sprint-1/fix/median.js
@@ -6,31 +6,9 @@
 // or 'list' has mixed values (the function is expected to sort only numbers).
 
 function calculateMedian(list) {
-  // Validate input: must be an array
-  if (!Array.isArray(list)) {
-    return null;
-  }
-
-  // Filter out non-numeric values
-  const numbers = list.filter((item) => typeof item === "number");
-
-  // If no numeric values, return null
-  if (numbers.length === 0) {
-    return null;
-  }
-
-  // Sort numbers without mutating original list
-  const sorted = [...numbers].sort((a, b) => a - b);
-
-  const middleIndex = Math.floor(sorted.length / 2);
-
-  // Odd length → return middle number
-  if (sorted.length % 2 !== 0) {
-    return sorted[middleIndex];
-  }
-
-  // Even length → average of two middle numbers
-  return (sorted[middleIndex - 1] + sorted[middleIndex]) / 2;
+  const middleIndex = Math.floor(list.length / 2);
+  const median = list.splice(middleIndex, 1)[0];
+  return median;
 }
 
 module.exports = calculateMedian;
diff --git a/Sprint-1/implement/dedupe.js b/Sprint-1/implement/dedupe.js
index 209465014..781e8718a 100644
--- a/Sprint-1/implement/dedupe.js
+++ b/Sprint-1/implement/dedupe.js
@@ -1,5 +1 @@
-function dedupe(arr) {
-  return [...new Set(arr)];
-}
-
-module.exports = dedupe;
\ No newline at end of file
+function dedupe() {}
diff --git a/Sprint-1/implement/max.js b/Sprint-1/implement/max.js
index ceacf7c5a..6dd76378e 100644
--- a/Sprint-1/implement/max.js
+++ b/Sprint-1/implement/max.js
@@ -1,9 +1,4 @@
 function findMax(elements) {
-  const numbers = elements.filter((item) => typeof item === "number");
-  if (numbers.length === 0) {
-    return -Infinity;
-  }
-  return Math.max(...numbers);
 }
 
 module.exports = findMax;
diff --git a/Sprint-1/implement/max.test.js b/Sprint-1/implement/max.test.js
index ebfabf0fa..82f18fd88 100644
--- a/Sprint-1/implement/max.test.js
+++ b/Sprint-1/implement/max.test.js
@@ -16,49 +16,28 @@ const findMax = require("./max.js");
 // 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");
-test("given an empty array, returns -Infinity", () => {
-  expect(findMax([])).toBe(-Infinity);
-});
+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
-test("given an array with one number, returns that number", () => {
-  expect(findMax([42])).toBe(42);
-});
 
 // 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 positive and negative numbers, returns the largest", () => {
-  expect(findMax([-10, 5, 3, -2])).toBe(5);
-});
 
 // 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 only negative numbers, returns the closest to zero", () => {
-  expect(findMax([-10, -3, -20])).toBe(-3);
-});
 
 // 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", () => {
-  expect(findMax([1.2, 3.5, 2.8])).toBe(3.5);
-});
 
 // 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, ignores them and returns the max", () => {
-  expect(findMax(['hey', 10, 'hi', 60, 10])).toBe(60);
-});
 
 // 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(['a', 'b', null, undefined])).toBe(-Infinity);
-});
\ No newline at end of file
diff --git a/Sprint-1/implement/sum.js b/Sprint-1/implement/sum.js
index 7d4ec4186..9062aafe3 100644
--- a/Sprint-1/implement/sum.js
+++ b/Sprint-1/implement/sum.js
@@ -1,9 +1,4 @@
 function sum(elements) {
-  const numbers = elements.filter((item) => typeof item === "number");
-  if (numbers.length === 0) {
-    return 0;
-  }
-  return numbers.reduce((total, num) => total + num, 0);
 }
 
 module.exports = sum;
diff --git a/Sprint-1/implement/sum.test.js b/Sprint-1/implement/sum.test.js
index 32bab4ade..dd0a090ca 100644
--- a/Sprint-1/implement/sum.test.js
+++ b/Sprint-1/implement/sum.test.js
@@ -13,42 +13,24 @@ const sum = require("./sum.js");
 // Given an empty array
 // When passed to the sum function
 // Then it should return 0
-// test.todo("given an empty array, returns 0")
-test("given an empty array, returns 0", () => {
-  expect(sum([])).toBe(0);
-});
+test.todo("given an empty array, returns 0")
 
 // Given an array with just one number
 // When passed to the sum function
 // Then it should return that number
-test("given an array with one number, returns that number", () => {
-  expect(sum([42])).toBe(42);
-});
 
 // Given an array containing negative numbers
 // When passed to the sum function
 // Then it should still return the correct total sum
-test("given an array with negative numbers, returns the correct total", () => {
-  expect(sum([-5, 10, -3])).toBe(2);
-});
 
 // Given an array with decimal/float numbers
 // When passed to the sum function
 // Then it should return the correct total sum
-test("given an array with decimal numbers, returns the correct total", () => {
-  expect(sum([1.5, 2.5, 3.1])).toBe(7.1);
-});
 
 // Given an array containing non-number values
 // When passed to the sum function
 // Then it should ignore the non-numerical values and return the sum of the numerical elements
-test("given an array with non-number values, ignores them and returns the sum of numbers", () => {
-  expect(sum(["hey", 10, "hi", 60, 10])).toBe(80);
-});
 
 // Given an array with only non-number values
 // When passed to the sum 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 0", () => {
-  expect(sum(["a", null, undefined, "b"])).toBe(0);
-});
diff --git a/Sprint-1/refactor/includes.js b/Sprint-1/refactor/includes.js
index 8c9ae2e66..29dad81f0 100644
--- a/Sprint-1/refactor/includes.js
+++ b/Sprint-1/refactor/includes.js
@@ -1,7 +1,8 @@
 // Refactor the implementation of includes to use a for...of loop
 
 function includes(list, target) {
-  for (const element of list) {
+  for (let index = 0; index < list.length; index++) {
+    const element = list[index];
     if (element === target) {
       return true;
     }

From 99e811b47f1d33534e1e22255cb6a7251132d5fe Mon Sep 17 00:00:00 2001
From: ChinweP <exponentialresults1973@gmail.com>
Date: Sat, 1 Aug 2026 14:24:16 +0100
Subject: [PATCH 14/17] Fix revert files for Sprint 2

---
 Sprint-2/debug/address.js           | 15 +--------------
 Sprint-2/debug/author.js            | 20 +-------------------
 Sprint-2/debug/recipe.js            | 25 +------------------------
 Sprint-2/implement/contains.js      |  9 +--------
 Sprint-2/implement/contains.test.js | 29 +----------------------------
 Sprint-2/interpret/invert.js        | 26 +++++---------------------
 6 files changed, 10 insertions(+), 114 deletions(-)

diff --git a/Sprint-2/debug/address.js b/Sprint-2/debug/address.js
index a3f63cd92..940a6af83 100644
--- a/Sprint-2/debug/address.js
+++ b/Sprint-2/debug/address.js
@@ -4,19 +4,6 @@
 // but it isn't working...
 // Fix anything that isn't working
 
-/*const address = {
-  houseNumber: 42,
-  street: "Imaginary Road",
-  city: "Manchester",
-  country: "England",
-  postcode: "XYZ 123",
-};
-
-console.log(`My house number is ${address[0]}`); */
-
-// The line "address[0]" is wrong. Because address is an object not an array, so "address[0]" is undefined. The output will be "My house number is undefined".
-
-// Corrected code
 const address = {
   houseNumber: 42,
   street: "Imaginary Road",
@@ -25,4 +12,4 @@ const address = {
   postcode: "XYZ 123",
 };
 
-console.log(`My house number is ${address.houseNumber}`);
\ No newline at end of file
+console.log(`My house number is ${address[0]}`);
diff --git a/Sprint-2/debug/author.js b/Sprint-2/debug/author.js
index 37a05131d..8c2125977 100644
--- a/Sprint-2/debug/author.js
+++ b/Sprint-2/debug/author.js
@@ -3,7 +3,7 @@
 // This program attempts to log out all the property values in the object.
 // But it isn't working. Explain why first and then fix the problem
 
-/* const author = {
+const author = {
   firstName: "Zadie",
   lastName: "Smith",
   occupation: "writer",
@@ -14,21 +14,3 @@
 for (const value of author) {
   console.log(value);
 }
-*/
-
-// Prediction and Explanation
-/* The line "for (const value of author) {console.log(value);}" will not work because for-of is used only for iterable things, e.g: arrays, strings, maps, sets.
-But author is a plain object (created with {}), and plain objects are not iterable. So Javascript throws: TypeError: author is not iterable because objects don't have a natural order to loop through. */
-
-// Corrected code
-const author = {
-  firstName: "Zadie",
-  lastName: "Smith",
-  occupation: "writer",
-  age: 40,
-  alive: true,
-};
-
-for (const value of Object.values(author)) {
-  console.log(value);
-} 
diff --git a/Sprint-2/debug/recipe.js b/Sprint-2/debug/recipe.js
index ab5c9d0cd..6cbdd22cd 100644
--- a/Sprint-2/debug/recipe.js
+++ b/Sprint-2/debug/recipe.js
@@ -4,7 +4,7 @@
 // Each ingredient should be logged on a new line
 // How can you fix it?
 
-/*const recipe = {
+const recipe = {
   title: "bruschetta",
   serves: 2,
   ingredients: ["olive oil", "tomatoes", "salt", "pepper"],
@@ -13,26 +13,3 @@
 console.log(`${recipe.title} serves ${recipe.serves}
   ingredients:
 ${recipe}`);
-*/
-
-// Prediction and Explanation
-/* 
-The line inside the template string: ${recipe} does not print the ingredients. 
-Instead, JavaScript converts the entire recipe object into a string, which becomes "[object Object]". This happens 
-because plain objects {} are not automatically converted into readable text. The program should print each ingredient on a new line, but the current code 
-never loops through recipe.ingredients, so nothing is printed correctly.
-*/
-
-// Corrected code
-const recipe = {
-  title: "bruschetta",
-  serves: 2,
-  ingredients: ["olive oil", "tomatoes", "salt", "pepper"],
-};
-
-console.log(`${recipe.title} serves ${recipe.serves}
-ingredients:`);
-
-for (const ingredient of recipe.ingredients) {
-  console.log(ingredient);
-}
diff --git a/Sprint-2/implement/contains.js b/Sprint-2/implement/contains.js
index 05f18e140..cd779308a 100644
--- a/Sprint-2/implement/contains.js
+++ b/Sprint-2/implement/contains.js
@@ -1,10 +1,3 @@
-function contains(obj, prop) {
-  if (typeof obj !== "object" || obj === null || Array.isArray(obj)) {
-    return false;
-  }
-
-  return obj.hasOwnProperty(prop);
-}
+function contains() {}
 
 module.exports = contains;
-
diff --git a/Sprint-2/implement/contains.test.js b/Sprint-2/implement/contains.test.js
index 9a232aa56..326bdb1f2 100644
--- a/Sprint-2/implement/contains.test.js
+++ b/Sprint-2/implement/contains.test.js
@@ -16,47 +16,20 @@ as the object doesn't contains a key of 'c'
 // Given a contains function
 // When passed an object and a property name
 // Then it should return true if the object contains the property, false otherwise
-test("returns true when object contains the property", () => {
-  const obj = { a: 1, b: 2 };
-  expect(contains(obj, "a")).toBe(true);
-});
-
-test("returns false when object does not contain the property", () => {
-  const obj = { a: 1, b: 2 };
-  expect(contains(obj, "c")).toBe(false);
-});
-
 
 // Given an empty object
 // When passed to contains
 // Then it should return false
-// test.todo("contains on empty object returns false");
-test("contains on empty object returns false", () => {
-  expect(contains({}, "a")).toBe(false);
-});
+test.todo("contains on empty object returns false");
 
 // Given an object with properties
 // When passed to contains with an existing property name
 // Then it should return true
-test("returns true when object contains the property", () => {
-  const obj = { a: 1, b: 2 };
-  expect(contains(obj, "a")).toBe(true);
-});
 
 // Given an object with properties
 // When passed to contains with a non-existent property name
 // Then it should return false
-test("returns false when object does not contain the property", () => {
-  const obj = { a: 1, b: 2 };
-  expect(contains(obj, "c")).toBe(false);
-});
 
 // Given invalid parameters like an array
 // When passed to contains
 // Then it should return false or throw an error
-test("returns false for invalid parameters like an array", () => {
-  expect(contains([], "a")).toBe(false);
-  expect(contains(null, "a")).toBe(false);
-  expect(contains(123, "a")).toBe(false);
-  expect(contains("hello", "a")).toBe(false);
-});
diff --git a/Sprint-2/interpret/invert.js b/Sprint-2/interpret/invert.js
index 48466c281..bb353fb1f 100644
--- a/Sprint-2/interpret/invert.js
+++ b/Sprint-2/interpret/invert.js
@@ -5,7 +5,7 @@
 // Then it should swap the keys and values in the object
 
 // E.g. invert({x : 10, y : 20}), target output: {"10": "x", "20": "y"}
-/*
+
 function invert(obj) {
   const invertedObj = {};
 
@@ -15,31 +15,15 @@ function invert(obj) {
 
   return invertedObj;
 }
-  */
 
-// a) What is the current return value when invert is called with { a : 1 } - The current return value is { key: 1}
+// a) What is the current return value when invert is called with { a : 1 }
 
-// b) What is the current return value when invert is called with { a: 1, b: 2 } - The current return value is { key: 2}
+// b) What is the current return value when invert is called with { a: 1, b: 2 }
 
-// c) What is the target return value when invert is called with {a : 1, b: 2} - The target return value is { "1": "a", "2": "b" }
+// c) What is the target return value when invert is called with {a : 1, b: 2}
 
 // c) What does Object.entries return? Why is it needed in this program?
-/* Object.entries(obj) returns an array of [key, value] pairs. Example: Object.entries({ a:1, b:2 }) returns [["a", 1], ["b", 2]].
-It is needed because the loop "for (const [key, value] of Object.entries(obj))" let's each pair destructure easily.
-*/
 
-// d) Explain why the current return value is different from the target output - It is because this line "invertedObj.key = value;" uses literal string "key instead of the variable key" which is a wrong property name,
-// instead of this "invertedObj[value] = key;"
+// d) Explain why the current return value is different from the target output
 
 // e) Fix the implementation of invert (and write tests to prove it's fixed!)
-function invert(obj) {
-  const invertedObj = {};
-
-  for (const [key, value] of Object.entries(obj)) {
-    invertedObj[value] = key;
-  }
-
-  return invertedObj;
-}
-
-module.exports = invert;

From 6d1118e56706de9961f42b16cf58553684e1d540 Mon Sep 17 00:00:00 2001
From: ChinweP <exponentialresults1973@gmail.com>
Date: Sat, 1 Aug 2026 15:58:53 +0100
Subject: [PATCH 15/17] Remove obsolete invert tests from Sprint 2

---
 Sprint-2/interpret/invert.test.js | 17 -----------------
 1 file changed, 17 deletions(-)
 delete mode 100644 Sprint-2/interpret/invert.test.js

diff --git a/Sprint-2/interpret/invert.test.js b/Sprint-2/interpret/invert.test.js
deleted file mode 100644
index f8643a570..000000000
--- a/Sprint-2/interpret/invert.test.js
+++ /dev/null
@@ -1,17 +0,0 @@
-const invert = require("./invert.js");
-
-test("inverts a single key-value pair", () => {
-  expect(invert({ a: 1 })).toEqual({ 1: "a" });
-});
-
-test("inverts multiple key-value pairs", () => {
-  expect(invert({ a: 1, b: 2 })).toEqual({ 1: "a", 2: "b" });
-});
-
-test("handles empty objects", () => {
-  expect(invert({})).toEqual({});
-});
-
-test("handles string values", () => {
-  expect(invert({ x: "hello" })).toEqual({ hello: "x" });
-});

From 86a7e7d8a4d012819ba9ddb04c1fcb888ef0c333 Mon Sep 17 00:00:00 2001
From: ChinweP <exponentialresults1973@gmail.com>
Date: Sat, 1 Aug 2026 16:13:00 +0100
Subject: [PATCH 16/17] restore package.json

---
 Sprint-3/package.json | 27 ++++++---------------------
 1 file changed, 6 insertions(+), 21 deletions(-)

diff --git a/Sprint-3/package.json b/Sprint-3/package.json
index d7728b8bb..e1331e071 100644
--- a/Sprint-3/package.json
+++ b/Sprint-3/package.json
@@ -1,32 +1,17 @@
 {
-  "name": "module-data-groups",
+  "name": "alarmclock",
   "version": "1.0.0",
   "license": "CC-BY-SA-4.0",
-  "description": "",
+  "description": "You must update this package",
   "scripts": {
-    "test": "jest",
-    "format": "prettier --write ."
+    "test": "jest --config=../jest.config.js alarmclock"
   },
-  "workspaces": [
-    "alarmclock",
-    "quote-generator",
-    "reading-list",
-    "slideshow",
-    "todo-list"
-  ],
   "repository": {
     "type": "git",
-    "url": "git+https://github.com/CodeYourFuture/Module-Data-Groups.git"
+    "url": "git+https://github.com/CodeYourFuture/CYF-Coursework-Template.git"
   },
   "bugs": {
-    "url": "https://github.com/CodeYourFuture/Module-Data-Groups/issues"
+    "url": "https://github.com/CodeYourFuture/CYF-Coursework-Template/issues"
   },
-  "homepage": "https://github.com/CodeYourFuture/Module-Data-Groups#readme",
-  "devDependencies": {
-    "@testing-library/dom": "^10.4.0",
-    "@testing-library/jest-dom": "^6.6.3",
-    "@testing-library/user-event": "^14.6.1",
-    "jest": "^28.1.3",
-    "jest-environment-jsdom": "^30.0.4"
-  }
+  "homepage": "https://github.com/CodeYourFuture/CYF-Coursework-Template#readme"
 }

From d724dbc6af47d21587a355f7bd918e07cba7f5a7 Mon Sep 17 00:00:00 2001
From: ChinweP <exponentialresults1973@gmail.com>
Date: Sat, 1 Aug 2026 16:17:49 +0100
Subject: [PATCH 17/17] Update package.json for module-data-groups
 configuration

---
 Sprint-3/package.json | 27 +++++++++++++++++++++------
 1 file changed, 21 insertions(+), 6 deletions(-)

diff --git a/Sprint-3/package.json b/Sprint-3/package.json
index e1331e071..711a5390f 100644
--- a/Sprint-3/package.json
+++ b/Sprint-3/package.json
@@ -1,17 +1,32 @@
 {
-  "name": "alarmclock",
+  "name": "module-data-groups",
   "version": "1.0.0",
   "license": "CC-BY-SA-4.0",
-  "description": "You must update this package",
+  "description": "",
   "scripts": {
-    "test": "jest --config=../jest.config.js alarmclock"
+    "test": "jest",
+    "format": "prettier --write ."
   },
+  "workspaces": [
+    "alarmclock",
+    "quote-generator",
+    "reading-list",
+    "slideshow",
+    "todo-list"
+  ],
   "repository": {
     "type": "git",
-    "url": "git+https://github.com/CodeYourFuture/CYF-Coursework-Template.git"
+    "url": "git+https://github.com/CodeYourFuture/Module-Data-Groups.git"
   },
   "bugs": {
-    "url": "https://github.com/CodeYourFuture/CYF-Coursework-Template/issues"
+    "url": "https://github.com/CodeYourFuture/Module-Data-Groups/issues"
   },
-  "homepage": "https://github.com/CodeYourFuture/CYF-Coursework-Template#readme"
+  "homepage": "https://github.com/CodeYourFuture/Module-Data-Groups#readme",
+  "devDependencies": {
+    "@testing-library/dom": "^10.4.0",
+    "@testing-library/jest-dom": "^6.6.3",
+    "@testing-library/user-event": "^14.6.1",
+    "jest": "^30.0.4",
+    "jest-environment-jsdom": "^30.0.4"
+  }
 }