Skip to content

Sheffield | 26-ITP-Jan | Mahammad Osman | Sprint 3 | coursework/sprint-3-implement-and-rewrite#1235

Open
Darkidd77 wants to merge 5 commits intoCodeYourFuture:mainfrom
Darkidd77:coursework/sprint-3-implement-and-rewrite
Open

Sheffield | 26-ITP-Jan | Mahammad Osman | Sprint 3 | coursework/sprint-3-implement-and-rewrite#1235
Darkidd77 wants to merge 5 commits intoCodeYourFuture:mainfrom
Darkidd77:coursework/sprint-3-implement-and-rewrite

Conversation

@Darkidd77
Copy link

Learners, PR Template

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

Changelist

  • Jest Setup: Installed and configured Jest as the primary test runner.
  • Defensive Programming: The function now throws descriptive errors for invalid inputs.
  • Unit Testing: Replaced try...catch manual tests with Jest's .toThrow() and .toBe() assertions for cleaner, more readable test code.

@Darkidd77 Darkidd77 added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Mar 9, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant