redmine_rate 0.2.0

data/test/functional/rates_controller_test.rb

@@ -0,0 +1,401 @@
+require File.dirname(__FILE__) + '/../test_helper'
+
+class RatesControllerTest < ActionController::TestCase
+
+  def self.should_be_unauthorized(&block)
+    should 'should return a forbidden status code' do
+      instance_eval(&block)
+      assert_response :forbidden
+    end
+    
+    should 'should display the standard unauthorized page' do
+      instance_eval(&block)
+      assert_template 'common/error'
+    end
+    
+    context "with mime type of xml" do
+      
+      should "should return an forbidden error" do
+        @request.env["HTTP_ACCEPT"] = "application/xml"
+        instance_eval(&block)
+        assert_response :forbidden
+      end
+    end
+
+  end
+
+  def mock_rate(stubs={})
+    @project = Project.generate!
+    stubs = {
+      :date_in_effect => Date.today,
+      :project => @project,
+      :amount => 100.0,
+      :user => @user
+    }.merge(stubs)
+    @mock_rate = Rate.generate(stubs)
+  end
+
+  def mock_locked_rate(stubs={})
+    @mock_rate = mock_rate
+    @mock_rate.time_entries << TimeEntry.generate!
+    @mock_rate
+  end
+
+  context "as regular user" do
+    setup do
+      @user = User.generate!
+      @request.session[:user_id] = @user.id
+    end
+    
+    context "responding to GET index" do
+      should_be_unauthorized { get :index }
+    end
+
+    context "responding to GET show" do
+      should_be_unauthorized { get :show, :id => "37" }
+    end
+
+    context "responding to GET new" do
+      should_be_unauthorized { get :new }
+    end
+
+    context "responding to GET edit" do
+      should_be_unauthorized { get :edit, :id => "37" }
+    end
+
+    context "responding to POST create" do
+      should_be_unauthorized { post :create, :rate => {:these => 'params'} }
+    end
+
+    context "responding to PUT update" do
+      should_be_unauthorized { put :update, :id => "37", :rate => {:these => 'params'} }
+    end
+
+    context "responding to DELETE destroy" do
+      should_be_unauthorized { delete :destroy, :id => "37" }
+    end
+  end
+
+
+  context "as an administrator" do
+    
+    setup do
+      @user = User.generate!(:admin => true)
+      @request.session[:user_id] = @user.id
+    end
+    
+    context "responding to GET index" do
+
+      should "should redirect to the homepage" do
+        get :index
+        assert_redirected_to home_url
+      end
+      
+      should "should display an error flash message" do
+        get :index
+        assert_match /not found/, flash[:error]
+      end
+
+      context "with mime type of xml" do
+        
+        should "should return a 404 error" do
+          @request.env["HTTP_ACCEPT"] = "application/xml"
+          get :index
+          assert_response :not_found
+        end
+        
+      end
+
+    end
+
+    context "responding to GET index with user" do
+      setup do
+        mock_rate
+      end
+
+      should "should expose all historic rates for the user as @rates" do
+        get :index, :user_id => @user.id
+        assert_equal assigns(:rates), [@mock_rate]
+      end
+
+      context "with mime type of xml" do
+        
+        should "should render all rates as xml" do
+          @request.env["HTTP_ACCEPT"] = "application/xml"
+          get :index, :user_id => @user.id
+
+          assert_select 'rates' do
+            assert_select 'rate' do
+              assert_select 'id', :text => @mock_rate.id
+            end
+          end
+          
+        end
+        
+      end
+
+    end
+
+    context "responding to GET show" do
+      setup do
+        mock_rate
+      end
+
+      should "should expose the @requested rate as @rate" do
+        get :show, :id => @mock_rate.id
+        assert_equal assigns(:rate), @mock_rate
+      end
+      
+      context "with mime type of xml" do
+
+        should "should render the requested rate as xml" do
+          @request.env["HTTP_ACCEPT"] = "application/xml"
+          get :show, :id => @mock_rate.id
+
+          assert_select 'rate' do
+            assert_select 'id', :text => @mock_rate.id
+            assert_select 'amount', :text => /100/
+          end
+
+        end
+
+      end
+      
+    end
+
+    context "responding to GET new" do
+      
+      should "should redirect to the homepage" do
+        get :new
+        assert_redirected_to home_url
+      end
+      
+      should "should display an error flash message" do
+        get :new
+        assert_match /not found/, flash[:error]
+      end
+
+      context "with mime type of xml" do
+        
+        should "should return a 404 error" do
+          @request.env["HTTP_ACCEPT"] = "application/xml"
+          get :new
+          assert_response :not_found
+        end
+        
+      end
+    end
+
+    context "responding to GET new with user" do
+      should 'should be successful' do
+        get :new, :user_id => @user.id
+        assert_response :success
+      end
+      
+      should "should expose a new rate as @rate" do
+        get :new, :user_id => @user.id
+        assert assigns(:rate)
+        assert assigns(:rate).new_record?
+      end
+
+    end
+
+    context "responding to GET edit" do
+      setup do
+        mock_rate
+      end
+      
+      should "should expose the requested rate as @rate" do
+        get :edit, :id => @mock_rate.id
+        assert_equal assigns(:rate), @mock_rate
+      end
+      
+      context "on a locked rate" do
+        setup do
+          mock_locked_rate
+        end
+        
+        should 'should not have a Update button' do
+          get :edit, :id => @mock_rate.id
+          assert_select "input[type=submit]", :count => 0
+        end
+
+        should 'should show the locked icon' do
+          get :edit, :id => @mock_rate.id
+          assert_select "img[src*=locked.png]"
+        end
+      end
+
+    end
+
+    context "responding to POST create" do
+
+      context "with valid params" do
+        setup do
+          @project = Project.generate!
+        end
+        
+        should "should expose a newly created rate as @rate" do
+          post :create, :rate => {:project_id => @project.id, :amount => '50', :date_in_effect => Date.today.to_s, :user_id => @user.id}
+          assert assigns(:rate)
+        end
+
+        should "should redirect to the rate list" do
+          post :create, :rate => {:project_id => @project.id, :amount => '50', :date_in_effect => Date.today.to_s, :user_id => @user.id}
+
+          assert_redirected_to rates_url(:user_id => @user.id)
+        end
+        
+        should 'should redirect to the back_url if set' do
+          back_url = '/rates'
+          post :create, :rate => {:project_id => @project.id, :amount => '50', :date_in_effect => Date.today.to_s, :user_id => @user.id}, :back_url => back_url
+
+          assert_redirected_to back_url
+        end
+        
+      end
+      
+      context "with invalid params" do
+        should "should expose a newly created but unsaved rate as @rate" do
+          post :create, :rate => {}
+          assert assigns(:rate).new_record?
+        end
+
+        should "should re-render the 'new' template" do
+          post :create, :rate => {}
+          assert_template 'new'
+        end
+        
+      end
+      
+    end
+
+    context "responding to PUT udpate" do
+
+      context "with valid params" do
+        setup do
+          mock_rate
+        end
+
+        should "should update the requested rate" do
+          put :update, :id => @mock_rate.id, :rate => {:amount => '150'}
+
+          assert_equal 150.0, @mock_rate.reload.amount
+        end
+
+        should "should expose the requested rate as @rate" do
+          put :update, :id => @mock_rate.id
+          
+          assert_equal assigns(:rate), @mock_rate
+        end
+
+        should "should redirect to the rate list" do
+          put :update, :id => "1"
+
+          assert_redirected_to rates_url(:user_id => @user.id)
+        end
+
+        should 'should redirect to the back_url if set' do
+          back_url = '/rates'
+          put :update, :id => "1", :back_url => back_url
+
+          assert_redirected_to back_url
+        end
+
+      end
+      
+      context "with invalid params" do
+        setup do
+          mock_rate
+        end
+
+        should "should not update the requested rate" do
+          put :update, :id => @mock_rate.id, :rate => {:amount => 'asdf'}
+
+          assert_equal 100.0, @mock_rate.reload.amount
+        end
+
+        should "should expose the rate as @rate" do
+          put :update, :id => @mock_rate.id, :rate => {:amount => 'asdf'}
+          
+          assert_equal assigns(:rate), @mock_rate
+        end
+
+        should "should re-render the 'edit' template" do
+          put :update, :id => @mock_rate.id, :rate => {:amount => 'asdf'}
+
+          assert_template 'edit'
+        end
+
+      end
+      
+      context "on a locked rate" do
+        setup do
+          mock_locked_rate
+        end
+        
+        should "should not save the rate" do
+          put :update, :id => @mock_rate.id, :rate => {:amount => '150'}
+
+          assert_equal 100, @mock_rate.reload.amount
+        end
+
+        should "should set the locked rate as @rate" do
+          put :update, :id => @mock_rate.id, :rate => { :amount => 200.0 }
+
+          assert_equal assigns(:rate), @mock_rate
+        end
+        
+        should "should re-render the 'edit' template" do
+          put :update, :id => @mock_rate.id
+
+          assert_template 'edit'
+        end
+        
+        should "should render an error message" do
+          put :update, :id => @mock_rate.id
+          
+          assert_match /locked/, flash[:error]
+        end
+      end
+
+    end
+
+    context "responding to DELETE destroy" do
+      setup do
+        mock_rate
+      end
+
+      should "should destroy the requested rate" do
+        assert_difference('Rate.count', -1) do
+          delete :destroy, :id => @mock_rate.id
+        end
+      end
+      
+      should "should redirect to the user's rates list" do
+        delete :destroy, :id => @mock_rate.id
+        assert_redirected_to rates_url(:user_id => @user.id)
+      end
+
+      should 'should redirect to the back_url if set' do
+        back_url = '/rates'
+        delete :destroy, :id => "1", :back_url => back_url
+        
+        assert_redirected_to back_url
+      end
+      
+      context "on a locked rate" do
+        setup do
+          mock_locked_rate
+        end
+        
+        should "should display an error message" do
+          delete :destroy, :id => @mock_rate.id
+          assert_match /locked/, flash[:error]
+        end
+      end
+
+    end
+
+  end
+end

