Refactor toPounds function for clarity and return value; enhance time-format.js comments for accuracy; add additional test cases in format-time.js

Author: JohnRob-jnr

Sprint-2/3-mandatory-implement/3-to-pounds.js

@@ -6,19 +6,25 @@
 // You should call this function a number of times to check it works for different inputs
 
 function toPounds(penceString) {
-  const penceStringWithoutTrailingP = penceString.substring(
-    0,
-    penceString.length - 1
-  );
+    const penceStringWithoutTrailingP = penceString.substring(
+        0,
+        penceString.length - 1
+    );
 
-const paddedPenceNumberString = penceStringWithoutTrailingP.padStart(3, "0");
-const pounds = paddedPenceNumberString.substring(
-  0,
-  paddedPenceNumberString.length - 2
-);
+    const paddedPenceNumberString = penceStringWithoutTrailingP.padStart(3, "0");
+    const pounds = paddedPenceNumberString.substring(
+        0,
+        paddedPenceNumberString.length - 2
+    );
 
-const pence = paddedPenceNumberString
-  .substring(paddedPenceNumberString.length - 2)
-  .padEnd(2, "0");
+    const pence = paddedPenceNumberString
+        .substring(paddedPenceNumberString.length - 2)
+        .padEnd(2, "0");
 
-console.log(`£${pounds}.${pence}`);
+    return `£${pounds}.${pence}`;
+}
+
+console.log(toPounds("1p")); 
+console.log(toPounds("90p"));
+console.log(toPounds("303p"));
+console.log(toPounds("23456p"));

Sprint-2/4-mandatory-interpret/time-format.js

@@ -21,15 +21,15 @@ function formatTimeDisplay(seconds) {
 // Questions
 
 // a) When formatTimeDisplay is called how many times will pad be called?
-// =============> write your answer here
+// =============> 3
 
 // Call formatTimeDisplay with an input of 61, now answer the following:
 
 // b) What is the value assigned to num when pad is called for the first time?
-// =============> write your answer here
+// =============> 1
 
 // c) What is the return value of pad is called for the first time?
-// =============> write your answer here
+// =============> "01"
 
 // d) What is the value assigned to num when pad is called for the last time in this program?  Explain your answer
 // =============> write your answer here

Sprint-2/5-stretch-extend/format-time.js

@@ -23,3 +23,17 @@ console.assert(
   currentOutput2 === targetOutput2,
   `current output: ${currentOutput2}, target output: ${targetOutput2}`
 );
+
+const currentOutput3 = formatAs12HourClock("13:15");
+const targetOutput3 = "1:15 pm";
+console.assert(
+  currentOutput3 === targetOutput3,
+  `current output: ${currentOutput3}, target output: ${targetOutput3}`
+);
+
+const currentOutput4 = formatAs12HourClock("12:00");
+const targetOutput4 = "12:00 am";
+console.assert(
+  currentOutput4 === targetOutput4,
+  `current output: ${currentOutput4}, target output: ${targetOutput4}`
+);