@@ -35,7 +35,7 @@ class <%= controller_class_name %>Controller < ApplicationController # GET /<%=nesting_owner.pluralize%>/:<%=nesting_owner%>_id/<%= table_name %>/new # GET /<%=nesting_owner.pluralize%>/:<%=nesting_owner%>_id/<%= table_name %>/new.xml def new - @<%= file_name %> = @<%= nesting_owner %>.<%= table_name %>.new + @<%= file_name %> = @<%= nesting_owner %>.<%= table_name %>.build respond_to do |format| format.html # new.html.erb @@ -51,7 +51,7 @@ class <%= controller_class_name %>Controller < ApplicationController # POST /<%=nesting_owner.pluralize%>/:<%=nesting_owner%>_id/<%= table_name %> # POST /<%=nesting_owner.pluralize%>/:<%=nesting_owner%>_id/<%= table_name %>.xml def create - @<%= file_name %> = @<%= nesting_owner %>.<%= table_name %>.new(params[:<%= file_name %>]) + @<%= file_name %> = @<%= nesting_owner %>.<%= table_name %>.build(params[:<%= file_name %>]) respond_to do |format| if @<%= file_name %>.save generators/rspec_nested_scaffold/templates/controller.rb @@ -61,7 +61,7 @@ describe <%= controller_class_name %>Controller do describe "responding to GET new" do it "should expose a new <%= file_name %> as @<%= file_name %>" do - mock_<%= nesting_owner %>.<%= table_name %>.should_receive(:new).and_return(mock_<%= file_name %>) + mock_<%= nesting_owner %>.<%= table_name %>.should_receive(:build).and_return(mock_<%= file_name %>) get :new, :<%= nesting_owner %>_id => "1" assigns[:<%= file_name %>].should equal(mock_<%= file_name %>) end @@ -83,13 +83,13 @@ describe <%= controller_class_name %>Controller do describe "with valid params" do it "should expose a newly created <%= file_name %> as @<%= file_name %>" do - mock_<%= nesting_owner %>.<%= table_name %>.should_receive(:new).with({'these' => 'params'}).and_return(mock_<%= file_name %>(:save => true)) + mock_<%= nesting_owner %>.<%= table_name %>.should_receive(:build).with({'these' => 'params'}).and_return(mock_<%= file_name %>(:save => true)) post :create, :<%= file_name %> => {:these => 'params'}, :<%= nesting_owner %>_id => "1" assigns(:<%= file_name %>).should equal(mock_<%= file_name %>) end it "should redirect to the created <%= file_name %>" do - mock_<%= nesting_owner %>.<%= table_name %>.stub!(:new).and_return(mock_<%= file_name %>(:save => true)) + mock_<%= nesting_owner %>.<%= table_name %>.stub!(:build).and_return(mock_<%= file_name %>(:save => true)) post :create, :<%= file_name %> => {}, :<%= nesting_owner %>_id => "1" response.should redirect_to(<%= nesting_owner %>_<%= file_name %>_url(mock_<%= nesting_owner %>, mock_<%= file_name %>)) end @@ -99,13 +99,13 @@ describe <%= controller_class_name %>Controller do describe "with invalid params" do it "should expose a newly created but unsaved <%= file_name %> as @<%= file_name %>" do - mock_<%= nesting_owner %>.<%= table_name %>.stub!(:new).with({'these' => 'params'}).and_return(mock_<%= file_name %>(:save => false)) + mock_<%= nesting_owner %>.<%= table_name %>.stub!(:build).with({'these' => 'params'}).and_return(mock_<%= file_name %>(:save => false)) post :create, :<%= file_name %> => {:these => 'params'}, :<%= nesting_owner %>_id => "1" assigns(:<%= file_name %>).should equal(mock_<%= file_name %>) end it "should re-render the 'new' template" do - mock_<%= nesting_owner %>.<%= table_name %>.stub!(:new).and_return(mock_<%= file_name %>(:save => false)) + mock_<%= nesting_owner %>.<%= table_name %>.stub!(:build).and_return(mock_<%= file_name %>(:save => false)) post :create, :<%= file_name %> => {}, :<%= nesting_owner %>_id => "1" response.should render_template('new') end generators/rspec_nested_scaffold/templates/controller_spec.rb 47346461a246a366d723068b90d34bedd84a8267 Paul Horsfall horsfallp@gmail.com http://github.com/phorsfall/rspec_on_rails_nested_scaffold/commit/f2e1f871b8454585d375cefa18270a4d4c766bb2 f2e1f871b8454585d375cefa18270a4d4c766bb2 2008-10-24T11:38:26-07:00 2008-10-24T11:38:26-07:00 Fixed that controllers used new rather than build on associations. 6be51af351c0f02d03a3cd58ce5ea6500a59f2be Paul Horsfall horsfallp@gmail.com