Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix Minitest capitalization #1818

Merged
merged 1 commit into from
Jul 27, 2023
Merged

Conversation

mhashizume
Copy link
Contributor

Minitest v5.19.0, released today, includes a commit[1] that only calls its compatibility layer when using an environment variable ("MT_COMPAT").

This means that calling "MiniTest" (with a capital "T") instead of "Minitest" (with all lowercase "T"s) no longer works.

This commit replaces all instances of "MiniTest" with "Minitest."

[1] minitest/minitest@a2c6c18

@codecov
Copy link

codecov bot commented Jul 26, 2023

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (6e73071) 74.91% compared to head (c301dc7) 74.91%.

❗ Current head c301dc7 differs from pull request most recent head 7f9e73b. Consider uploading reports for the commit 7f9e73b to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##              4.x    #1818   +/-   ##
=======================================
  Coverage   74.91%   74.91%           
=======================================
  Files          82       82           
  Lines        4863     4863           
=======================================
  Hits         3643     3643           
  Misses       1220     1220           
Files Changed Coverage Δ
lib/beaker.rb 90.90% <ø> (ø)
lib/beaker/test_case.rb 91.35% <100.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Minitest v5.19.0, released today, includes a commit[1] that only calls
its compatibility layer when using an environment variable
("MT_COMPAT").

This means that calling "MiniTest" (with a capital "T") instead of
"Minitest" (with all lowercase "T"s) no longer works.

This commit replaces all instances of "MiniTest" with "Minitest."

[1] minitest/minitest@a2c6c18
@bastelfreak
Copy link
Member

do we have to merge this into the 4.x branch? that's kinda dead and maintaining and releasing it is tricky.

@mhashizume
Copy link
Contributor Author

We're still using 4.x internally at Puppet, so a 4.x release would be much appreciated

Copy link
Member

@smortex smortex left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's try it :)

@smortex smortex added the bug label Jul 27, 2023
@smortex smortex merged commit d6bbb00 into voxpupuli:4.x Jul 27, 2023
7 checks passed
@smortex smortex changed the title (maint) Update Minitest capitalization Fix Minitest capitalization Jul 27, 2023
@smortex
Copy link
Member

smortex commented Jul 27, 2023

Let's try it :)

Note for myself next time:

run away!

@mhashizume mhashizume deleted the maint/4.x/minitest branch July 27, 2023 17:33
@mhashizume mhashizume restored the maint/4.x/minitest branch July 27, 2023 18:22
@mhashizume mhashizume deleted the maint/4.x/minitest branch September 18, 2023 17:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants