Skip to content

London | 26-ITP-Jan | Kayanat Suleman | Sprint 2 | Data Groups#973

Open
KayanatSuleman wants to merge 31 commits intoCodeYourFuture:mainfrom
KayanatSuleman:sprint-2
Open

London | 26-ITP-Jan | Kayanat Suleman | Sprint 2 | Data Groups#973
KayanatSuleman wants to merge 31 commits intoCodeYourFuture:mainfrom
KayanatSuleman:sprint-2

Conversation

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

Implemented all Sprint 2 Data Groups exercises:

  • Fixed object access issues in debug tasks (address, author, recipe).
  • Implemented contains with validation and full test coverage.
  • Implemented createLookup and added tests.
  • Fixed parseQueryString to correctly handle values containing "=" and added edge case tests.
  • Implemented tally with error handling and frequency counting.
  • Fixed invert to correctly swap keys and values using dynamic bracket notation.
  • Separated test files from implementation files for clarity.

All tests pass locally using npm test, and have used Node REPL for further testing.

@KayanatSuleman KayanatSuleman added 📅 Sprint 2 Assigned during Sprint 2 of this module Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. Module-Data-Groups The name of the module. labels Mar 2, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Module-Data-Groups The name of the module. Needs Review Trainee to add when requesting review. PRs without this label will not be reviewed. 📅 Sprint 2 Assigned during Sprint 2 of this module

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant