josevalim-inherited_resources 0.1 → 0.1.1

data/CHANGELOG

@@ -1,4 +1,8 @@
-Version 0.1
+# Version 0.1.1
+
+* Added Rails 2.3.0 and changed tests to work with ActionController::TestCase.
+
+# Version 0.1.0
 
 * Initial release. Support to I18n, singleton controllers, polymorphic
 controllers, belongs_to, nested_belongs_to and url helpers.

data/README

@@ -1,6 +1,6 @@
 Inherited Resources
 License: MIT
-Version: 0.1
+Version: 0.1.1
 
 You can also read this README in pretty html at the GitHub project Wiki page:
 
@@ -13,6 +13,11 @@ Inherited Resources speeds up development by making your controllers inherit
 all restful actions so you just have to focus on what is important. It makes
 your controllers more powerful and cleaner at the same time.
 
+Plus, making your controllers follow a pattern, it helps you to write better
+code by following fat models and skinny controllers convention.
+
+Inherited Resources is tested and compatible with Rails 2.2.2 and Rails 2.3.0.
+
 keywords: resources, controller, singleton, belongs_to, polymorphic and I18n
 
 Installation
@@ -258,12 +263,11 @@ practices says that you should never nest more than two resources, but sometimes
 you have to for security reasons. So this is an example of how you can do it:
 
    class CommentsController < InheritedResources::Base
-     nested_belongs_to :project
-     nested_belongs_to :task
+     nested_belongs_to :project, :task
    end
 
-nested_belongs_to is actually just an alias to belongs_to. This is an
-alternative declaration:
+nested_belongs_to is actually just an alias to belongs_to created to make clear
+what is happening. You can also declare nested_belongs_to like this:
 
    class CommentsController < InheritedResources::Base
      belongs_to :project do

data/lib/inherited_resources/respond_to.rb

@@ -255,6 +255,18 @@ module ActionController #:nodoc:
         head :not_acceptable
         return false
       end
+
+    private
+
+      # Define template_exists? for Rails 2.3
+      unless ActionController::Base.private_instance_methods.include? 'template_exists?'
+        def template_exists?
+          self.view_paths.find_template("#{controller_name}/#{action_name}", response.template.template_format)
+        rescue ActionView::MissingTemplate
+          false
+        end
+      end
+
   end
 
   module MimeResponds #:nodoc:

data/test/aliases_test.rb

@@ -19,7 +19,7 @@ class StudentsController < InheritedResources::Base
 
 end
 
-class AliasesBaseTest < Test::Unit::TestCase
+class AliasesBaseTest < TEST_CLASS
 
   def setup
     @controller          = StudentsController.new

data/test/base_helpers_test.rb

@@ -11,7 +11,7 @@ class AddressesController < InheritedResources::Base
     end
 end
 
-class FlashBaseHelpersTest < Test::Unit::TestCase
+class FlashBaseHelpersTest < TEST_CLASS
 
   def setup
     @controller          = AddressesController.new
@@ -75,7 +75,7 @@ class PetsController < InheritedResources::Base
     end
 end
 
-class AssociationChainBaseHelpersTest < Test::Unit::TestCase
+class AssociationChainBaseHelpersTest < TEST_CLASS
 
   def setup
     @controller              = PetsController.new

data/test/base_test.rb

@@ -24,7 +24,7 @@ module UserTestHelper
     end
 end
 
-class IndexActionBaseTest < Test::Unit::TestCase
+class IndexActionBaseTest < TEST_CLASS
   include UserTestHelper
 
   def test_expose_all_users_as_instance_variable
@@ -50,7 +50,7 @@ class IndexActionBaseTest < Test::Unit::TestCase
   end
 end
 
-class ShowActionBaseTest < Test::Unit::TestCase
+class ShowActionBaseTest < TEST_CLASS
   include UserTestHelper
 
   def test_expose_the_resquested_user
@@ -76,7 +76,7 @@ class ShowActionBaseTest < Test::Unit::TestCase
   end
 end
 
-class NewActionBaseTest < Test::Unit::TestCase
+class NewActionBaseTest < TEST_CLASS
   include UserTestHelper
 
   def test_expose_a_new_user
@@ -102,7 +102,7 @@ class NewActionBaseTest < Test::Unit::TestCase
   end
 end
 
-class EditActionBaseTest < Test::Unit::TestCase
+class EditActionBaseTest < TEST_CLASS
   include UserTestHelper
 
   def test_expose_the_resquested_user
@@ -120,7 +120,7 @@ class EditActionBaseTest < Test::Unit::TestCase
   end
 end
 
-class CreateActionBaseTest < Test::Unit::TestCase
+class CreateActionBaseTest < TEST_CLASS
   include UserTestHelper
 
   def test_expose_a_newly_create_user_when_saved_with_success
@@ -146,7 +146,7 @@ class CreateActionBaseTest < Test::Unit::TestCase
     User.stubs(:new).returns(mock_user(:save => false, :errors => []))
     post :create
     assert_response :success
-    assert_template 'new'
+    assert_template :new
   end
 
   def test_dont_show_flash_message_when_user_cannot_be_saved
@@ -156,7 +156,7 @@ class CreateActionBaseTest < Test::Unit::TestCase
   end
 end
 
-class UpdateActionBaseTest < Test::Unit::TestCase
+class UpdateActionBaseTest < TEST_CLASS
   include UserTestHelper
 
   def test_update_the_requested_object
@@ -183,7 +183,7 @@ class UpdateActionBaseTest < Test::Unit::TestCase
     User.stubs(:find).returns(mock_user(:update_attributes => false, :errors => []))
     put :update
     assert_response :success
-    assert_template 'edit'
+    assert_template :edit
   end
 
   def test_dont_show_flash_message_when_user_cannot_be_saved
@@ -193,7 +193,7 @@ class UpdateActionBaseTest < Test::Unit::TestCase
   end
 end
 
-class DestroyActionBaseTest < Test::Unit::TestCase
+class DestroyActionBaseTest < TEST_CLASS
   include UserTestHelper
   
   def test_the_resquested_user_is_destroyed

data/test/belongs_to_base_test.rb

@@ -32,7 +32,7 @@ module CommentTestHelper
     end
 end
 
-class IndexActionBelongsToTest < Test::Unit::TestCase
+class IndexActionBelongsToTest < TEST_CLASS
   include CommentTestHelper
 
   def test_expose_all_comments_as_instance_variable
@@ -64,7 +64,7 @@ class IndexActionBelongsToTest < Test::Unit::TestCase
   end
 end
 
-class ShowActionBelongsToTest < Test::Unit::TestCase
+class ShowActionBelongsToTest < TEST_CLASS
   include CommentTestHelper
 
   def test_expose_the_resquested_comment
@@ -96,7 +96,7 @@ class ShowActionBelongsToTest < Test::Unit::TestCase
   end
 end
 
-class NewActionBelongsToTest < Test::Unit::TestCase
+class NewActionBelongsToTest < TEST_CLASS
   include CommentTestHelper
 
   def test_expose_a_new_comment
@@ -128,7 +128,7 @@ class NewActionBelongsToTest < Test::Unit::TestCase
   end
 end
 
-class EditActionBelongsToTest < Test::Unit::TestCase
+class EditActionBelongsToTest < TEST_CLASS
   include CommentTestHelper
 
   def test_expose_the_resquested_comment
@@ -150,7 +150,7 @@ class EditActionBelongsToTest < Test::Unit::TestCase
   end
 end
 
-class CreateActionBelongsToTest < Test::Unit::TestCase
+class CreateActionBelongsToTest < TEST_CLASS
   include CommentTestHelper
 
   def test_expose_a_newly_create_comment_when_saved_with_success
@@ -182,7 +182,7 @@ class CreateActionBelongsToTest < Test::Unit::TestCase
     Comment.stubs(:build).returns(mock_comment(:save => false, :errors => []))
     post :create
     assert_response :success
-    assert_template 'new'
+    assert_template :new
   end
 
   def test_dont_show_flash_message_when_comment_cannot_be_saved
@@ -193,7 +193,7 @@ class CreateActionBelongsToTest < Test::Unit::TestCase
   end
 end
 
-class UpdateActionBelongsToTest < Test::Unit::TestCase
+class UpdateActionBelongsToTest < TEST_CLASS
   include CommentTestHelper
 
   def test_update_the_requested_object
@@ -226,7 +226,7 @@ class UpdateActionBelongsToTest < Test::Unit::TestCase
     Comment.stubs(:find).returns(mock_comment(:update_attributes => false, :errors => []))
     put :update
     assert_response :success
-    assert_template 'edit'
+    assert_template :edit
   end
 
   def test_dont_show_flash_message_when_comment_cannot_be_saved
@@ -237,7 +237,7 @@ class UpdateActionBelongsToTest < Test::Unit::TestCase
   end
 end
 
-class DestroyActionBelongsToTest < Test::Unit::TestCase
+class DestroyActionBelongsToTest < TEST_CLASS
   include CommentTestHelper
   
   def test_the_resquested_comment_is_destroyed

data/test/belongs_to_test.rb

@@ -19,7 +19,7 @@ class ProfessorsController < InheritedResources::Base
 end
 
 # Create a TestHelper module with some helpers
-class BelongsToTest < Test::Unit::TestCase
+class BelongsToTest < TEST_CLASS
   def setup
     @controller          = ProfessorsController.new
     @controller.request  = @request  = ActionController::TestRequest.new

data/test/class_methods_test.rb

@@ -1,41 +1,41 @@
-require File.dirname(__FILE__) + '/test_helper'
+require File.dirname(__FILE__) + '/test_helper'
 
 class Book; end
 class Folder; end
-
-class BooksController < InheritedResources::Base
+
+class BooksController < InheritedResources::Base
   actions :index, :show
-end
-
-class ReadersController < InheritedResources::Base
-  actions :all, :except => [ :edit, :update ]
+end
+
+class ReadersController < InheritedResources::Base
+  actions :all, :except => [ :edit, :update ]
 end
 
 class FoldersController < InheritedResources::Base
-end
-
-class ActionsTest < Test::Unit::TestCase
-  def test_actions_are_undefined_when_only_option_is_given
-    action_methods = BooksController.send(:action_methods)
-    assert_equal 2, action_methods.size
-
-    ['index', 'show'].each do |action|
-      assert action_methods.include? action
-    end
-  end
-
-  def test_actions_are_undefined_when_except_option_is_given
-    action_methods = ReadersController.send(:action_methods)
-    assert_equal 5, action_methods.size
-
-    ['index', 'new', 'show', 'create', 'destroy'].each do |action|
-      assert action_methods.include? action
-    end
+end
+
+class ActionsTest < ActiveSupport::TestCase
+  def test_actions_are_undefined_when_only_option_is_given
+    action_methods = BooksController.send(:action_methods)
+    assert_equal 2, action_methods.size
+
+    ['index', 'show'].each do |action|
+      assert action_methods.include? action
+    end
+  end
+
+  def test_actions_are_undefined_when_except_option_is_given
+    action_methods = ReadersController.send(:action_methods)
+    assert_equal 5, action_methods.size
+
+    ['index', 'new', 'show', 'create', 'destroy'].each do |action|
+      assert action_methods.include? action
+    end
   end
 end
 
-class DefaultsTest < Test::Unit::TestCase
-
  def test_resource_class_is_set_to_nil_when_resource_model_cannot_be_found
+class DefaultsTest < ActiveSupport::TestCase
+  def test_resource_class_is_set_to_nil_when_resource_model_cannot_be_found
     assert_nil ReadersController.send(:resource_class)
   end
 

data/test/nested_belongs_to_test.rb

@@ -15,7 +15,7 @@ class CitiesController < InheritedResources::Base
 end
 
 # Create a TestHelper module with some helpers
-class NestedBelongsToTest < Test::Unit::TestCase
+class NestedBelongsToTest < TEST_CLASS
 
   def setup
     @controller          = CitiesController.new

data/test/polymorphic_base_test.rb

@@ -1,282 +1,282 @@
-require File.dirname(__FILE__) + '/test_helper'
-
+require File.dirname(__FILE__) + '/test_helper'
+
 class Factory; end
-class Company; end
-
-class Employee
-  def self.human_name; 'Employee'; end
-end
-
-class EmployeesController < InheritedResources::Base
-  belongs_to :factory, :company, :polymorphic => true
-end
-
-# Create a TestHelper module with some helpers
-module EmployeeTestHelper
-  def setup
-    @controller          = EmployeesController.new
-    @controller.request  = @request  = ActionController::TestRequest.new
-    @controller.response = @response = ActionController::TestResponse.new
-  end
-
-  protected
-    def mock_factory(stubs={})
-      @mock_factory ||= mock(stubs)
-    end
-
-    def mock_employee(stubs={})
-      @mock_employee ||= mock(stubs)
-    end
-end
-
-class IndexActionPolymorphicTest < Test::Unit::TestCase
-  include EmployeeTestHelper
-
-  def test_expose_all_employees_as_instance_variable
-    Factory.expects(:find).with('37').returns(mock_factory)
-    mock_factory.expects(:employees).returns(Employee)
-    Employee.expects(:find).with(:all).returns([mock_employee])
-    get :index, :factory_id => '37'
-    assert_equal mock_factory, assigns(:factory)
-    assert_equal [mock_employee], assigns(:employees)
-  end
-
-  def test_controller_should_render_index
-    Factory.stubs(:find).returns(mock_factory(:employees => Employee))
-    Employee.stubs(:find).returns([mock_employee])
-    get :index, :factory_id => '37'
-    assert_response :success
-    assert_equal 'Index HTML', @response.body.strip
-  end
-
-  def test_render_all_employees_as_xml_when_mime_type_is_xml
-    @request.accept = 'application/xml'
-    Factory.expects(:find).with('37').returns(mock_factory)
-    mock_factory.expects(:employees).returns(Employee)
-    Employee.expects(:find).with(:all).returns(mock_employee)
-    mock_employee.expects(:to_xml).returns('Generated XML')
-    get :index, :factory_id => '37'
-    assert_response :success
-    assert_equal 'Generated XML', @response.body
-  end
-end
-
-class ShowActionPolymorphicTest < Test::Unit::TestCase
-  include EmployeeTestHelper
-
-  def test_expose_the_resquested_employee
-    Factory.expects(:find).with('37').returns(mock_factory)
-    mock_factory.expects(:employees).returns(Employee)
-    Employee.expects(:find).with('42').returns(mock_employee)
-    get :show, :id => '42', :factory_id => '37'
-    assert_equal mock_factory, assigns(:factory)
-    assert_equal mock_employee, assigns(:employee)
-  end
-
-  def test_controller_should_render_show
-    Factory.stubs(:find).returns(mock_factory(:employees => Employee))
-    Employee.stubs(:find).returns(mock_employee)
-    get :show, :factory_id => '37'
-    assert_response :success
-    assert_equal 'Show HTML', @response.body.strip
-  end
-
-  def test_render_exposed_employee_as_xml_when_mime_type_is_xml
-    @request.accept = 'application/xml'
-    Factory.expects(:find).with('37').returns(mock_factory)
-    mock_factory.expects(:employees).returns(Employee)
-    Employee.expects(:find).with('42').returns(mock_employee)
-    mock_employee.expects(:to_xml).returns("Generated XML")
-    get :show, :id => '42', :factory_id => '37'
-    assert_response :success
-    assert_equal 'Generated XML', @response.body
-  end
-end
-
-class NewActionPolymorphicTest < Test::Unit::TestCase
-  include EmployeeTestHelper
-
-  def test_expose_a_new_employee
-    Factory.expects(:find).with('37').returns(mock_factory)
-    mock_factory.expects(:employees).returns(Employee)
-    Employee.expects(:build).returns(mock_employee)
-    get :new, :factory_id => '37'
-    assert_equal mock_factory, assigns(:factory)
-    assert_equal mock_employee, assigns(:employee)
-  end
-
-  def test_controller_should_render_new
-    Factory.stubs(:find).returns(mock_factory(:employees => Employee))
-    Employee.stubs(:build).returns(mock_employee)
-    get :new, :factory_id => '37'
-    assert_response :success
-    assert_equal 'New HTML', @response.body.strip
-  end
-
-  def test_render_exposed_a_new_employee_as_xml_when_mime_type_is_xml
-    @request.accept = 'application/xml'
-    Factory.expects(:find).with('37').returns(mock_factory)
-    mock_factory.expects(:employees).returns(Employee)
-    Employee.expects(:build).returns(mock_employee)
-    mock_employee.expects(:to_xml).returns("Generated XML")
-    get :new, :factory_id => '37'
-    assert_equal 'Generated XML', @response.body
-    assert_response :success
-  end
-end
-
-class EditActionPolymorphicTest < Test::Unit::TestCase
-  include EmployeeTestHelper
-
-  def test_expose_the_resquested_employee
-    Factory.expects(:find).with('37').returns(mock_factory)
-    mock_factory.expects(:employees).returns(Employee)
-    Employee.expects(:find).with('42').returns(mock_employee)
-    get :edit, :id => '42', :factory_id => '37'
-    assert_equal mock_factory, assigns(:factory)
-    assert_equal mock_employee, assigns(:employee)
-    assert_response :success
-  end
-
-  def test_controller_should_render_edit
-    Factory.stubs(:find).returns(mock_factory(:employees => Employee))
-    Employee.stubs(:find).returns(mock_employee)
-    get :edit, :factory_id => '37'
-    assert_response :success
-    assert_equal 'Edit HTML', @response.body.strip
-  end
-end
-
-class CreateActionPolymorphicTest < Test::Unit::TestCase
-  include EmployeeTestHelper
-
-  def test_expose_a_newly_create_employee_when_saved_with_success
-    Factory.expects(:find).with('37').returns(mock_factory)
-    mock_factory.expects(:employees).returns(Employee)
-    Employee.expects(:build).with({'these' => 'params'}).returns(mock_employee(:save => true))
-    post :create, :factory_id => '37', :employee => {:these => 'params'}
-    assert_equal mock_factory, assigns(:factory)
-    assert_equal mock_employee, assigns(:employee)
-  end
-
-  def test_redirect_to_the_created_employee
-    Factory.stubs(:find).returns(mock_factory(:employees => Employee))
-    Employee.stubs(:build).returns(mock_employee(:save => true))
-    @controller.expects(:resource_url).returns('http://test.host/').times(2)
-    post :create, :factory_id => '37'
-    assert_redirected_to 'http://test.host/'
-  end
-
-  def test_show_flash_message_when_success
-    Factory.stubs(:find).returns(mock_factory(:employees => Employee))
-    Employee.stubs(:build).returns(mock_employee(:save => true))
-    post :create, :factory_id => '37'
-    assert_equal flash[:notice], 'Employee was successfully created.'
-  end
-
-  def test_render_new_template_when_employee_cannot_be_saved
-    Factory.stubs(:find).returns(mock_factory(:employees => Employee))
-    Employee.stubs(:build).returns(mock_employee(:save => false, :errors => []))
-    post :create, :factory_id => '37'
-    assert_response :success
-    assert_template 'new'
-  end
-
-  def test_dont_show_flash_message_when_employee_cannot_be_saved
-    Factory.stubs(:find).returns(mock_factory(:employees => Employee))
-    Employee.stubs(:build).returns(mock_employee(:save => false, :errors => []))
-    post :create, :factory_id => '37'
-    assert flash.empty?
-  end
-end
-
-class UpdateActionPolymorphicTest < Test::Unit::TestCase
-  include EmployeeTestHelper
-
-  def test_update_the_requested_object
-    Factory.expects(:find).with('37').returns(mock_factory)
-    mock_factory.expects(:employees).returns(Employee)
-    Employee.expects(:find).with('42').returns(mock_employee)
-    mock_employee.expects(:update_attributes).with({'these' => 'params'}).returns(true)
-    put :update, :id => '42', :factory_id => '37', :employee => {:these => 'params'}
-    assert_equal mock_factory, assigns(:factory)
-    assert_equal mock_employee, assigns(:employee)
-  end
-
-  def test_redirect_to_the_created_employee
-    Factory.stubs(:find).returns(mock_factory(:employees => Employee))
-    Employee.stubs(:find).returns(mock_employee(:update_attributes => true))
-    @controller.expects(:resource_url).returns('http://test.host/')
-    put :update, :factory_id => '37'
-    assert_redirected_to 'http://test.host/'
-  end
-
-  def test_show_flash_message_when_success
-    Factory.stubs(:find).returns(mock_factory(:employees => Employee))
-    Employee.stubs(:find).returns(mock_employee(:update_attributes => true))
-    put :update, :factory_id => '37'
-    assert_equal flash[:notice], 'Employee was successfully updated.'
-  end
-
-  def test_render_edit_template_when_employee_cannot_be_saved
-    Factory.stubs(:find).returns(mock_factory(:employees => Employee))
-    Employee.stubs(:find).returns(mock_employee(:update_attributes => false, :errors => []))
-    put :update, :factory_id => '37'
-    assert_response :success
-    assert_template 'edit'
-  end
-
-  def test_dont_show_flash_message_when_employee_cannot_be_saved
-    Factory.stubs(:find).returns(mock_factory(:employees => Employee))
-    Employee.stubs(:find).returns(mock_employee(:update_attributes => false, :errors => []))
-    put :update, :factory_id => '37'
-    assert flash.empty?
-  end
-end
-
-class DestroyActionPolymorphicTest < Test::Unit::TestCase
-  include EmployeeTestHelper
-  
-  def test_the_resquested_employee_is_destroyed
-    Factory.expects(:find).with('37').returns(mock_factory)
-    mock_factory.expects(:employees).returns(Employee)
-    Employee.expects(:find).with('42').returns(mock_employee)
-    mock_employee.expects(:destroy)
-    delete :destroy, :id => '42', :factory_id => '37'
-    assert_equal mock_factory, assigns(:factory)
-    assert_equal mock_employee, assigns(:employee)
-  end
-
-  def test_show_flash_message
-    Factory.stubs(:find).returns(mock_factory(:employees => Employee))
-    Employee.stubs(:find).returns(mock_employee(:destroy => true))
-    delete :destroy, :factory_id => '37'
-    assert_equal flash[:notice], 'Employee was successfully destroyed.'
-  end
-
-  def test_redirects_to_employees_list
-    Factory.stubs(:find).returns(mock_factory(:employees => Employee))
-    Employee.stubs(:find).returns(mock_employee(:destroy => true))
-    @controller.expects(:collection_url).returns('http://test.host/')
-    delete :destroy, :factory_id => '37'
-    assert_redirected_to 'http://test.host/'
-  end
-end
-
-  include EmployeeTestHelper
-
+class Company; end
+
+class Employee
+  def self.human_name; 'Employee'; end
+end
+
+class EmployeesController < InheritedResources::Base
+  belongs_to :factory, :company, :polymorphic => true
+end
+
+# Create a TestHelper module with some helpers
+module EmployeeTestHelper
+  def setup
+    @controller          = EmployeesController.new
+    @controller.request  = @request  = ActionController::TestRequest.new
+    @controller.response = @response = ActionController::TestResponse.new
+  end
+
+  protected
+    def mock_factory(stubs={})
+      @mock_factory ||= mock(stubs)
+    end
+
+    def mock_employee(stubs={})
+      @mock_employee ||= mock(stubs)
+    end
+end
+
+class IndexActionPolymorphicTest < TEST_CLASS
+  include EmployeeTestHelper
+
+  def test_expose_all_employees_as_instance_variable
+    Factory.expects(:find).with('37').returns(mock_factory)
+    mock_factory.expects(:employees).returns(Employee)
+    Employee.expects(:find).with(:all).returns([mock_employee])
+    get :index, :factory_id => '37'
+    assert_equal mock_factory, assigns(:factory)
+    assert_equal [mock_employee], assigns(:employees)
+  end
+
+  def test_controller_should_render_index
+    Factory.stubs(:find).returns(mock_factory(:employees => Employee))
+    Employee.stubs(:find).returns([mock_employee])
+    get :index, :factory_id => '37'
+    assert_response :success
+    assert_equal 'Index HTML', @response.body.strip
+  end
+
+  def test_render_all_employees_as_xml_when_mime_type_is_xml
+    @request.accept = 'application/xml'
+    Factory.expects(:find).with('37').returns(mock_factory)
+    mock_factory.expects(:employees).returns(Employee)
+    Employee.expects(:find).with(:all).returns(mock_employee)
+    mock_employee.expects(:to_xml).returns('Generated XML')
+    get :index, :factory_id => '37'
+    assert_response :success
+    assert_equal 'Generated XML', @response.body
+  end
+end
+
+class ShowActionPolymorphicTest < TEST_CLASS
+  include EmployeeTestHelper
+
+  def test_expose_the_resquested_employee
+    Factory.expects(:find).with('37').returns(mock_factory)
+    mock_factory.expects(:employees).returns(Employee)
+    Employee.expects(:find).with('42').returns(mock_employee)
+    get :show, :id => '42', :factory_id => '37'
+    assert_equal mock_factory, assigns(:factory)
+    assert_equal mock_employee, assigns(:employee)
+  end
+
+  def test_controller_should_render_show
+    Factory.stubs(:find).returns(mock_factory(:employees => Employee))
+    Employee.stubs(:find).returns(mock_employee)
+    get :show, :factory_id => '37'
+    assert_response :success
+    assert_equal 'Show HTML', @response.body.strip
+  end
+
+  def test_render_exposed_employee_as_xml_when_mime_type_is_xml
+    @request.accept = 'application/xml'
+    Factory.expects(:find).with('37').returns(mock_factory)
+    mock_factory.expects(:employees).returns(Employee)
+    Employee.expects(:find).with('42').returns(mock_employee)
+    mock_employee.expects(:to_xml).returns("Generated XML")
+    get :show, :id => '42', :factory_id => '37'
+    assert_response :success
+    assert_equal 'Generated XML', @response.body
+  end
+end
+
+class NewActionPolymorphicTest < TEST_CLASS
+  include EmployeeTestHelper
+
+  def test_expose_a_new_employee
+    Factory.expects(:find).with('37').returns(mock_factory)
+    mock_factory.expects(:employees).returns(Employee)
+    Employee.expects(:build).returns(mock_employee)
+    get :new, :factory_id => '37'
+    assert_equal mock_factory, assigns(:factory)
+    assert_equal mock_employee, assigns(:employee)
+  end
+
+  def test_controller_should_render_new
+    Factory.stubs(:find).returns(mock_factory(:employees => Employee))
+    Employee.stubs(:build).returns(mock_employee)
+    get :new, :factory_id => '37'
+    assert_response :success
+    assert_equal 'New HTML', @response.body.strip
+  end
+
+  def test_render_exposed_a_new_employee_as_xml_when_mime_type_is_xml
+    @request.accept = 'application/xml'
+    Factory.expects(:find).with('37').returns(mock_factory)
+    mock_factory.expects(:employees).returns(Employee)
+    Employee.expects(:build).returns(mock_employee)
+    mock_employee.expects(:to_xml).returns("Generated XML")
+    get :new, :factory_id => '37'
+    assert_equal 'Generated XML', @response.body
+    assert_response :success
+  end
+end
+
+class EditActionPolymorphicTest < TEST_CLASS
+  include EmployeeTestHelper
+
+  def test_expose_the_resquested_employee
+    Factory.expects(:find).with('37').returns(mock_factory)
+    mock_factory.expects(:employees).returns(Employee)
+    Employee.expects(:find).with('42').returns(mock_employee)
+    get :edit, :id => '42', :factory_id => '37'
+    assert_equal mock_factory, assigns(:factory)
+    assert_equal mock_employee, assigns(:employee)
+    assert_response :success
+  end
+
+  def test_controller_should_render_edit
+    Factory.stubs(:find).returns(mock_factory(:employees => Employee))
+    Employee.stubs(:find).returns(mock_employee)
+    get :edit, :factory_id => '37'
+    assert_response :success
+    assert_equal 'Edit HTML', @response.body.strip
+  end
+end
+
+class CreateActionPolymorphicTest < TEST_CLASS
+  include EmployeeTestHelper
+
+  def test_expose_a_newly_create_employee_when_saved_with_success
+    Factory.expects(:find).with('37').returns(mock_factory)
+    mock_factory.expects(:employees).returns(Employee)
+    Employee.expects(:build).with({'these' => 'params'}).returns(mock_employee(:save => true))
+    post :create, :factory_id => '37', :employee => {:these => 'params'}
+    assert_equal mock_factory, assigns(:factory)
+    assert_equal mock_employee, assigns(:employee)
+  end
+
+  def test_redirect_to_the_created_employee
+    Factory.stubs(:find).returns(mock_factory(:employees => Employee))
+    Employee.stubs(:build).returns(mock_employee(:save => true))
+    @controller.expects(:resource_url).returns('http://test.host/').times(2)
+    post :create, :factory_id => '37'
+    assert_redirected_to 'http://test.host/'
+  end
+
+  def test_show_flash_message_when_success
+    Factory.stubs(:find).returns(mock_factory(:employees => Employee))
+    Employee.stubs(:build).returns(mock_employee(:save => true))
+    post :create, :factory_id => '37'
+    assert_equal flash[:notice], 'Employee was successfully created.'
+  end
+
+  def test_render_new_template_when_employee_cannot_be_saved
+    Factory.stubs(:find).returns(mock_factory(:employees => Employee))
+    Employee.stubs(:build).returns(mock_employee(:save => false, :errors => []))
+    post :create, :factory_id => '37'
+    assert_response :success
+    assert_template :new
+  end
+
+  def test_dont_show_flash_message_when_employee_cannot_be_saved
+    Factory.stubs(:find).returns(mock_factory(:employees => Employee))
+    Employee.stubs(:build).returns(mock_employee(:save => false, :errors => []))
+    post :create, :factory_id => '37'
+    assert flash.empty?
+  end
+end
+
+class UpdateActionPolymorphicTest < TEST_CLASS
+  include EmployeeTestHelper
+
+  def test_update_the_requested_object
+    Factory.expects(:find).with('37').returns(mock_factory)
+    mock_factory.expects(:employees).returns(Employee)
+    Employee.expects(:find).with('42').returns(mock_employee)
+    mock_employee.expects(:update_attributes).with({'these' => 'params'}).returns(true)
+    put :update, :id => '42', :factory_id => '37', :employee => {:these => 'params'}
+    assert_equal mock_factory, assigns(:factory)
+    assert_equal mock_employee, assigns(:employee)
+  end
+
+  def test_redirect_to_the_created_employee
+    Factory.stubs(:find).returns(mock_factory(:employees => Employee))
+    Employee.stubs(:find).returns(mock_employee(:update_attributes => true))
+    @controller.expects(:resource_url).returns('http://test.host/')
+    put :update, :factory_id => '37'
+    assert_redirected_to 'http://test.host/'
+  end
+
+  def test_show_flash_message_when_success
+    Factory.stubs(:find).returns(mock_factory(:employees => Employee))
+    Employee.stubs(:find).returns(mock_employee(:update_attributes => true))
+    put :update, :factory_id => '37'
+    assert_equal flash[:notice], 'Employee was successfully updated.'
+  end
+
+  def test_render_edit_template_when_employee_cannot_be_saved
+    Factory.stubs(:find).returns(mock_factory(:employees => Employee))
+    Employee.stubs(:find).returns(mock_employee(:update_attributes => false, :errors => []))
+    put :update, :factory_id => '37'
+    assert_response :success
+    assert_template :edit
+  end
+
+  def test_dont_show_flash_message_when_employee_cannot_be_saved
+    Factory.stubs(:find).returns(mock_factory(:employees => Employee))
+    Employee.stubs(:find).returns(mock_employee(:update_attributes => false, :errors => []))
+    put :update, :factory_id => '37'
+    assert flash.empty?
+  end
+end
+
+class DestroyActionPolymorphicTest < TEST_CLASS
+  include EmployeeTestHelper
+
+  def test_the_resquested_employee_is_destroyed
+    Factory.expects(:find).with('37').returns(mock_factory)
+    mock_factory.expects(:employees).returns(Employee)
+    Employee.expects(:find).with('42').returns(mock_employee)
+    mock_employee.expects(:destroy)
+    delete :destroy, :id => '42', :factory_id => '37'
+    assert_equal mock_factory, assigns(:factory)
+    assert_equal mock_employee, assigns(:employee)
+  end
+
+  def test_show_flash_message
+    Factory.stubs(:find).returns(mock_factory(:employees => Employee))
+    Employee.stubs(:find).returns(mock_employee(:destroy => true))
+    delete :destroy, :factory_id => '37'
+    assert_equal flash[:notice], 'Employee was successfully destroyed.'
+  end
+
+  def test_redirects_to_employees_list
+    Factory.stubs(:find).returns(mock_factory(:employees => Employee))
+    Employee.stubs(:find).returns(mock_employee(:destroy => true))
+    @controller.expects(:collection_url).returns('http://test.host/')
+    delete :destroy, :factory_id => '37'
+    assert_redirected_to 'http://test.host/'
+  end
+end
+class PolymorphicHelpersTest < TEST_CLASS
+  include EmployeeTestHelper
+
   def test_polymorphic_helpers
