-
Notifications
You must be signed in to change notification settings - Fork 9
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
Check for unused stubs #16
Comments
Awesome! Glad it was helpful. :D
Yeah, I think that's actually a really good idea. I think it'd be fairly manageable to support in at least a few cases if not all. The tricky part is that stubbing is chainable so it might be hard to determine if the whole expression is useless, but I imagine I can figure something out. On a related note, I've always planned to handle this for doubles--it's common to make the same mistake there where you do something like |
I think this is now roughly three categories of features I want:
I may break this into other issues later, but I think this suffices for now. |
This issue now just tracks this--I split out the other two. |
Today, I used rspectre to remove 250 unused lines from our spec suite. So: Thank you!
Quite a few of these removed lets were actually still being referenced in blocks in unused stubs, such as:
Would it be possible for rspectre to also check for unused stubs, or it that a job for another tool?
The text was updated successfully, but these errors were encountered: