Rails%203%20In%20Action

96 CHAPTER 4 Oh CRUD!
F
When the test runs, you get a failure:
1) ProjectsController displays an error
message when asked for a missing project
Failure/Error: get :show, :id => "not-here"
Couldn't find Project with ID=not-here
This is the same failure you saw when you tried running the application using rails
server. Now that you have a failing test, you can fix it.
Open the app/controllers/projects_controller.rb file, and put the code from the
following listing underneath the last action in the controller but before the end of the
class.
Listing 4.17 app/controllers/projects_controller.rb
private
def find_project
@project = Project.find(params[:id])
rescue ActiveRecord::RecordNotFound
flash[:alert] = "The project you were looking" +
" for could not be found."
redirect_to projects_path
end
This method has the private method before it, so the controller doesn’t respond to
this method as an action. To call this method before every action, use the
before_filter method. Place these lines directly under the class Projects-
Controller definition:
before_filter :find_project, :only => [:show,
:edit,
:update,
:destroy]
What does all this mean? Let’s start with the before_filter. before_filters are run
before all the actions in your controller unless you specify either the :except or :only
option. Here you have the :only option defining actions you want the before_filter
to run for. The :except option is the opposite of the :only option, specifying the
actions you do not want the before_filter to run for. The before_filter calls the
find_project method before the specified actions, setting up the @project variable
for you. This means you can remove the following line from all four of your actions
(show, edit, update, and destroy):
@project = Project.find(params[:id])
By doing this, you make the show and edit actions empty. If you remove these actions
and run rake cucumber:ok again, all the scenarios still pass. Controller actions don’t
need to exist in the controllers if there are templates corresponding to those actions,
which you have for these actions. For readability’s sake, it’s best to leave these in the