-
Notifications
You must be signed in to change notification settings - Fork 58
uninitialized constant HTMLSelector::Minitest after update to 2.1.0 #107
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
Comments
I am also getting the |
Im pretty sure the error was introduced with #106. The workaround is to |
I'm also getting this error over here, I'm upgrading the Ruby version of an application, and this error was returned when bumping from Ruby version 2.6.8 to 2.6.9. During the execution of the tests I don't have any problems, but when running the server the error |
I got the problem after upgrading dartsass-rails to 0.5.0, which somewhat triggered some additional upgrade on this gem, rails-dom-testing.
|
After performing some tests in my application, I was able to run the server again successfully, but I had to add the gem explicitly to group :test do
gem 'rails-dom-testing', '~> 2.0.3'
end |
it looks like adding Here is a sample repo: https://github.com/joeadcock/rails-dom-testing-2.1.0-bug |
@iagopuccini yeah, we're currently on 2.0.3 and it works, but locking to that version prevents a bunch of other gems from updating. |
@xxx Adding |
@joeadcock thanks, that does work, though I guess I don't understand why specifying the version that way allows other gems to be updated, while locking to 2.0.3 doesn't. |
@xxx Yeah, that is strange... |
wow, that was quick. Thank you folks for reporting 2.1.1 should have fixed it. Let me know if it is still happening. |
I can confirm that 2.1.1 fixes me. thanks! |
Hi, I'm doing some gem updates today, and starting seeing this when I try to run tests. rails-dom-testing is in our bundle as a dependency of other packages, but we do not use it ourselves directly.
Is there some new configuration I need to add? I didn't see anything documented.
full trace:
The text was updated successfully, but these errors were encountered: