Manchester| 25-ITP-September | Fithi Teklom| Sprint 3 | 2-practice-tdd

[Fithi-Teklom]

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

[github-actions]

Your PR's title isn't in the expected format.

Please check the expected title format, and update yours to match.

Reason: Wrong number of parts separated by |s

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

[github-actions]

Your PR's title isn't in the expected format.

Please check the expected title format, and update yours to match.

Reason: Wrong number of parts separated by |s

If this PR is not coursework, please add the NotCoursework label (and message on Slack in #cyf-curriculum or it will probably not be noticed).

[cjyuan]

The test in get-oridinal-number.test.js could use some improvement.

All other changes look good.

[Fithi-Teklom]

I have cleaned my branch.

[cjyuan]

Branch is clean. Well done!

You placed Jest test script for getOridinalNumber() in Sprint-3/2-practice-tdd/get-ordinal-number.js instead of in Sprint-3/2-practice-tdd/get-ordinal-number.test.js. Can you move the test script to the .test.js file?

[cjyuan]

Have you tried running this test script to make sure it can run properly?

[Fithi-Teklom]

Now I have made sure that the tests have all passed.

[cjyuan]

Looks good.