-
Notifications
You must be signed in to change notification settings - Fork 4
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
Cucumber feature subdirs #6
base: master
Are you sure you want to change the base?
Conversation
This is quite a large change to make to the application and the book this far along. However, I do agree with you on this. Don't think that I am ignoring this pull request, just rather letting it marinate while I come up with the motivation to fix this in the book as well as in ticketee. |
No worries. It is true that it is a big change that has impact on the book as you would have to change every listing that references the features path and add the subdir to it. Maybe you can mention the idea in a tip box (like the one about "Combining redirect_to flash" on page 86)? That would be a lot less changes to do in the book :) |
This works fine for running the entire feature set but when attempting to run a single feature using Did you try this? |
I did not try this. Out of habit, I just run I tried I will investigate and get back to you |
So after digging a little, it seems that the Without that, it does not load the code under support (you can see that with --verbose). I'll do more testing and update the pull request Cheers |
Here are my observations:
TL;DR: If you want to target a specific feature, you need to have a Cheers |
I re-organized the features into sub-directories. This is a tip I read in the Rspec book which I find useful. As a project grows, it makes it easier to navigate through the features.