-    new_factory = Factory.new
-    Factory.expects(:find).with('37').returns(new_factory)
-    new_factory.expects(:employees).returns(Employee)
-    Employee.expects(:find).with(:all).returns([mock_employee])
+    new_factory = Factory.new
+    Factory.expects(:find).with('37').returns(new_factory)
+    new_factory.expects(:employees).returns(Employee)
+    Employee.expects(:find).with(:all).returns([mock_employee])
     get :index, :factory_id => '37'
-
+
     assert @controller.send(:parent?)
     assert_equal :factory, assigns(:parent_type)
     assert_equal :factory, @controller.send(:parent_type)
     assert_equal Factory, @controller.send(:parent_class)
     assert_equal new_factory, assigns(:factory)
-    assert_equal new_factory, @controller.send(:parent_instance)
+    assert_equal new_factory, @controller.send(:parent_instance)
   end
 end

data/test/respond_to_test.rb

@@ -51,7 +51,7 @@ end
 class SuperProjectsController < ProjectsController
 end
 
-class RespondToUnitTest < Test::Unit::TestCase
+class RespondToUnitTest < TEST_CLASS
   def setup(class_controller = ProjectsController)
     @controller          = class_controller.new
     @controller.request  = @request  = ActionController::TestRequest.new
@@ -172,7 +172,7 @@ class RespondToUnitTest < Test::Unit::TestCase
     end
 end
 
