Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions format-clock-edge-cases/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -13,3 +13,4 @@ function formatAs12HourClock(time) {
```

It still has bugs. Think of as many edge-cases as you can with this code. Write tests for all of them, and fix this code so that it works correctly for all valid inputs. You don't need to worry about invalid inputs (e.g. `"25:00"`).

20 changes: 17 additions & 3 deletions format-clock-edge-cases/timeConverter.js
Original file line number Diff line number Diff line change
@@ -1,11 +1,25 @@
function formatAs12HourClock(time) {

const hours = Number(time.slice(0, 2));
const minutes = time.slice(3, 5);

if (hours > 12) {
return `${hours - 12}:00 pm`;
const convertedHours = hours - 12;
// Add leading zero if convertedHours is a single digit
const formattedHours =
convertedHours < 10 ? `0${convertedHours}` : `${convertedHours}`;
return `${formattedHours}:${minutes} pm`;
}
//handling noon.
if (hours === 12) {
return `12:${minutes} pm`;
}

//handling midnight
if (hours === 0) {
return `12:${minutes} am`;
}
//handling morning times
return `${time} am`;
}

export {formatAs12HourClock};
export { formatAs12HourClock };
34 changes: 29 additions & 5 deletions format-clock-edge-cases/timeConverter.test.js

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The test cases you have included are:

  • 00:00 midnight
  • 12:00 noon
  • 01:30 single digit hour in the morning
  • 13:45 single digit hour in the afternoon
  • 23:00 double digits hour in the afternoon

It seems that there is still one case missing. Please add it back, thanks a lot.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for taking your time to review my code, I have added the "double-digit hour in the afternoon" test case.

@hackertainment hackertainment Nov 11, 2026 •

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good try, but please read the list of test cases carefully. Since you already have both single-digit and double-digit hour in the afternoon in your original tests, the one you have missed would not be in the afternoon anyway.

Original file line number Diff line number Diff line change
@@ -1,11 +1,35 @@
import {formatAs12HourClock} from "./timeConverter.js";
import { formatAs12HourClock } from "./timeConverter.js";
import assert from "node:assert";
import test from "node:test";

test("correctly convert time after 12:00", function(){
assert.equal(formatAs12HourClock("23:00"), "11:00 pm");
test("correctly convert time after 12:00", function () {
assert.equal(formatAs12HourClock("23:00"), "11:00 pm");
});

test("can correctly convert morning time", function() {
assert.equal(formatAs12HourClock("08:00"), "08:00 am");
test("can correctly convert morning time", function () {
assert.equal(formatAs12HourClock("08:00"), "08:00 am");
});

test("correctly convert midnight", function () {
assert.equal(formatAs12HourClock("00:00"), "12:00 am");
});

test("correctly convert noon", function () {
assert.equal(formatAs12HourClock("12:00"), "12:00 pm");
});

test("correctly convert single digit hour in the morning", function () {
assert.equal(formatAs12HourClock("01:30"), "01:30 am");
});

test("correctly convert single digit hour in the afternoon", function () {
assert.equal(formatAs12HourClock("13:45"), "01:45 pm");
});

test("correctly convert time with leading zeros", function () {
assert.equal(formatAs12HourClock("09:05"), "09:05 am");
});

test("correctly convert double digit hour in the afternoon", function () {
assert.equal(formatAs12HourClock("15:20"), "03:20 pm");
});
Loading