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

How to use this in Turnip (instead of Cucumber)? #21

Closed
jmuheim opened this issue Jan 2, 2014 · 1 comment
Closed

How to use this in Turnip (instead of Cucumber)? #21

jmuheim opened this issue Jan 2, 2014 · 1 comment

Comments

@jmuheim
Copy link

jmuheim commented Jan 2, 2014

Your gem looks very promising. Sadly it doesn't seem to do anything when using Turnip instead of Cucumber, although it gives some output:

rake spec DEADWEIGHT=true
[deprecated] I18n.enforce_available_locales will default to true in the future. If you really want to skip validation of your locale you can set I18n.enforce_available_locales = false to avoid this message.
/Users/josh/.rvm/rubies/ruby-2.0.0-p353/bin/ruby -S rspec ./spec/acceptance/members/confirm_registration.feature ./spec/acceptance/members/member_edit.feature ./spec/acceptance/members/member_show.feature ./spec/acceptance/members/sign_in.feature ./spec/acceptance/members/sign_out.feature ./spec/acceptance/members/sign_up.feature ./spec/cells/mini_hub_cell_spec.rb ./spec/controllers/dashboards_controller_spec.rb ./spec/models/member_spec.rb ./spec/routing/members_routing_spec.rb

found 0 unused selectors out of 0 total

The specs are successfully run, and I added some unused CSS selectors to my code so Deadweight could find them, but nothing happens.

Any hint on how to debug this? I have activated @js, @chrome and @selenium for some of the feature tests, so the CSS is definitely compiled and available.

Thanks for help.

@jmuheim
Copy link
Author

jmuheim commented Jan 5, 2014

I fixed it myself, here's the PR: #22.

Didn't have anything to do with Turnip, I just guess that your gem is pretty old already and wasn't updated a long time.

Would be nice to have it merged as it's functionality is really useful.

@jmuheim jmuheim closed this as completed Jan 22, 2014
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

No branches or pull requests

1 participant