-class RespondToFunctionalTest < Test::Unit::TestCase
+class RespondToFunctionalTest < TEST_CLASS
   def setup
     @controller          = ProjectsController.new
     @controller.request  = @request  = ActionController::TestRequest.new

data/test/singleton_base_test.rb

@@ -32,7 +32,7 @@ module ManagerTestHelper
     end
 end
 
-class ShowActionSingletonTest < Test::Unit::TestCase
+class ShowActionSingletonTest < TEST_CLASS
   include ManagerTestHelper
 
   def test_expose_the_resquested_manager
@@ -61,7 +61,7 @@ class ShowActionSingletonTest < Test::Unit::TestCase
   end
 end
 
-class NewActionSingletonTest < Test::Unit::TestCase
+class NewActionSingletonTest < TEST_CLASS
   include ManagerTestHelper
 
   def test_expose_a_new_manager
@@ -91,7 +91,7 @@ class NewActionSingletonTest < Test::Unit::TestCase
   end
 end
 
-class EditActionSingletonTest < Test::Unit::TestCase
+class EditActionSingletonTest < TEST_CLASS
   include ManagerTestHelper
 
   def test_expose_the_resquested_manager
@@ -112,7 +112,7 @@ class EditActionSingletonTest < Test::Unit::TestCase
   end
 end
 
-class CreateActionSingletonTest < Test::Unit::TestCase
+class CreateActionSingletonTest < TEST_CLASS
   include ManagerTestHelper
 
   def test_expose_a_newly_create_manager_when_saved_with_success
@@ -143,7 +143,7 @@ class CreateActionSingletonTest < Test::Unit::TestCase
     mock_store.stubs(:build_manager).returns(mock_manager(:save => false, :errors => []))
     post :create
     assert_response :success
-    assert_template 'new'
+    assert_template :new
   end
 
   def test_dont_show_flash_message_when_manager_cannot_be_saved
@@ -154,7 +154,7 @@ class CreateActionSingletonTest < Test::Unit::TestCase
   end
 end
 
-class UpdateActionSingletonTest < Test::Unit::TestCase
+class UpdateActionSingletonTest < TEST_CLASS
   include ManagerTestHelper
 
   def test_update_the_requested_object
@@ -185,7 +185,7 @@ class UpdateActionSingletonTest < Test::Unit::TestCase
     mock_manager.stubs(:update_attributes).returns(false)
     put :update
     assert_response :success
-    assert_template 'edit'
+    assert_template :edit
   end
 
   def test_dont_show_flash_message_when_manager_cannot_be_saved
@@ -196,7 +196,7 @@ class UpdateActionSingletonTest < Test::Unit::TestCase
   end
 end
 
