giraffesoft-attribute_fu 0.2

data/test/app/controllers/application.rb

@@ -0,0 +1,10 @@
+# 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

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

data/test/app/models/comment.rb

@@ -0,0 +1,8 @@
+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

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

data/test/config/boot.rb

@@ -0,0 +1,97 @@
+# 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

@@ -0,0 +1,15 @@
+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

@@ -0,0 +1,15 @@
+$:.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

@@ -0,0 +1,18 @@
+# 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

@@ -0,0 +1,22 @@
+# 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

@@ -0,0 +1,35 @@
+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

@@ -0,0 +1,14 @@
+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

@@ -0,0 +1,15 @@
+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/db/schema.rb

@@ -0,0 +1,29 @@
+# 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"
+  end
+
+  create_table "photos", :force => true do |t|
+    t.string   "title"
+    t.text     "description"
+    t.datetime "created_at"
+    t.datetime "updated_at"
+  end
+
+end

data/test/script/console

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

data/test/script/destroy

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

data/test/script/generate

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

data/test/script/server

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

data/test/test/test_helper.rb

@@ -0,0 +1,6 @@
+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

@@ -0,0 +1,376 @@
+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