Skip to content

Comments

London | 26-ITP-Jan | Kris Oldrini | Sprint 3 | Implement And Rewrite Tests#1045

Open
XiaoQuark wants to merge 9 commits intoCodeYourFuture:mainfrom
XiaoQuark:coursework/sprint-3
Open

London | 26-ITP-Jan | Kris Oldrini | Sprint 3 | Implement And Rewrite Tests#1045
XiaoQuark wants to merge 9 commits intoCodeYourFuture:mainfrom
XiaoQuark:coursework/sprint-3

Conversation

@XiaoQuark
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

1 Implement solutions

  • Implemented required functions for files:
    • 1-get-angle-type.js
    • 2-is-proper-fraction.js
    • 3-get-card-value.js
  • Added tests in the above files to cover all possible cases (including error handling).
  • Verified all tests passed.

2 Rewrite tests with Jest

  • Rewrote equivalent test suites in Jest
  • Verified all tests pass via npm test

@XiaoQuark XiaoQuark added the Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. label Feb 24, 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