iain-pizza-generators 0.1.2 → 0.1.3

data/rails_generators/pizza_scaffold/templates/controller.rb

@@ -1,10 +1,4 @@
-class <%= plural_class_name %>Controller < ResourceController::Base
-
-  <%- if controller_actions.size < 3 -%>
-  actions <%= controller_actions.map(&:to_sym).map(&:inspect).join(', ') %>
-  <%- elsif controller_actions.size < 7  -%>
-  actions :all, :except => [ <%= not_used_actions.map(&:to_sym).map(&:inspect).join(', ') %> ]
-  <%- end -%>
+class <%= plural_class_name %>Controller < ApplicationController
 
   <%= controller_methods :actions %>
 

data/rails_generators/pizza_scaffold/templates/tests/rspec/actions/create.rb

@@ -1,15 +1,83 @@
-  describe "create action" do
+  describe "the create action" do
 
-    it "should render new template when model is invalid" do
-      <%= class_name %>.any_instance.stubs(:valid?).returns(false)
-      post :create
-      response.should render_template(:new)
+    before :each do
+      @<%= singular_name %> = <%= class_name %>.new
+      mock(<%= class_name %>).new("these" => "params") { @<%= singular_name %> }
     end
-    
-    it "should redirect when model is valid" do
-      <%= class_name %>.any_instance.stubs(:valid?).returns(true)
-      post :create
-      response.should redirect_to(<%= item_path_for_spec('url') %>)
+
+    context "with valid data" do
+
+      before :each do
+        mock(@<%= singular_name %>).save { true }
+        stub(@<%= singular_name %>).id { 1337 }
+        stub(@<%= singular_name %>).new_record? { false }
+      end
+
+      context "in html format" do
+
+        before :each do
+          post :create, :<%= singular_name %> => { "these" => "params" }
+        end
+
+        it { should route(:post, <%= plural_name %>_path).to(:action => :create) }
+        it { should set_the_flash.to("<%= class_name.titleize %> was successfully created.") }
+        it { should respond_with(:redirect) }
+        it { should redirect_to(<%= item_path('url') %>) }
+        it { should assign_to(:<%= singular_name %>).with(@<%= singular_name %>) }
+
+      end
+
+      context "in xml format" do
+
+        before :each do
+          mock(@<%= singular_name %>).to_xml { "Generated XML" }
+          post :create, :<%= singular_name %> => { "these" => "params" }, :format => "xml"
+        end
+
+        it { should route(:post, <%= plural_name %>_path(:format => :xml)).to(:action => :create, :format => :xml) }
+        it { should respond_with_content_type(:xml) }
+        it { should respond_with(:created) }
+        it { should assign_to(:<%= singular_name %>).with(@<%= singular_name %>) }
+        specify { response.location.should == <%= singular_name %>_url(@<%= singular_name %>) }
+        specify { response.should have_text("Generated XML") }
+
+      end
+
+    end
+
+    context "with invalid data" do
+
+      before :each do
+        mock(@<%= singular_name %>).save { false }
+      end
+
+      context "in html format" do
+
+        before :each do
+          post :create, :<%= singular_name %> => { "these" => "params" }
+        end
+
+        it { should respond_with(:success) }
+        it { should render_template(:new) }
+        it { should assign_to(:<%= singular_name %>).with(@<%= singular_name %>) }
+        it { should_not set_the_flash }
+
+      end
+
+      context "in xml format" do
+
+        before :each do
+          mock(@<%= singular_name %>.errors).to_xml { "XML errors" }
+          post :create, :<%= singular_name %> => { "these" => "params" }, :format => "xml"
+        end
+
+        it { should respond_with_content_type(:xml) }
+        it { should respond_with(:unprocessable_entity) }
+        it { should assign_to(:<%= singular_name %>).with(@<%= singular_name %>) }
+        specify { response.should have_text("XML errors") }
+
+      end
+
     end
 
   end

data/rails_generators/pizza_scaffold/templates/tests/rspec/actions/destroy.rb

@@ -1,10 +1,32 @@
-  describe "destroy action" do
+  describe "the destroy action" do
+
+    before :each do
+      @<%= singular_name %> = mock_record <%= class_name %>, :destroy => true
+    end
+
+    context "in html format" do
+
+      before :each do
+        delete :destroy, :id => @<%= singular_name %>.to_param
+      end
+
+      it { should respond_with(:redirect) }
+      it { should redirect_to(<%= plural_name %>_url) }
+      it { should assign_to(:<%= singular_name %>).with(@<%= singular_name %>) }
+      it { should_not set_the_flash }
+
+    end
+
+    context "in xml format" do
+
+      before :each do
+        delete :destroy, :id => @<%= singular_name %>.to_param, :format => "xml"
+      end
+
+      it { should respond_with_content_type(:xml) }
+      it { should respond_with(:ok) }
+      it { should assign_to(:<%= singular_name %>).with(@<%= singular_name %>) }
 
-    it "should destroy model and redirect to index action" do
-      <%= singular_name %> = <%= class_name %>.first
-      delete :destroy, :id => <%= singular_name %>
-      response.should redirect_to(<%= plural_name %>_url)
-      <%= class_name %>.exists?(<%= singular_name %>.id).should be_false
     end
 
   end

data/rails_generators/pizza_scaffold/templates/tests/rspec/actions/edit.rb

@@ -1,8 +1,16 @@
-  describe "edit action" do
+  describe "the edit page" do
 
-    it "should render edit template" do
-      get :edit, :id => <%= class_name %>.first
-      response.should render_template(:edit)
+    before :each do
+      @<%= singular_name %> = mock_record <%= class_name %>
+      get :edit, :id => @<%= singular_name %>.to_param
     end
 
+    it { should route(:get, edit_<%= singular_name %>_path(@<%= singular_name %>)).to(:action => :edit, :id => @<%= singular_name %>.to_param) }
+    it { should respond_with(:success) }
+    it { should render_template(:edit) }
+    it { should_not set_the_flash }
+    it { should assign_to(:<%= singular_name %>).with(@<%= singular_name %>) }
+    it { should have_form_to(<%= singular_name %>_path(@<%= singular_name %>)) }
+    it { should have_link_to(<%= plural_name %>_path) }
+
   end

data/rails_generators/pizza_scaffold/templates/tests/rspec/actions/index.rb

@@ -1,8 +1,33 @@
-  describe "index action" do
+  describe "the index page" do
 
-    it "should render index template" do
-      get :index
-      response.should render_template(:index)
+    before :each do
+      @<%= singular_name %> = mock_record <%= class_name %>
+      @<%= plural_name %> = [@<%= singular_name %>].paginate
+      mock(<%= class_name %>).paginate(:page => "100") { @<%= plural_name %> }
+    end
+
+    context "in html format" do
+      before(:each) { get :index, :page => "100" }
+      it { should route(:get, <%= plural_name %>_path).to(:action => :index) }
+      it { should respond_with(:success) }
+      it { should render_template(:index) }
+      it { should_not set_the_flash }
+      it { should assign_to(:<%= plural_name %>).with(@<%= plural_name %>) }
+      it { should have_link_to(new_<%= singular_name %>_path) }
+      it { should have_link_to(edit_<%= singular_name %>_path(@<%= singular_name %>)) }
+      it { should have_link_to(<%= singular_name %>_path(@<%= singular_name %>)) }
+    end
+
+    context "in xml format" do
+      before(:each) do
+        mock(@<%= plural_name %>).to_xml { "Generated XML" }
+        get :index, :page => "100", :format => "xml"
+      end
+      it { should route(:get, <%= plural_name %>_path(:format => :xml)).to(:action => :index, :format => :xml) }
+      it { should respond_with(:success) }
+      it { should respond_with_content_type(:xml) }
+      it { should assign_to(:<%= plural_name %>).with(@<%= plural_name %>) }
+      specify { response.should have_text("Generated XML") }
     end
 
   end

data/rails_generators/pizza_scaffold/templates/tests/rspec/actions/new.rb

@@ -1,8 +1,33 @@
-  describe "new action" do
+  describe "the new page" do
+
+    before :each do
+      @<%= singular_name %> = <%= class_name %>.new
+      mock(<%= class_name %>).new { @<%= singular_name %> }
+    end
+
+    context "in html format" do
+      before(:each) { get :new }
+      it { should route(:get, new_<%= singular_name %>_path).to(:action => :new) }
+      it { should respond_with(:success) }
+      it { should render_template(:new) }
+      it { should_not set_the_flash }
+      it { should assign_to(:<%= singular_name %>).with(@<%= singular_name %>) }
+      it { should have_form_to(<%= plural_name %>_path) }
+    end
+
+    context "in xml format" do
+
+      before :each do
+        mock(@<%= singular_name %>).to_xml { "Generated XML" }
+        get :new, :format => "xml"
+      end
+
+      it { should route(:get, new_<%= singular_name %>_path(:format => :xml)).to(:action => :new, :format => :xml) }
+      it { should respond_with(:success) }
+      it { should respond_with_content_type(:xml) }
+      it { should assign_to(:<%= singular_name %>).with(@<%= singular_name %>) }
+      specify { response.should have_text("Generated XML") }
 
-    it "should render new template" do
-      get :new
-      response.should render_template(:new)
     end
 
   end

data/rails_generators/pizza_scaffold/templates/tests/rspec/actions/show.rb

@@ -1,8 +1,30 @@
-  describe "show action" do
+  describe "the show page" do
 
-    it "should render show template" do
-      get :show, :id => <%= class_name %>.first
-      response.should render_template(:show)
+    before :each do
+      @<%= singular_name %> = mock_record <%= class_name %>
+    end
+
+    context "in html format" do
+      before(:each) { get :show, :id => @<%= singular_name %>.to_param }
+      it { should route(:get, <%= singular_name %>_path(@<%= singular_name %>)).to(:action => :show, :id => @<%= singular_name %>.to_param) }
+      it { should respond_with(:success) }
+      it { should render_template(:show) }
+      it { should_not set_the_flash }
+      it { should assign_to(:<%= singular_name %>).with(@<%= singular_name %>) }
+      it { should have_link_to(edit_<%= singular_name %>_path(@<%= singular_name %>)) }
+      it { should have_link_to(<%= plural_name %>_path) }
+    end
+
+    context "in xml format" do
+      before(:each) do
+        mock(@<%= singular_name %>).to_xml { "Generated XML" }
+        get :show, :id => @<%= singular_name %>.to_param, :format => "xml"
+      end
+      it { should route(:get, <%= singular_name %>_path(@<%= singular_name %>, :format => :xml)).to(:action => :show, :id => @<%= singular_name %>.to_param, :format => :xml) }
+      it { should respond_with(:success) }
+      it { should respond_with_content_type(:xml) }
+      it { should assign_to(:<%= singular_name %>).with(@<%= singular_name %>) }
+      specify { response.should have_text("Generated XML") }
     end
 
   end

data/rails_generators/pizza_scaffold/templates/tests/rspec/actions/update.rb

@@ -1,15 +1,77 @@
-  describe "update action" do
+  describe "the update action" do
 
-    it "should render edit template when model is invalid" do
-      <%= class_name %>.any_instance.stubs(:valid?).returns(false)
-      put :update, :id => <%= class_name %>.first
-      response.should render_template(:edit)
+    before :each do
+      @<%= singular_name %> = mock_record <%= class_name %>
     end
-    
-    it "should redirect when model is valid" do
-      <%= class_name %>.any_instance.stubs(:valid?).returns(true)
-      put :update, :id => <%= class_name %>.first
-      response.should redirect_to(<%= item_path_for_spec('url') %>)
+
+    context "with valid data" do
+
+      before :each do
+        mock(@<%= singular_name %>).update_attributes("these" => "params") { true }
+      end
+
+      context "in html format" do
+
+        before :each do
+          put :update, :id => @<%= singular_name %>.to_param, :<%= singular_name %> => { "these" => "params" }
+        end
+
+        it { should route(:put, <%= singular_name %>_path(@<%= singular_name %>)).to(:action => :update, :id => @<%= singular_name %>.to_param) }
+        it { should respond_with(:redirect) }
+        it { should redirect_to(<%= singular_name %>_url(@<%= singular_name %>)) }
+        it { should set_the_flash.to("<%= class_name.titleize %> was successfully updated.") }
+        it { should assign_to(:<%= singular_name %>).with(@<%= singular_name %>) }
+
+      end
+
+      context "in xml format" do
+
+        before :each do
+          put :update, :id => @<%= singular_name %>.to_param, :<%= singular_name %> => { "these" => "params" }, :format => "xml"
+        end
+
+        it { should route(:put, <%= singular_name %>_path(@<%= singular_name %>, :format => :xml)).to(:action => :update, :id => @<%= singular_name %>.to_param, :format => :xml) }
+        it { should respond_with_content_type(:xml) }
+        it { should respond_with(:ok) }
+        it { should assign_to(:<%= singular_name %>).with(@<%= singular_name %>) }
+      end
+
+    end
+
+    context "with invalid data" do
+
+      before :each do
+        mock(@<%= singular_name %>).update_attributes("these" => "params") { false }
+      end
+
+      context "in html format" do
+
+        before :each do
+          put :update, :id => @<%= singular_name %>.to_param, :<%= singular_name %> => { "these" => "params" }
+        end
+
+        it { should respond_with(:success) }
+        it { should render_template(:edit) }
+        it { should_not set_the_flash }
+        it { should assign_to(:<%= singular_name %>).with(@<%= singular_name %>) }
+        it { should have_form_to(<%= singular_name %>_path(@<%= singular_name %>)) }
+
+      end
+
+      context "in xml format" do
+
+        before :each do
+          mock(@<%= singular_name %>.errors).to_xml { "XML errors" }
+          put :update, :id => @<%= singular_name %>.to_param, :<%= singular_name %> => { "these" => "params" }, :format => "xml"
+        end
+
+        it { should respond_with_content_type(:xml) }
+        it { should respond_with(:unprocessable_entity) }
+        it { should assign_to(:<%= singular_name %>).with(@<%= singular_name %>) }
+        specify { response.should have_text("XML errors") }
+
+      end
+
     end
 
   end

data/rails_generators/pizza_scaffold/templates/tests/rspec/controller.rb

@@ -1,8 +1,8 @@
 require File.dirname(__FILE__) + '/../spec_helper'
- 
+
 describe <%= plural_class_name %>Controller do
-  fixtures :all
+
   integrate_views
-  
+
   <%= controller_methods 'tests/rspec/actions' %>
 end

data/rails_generators/pizza_scaffold/templates/views/haml/_form.html.haml

@@ -1,10 +1,9 @@
-- form_for @<%= singular_name %>, :url => form_url do |f|
-  = f.error_messages
-  - f.fieldset do
+- form_for @<%= singular_name %> do |f|
+
+  = f.fieldset do
+
     <%- for attribute in attributes -%>
     - f.field :<%= attribute.name %>
     <%- end -%>
     - f.field do
-      %label &nbsp;
       = f.submit t(:submit)
-      = cancel_link

data/rails_generators/pizza_scaffold/templates/views/haml/edit.html.haml

@@ -1,13 +1,11 @@
-- title t(:title, :model => <%= class_name %>.human_name)
-
-<%= render_form :object_path %>
+<%= render_form %>
 
 <%- if actions? :show, :index -%>
 %ol.actions
   <%- if action? :show -%>
-  %li.show= link_to t(:show), object_path
+  %li.show=  link_to t(:show), <%= item_path %>
   <%- end -%>
   <%- if action? :index -%>
-  %li.index= link_to t(:index), collection_path
+  %li.index= link_to t(:index), <%= plural_name %>_path
   <%- end -%>
 <%- end -%>

data/rails_generators/pizza_scaffold/templates/views/haml/index.html.haml

@@ -1,31 +1,32 @@
-- title t(:title, :model => <%= class_name %>.human_name)
-
 - table_for @<%= plural_name %> do |t|
+
   - t.head do
     <%- for attribute in attributes -%>
     - t.th :<%= attribute.name %>
     <%- end -%>
     - t.th t("actions"), :colspan => <%= table_link_count %>
+
   - unless will_paginate(@<%= plural_name %>).blank?
     - t.foot do
       - t.tr do
         - t.td will_paginate(@<%= plural_name %>), :colspan => t.width
+
   - t.body do |<%= singular_name %>|
     - t.tr do
       <%- for attribute in attributes -%>
       - t.td_h :<%= attribute.name %>
       <%- end -%>
       <%- if action? :show -%>
-      - t.td link_to(t(:show), object_path(<%= singular_name %>))
+      - t.td link_to(t(:show), <%= item_path %>)
       <%- end -%>
       <%- if action? :edit -%>
-      - t.td link_to(t(:edit), edit_object_path(<%= singular_name %>))
+      - t.td link_to(t(:edit), edit_<%= singular_name %>_path(<%= singular_name %>))
       <%- end -%>
       <%- if action? :destroy -%>
-      - t.td link_to(t(:destroy), object_path(<%= singular_name %>), :confirm => h(t(:confirm)), :method => :delete)
+      - t.td link_to(t(:destroy), <%= singular_name %>, :confirm => h(t(:confirm)), :method => :delete)
       <%- end -%>
 
 <%- if actions? :new -%>
 %ol.actions
-  %li.new= link_to t(:new), new_object_path
+  %li.new= link_to t(:new), new_<%= singular_name %>_path
   <%- end -%>

data/rails_generators/pizza_scaffold/templates/views/haml/new.html.haml

@@ -1,8 +1,6 @@
-- title t(:title, :model => <%= class_name %>.human_name)
-
-<%= render_form :collection_url %>
+<%= render_form %>
 
 <%- if action? :index -%>
 %ol.actions
-  %li.index= link_to t(:index), collection_path
+  %li.index= link_to t(:index), <%= plural_name %>_path
   <%- end -%>

data/rails_generators/pizza_scaffold/templates/views/haml/show.html.haml

@@ -1,5 +1,3 @@
-- title t(:title, :model => <%= class_name %>.human_name)
-
 - definition_list_for @<%= singular_name %> do |dl|
   <%- for attribute in attributes -%>
   - dl.show_h :<%= attribute.name %>
@@ -7,11 +5,11 @@
 
 %ol.actions
   <%- if action? :edit -%>
-  %li.edit= link_to t(:edit), edit_object_path
+  %li.edit=    link_to t(:edit), edit_<%= singular_name %>_path(@<%= singular_name %>)
   <%- end -%>
   <%- if action? :destroy -%>
-  %li.destroy= link_to t(:destroy), object_path, :confirm => h(t(:confirm)), :method => :delete
+  %li.destroy= link_to t(:destroy), @<%= singular_name %>, :confirm => h(t(:confirm)), :method => :delete
   <%- end -%>
   <%- if action? :index -%>
-  %li.index= link_to t(:index), collection_path
+  %li.index=   link_to t(:index), <%= plural_name %>_path
   <%- end -%>