Rails%203%20In%20Action

Fixing what you broke
else
Project.readable_by(current_user)
end
This statement is useful when you have short conditional statements like this one, but
it shouldn’t be (ab)used for longer conditional statements. As a general rule of
thumb, if the line for a ternary statement is longer than 80 characters, it’s probably
best to split it out over multiple lines for better readability.
In the find_project method, you can call find on what this new for method
returns, and now in the index method, you can use it in an identical fashion, but just
replace the call to find with all, like this:
def index
@projects = Project.for(current_user).all
end
Because you are referencing current_user in this action, you must modify the
before_filter line that references authenticate_user! to ensure that users are
signed in before they visit this page. Let’s change it to this:
before_filter :authenticate_user!, :only => [:index, :show]
When you run the feature again with bin/cucumber features/viewing_projects
.feature, it passes:
1 scenario (1 passed)
9 steps (9 passed)
Ensure that everything is working as it should by running rake cucumber:ok spec.
Oops! You broke some of the scenarios inside the Hidden Links feature, as shown by
this output from the cucumber:ok part of the command you just ran:
cucumber features/hidden_links.feature:23
cucumber features/hidden_links.feature:27
cucumber features/hidden_links.feature:36
cucumber features/hidden_links.feature:40
If you run the first one of these features with bin/cucumber features/hidden_links
.feature:23, you’ll see that it can’t find the TextMate 2 link.
When I follow "TextMate 2"
no link with title, id or text 'TextMate 2' found (Capybara::ElementNotFound)
This error occurs because the user in this feature doesn’t have permission to view the
TextMate 2 project. But if you look at this scenario, it’s for users who are not signed
in—users who can no longer visit this page because, when they do, they are redirected
to the sign-in page. This means that the following scenarios no longer apply and can
be deleted:
Scenario: Edit project link is hidden for non-signed-in users
Scenario: Delete project link is hidden for non-signed-in users
177