giraffesoft-attribute_fu 0.2 → 0.2.1

data/README

@@ -105,6 +105,12 @@ Come join the discussion on the {mailing list}[link:http://groups.google.com/gro
 
 Updates will be available {here}[http://jamesgolick.com/attribute_fu]
 
+= Running the tests
+
+To run the tests, you need Shoulda, mocha and multi-rails:
+
+  $ sudo gem install thoughtbot-shoulda --source http://gems.github.com/
+  $ sudo gem install mocha multi_rails
 
 == Credits
 

data/Rakefile

@@ -1,4 +1,5 @@
 require 'rake'
+require "load_multi_rails_rake_tasks" 
 require 'rake/testtask'
 require 'rake/rdoctask'
 

data/lib/attribute_fu/associated_form_helper.rb

@@ -89,7 +89,7 @@ module AttributeFu
       
       @template.link_to_function(name, opts) do |page|
         page << "if (typeof #{variable} == 'undefined') #{variable} = 0;"
-        page << "new Insertion.Bottom(#{container}, new Template("+form_builder.render_associated_form(object, :fields_for => { :javascript => true }, :partial => partial).to_json+").evaluate({'number': --#{variable}}))"
+        page << "new Insertion.Bottom(#{container}, new Template("+form_builder.render_associated_form(object, :fields_for => { :javascript => true }, :partial => partial).to_json+").evaluate({'number': --#{variable}}).gsub(/__number_/, #{variable}))"
       end
     end
     

data/lib/attribute_fu/associations.rb

@@ -62,12 +62,14 @@ module AttributeFu
       def save_managed_associations #:nodoc:
         if managed_association_attributes != nil
           managed_association_attributes.keys.each do |association_id|
-            association = send(association_id)
-            association.each(&:save)
+            if send(association_id).loaded? # don't save what we haven't even loaded
+              association = send(association_id)
+              association.each(&:save)
 
-            unless (objects_to_remove = instance_variable_get removal_variable_name(association_id)).nil?
-              objects_to_remove.each { |remove_id| association.delete association.detect { |obj| obj.id.to_s == remove_id.to_s } }
-              instance_variable_set removal_variable_name(association_id), nil
+              unless (objects_to_remove = instance_variable_get removal_variable_name(association_id)).nil?
+                objects_to_remove.each { |remove_id| association.delete association.detect { |obj| obj.id.to_s == remove_id.to_s } }
+                instance_variable_set removal_variable_name(association_id), nil
+              end
             end
           end
         end
@@ -114,10 +116,20 @@ module AttributeFu
             discard_if = discard_if.to_proc if discard_if.is_a?(Symbol)
             managed_association_attributes[association_id][:discard_if] = discard_if
           end
+          collection_with_attributes_writer(association_id)
         end
         
         has_many_without_association_option(association_id, options, &extension)
       end
+      
+    private
+    
+      def collection_with_attributes_writer(association_name)
+        define_method("#{association_name.to_s.singularize}_attributes=") do |attributes|
+          has_many_attributes association_name, attributes
+        end
+      end
+      
     end
     
   end # Associations

data/test/db/schema.rb

@@ -1,29 +1,16 @@
-# This file is auto-generated from the current state of the database. Instead of editing this file, 
-# please use the migrations feature of ActiveRecord to incrementally modify your database, and
-# then regenerate this schema definition.
-#
-# Note that this schema.rb definition is the authoritative source for your database schema. If you need
-# to create the application database on another system, you should be using db:schema:load, not running
-# all the migrations from scratch. The latter is a flawed and unsustainable approach (the more migrations
-# you'll amass, the slower it'll run and the greater likelihood for issues).
-#
-# It's strongly recommended to check this file into your version control system.
-
 ActiveRecord::Schema.define(:version => 2) do
 
-  create_table "comments", :force => true do |t|
-    t.integer  "photo_id"
-    t.string   "author"
-    t.text     "body"
-    t.datetime "created_at"
-    t.datetime "updated_at"
+  create_table :comments, :force => true do |t|
+    t.integer :photo_id
+    t.string :author
+    t.text :body
+    t.timestamps
   end
 
-  create_table "photos", :force => true do |t|
-    t.string   "title"
-    t.text     "description"
-    t.datetime "created_at"
-    t.datetime "updated_at"
+  create_table :photos, :force => true do |t|
+    t.string :title
+    t.text   :description
+    t.timestamps
   end
 
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: giraffesoft-attribute_fu
 version: !ruby/object:Gem::Version 
-  version: "0.2"
+  version: 0.2.1
 platform: ruby
 authors: 
 - James Golick

data/test/Rakefile

@@ -1,10 +0,0 @@
-# Add your own tasks in files placed in lib/tasks ending in .rake,
-# for example lib/tasks/capistrano.rake, and they will automatically be available to Rake.
-
-require(File.join(File.dirname(__FILE__), 'config', 'boot'))
-
-require 'rake'
-require 'rake/testtask'
-require 'rake/rdoctask'
-
-require 'tasks/rails'

data/test/app/controllers/application.rb

@@ -1,10 +0,0 @@
-# Filters added to this controller apply to all controllers in the application.
-# Likewise, all the methods added will be available for all controllers.
-
-class ApplicationController < ActionController::Base
-  helper :all # include all helpers, all the time
-
-  # See ActionController::RequestForgeryProtection for details
-  # Uncomment the :secret if you're not using the cookie session store
-  protect_from_forgery # :secret => '012cbaf0c3d36504f3b1bc397b838d24'
-end

data/test/app/helpers/application_helper.rb

@@ -1,3 +0,0 @@
-# Methods added to this helper will be available to all templates in the application.
-module ApplicationHelper
-end

data/test/app/models/comment.rb

@@ -1,8 +0,0 @@
-class Comment < ActiveRecord::Base
-  belongs_to :photo
-  validates_presence_of :author, :body
-  
-  def blank?
-    author.blank? && body.blank?
-  end
-end

data/test/app/models/photo.rb

@@ -1,3 +0,0 @@
-class Photo < ActiveRecord::Base
-  has_many :comments, :attributes => true
-end

data/test/config/boot.rb

@@ -1,97 +0,0 @@
-# Don't change this file!
-# Configure your app in config/environment.rb and config/environments/*.rb
-
-RAILS_ROOT = "#{File.dirname(__FILE__)}/.." unless defined?(RAILS_ROOT)
-
-module Rails
-  class << self
-    def boot!
-      pick_boot.run unless booted?
-    end
-
-    def booted?
-      defined? Rails::Initializer
-    end
-
-    def pick_boot
-      (vendor_rails? ? VendorBoot : GemBoot).new
-    end
-
-    def vendor_rails?
-      File.exist?("#{RAILS_ROOT}/vendor/rails")
-    end
-  end
-
-  class Boot
-    def run
-      load_initializer
-      Rails::Initializer.run(:set_load_path)
-    end
-  end
-
-  class VendorBoot < Boot
-    def load_initializer
-      require "#{RAILS_ROOT}/vendor/rails/railties/lib/initializer"
-    end
-  end
-
-  class GemBoot < Boot
-    def load_initializer
-      self.class.load_rubygems
-      load_rails_gem
-      require 'initializer'
-    end
-
-    def load_rails_gem
-      if version = self.class.gem_version
-        gem 'rails', "=#{version}"
-      else
-        gem 'rails'
-      end
-    rescue Gem::LoadError => load_error
-      $stderr.puts %(Missing the Rails #{version} gem. Please `gem install -v=#{version} rails`, update your RAILS_GEM_VERSION setting in config/environment.rb for the Rails version you do have installed, or comment out RAILS_GEM_VERSION to use the latest version installed.)
-      exit 1
-    end
-
-    class << self
-      def rubygems_version
-        Gem::RubyGemsVersion if defined? Gem::RubyGemsVersion
-      end
-
-      def gem_version
-        if defined? RAILS_GEM_VERSION
-          RAILS_GEM_VERSION
-        elsif ENV.include?('RAILS_GEM_VERSION')
-          ENV['RAILS_GEM_VERSION']
-        else
-          parse_gem_version(read_environment_rb)
-        end
-      end
-
-      def load_rubygems
-        require 'rubygems'
-
-        unless rubygems_version >= '0.9.4'
-          $stderr.puts %(Rails requires RubyGems >= 0.9.4 (you have #{rubygems_version}). Please `gem update --system` and try again.)
-          exit 1
-        end
-
-      rescue LoadError
-        $stderr.puts %(Rails requires RubyGems >= 0.9.4. Please install RubyGems and try again: http://rubygems.rubyforge.org)
-        exit 1
-      end
-
-      def parse_gem_version(text)
-        $1 if text =~ /^[^#]*RAILS_GEM_VERSION\s*=\s*'([\d.]+)'/
-      end
-
-      private
-        def read_environment_rb
-          File.read("#{RAILS_ROOT}/config/environment.rb")
-        end
-    end
-  end
-end
-
-# All that for this:
-Rails.boot!

data/test/config/database.yml

@@ -1,15 +0,0 @@
-development:
-  adapter: mysql
-  encoding: utf8
-  database: attribute_fu_test
-  username: root
-  password:
-  socket: /tmp/mysql.sock
-  
-test:
-  adapter: mysql
-  encoding: utf8
-  database: attribute_fu_test
-  username: root
-  password:
-  socket: /tmp/mysql.sock

data/test/config/environment.rb

@@ -1,15 +0,0 @@
-$:.reject! { |e| e.include? 'TextMate' }
-RAILS_GEM_VERSION = '2.0.2' unless defined? RAILS_GEM_VERSION
-
-require File.join(File.dirname(__FILE__), 'boot')
-
-Rails::Initializer.run do |config|
-  config.action_controller.session = {
-    :session_key => '_test_session',
-    :secret      => '012cbaf0c3d36504f3b1bc397b838d24'
-  }
-
-  config.load_paths += %W( #{RAILS_ROOT}/../lib )
-end
-
-require "#{RAILS_ROOT}/../init"

data/test/config/environments/development.rb

@@ -1,18 +0,0 @@
-# Settings specified here will take precedence over those in config/environment.rb
-
-# In the development environment your application's code is reloaded on
-# every request.  This slows down response time but is perfect for development
-# since you don't have to restart the webserver when you make code changes.
-config.cache_classes = false
-
-# Log error messages when you accidentally call methods on nil.
-config.whiny_nils = true
-
-# Show full error reports and disable caching
-config.action_controller.consider_all_requests_local = true
-config.action_controller.perform_caching             = false
-config.action_view.cache_template_extensions         = false
-config.action_view.debug_rjs                         = true
-
-# Don't care if the mailer can't send
-config.action_mailer.raise_delivery_errors = false

data/test/config/environments/test.rb

@@ -1,22 +0,0 @@
-# Settings specified here will take precedence over those in config/environment.rb
-
-# The test environment is used exclusively to run your application's
-# test suite.  You never need to work with it otherwise.  Remember that
-# your test database is "scratch space" for the test suite and is wiped
-# and recreated between test runs.  Don't rely on the data there!
-config.cache_classes = true
-
-# Log error messages when you accidentally call methods on nil.
-config.whiny_nils = true
-
-# Show full error reports and disable caching
-config.action_controller.consider_all_requests_local = true
-config.action_controller.perform_caching             = false
-
-# Disable request forgery protection in test environment
-config.action_controller.allow_forgery_protection    = false
-
-# Tell ActionMailer not to deliver emails to the real world.
-# The :test delivery method accumulates sent emails in the
-# ActionMailer::Base.deliveries array.
-config.action_mailer.delivery_method = :test

data/test/config/routes.rb

@@ -1,35 +0,0 @@
-ActionController::Routing::Routes.draw do |map|
-  # The priority is based upon order of creation: first created -> highest priority.
-
-  # Sample of regular route:
-  #   map.connect 'products/:id', :controller => 'catalog', :action => 'view'
-  # Keep in mind you can assign values other than :controller and :action
-
-  # Sample of named route:
-  #   map.purchase 'products/:id/purchase', :controller => 'catalog', :action => 'purchase'
-  # This route can be invoked with purchase_url(:id => product.id)
-
-  # Sample resource route (maps HTTP verbs to controller actions automatically):
-  #   map.resources :products
-
-  # Sample resource route with options:
-  #   map.resources :products, :member => { :short => :get, :toggle => :post }, :collection => { :sold => :get }
-
-  # Sample resource route with sub-resources:
-  #   map.resources :products, :has_many => [ :comments, :sales ], :has_one => :seller
-
-  # Sample resource route within a namespace:
-  #   map.namespace :admin do |admin|
-  #     # Directs /admin/products/* to Admin::ProductsController (app/controllers/admin/products_controller.rb)
-  #     admin.resources :products
-  #   end
-
-  # You can have the root of your site routed with map.root -- just remember to delete public/index.html.
-  # map.root :controller => "welcome"
-
-  # See how all your routes lay out with "rake routes"
-
-  # Install the default routes as the lowest priority.
-  map.connect ':controller/:action/:id'
-  map.connect ':controller/:action/:id.:format'
-end

data/test/db/migrate/001_create_photos.rb

@@ -1,14 +0,0 @@
-class CreatePhotos < ActiveRecord::Migration
-  def self.up
-    create_table :photos do |t|
-      t.string :title
-      t.text   :description
-
-      t.timestamps
-    end
-  end
-
-  def self.down
-    drop_table :photos
-  end
-end

data/test/db/migrate/002_create_comments.rb

@@ -1,15 +0,0 @@
-class CreateComments < ActiveRecord::Migration
-  def self.up
-    create_table :comments do |t|
-      t.integer :photo_id
-      t.string :author
-      t.text :body
-
-      t.timestamps
-    end
-  end
-
-  def self.down
-    drop_table :comments
-  end
-end

data/test/script/console

@@ -1,3 +0,0 @@
-#!/usr/bin/env ruby
-require File.dirname(__FILE__) + '/../config/boot'
-require 'commands/console'

data/test/script/destroy

@@ -1,3 +0,0 @@
-#!/usr/bin/env ruby
-require File.dirname(__FILE__) + '/../config/boot'
-require 'commands/destroy'

data/test/script/generate

@@ -1,3 +0,0 @@
-#!/usr/bin/env ruby
-require File.dirname(__FILE__) + '/../config/boot'
-require 'commands/generate'

data/test/script/server

@@ -1,3 +0,0 @@
-#!/usr/bin/env ruby
-require File.dirname(__FILE__) + '/../config/boot'
-require 'commands/server'

data/test/test/test_helper.rb

@@ -1,6 +0,0 @@
-ENV["RAILS_ENV"] = "test"
-require File.expand_path(File.dirname(__FILE__) + "/../config/environment")
-require 'test_help'
-require 'mocha'
-
-class Test::Unit::TestCase; end

data/test/test/unit/associated_form_helper_test.rb

@@ -1,376 +0,0 @@
-require File.dirname(__FILE__)+'/../test_helper'
-
-class AssociatedFormHelperTest < Test::Unit::TestCase
-  include ActionView::Helpers::FormHelper
-  include ActionView::Helpers::FormTagHelper
-  include ActionView::Helpers::UrlHelper
-  include ActionView::Helpers::TagHelper
-  include ActionView::Helpers::TextHelper
-  include AttributeFu::AssociatedFormHelper
-    
-  def setup
-    @photo   = Photo.create
-    @controller = mock()
-    @controller.stubs(:url_for).returns 'asdf'
-    @controller.stubs(:protect_against_forgery?).returns false
-    stubs(:protect_against_forgery?).returns false
-  end
-    
-  context "fields for associated" do
-    context "with existing object" do
-      setup do
-        @photo.comments.create :author => "Barry", :body => "Oooh I did good today..."
-
-        @erbout = assoc_output @photo.comments.first
-      end
-
-      should "name field with attribute_fu naming conventions" do
-        assert_match "photo[comment_attributes][#{@photo.comments.first.id}]", @erbout
-      end
-    end
-
-    context "with non-existent object" do
-      setup do      
-        @erbout = assoc_output(@photo.comments.build) do |f|
-          f.fields_for_associated(@photo.comments.build) do |comment|
-            comment.text_field(:author)
-          end
-        end
-      end
-
-      should "name field with attribute_fu naming conventions" do
-        assert_match "photo[comment_attributes][new][0]", @erbout
-      end
-
-      should "maintain the numbering of the new object if called again" do
-        assert_match "photo[comment_attributes][new][1]", @erbout
-      end
-    end
-    
-    context "with overridden name" do
-      setup do
-        _erbout = ''
-        fields_for(:photo) do |f|
-          f.fields_for_associated(@photo.comments.build, :name => :something_else) do |comment|
-            _erbout.concat comment.text_field(:author)
-          end
-        end
-        
-        @erbout = _erbout
-      end
-
-      should "use override name" do
-        assert_dom_equal "<input name='photo[something_else_attributes][new][0][author]' size='30' type='text' id='photo_something_else_attributes__new__0_author' />", @erbout
-      end
-    end
-  end
-  
-  context "remove link" do
-    context "with just a name" do
-      setup do
-        remove_link "remove"
-      end
-
-      should "create a link" do
-        assert_match ">remove</a>", @erbout
-      end
-
-      should "infer the name of the current @object in fields_for" do
-        assert_match "$(this).up('.comment').remove()", @erbout
-      end
-    end
-    
-    context "with an alternate CSS selector" do
-      setup do
-        remove_link "remove", :selector => '.blah'
-      end
-
-      should "use the alternate selector" do
-        assert_match "$(this).up('.blah').remove()", @erbout
-      end
-    end
-    
-    context "with an extra function" do
-      setup do
-        @other_function = "$('asdf').blah();"
-        remove_link "remove", :function => @other_function
-      end
-
-      should "still infer the name of the current @object in fields_for, and create the function as usual" do
-        assert_match "$(this).up('.comment').remove()", @erbout
-      end
-      
-      should "append the secondary function" do
-        assert_match @other_function, @erbout
-      end
-    end
-  end
-  
-  context "with javascript flag" do
-    setup do
-      _erbout = ''
-      fields_for(:photo) do |f|
-        _erbout.concat(f.fields_for_associated(@photo.comments.build, :javascript => true) do |comment|
-          comment.text_field(:author)
-        end)
-      end
-      
-      @erbout = _erbout
-    end
-    
-    should "use placeholders instead of numbers" do
-      assert_match 'photo[comment_attributes][new][#{number}]', @erbout
-    end
-  end
-  
-  context "add_associated_link " do
-    setup do
-      comment = @photo.comments.build
-      
-      _erbout = ''
-      fields_for(:photo) do |f|
-        f.stubs(:render_associated_form).with(comment, :fields_for => {:javascript => true}, :partial => 'comment')
-        _erbout.concat f.add_associated_link("Add Comment", comment, :class => 'something')
-      end
-      
-      @erbout = _erbout
-    end
-
-    should "create link" do
-      assert_match ">Add Comment</a>", @erbout
-    end
-    
-    should "insert into the bottom of the parent container by default" do
-      assert_match "Insertion.Bottom('comments'", @erbout
-    end
-    
-    should "wrap the partial in a prototype template" do
-      assert_match "new Template", @erbout
-      assert_match "evaluate", @erbout
-    end
-    
-    should "name the variable correctly" do
-      assert_match "attribute_fu_comment_count", @erbout
-    end
-    
-    should "pass along the additional options to the link_to_function call" do
-      assert_match 'class="something"', @erbout
-    end
-    
-    should "produce the following link" do
-      # this is a way of testing the whole link
-      assert_equal %{
-        <a class=\"something\" href=\"#\" onclick=\"if (typeof attribute_fu_comment_count == 'undefined') attribute_fu_comment_count = 0;\nnew Insertion.Bottom('comments', new Template(null).evaluate({'number': --attribute_fu_comment_count})); return false;\">Add Comment</a>
-      }.strip, @erbout
-    end
-  end
-  
-  context "add_associated_link with parameters" do
-    setup do
-      comment = @photo.comments.build
-      
-      _erbout = ''
-      fields_for(:photo) do |f|
-        f.stubs(:render_associated_form).with(comment, :fields_for => {:javascript => true}, :partial => 'some_other_partial')
-        _erbout.concat f.add_associated_link("Add Comment", comment, :container => 'something_comments', :partial => 'some_other_partial')
-      end
-      
-      @erbout = _erbout
-    end
-
-    should "create link" do
-      assert_match ">Add Comment</a>", @erbout
-    end
-    
-    should "insert into the bottom of the container specified" do
-      assert_match "Insertion.Bottom('something_comments'", @erbout
-    end
-    
-    should "wrap the partial in a prototype template" do
-      assert_match "new Template", @erbout
-      assert_match "evaluate", @erbout
-    end
-    
-    should "name the variable correctly" do
-      assert_match "attribute_fu_comment_count", @erbout
-    end
-    
-    should "produce the following link" do
-      # this is a way of testing the whole link
-      assert_equal %{
-        <a href=\"#\" onclick=\"if (typeof attribute_fu_comment_count == 'undefined') attribute_fu_comment_count = 0;\nnew Insertion.Bottom('something_comments', new Template(null).evaluate({'number': --attribute_fu_comment_count})); return false;\">Add Comment</a>
-      }.strip, @erbout
-    end
-  end
-  
-  context "add associated link with expression parameter" do
-    setup do
-      comment = @photo.comments.build
-      
-      _erbout = ''
-      fields_for(:photo) do |f|
-        f.stubs(:render_associated_form).with(comment, :fields_for => {:javascript => true}, :partial => 'some_other_partial')
-        _erbout.concat f.add_associated_link("Add Comment", comment, :expression => '$(this).up(".something_comments")', :partial => 'some_other_partial')
-      end
-      
-      @erbout = _erbout
-    end
-
-    should "create link" do
-      assert_match ">Add Comment</a>", @erbout
-    end
-    
-    should "use the javascript expression provided instead of passing the ID in" do
-      assert_match "Insertion.Bottom($(this).up(&quot;.something_comments&quot;)", @erbout
-    end
-    
-    should "wrap the partial in a prototype template" do
-      assert_match "new Template", @erbout
-      assert_match "evaluate", @erbout
-    end
-    
-    should "name the variable correctly" do
-      assert_match "attribute_fu_comment_count", @erbout
-    end
-    
-    should "produce the following link" do
-      # this is a way of testing the whole link
-      assert_equal %{
-        <a href=\"#\" onclick=\"if (typeof attribute_fu_comment_count == 'undefined') attribute_fu_comment_count = 0;\nnew Insertion.Bottom($(this).up(&quot;.something_comments&quot;), new Template(null).evaluate({'number': --attribute_fu_comment_count})); return false;\">Add Comment</a>
-      }.strip, @erbout
-    end    
-  end
-  
-  context "render_associated_form" do
-    setup do
-      comment = @photo.comments.build
-      
-      associated_form_builder = mock()
-      
-      _erbout = ''
-      fields_for(:photo) do |f|
-        f.stubs(:fields_for_associated).yields(associated_form_builder)
-        expects(:render).with(:partial => "comment", :locals => { :comment => comment, :f => associated_form_builder })
-        _erbout.concat f.render_associated_form(comment).to_s
-      end
-      
-      @erbout = _erbout
-    end
-    
-    should "extract the correct parameters for render" do
-      # assertions in mock
-    end
-  end
-  
-  context "render_associated_form with specified partial name" do
-    setup do
-      comment = @photo.comments.build
-      
-      associated_form_builder = mock()
-      
-      _erbout = ''
-      fields_for(:photo) do |f|
-        f.stubs(:fields_for_associated).yields(associated_form_builder)
-        expects(:render).with(:partial => "somewhere/something.html.erb", :locals => { :something => comment, :f => associated_form_builder })
-        _erbout.concat f.render_associated_form(comment, :partial => "somewhere/something.html.erb").to_s
-      end
-      
-      @erbout = _erbout
-    end
-    
-    should "extract the correct parameters for render" do
-      # assertions in mock
-    end
-  end
-  
-  context "render_associated_form with collection" do
-    setup do
-      associated_form_builder = mock()
-      new_comment = Comment.new
-      @photo.comments.expects(:build).returns(new_comment).times(3)
-      @photo.comments.stubs(:empty?).returns(false)
-      @photo.comments.stubs(:first).returns(new_comment)
-      @photo.comments.stubs(:map).yields(new_comment)
-      
-      _erbout = ''
-      fields_for(:photo) do |f|
-        f.stubs(:fields_for_associated).yields(associated_form_builder)
-        expects(:render).with(:partial => "comment", :locals => { :comment => new_comment, :f => associated_form_builder })
-        _erbout.concat f.render_associated_form(@photo.comments, :new => 3).to_s
-      end
-      
-      @erbout = _erbout
-    end
-    
-    should "extract the correct parameters for render" do
-      # assertions in mock
-    end
-  end
-  
-  context "render_associated_form with collection that already has a couple of new objects in it" do
-    setup do
-      associated_form_builder = mock()
-      2.times { @photo.comments.build }
-      new_comment = Comment.new
-      @photo.comments.expects(:build).returns(new_comment)
-      @photo.comments.stubs(:empty?).returns(false)
-      @photo.comments.stubs(:first).returns(new_comment)
-      @photo.comments.stubs(:map).yields(new_comment)
-      
-      _erbout = ''
-      fields_for(:photo) do |f|
-        f.stubs(:fields_for_associated).yields(associated_form_builder)
-        expects(:render).with(:partial => "comment", :locals => { :comment => new_comment, :f => associated_form_builder })
-        _erbout.concat f.render_associated_form(@photo.comments, :new => 3).to_s
-      end
-      
-      @erbout = _erbout
-    end
-    
-    should "extract the correct parameters for render" do
-      # assertions in mock
-    end
-  end
-  
-  context "render_associated_form with overridden name" do
-    setup do
-      associated_form_builder = mock()
-      comment = @photo.comments.build
-      
-      _erbout = ''
-      fields_for(:photo) do |f|
-        f.stubs(:fields_for_associated).with(comment, :name => 'something_else').yields(associated_form_builder)
-        expects(:render).with(:partial => "something_else", :locals => { :something_else => comment, :f => associated_form_builder })
-        _erbout.concat f.render_associated_form(@photo.comments, :name => :something_else).to_s
-      end
-      
-      @erbout = _erbout
-    end
-    
-    should "render with correct parameters" do
-      # assertions in mock
-    end
-  end
-  
-  private
-    def assoc_output(comment, &block)
-      _erbout = ''
-      fields_for(:photo) do |f|
-        _erbout.concat(f.fields_for_associated(comment) do |comment|
-          comment.text_field(:author)
-        end)
-        
-        _erbout.concat yield(f) if block_given?
-      end
-      
-      _erbout
-    end
-    
-    def remove_link(*args)
-      @erbout = assoc_output(@photo.comments.build) do |f|
-        f.fields_for_associated(@photo.comments.build) do |comment|
-          comment.remove_link *args
-        end
-      end
-    end
-end