admin_assistant 2.0.0 → 2.0.1

data/rails_2_test/test/integration/admin/products_integration_test.rb

@@ -0,0 +1,210 @@
+require File.expand_path(File.dirname(__FILE__) + "/../../test_helper")
+
+class Admin::ProductsIntegrationTest < ActionController::IntegrationTest
+  def test_destroy
+    @product = Product.find_or_create_by_name 'Chocolate bar'
+    post "/admin/products/destroy/#{@product.id}"
+    
+    # should call the custom destroy block
+    @product.reload
+    assert_not_nil(@product.deleted_at)
+  end
+  
+  def test_index_with_at_least_one_Product
+    @product = (
+      Product.find(:first) || Product.create!(:name => 'product name')
+    )
+    get "/admin/products"
+    
+    # should have a search form that allows you to specify equals or other options
+    assert_select('form[id=search_form][method=get]') do
+      assert_select("select[name=?]", "search[price(comparator)]") do
+        assert_select("option[value='>']", :text => 'greater than')
+        assert_select("option[value='>=']", :text => 'greater than or equal to')
+        assert_select("option[value='='][selected=selected]", :text => 'equal to')
+        assert_select("option[value='<=']", :text => 'less than or equal to')
+        assert_select("option[value='<']", :text => 'less than')
+      end
+      assert_select('input[name=?]', 'search[price]')
+    end
+    
+    # should not have a comparator for the name search field
+    assert_select('form[id=search_form][method=get]') do
+      assert_select("select[name=?]", "search[name(comparator)]", false)
+    end
+    
+    # should not have a Show link
+    assert_no_match(%r|<a[^>]*>Show</a>|, response.body)
+  end
+  
+  def test_index_search_by_price
+    Product.destroy_all
+    Product.create!(:name => 'Chocolate bar', :price => 200)
+    Product.create!(:name => 'Diamond ring', :price => 200_000)
+    get "/admin/products", :search => {"price(comparator)" => '>', :price => 1000}
+    
+    # should only show products more expensive than $10
+    assert_no_match(%r|<td[^>]*>Chocolate bar</td>|, response.body)
+    assert_select('td', :text => 'Diamond ring')
+    
+    # should show the price comparison in the search form
+    assert_select('form[id=search_form][method=get]') do
+      assert_select("select[name=?]", "search[price(comparator)]") do
+        assert_select(
+          "option[value='>'][selected=selected]", :text => 'greater than'
+        )
+        assert_select("option[value='>=']", :text => 'greater than or equal to')
+        assert_select("option[value='=']", :text => 'equal to')
+        assert_select("option[value='<=']", :text => 'less than or equal to')
+        assert_select("option[value='<']", :text => 'less than')
+      end
+      assert_select('input[name=?]', 'search[price]')
+    end
+  end
+  
+  def test_index_sort_by_product_category
+    ProductCategory.destroy_all
+    Product.destroy_all
+    diamond_cat = ProductCategory.create!(
+      :category_name => 'diamond', :position => 1
+    )
+    @diamond = Product.create!(
+      :name => 'Diamond', :price => 200_000, :product_category => diamond_cat
+    )
+    choc_cat = ProductCategory.create!(
+      :category_name => 'chocolate', :position => 2
+    )
+    @chocolate = Product.create!(
+      :name => 'Chocolate bar', :price => 200, :product_category => choc_cat
+    )
+    assert(diamond_cat.id < choc_cat.id)
+    get "/admin/products", :sort_order => "asc", :sort => "product_category"
+    
+    # should be a success
+    assert_response :success
+    
+    # should order by product_category_id
+    assert_match(
+      %r|product_#{@diamond.id}.*product_#{@chocolate.id}|m, response.body
+    )
+  end
+  
+  def test_new
+    get "/admin/products/new"
+    assert_response :success
+    
+    # should not render the default price input
+    assert_select("input[name=?]", 'product[price]', false)
+    
+    # should render the custom price input from _price_input.html.erb
+    assert_select("input[name=?]", 'product[price][dollars]')
+    
+    # should show clear links for sale_starts_at and sale_ends_at
+    assert_select('a', :text => "Not on sale")
+    assert_select('a', :text => "Sale doesn't end")
+  end
+  
+  def test_edit
+    @product = Product.find_or_create_by_name 'a bird'
+    @product.update_attributes(
+      :name => 'a bird', :price => 100_00,
+      :file_column_image => File.open("./test/data/ruby_throated.jpg"),
+      :percent_off => 25
+    )
+    get "/admin/products/edit/#{@product.id}"
+    assert_response :success
+    
+    # should have a multipart form
+    assert_select('form[enctype=multipart/form-data]')
+    
+    # should have a file input for image
+    assert_select(
+      'input[name=?][type=file]', 'product[file_column_image]'
+    )
+    
+    # should show the current image
+    assert_select(
+      "img[src^=?][height=100][width=100]",
+      "/product/file_column_image/#{@product.id}/ruby_throated.jpg"
+    )
+    
+    # should have a remove-image option
+    assert_select(
+      "input[type=checkbox][name=?]", 'product[file_column_image(destroy)]'
+    )
+    
+    # should the percent_off value pre-filled
+    assert_select(
+      'input[name=?][value=25]', 'product[percent_off]'
+    )
+  end
+  
+  def test_update_while_updating_a_current_image
+    @product = Product.find_or_create_by_name 'a bird'
+    @product.update_attributes(
+      :name => 'a bird', :price => 100_00,
+      :file_column_image => File.open("./test/data/ruby_throated.jpg")
+    )
+    file = File.new './test/data/tweenbot.jpg'
+    post(
+      "/admin/products/update/#{@product.id}", 
+      :product => {
+        :file_column_image => fixture_file_upload(
+          "../../test/data/tweenbot.jpg"
+        )
+      },
+      :html => {:multipart => true}
+    )
+      
+    # should update the image
+    product_prime = Product.find_by_id @product.id
+    assert_match(/tweenbot/, product_prime.file_column_image)
+    
+    # should save the image locally
+    assert(
+      File.exist?(
+        "./public/product/file_column_image/#{@product.id}/tweenbot.jpg"
+      )
+    )
+  end    
+    
+  def test_update_while_removing_the_current_image
+    @product = Product.find_or_create_by_name 'a bird'
+    @product.update_attributes(
+      :name => 'a bird', :price => 100_00,
+      :file_column_image => File.open("./test/data/ruby_throated.jpg")
+    )
+    post(
+      "/admin/products/update/#{@product.id}",
+      :product => {
+        'file_column_image(destroy)' => '1', :file_column_image => ''
+      }
+    )
+  
+    # should remove the existing image
+    product_prime = Product.find_by_id @product.id
+    assert_nil product_prime.file_column_image
+  end
+  
+  def test_update_while_trying_to_update_and_remove_at_the_same_time
+    @product = Product.find_or_create_by_name 'a bird'
+    @product.update_attributes(
+      :name => 'a bird', :price => 100_00,
+      :file_column_image => File.open("./test/data/ruby_throated.jpg")
+    )
+    post(
+      "/admin/products/update/#{@product.id}", 
+      :product => {
+        :file_column_image => fixture_file_upload(
+          "../../test/data/tweenbot.jpg"
+        ),
+        'file_column_image(destroy)' => '1'
+      },
+      :html => {:multipart => true}
+    )
+
+    # should assume you meant to update
+    product_prime = Product.find_by_id @product.id
+    assert_match(/tweenbot/, product_prime.file_column_image)
+  end
+end

data/rails_2_test/test/integration/admin/television_airings_integration_test.rb

@@ -0,0 +1,25 @@
+require File.expand_path(File.dirname(__FILE__) + "/../../test_helper")
+
+class Admin::TelevisionAiringsIntegrationTest < 
+      ActionController::IntegrationTest
+  def test_new
+    TelevisionTimeSlot.destroy_all
+    @tv_time_slot1 = TelevisionTimeSlot.create!(
+      :time => Time.utc(2010, 2, 14, 9)
+    )
+    @tv_time_slot2 = TelevisionTimeSlot.create!(
+      :time => Time.utc(2010, 2, 14, 14)
+    )
+    get "/admin/television_airings/new"
+    assert_response :success
+    
+    # should sort television time slots by time, not by the time string
+    assert_select(
+      'select[name=?]', 'television_airing[television_time_slot_id]'
+    ) do
+      assert_select 'option:first-child[value=""]'
+      assert_select 'option:nth-child(2)[value=?]', @tv_time_slot1.id
+      assert_select 'option:nth-child(3)[value=?]', @tv_time_slot2.id
+    end
+  end
+end

data/rails_2_test/test/integration/admin/users2_integration_test.rb

@@ -0,0 +1,48 @@
+require File.expand_path(File.dirname(__FILE__) + "/../../test_helper")
+
+class Admin::Users2IntegrationTest < ActionController::IntegrationTest
+  def test_index
+    User.create!(:username => random_word) if User.count == 0
+    get "/admin/users2"
+  
+    # should show the empty search form
+    assert_select('form#search_form') do
+      assert_select('input[name=?]', 'search[blog_posts]')
+    end
+  end
+
+  def test_index_when_searching_by_blog_post
+    User.destroy_all
+    @soren = User.create! :username => 'soren'
+    @jean_paul = User.create! :username => 'jean_paul'
+    BlogPost.create!(
+      :user => @jean_paul, :title => 'No Foobar',
+      :body => 'Hell is other foobars'
+    )
+    @friedrich = User.create! :username => 'friedrich'
+    BlogPost.create!(
+      :user => @friedrich, :title => 'Thus Spake Zarafoobar',
+      :body => 'Man is something that shall be overfoobared.'
+    )
+    BlogPost.create!(
+      :user => @friedrich, :title => 'Beyond Good and Foobar',
+      :body =>
+        'And when you gaze long into a foobar the foobar also gazes into you.'
+    )
+    get "/admin/users2", :search => {:blog_posts => 'foobar'}
+  
+    # should prefill the search form fields
+    assert_select('form#search_form') do
+      assert_select('input[name=?][value=?]', 'search[blog_posts]', 'foobar')
+    end
+    
+    # should not match a user without any matching blog posts
+    assert_no_match(%r|<td[^>]*>#{@soren.username}</td>|, response.body)
+    
+    # should match a user with one matching blog post
+    assert_select('td', :text => @jean_paul.username)
+    
+    # should match a user with two matching blog posts, only presenting that user once
+    assert_select('td', :text => @friedrich.username, :count => 1)
+  end
+end

data/rails_2_test/test/integration/admin/users_integration_test.rb

@@ -0,0 +1,266 @@
+require File.expand_path(File.dirname(__FILE__) + "/../../test_helper")
+
+class Admin::UsersIntegrationTest < ActionController::IntegrationTest
+  def setup
+    @user = User.find_or_create_by_username 'betty'
+    @user.update_attribute :password, 'crocker'
+  end
+  
+  def test_create
+    post(
+      "/admin/users/create", :user => { :username => 'bill', :password => '' }
+    )
+    
+    # should assign a new random password
+    user = User.find_by_username 'bill'
+    assert_not_nil(user)
+    assert_not_equal(user.password, '')
+  end
+  
+  def test_create_with_the_same_username
+    @user_count = User.count
+    post(
+      "/admin/users/create", :user => {:username => 'betty', :password => ''}
+    )
+    
+    # should not save a new user
+    assert_equal(User.count, @user_count)
+    
+    # should not call after_save
+    assert_response :success
+  end
+  
+  def test_destroy
+    post "/admin/users/destroy/#{@user.id}"
+    
+    # should destroy the user
+    assert_nil(User.find_by_id(@user.id))
+  end
+  
+  def test_edit
+    @user.update_attributes(
+      :has_avatar => true, :avatar_version => 9,
+      :force_blog_posts_to_textile => true
+    )
+    get "/admin/users/edit/#{@user.id}"
+    
+    # should show the default text input for password
+    assert_match(%r|<input[^>]*name="user\[password\]"|, response.body)
+    
+    # should show a reset password checkbox
+    assert_select("input[type=checkbox][name=reset_password]")
+    
+    # should have a multipart form
+    assert_select('form[enctype=multipart/form-data]')
+    
+    # should have a file input for tmp_avatar
+    assert_select('input[name=?][type=file]', 'user[tmp_avatar]')
+    
+    # should show the current tmp_avatar with a custom src
+    assert_select(
+      "img[src^=?]", "http://my-image-server.com/users/#{@user.id}.jpg?v=9"
+    )
+    
+    # should have a remove-image option
+    assert_select("input[type=checkbox][name=?]", 'user[tmp_avatar(destroy)]')
+    
+    # should show a drop-down for force_blog_posts_to_textile
+    assert_select('select[name=?]', 'user[force_blog_posts_to_textile]') do
+      assert_select "option:not([selected])[value='']"
+      assert_select "option:not([selected])[value=0]", :text => 'false'
+      assert_select "option[selected=selected][value=1]", :text => 'true'
+    end
+  end
+  
+  def test_index
+    get "/admin/users"
+    
+    # should show a Delete link and a link to the profile page
+    assert_select('td') do
+      assert_select(
+        "a.destroy[href=?][data-method=?]",
+        "/admin/users/destroy/#{@user.id}", 'delete',
+        :text => 'Delete'
+      )
+      assert_select(
+        "a[href=?]",
+        "/admin/blog_posts/new?blog_post%5Buser_id%5D=#{@user.id}",
+        :text => "New blog post"
+      )
+    end
+  end
+  
+  def test_index_with_a_user_with_an_avatar
+    @user.update_attributes :has_avatar => true, :avatar_version => 9
+    get "/admin/users"
+    
+    # should show the avatar image in the avatar column
+    assert_select(
+      'img[src=?]', "http://my-image-server.com/users/#{@user.id}.jpg?v=9"
+    )
+  end
+
+  def test_index_search
+    User.destroy_all
+    @john_doe = User.create! :username => 'johndoe'
+    @jane_doe = User.create! :username => 'janedoe', :password => "ihatejohn"
+    get "/admin/users", :search => "john"
+    assert_response :success
+    
+    # should match username
+    assert_select('td', :text => 'johndoe')
+    
+    # should not match password
+    assert_no_match(%r|<td[^>]*>janedoe</td>|, response.body)
+  end
+  
+  def test_index_search_across_first_name_and_last_name
+    User.destroy_all
+    @john_doe = User.create!(
+      :username => 'johndoe', :first_name => 'john', :last_name => 'doe'
+    )
+    get "/admin/users", :search => "john doe"
+    
+    # should match
+    assert_select('td', :text => 'johndoe')
+  end
+  
+  def test_new
+    get "/admin/users/new"
+    
+    # should not show an input for password
+    assert_match(/autogenerated/, response.body)
+    assert_no_match(%r|<input[^>]*name="user\[password\]"|, response.body)
+    
+    # should not show a reset password checkbox
+    assert_select("input[type=checkbox][name=reset_password]", false)
+    
+    # should use date dropdowns with nil defaults for birthday
+    nums_and_dt_fields = {1 => :year, 2 => :month, 3 => :day}
+    nums_and_dt_fields.each do |num, dt_field|
+      name = "user[birthday(#{num}i)]"
+      assert_select('select[name=?]', name) do
+        assert_select "option[value='']"
+        assert_select(
+          "option:not([selected])[value=?]", Time.now.send(dt_field).to_s
+        )
+      end
+    end
+    
+    # should not try to set an hour or minute for birthday
+    nums_and_dt_fields = {4 => :hour, 5 => :min}
+    nums_and_dt_fields.each do |num, dt_field|
+      name = "blog_post[published_at(#{num}i)]"
+      assert_select('select[name=?]', name, false)
+    end
+    
+    # should respect start_year and end_year parameters
+    assert_select("select[name='user[birthday(1i)]']") do
+      (Time.now.year-100).upto(Time.now.year) do |year|
+        assert_select "option[value='#{year}']"
+      end
+    end
+    
+    # should show a drop-down for US states
+    assert_select('select[name=?]', 'user[state]') do
+      assert_select "option[value='']"
+      assert_select "option:not([selected])[value=AK]", :text => 'Alaska'
+      assert_select "option:not([selected])[value=NY]", :text => 'New York'
+      # blank option, 50 states, DC, Puerto Rico == 53 options
+      assert_select "option", :count => 53
+    end
+    
+    # should have a multipart form
+    assert_select('form[enctype=multipart/form-data]')
+    
+    # should have a file input for tmp_avatar
+    assert_match(
+      %r|<input[^>]*name="user\[tmp_avatar\]"[^>]*type="file"|, response.body
+    )
+    
+    # should show a drop-down for force_blog_posts_to_textile
+    assert_select('select[name=?]', 'user[force_blog_posts_to_textile]') do
+      assert_select "option[value='']"
+      assert_select "option:not([selected])[value=0]", :text => 'false'
+      assert_select "option:not([selected])[value=1]", :text => 'true'
+    end
+    
+    # should show a select for admin_level
+    assert_select("select[name=?]", "user[admin_level]") do
+      assert_select("option[value='']", false)
+      assert_select "option[value=normal][selected=selected]"
+      assert_select "option[value=admin]"
+      assert_select "option[value=superuser]"
+    end
+  end
+  
+  def test_update
+    post(
+      "/admin/users/update/#{@user.id}",
+      :user => {:username => 'bettie', :force_blog_posts_to_textile => ''}
+    )
+    @user.reload
+  
+    # should not assign a new random password
+    assert_equal(@user.password, 'crocker')
+  
+    # should know the difference between nil and false for force_blog_posts_to_textile
+    assert_nil(@user.force_blog_posts_to_textile)
+  end
+  
+  def test_update_while_resetting_password
+    post(
+      "/admin/users/update/#{@user.id}",
+      :user => {:username => 'bettie'}, :reset_password => '1'
+    )
+    
+    # should assign a new random password
+    @user.reload
+    assert_not_equal(@user.password, 'crocker')
+  end
+  
+  def test_update_while_updating_the_current_tmp_avatar
+    @user.update_attributes :has_avatar => true, :avatar_version => 9
+    post(
+      "/admin/users/update/#{@user.id}",
+      :user => {
+        :tmp_avatar => fixture_file_upload('../../test/data/tweenbot.jpg')
+      },
+      :html => {:multipart => true}
+    )
+    
+    # should increment the avatar_version through before_save
+    @user.reload
+    assert_equal(@user.avatar_version, 10)
+  end
+  
+  def test_update_while_removing_the_current_tmp_avatar
+    @user.update_attributes :has_avatar => true, :avatar_version => 9
+    post(
+      "/admin/users/update/#{@user.id}",
+      :user => {:tmp_avatar => '', 'tmp_avatar(destroy)' => '1'},
+      :html => {:multipart => true}
+    )
+      
+    # should set has_avatar to false
+    @user.reload
+    assert(!@user.has_avatar?)
+  end
+  
+  def test_while_trying_to_update_and_remove_tmp_avatar_at_the_same_time
+    @user.update_attributes :has_avatar => true, :avatar_version => 9
+    post(
+      "/admin/users/update/#{@user.id}",
+      :user => {
+        :tmp_avatar => fixture_file_upload('../../test/data/tweenbot.jpg'),
+        'tmp_avatar(destroy)' => '1'
+      },
+      :html => {:multipart => true}
+    )
+
+    # should assume you meant to update
+    @user.reload
+    assert_equal(@user.avatar_version, 10)
+    assert(@user.has_avatar?)
+  end
+end

data/rails_2_test/test/integration/blog_posts_integration_test.rb

@@ -0,0 +1,9 @@
+$: << 'test'
+require 'test_helper'
+
+class BlogPostsIntegrationTest < ActionController::IntegrationTest
+  def test_create_should_not_be_defined_by_admin_assistant
+    post "/blog_posts/create"
+    assert_response :not_found
+  end
+end

data/rails_2_test/test/test_helper.rb

@@ -0,0 +1,114 @@
+ENV["RAILS_ENV"] = "test"
+here = File.dirname(__FILE__)
+require File.expand_path(here + "/../config/environment")
+require 'test_help'
+
+module TestHelper
+  def assert_a_tag_with_get_args(
+    content, href_base, href_get_args, response_body
+  )
+    regex = %r|<a href="#{ href_base }\?([^"]*)"[^>]*>#{ content }</a>|
+    assert(
+      response_body =~ regex,
+      "#{response_body.inspect} expected to match #{regex.inspect}"
+    )
+    assert_get_args_equal( href_get_args, $1.gsub( /&amp;/, '&' ) )
+  end
+  
+  def assert_no_a_tag_with_get_args(
+    content, href_base, href_get_args, response_body
+  )
+    regex = %r|<a href="#{ href_base }\?([^"]*)"[^>]*>#{ content }</a>|
+    if response_body =~ regex
+      get_args_string = $1.gsub( /&amp;/, '&' )
+      response_h = HashWithIndifferentAccess.new
+      CGI::parse( get_args_string ).each do |key, values|
+        response_h[key] = values.first
+      end
+      if href_get_args.size == response_h.size
+        raise if href_get_args.all? { |key, value|
+          response_h[key] == value
+        }
+      end
+    end
+  end
+  
+  def assert_get_args_equal( expected_hash, get_args_string )
+    response_h = HashWithIndifferentAccess.new
+    CGI::parse( get_args_string ).each do |key, values|
+      response_h[key] = values.first
+    end
+    assert_equal(
+      expected_hash.size, response_h.size,
+      "<#{ expected_hash.inspect }> expected but was\n<#{ response_h.inspect }>."
+    )
+    expected_hash.each do |key, value|
+      assert_equal( value, response_h[key] )
+    end
+  end
+
+  def random_word( length = 25 )
+    letters = 'abcdefghijklmnopqrstuvwxyz_'.split //
+    ( 1..length ).to_a.map { letters[rand(letters.size)] }.join( '' )
+  end
+end
+
+class CacheStoreStub
+  def initialize
+    flush
+  end
+  
+  def expires_in(key)
+    @expirations[key.to_s]
+  end
+  
+  def flush
+    @cache = {}
+    @expirations = {}
+    @raise_on_write = false
+  end
+  
+  def raise_on_write
+    @raise_on_write = true
+  end
+  
+  def read(key, options = nil)
+    @cache[key.to_s]
+  end
+  
+  def write(key, value, options = nil)
+    raise if @raise_on_write
+    @cache[key.to_s] = value
+    @expirations[key.to_s] = options[:expires_in]
+  end
+end
+
+$cache = CacheStoreStub.new
+
+module Rails
+  mattr_accessor :cache
+  self.cache = $cache
+end
+
+if %w(2.1.0 2.1.2).include?(RAILS_GEM_VERSION)
+  class Test::Unit::TestCase
+    self.use_transactional_fixtures = true
+  
+    self.use_instantiated_fixtures  = false
+  
+    include TestHelper
+  end
+else
+  class ActiveSupport::TestCase
+    self.use_transactional_fixtures = true
+  
+    self.use_instantiated_fixtures  = false
+    
+    include TestHelper
+  end
+end
+
+Webrat.configure do |config|  
+  config.mode = :rails  
+end  
+