data/test/integration/admin_panel_test.rb

@@ -0,0 +1,81 @@
+require 'test_helper'
+
+class AdminPanelTest < ActionController::IntegrationTest
+  include Redmine::I18n
+  
+  def setup
+    @last_caching_run = 4.days.ago.to_s
+    @last_cache_clearing_run = 7.days.ago.to_s
+    
+    Setting.plugin_redmine_rate = {
+      'last_caching_run' => @last_caching_run,
+      'last_cache_clearing_run' => @last_cache_clearing_run
+    }
+    
+    @user = User.generate!(:admin => true, :password => 'rates', :password_confirmation => 'rates')
+    
+    login_as(@user.login, 'rates')
+  end
+
+  context "Rate Caches admin panel" do
+    should "be listed in the main Admin section" do
+      click_link "Administration"
+      assert_response :success
+
+      assert_select "#admin-menu" do
+        assert_select "a.rate-caches"
+      end
+      
+    end
+    
+    should "show the last run timestamp for the last caching run" do
+      click_link "Administration"
+      click_link "Rate Caches"
+      
+      assert_select '#caching-run' do
+        assert_select 'p', :text => /#{format_time(@last_caching_run)}/
+      end
+      
+    end
+
+    should "show the last run timestamp for the last cache clearing run" do
+      click_link "Administration"
+      click_link "Rate Caches"
+      
+      assert_select '#cache-clearing-run' do
+        assert_select 'p', :text => /#{format_time(@last_cache_clearing_run)}/
+      end
+      
+    end
+      
+    should "have a button to force a caching run" do
+      click_link "Administration"
+      click_link "Rate Caches"
+      click_button "Load Missing Caches"
+
+      assert_response :success
+
+      appx_clear_time = Date.today.strftime("%m/%d/%Y")
+      
+      assert_select '#caching-run' do
+        assert_select 'p', :text => /#{appx_clear_time}/
+      end
+
+    end
+
+    should "have a button to force a cache clearing run" do
+      click_link "Administration"
+      click_link "Rate Caches"
+      click_button "Clear and Load All Caches"
+
+      assert_response :success
+
+      appx_clear_time = Date.today.strftime("%m/%d/%Y")
+      
+      assert_select '#cache-clearing-run' do
+        assert_select 'p', :text => /#{appx_clear_time}/
+      end
+
+    end
+  end
+end

data/test/integration/routing_test.rb

@@ -0,0 +1,16 @@
+require "#{File.dirname(__FILE__)}/../test_helper"
+
+class RoutingTest < ActionController::IntegrationTest
+  context "routing rates" do
+    should_route :get, "/rates", :controller => "rates", :action => "index"
+    should_route :get, "/rates/new", :controller => "rates", :action => "new"
+    should_route :get, "/rates/1", :controller => "rates", :action => "show", :id => "1"
+    should_route :get, "/rates/1/edit", :controller => "rates", :action => "edit", :id => "1"
+
+    should_route :post, "/rates", :controller => "rates", :action => "create"
+
+    should_route :put, "/rates/1", :controller => "rates", :action => "update", :id => "1"
+
+    should_route :delete, "/rates/1", :controller => "rates", :action => "destroy", :id => "1"
+  end
+end

data/test/test_helper.rb

@@ -0,0 +1,43 @@
+# Load the normal Rails helper
+require File.expand_path(File.dirname(__FILE__) + '/../../../../test/test_helper')
+
+# Ensure that we are using the temporary fixture path
+Engines::Testing.set_fixture_path
+
+require "webrat"
+
+Webrat.configure do |config|
+  config.mode = :rails
+end
+
+module IntegrationTestHelper
+  def login_as(user="existing", password="existing")
+    visit "/login"
+    fill_in 'Login', :with => user
+    fill_in 'Password', :with => password
+    click_button 'login'
+    assert_response :success
+    assert User.current.logged?
+  end
+
+  def logout
+    visit '/logout'
+    assert_response :success
+    assert !User.current.logged?
+  end
+  
+  def assert_forbidden
+    assert_response :forbidden
+    assert_template 'common/error'
+  end
+
+  def assert_requires_login
+    assert_response :success
+    assert_template 'account/login'
+  end
+  
+end
+
+class ActionController::IntegrationTest
+  include IntegrationTestHelper
+end

data/test/unit/lib/rate_time_entry_patch_test.rb

@@ -0,0 +1,77 @@
+require File.dirname(__FILE__) + '/../../test_helper'
+
+class RateTimeEntryPatchTest < ActiveSupport::TestCase
+  def setup
+    @user = User.generate!
+    @project = Project.generate!
+    @date = Date.today.to_s
+    @time_entry = TimeEntry.new({:user => @user, :project => @project, :spent_on => @date, :hours => 10.0, :activity => TimeEntryActivity.generate!})
+    @rate = Rate.generate!(:user => @user, :project => @project, :date_in_effect => @date, :amount => 200.0)
+  end
+  
+  should 'should return 0.0 if there are no rates for the user' do
+    @rate.destroy
+    assert_equal 0.0, @time_entry.cost
+  end
+
+  context 'should return the product of hours by' do
+    should 'the results of Rate.amount_for' do
+      assert_equal((200.0 * @time_entry.hours), @time_entry.cost)
+    end
+
+    should 'the assigned rate' do
+      rate = Rate.generate!(:user => @user, :project => @project, :date_in_effect => @date, :amount => 100.0)
+      @time_entry.rate = rate
+      assert_equal rate.amount * @time_entry.hours, @time_entry.cost
+    end
+
+  end
+
+  context "#cost" do
+    setup do
+      @time_entry.save!
+    end
+    
+    context "without a cache" do
+      should "return the calculated cost" do
+        @time_entry.update_attribute(:cost, nil)
+        assert_equal 2000.0, @time_entry.cost
+      end
+      
+      should "cache the cost to the field" do
+        @time_entry.update_attribute(:cost, nil)
+        @time_entry.cost
+
+        assert_equal 2000.0, @time_entry.read_attribute(:cost)
+        assert_equal 2000.0, @time_entry.reload.read_attribute(:cost)
+      end
+      
+    end
+
+    context "with a cache" do
+      setup do
+        @time_entry.update_attribute(:cost, 2000.0)
+        @time_entry.reload
+      end
+      
+      should "return the cached cost" do
+        assert_equal 2000.0, @time_entry.read_attribute(:cost)
+        assert_equal 2000.0, @time_entry.cost
+      end
+      
+    end
+
+  end
+
+  context "before save" do
+    should "clear and recalculate the cache" do
+      assert_equal nil, @time_entry.read_attribute(:cost)
+
+      assert @time_entry.save
+
+      assert_equal 2000.0, @time_entry.read_attribute(:cost)
+    end
+  end
+  
+
+end

data/test/unit/lib/rate_users_helper_patch_test.rb

@@ -0,0 +1,37 @@
+require File.dirname(__FILE__) + '/../../test_helper'
+
+class UsersHelperWrapper
+  include UsersHelper
+end
+
+class RateUsersHelperPatchTest < ActiveSupport::TestCase
+  should 'should return 3 tabs' do
+    helper = UsersHelperWrapper.new
+    assert_equal 3, helper.user_settings_tabs.length
+  end
+  
+  should 'should include a rate tab at the end' do
+    helper = UsersHelperWrapper.new
+    rate_tab = helper.user_settings_tabs[-1]
+    assert_not_nil rate_tab
+  end
+
+  context 'rate tab' do
+    setup do
+      helper = UsersHelperWrapper.new
+      @rate_tab = helper.user_settings_tabs[-1]
+    end
+
+    should 'should have the name of "rates"' do
+      assert_equal 'rates', @rate_tab[:name]
+    end
+
+    should 'should use the rates partial' do
+      assert_equal 'users/rates', @rate_tab[:partial]
+    end
+
+    should 'should use the i18n rates label' do
+      assert_equal :rate_label_rate_history, @rate_tab[:label]
+    end
+  end
+end

data/test/unit/lib/redmine_rate/hooks/plugin_timesheet_view_timesheets_report_header_tags_hook_test.rb

@@ -0,0 +1,26 @@
+require File.dirname(__FILE__) + '/../../../../test_helper'
+
+class RedmineRate::Hooks::PluginTimesheetViewTimesheetsReportHeaderTagsTest < ActionController::TestCase
+  include Redmine::Hook::Helper
+
+  def controller
+    @controller ||= ApplicationController.new
+    @controller.response ||= ActionController::TestResponse.new
+    @controller
+  end
+
+  def request
+    @request ||= ActionController::TestRequest.new
+  end
+  
+  def hook(args={})
+    call_hook :plugin_timesheet_view_timesheets_report_header_tags, args
+  end
+
+  context "#plugin_timesheet_view_timesheets_report_header_tags" do
+    should "return a css string" do
+      @response.body = hook
+      assert_select "style", :text => /missing-rate/
+    end
+  end
+end