-class DestroyActionSingletonTest < Test::Unit::TestCase
+class DestroyActionSingletonTest < TEST_CLASS
   include ManagerTestHelper
 
   def test_the_resquested_manager_is_destroyed

data/test/test_helper.rb

@@ -4,11 +4,18 @@ require 'mocha'
 
 ENV["RAILS_ENV"] = "test"
 
+# Used to tests Rails 2.2.2 version
+# gem 'activesupport', '2.2.2'
+# gem 'actionpack', '2.2.2'
+# TEST_CLASS = Test::Unit::TestCase
+
 require 'active_support'
 require 'action_controller'
 require 'action_controller/test_case'
 require 'action_controller/test_process'
 
+TEST_CLASS = ActionController::TestCase
+
 I18n.load_path << File.join(File.dirname(__FILE__), 'fixtures', 'en.yml') 
 I18n.reload!
 

data/test/url_helpers_test.rb

@@ -45,7 +45,7 @@ class CentersController < InheritedResources::Base
 end
 
 # Create a TestHelper module with some helpers
-class UrlHelpersTest < Test::Unit::TestCase
+class UrlHelpersTest < ActiveSupport::TestCase
 
   def test_url_helpers_on_simple_inherited_resource
     controller = HousesController.new

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: josevalim-inherited_resources
 version: !ruby/object:Gem::Version 
-  version: "0.1"
+  version: 0.1.1
 platform: ruby
 authors: 
 - "Jos\xC3\xA9 Valim"