London | 26-ITP-SEPT | Carol Nassuna | Sprint 1 | Exhaustively test and fix formatAs12HourClock

[Mugs3]

Self checklist

• I have titled my PR with Region | Cohort | FirstName LastName | Sprint | Assignment Title
• My changes meet the requirements of the task
• I have tested my changes
• My changes follow the style guide

Task code

CYF-1197

Changelist

Updated the 2 files

[abdishakoor-dev]

You kept both starter tests exactly as they were, and your midnight test is doing its job: it fails, because the function still has a bug there. That's exactly what a test is for.

How to run your tests. You'll need this for every step below:

1. In VS Code, look at the file list on the left (the Explorer).
2. Right-click the folder called format-clock-edge-cases and choose Open in Integrated Terminal. A terminal panel opens at the bottom of the screen.
3. Click in that terminal panel, type node --test timeConverter.test.js and press Enter.
4. Each test is listed with a ✔ (passed) or a ✖ (failed). At the bottom you'll see pass and fail with a number next to each.

At the moment you'll see pass 8 and fail 1. The ✖ is next to "can correctly convert midnight".

What to change before I can mark this Complete:

1. Fix midnight in timeConverter.js, lines 5 and 6. My comment on line 5 explains how.

2. Fix the typo in timeConverter.test.js line 30. My comment there explains it.

3. Add three more tests at the bottom of timeConverter.test.js, after line 39. Copy one of your existing tests and change the time and the expected answer:

   • "00:30": half past midnight
   • "11:59": the last minute before am turns into pm
   • "13:00": one o'clock in the afternoon

   Work out what each should give on a 12-hour clock first, and write that as the expected answer.

4. Delete lines 16 to 20 of timeConverter.js (the five console.log lines).

When you've made the changes, run node --test timeConverter.test.js in the terminal again. Push only when it says fail 0. Then add the Needs Review label again. Once these four are done, I expect to mark this Complete.

[abdishakoor-dev]

This is why the midnight test fails.

Line 5 checks whether time is "00". But for midnight, time is the whole text "00:00", so the check is never true, and the code carries on down to line 12, which returns "00:00 am".

Line 2 already gives you the hour on its own, as a number, in the variable hours. Change line 5 so it checks hours instead of time. Remember hours is a number, so compare it with a number, not with text in quotes.

Then look at line 6. It returns 12:00: followed by the minutes, which gives 12:00:00 am, one part too many. Line 10 is how you did it for noon. Midnight needs the same shape, but with am instead of pm.

Run node --test timeConverter.test.js in the terminal again. The midnight test should now show ✔.

[abdishakoor-dev]

There's an extra 1 here: "12:001" should be "12:01". Delete the extra 1 so the test checks a real time. (It passes at the moment only by luck, because the function looks at the last two characters.)

[abdishakoor-dev]

These five lines were useful while you were working it out, but they print every time the tests run: you can see 11:00 pm, 2:00 pm and so on above your test results. Delete lines 16 to 20, then run node --test timeConverter.test.js again to check nothing else changed.