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 constant name in tests #791

Merged
merged 1 commit into from
Aug 1, 2023

Conversation

skipkayhil
Copy link
Member

Minitest 5.19.0 removed the "ancient MiniTest compatibility layer" that aliased Minitest as MiniTest. CI runs without a lock file so tests are failing due to this new release.

This commit renames all of the tests to the correct Minitest constant

Minitest 5.19.0 removed the "ancient MiniTest compatibility layer" that
aliased `Minitest` as `MiniTest`. CI runs without a lock file so tests
are failing due to this new release.

This commit renames all of the tests to the correct `Minitest` constant
@guilleiguaran guilleiguaran merged commit 6554b6d into rails:main Aug 1, 2023
13 of 14 checks passed
@skipkayhil skipkayhil deleted the hm-fix-minitest-casing branch August 1, 2023 21:36
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