Skip to content

Commit 7a70e63

Browse files
committed
Fix 12-hour time formatting for leading-zero hours
1 parent 8ee83a9 commit 7a70e63

1 file changed

Lines changed: 10 additions & 5 deletions

File tree

‎Sprint-3/5-stretch-extend/format-time.js‎

Lines changed: 10 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -12,10 +12,8 @@ function formatAs12HourClock(time) {
1212
suffix = "pm";
1313
}
1414

15-
let formattedHours = hours % 12;
16-
if (formattedHours === 0) {
17-
formattedHours = 12;
18-
}
15+
let formattedHours = String(hours % 12 || 12);
16+
formattedHours = formattedHours.padStart(2, "0");
1917

2018
return `${formattedHours}:${minutes} ${suffix}`;
2119
}
@@ -41,8 +39,15 @@ console.assert(
4139
`current output: ${midnightOutput}, target output: ${midnightTarget}`,
4240
);
4341

42+
const paddedHourOutput = formatAs12HourClock("08:00");
43+
const paddedHourTarget = "08:00 am";
44+
console.assert(
45+
paddedHourOutput === paddedHourTarget,
46+
`current output: ${paddedHourOutput}, target output: ${paddedHourTarget}`,
47+
);
48+
4449
const singleDigitMinuteOutput = formatAs12HourClock("9:5");
45-
const singleDigitMinuteTarget = "9:05 am";
50+
const singleDigitMinuteTarget = "09:05 am";
4651
console.assert(
4752
singleDigitMinuteOutput === singleDigitMinuteTarget,
4853
`current output: ${singleDigitMinuteOutput}, target output: ${singleDigitMinuteTarget}`,

0 commit comments

Comments
 (0)