There are no route matches {: action => "show" ,: controller => "user" ,: id => nil}

Question

Hi everyone, I have this problem (post title).

Failure/Error: get :create, :user => { :email => ' [e-mail] ', :name => 'userexample' } ActionController::RoutingError:No route matches {:action=>"show", :controller=>"user", :id=>nil} 

Here is my UserController .

 class UserController < ApplicationController def new end def create @user = User.new(params[:user]) @user.save redirect_to :action => :show, :id => @user.id end def show @user = User.find(params[:id]) end end 

Here are my .rb routes

 ::Application.routes.draw do devise_for :users root :to => "user#index" resources :user #match "newuser" => "user#new" get "user/new" post "user/create" get "user/:id" => "user#show" 

And here is my rspec test

 require 'spec/spec_helper' describe UserController do it "create new user" do get :create, :user => { :email => ' [e-mail] ', :name => 'userexample' } flash[:notice] = 'new user was successfully created.' end end end 

Answer 1

 if @user.save redirect_to :action => :show, :id => @user.id else ... end 

Answer 2

Most Rails applications use this type of content block to create and update records.

 def create @user = User.new(params[:user]) if @user.save redirect_to @user, :notice => "User was created successfully." else render :new end end 

Basically, when the create action is executed, it creates a new record or a new instance of the User model.

When .save is .save , it checks all checks. If the checks pass, then it returns true, otherwise it returns false.

If it returns false, you need to display the template :new again, so that the person has the opportunity to fix any validation errors.

If it returns true, you want to redirect to the show action. The great thing about rails is that it can automatically detect this by simply passing an instance of @user .

---

Ryan Bates has a tutorial on controllers with RSpec . Based on what he does, I will draw your controller.

 require 'spec/spec_helper' describe UserController do describe "POST 'create'" do it "should redirect to user on successful save" do User.any_instance.stub(:valid?) { true } post :user flash[:notice].should_not be_nil assigns(:user).should_not be_a_new_record response.should redirect_to(assigns(:user)) end it "should render new template on failed save" do User.any_instance.stub(:valid?) { false } post :user flash[:notice].should be_nil assigns(:user).should be_a_new_record response.should render_template(:new) end it "should pass params" do User.any_instance.stub(:valid?) { true } post :user, :user => { email_address: " [e-mail] " } assigns(:user).email_address.should == " [e-mail] " end end end