Skip to content

Conversation

@mao-sz
Copy link
Contributor

@mao-sz mao-sz commented Jan 27, 2026

Because

"works ..." doesn't communicate very well exactly what it means. Some learners have gotten confused with the expected behaviour (e.g. ESL speaker so "integer" isn't as immediately obvious).

This PR

  • Renames test titles to be more explicit

Issue

Closes #XXXXX

Additional Information

Pull Request Requirements

  • I have thoroughly read and understand The Odin Project Contributing Guide
  • The title of this PR follows the location of change: brief description of change format, e.g. 01_helloWorld: Update test cases
  • The Because section summarizes the reason for this PR
  • The This PR section has a bullet point list describing the changes in this PR
  • If this PR addresses an open issue, it is linked in the Issue section
  • If this PR includes any changes that affect the solution of an exercise, I've also updated the solution in the /solutions folder

"works ..." doesn't communicate very well exactly what it means
@mao-sz mao-sz merged commit 97796cf into TheOdinProject:main Jan 27, 2026
1 check passed
@mao-sz mao-sz deleted the totalIntegers-test-names branch January 27, 2026 17:59
ssgunnam added a commit to ssgunnam/javascript-exercises that referenced this pull request Jan 29, 2026
totalIntegers: Provide clearer test names (TheOdinProject#623)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants