Skip to content

Minitest update + code coverage#2666

Open
spamguy wants to merge 2 commits intofreeCodeCamp:mainfrom
spamguy:minitest
Open

Minitest update + code coverage#2666
spamguy wants to merge 2 commits intofreeCodeCamp:mainfrom
spamguy:minitest

Conversation

@spamguy
Copy link
Contributor

@spamguy spamguy commented Mar 25, 2026

In this PR:

  • Updates minitest to v6. This shuts up an ugly bundle install message.
  • Adds a minitest coverage reporter for core-level tests. Runnable by calling thor test:coverage.
  • Adds a pile of new unit tests to bring coverage up to 90.8%.
  • Fixes a frozen literal warning thrown during minitest's run.

Note: some unit tests were manually written by me to keep test writing skills fresh, but most were autogenerated by Claude.

Fix frozen literal warning; add tests.
@spamguy spamguy requested a review from a team as a code owner March 25, 2026 05:13
@spamguy spamguy changed the title Update minitest to v6. Add coverage reporter. Minitest update + code coverage Mar 25, 2026
@socket-security
Copy link

socket-security bot commented Mar 25, 2026

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Updatedminitest@​5.27.0 ⏵ 6.0.291 -2100100100100
Addedsimplecov@​0.22.097100100100100

View full report

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.

1 participant