zzak
zzak
Sorry I just saw #211 but I think this PR is better because specs are passing and doesn't break backwards compatibility.
@splittingred Sorry for the ping, would you mind taking a look at this when you have time? The linter was failing so I just rebased and fixed that a few...
@fatkodima I took the liberty to rebase this and get it green if you are interested in this still. :pray:
This looks too specific to your particular environment needs, and therefor not a good fit for the framework. :thinking:
Sorry I thought that was something specific to your setup, and now I realized rbenv is being used inside the devcontainer. I don't use it so I just wasn't familiar....
I think this is too unique a case, and that we shouldn't add an exception here only because this particular class doesn't behave with `#to_s` the way you expected. What...
If the name is generic enough like this, and it really only applies to ActiveModel objects, then it should only be available in something like `ActiveModel::TestCase` :thinking:
Yes, I realized that after I made the comment, sorry was just thinking about it. You're 100% right in that model tests inherit from AS::TestCase (by design). Would you mind...
I think GitHub auto-closed this because the message had "Fix #NNNNN" but was apart of the message for the revert.
I checked it and it looks like there was a fix made downstream so going to close this, please re-open if there was any issue. :bow: