-
Notifications
You must be signed in to change notification settings - Fork 554
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
not working with riot correctly? #80
Comments
I haven't tried SimpleCov with riot yet, but there's no reason it shouldn't work. I'll try it out. Maybe this is related to the bug in the Coverage lib as figured out in #60. |
yeah I think it's the Coverage lib as cover_me is churning out the same results :P |
@saiko-chriskun Could you please edit your issue description to hold the code sample you had posted on StackOverflow as it seems the question is no longer present there and I wanted to provide Ruby core maintainers with this example on my already filed bug report on this: http://redmine.ruby-lang.org/issues/5333 |
updated |
not sure what changed exactly but it's not catching any of my controller code now (it doesn't detect any files under the controller tab) although the models seem to be fine. |
Thanks for the report. I'm not using riot so I didn't realize the problem yet. It would be great if you could create a cucumber feature and example that tests usage with the riot framework similar to those already existing and submit it as a Pull Request so I could go about fixing those. Also, this might be related to #60. |
I'm not having any trouble using it with Riot... Although, I was. I'm trying to remember how I fixed it. I think what I actually ended up with was this: https://gist.github.com/1814276 Again, I'm fuzzy on the whole "fix". But I think it's the And yeah I'm pretty sure it's related to #60 |
Closing as stale. Please reopen if this is still an issue. (And take a look at #340 ) |
simplecov results: http://stuff.saikonet.org/images/misc/2011-09-18-150129_1024x768_scrot.png
my tests:
The text was updated successfully, but these errors were encountered: