From 38f2bcb17f9948c8c81a6ff9a28c3c058dd4b1b8 Mon Sep 17 00:00:00 2001 From: Eric Davis Date: Wed, 29 Sep 2010 10:44:54 -0700 Subject: [PATCH] [#4552] Add the Labor hours spent to the Deliverable details --- app/models/deliverable.rb | 5 ++ app/models/retainer_deliverable.rb | 13 +++++ app/views/deliverables/_details_row.html.erb | 6 +- test/integration/contracts_show_test.rb | 34 +++++++++++ test/unit/retainer_deliverable_test.rb | 60 ++++++++++++++++++++ 5 files changed, 116 insertions(+), 2 deletions(-) diff --git a/app/models/deliverable.rb b/app/models/deliverable.rb index 1001a3f..e6db222 100644 --- a/app/models/deliverable.rb +++ b/app/models/deliverable.rb @@ -86,6 +86,11 @@ class Deliverable < ActiveRecord::Base (overhead_budgets.sum(:hours) || 0.0) end + # OPTIMIZE: N+1 + def labor_hours_spent_total(date=nil) + issues.inject(0) {|total, issue| total += issue.billable_time_spent } # From redmine_overhead + end + # OPTIMIZE: N+1 def hours_spent_total issues.inject(0) {|total, issue| total += issue.spent_hours } diff --git a/app/models/retainer_deliverable.rb b/app/models/retainer_deliverable.rb index 572a8df..2237025 100644 --- a/app/models/retainer_deliverable.rb +++ b/app/models/retainer_deliverable.rb @@ -133,6 +133,19 @@ class RetainerDeliverable < HourlyDeliverable end end + def labor_hours_spent_total(date=nil) + case scope_date_status(date) + when :in + time_entries = issues.collect {|issue| issue.time_entries.all(:conditions => {:tyear => date.year, :tmonth => date.month}) }.flatten + + billable_hours_on_time_entries(time_entries) + when :out + 0 + else + super + end + end + def fixed_budget_total(date=nil) case scope_date_status(date) when :in diff --git a/app/views/deliverables/_details_row.html.erb b/app/views/deliverables/_details_row.html.erb index e41d9f0..856c7b6 100644 --- a/app/views/deliverables/_details_row.html.erb +++ b/app/views/deliverables/_details_row.html.erb @@ -54,13 +54,15 @@ Labor <%= h(format_value_field_for_contracts(deliverable.labor_budget_spent(validated_period))) %> <%= h(format_value_field_for_contracts(deliverable.labor_budget_total(validated_period))) %> - TODO: Release 2 / TODO hrs + + <%= h(format_value_field_for_contracts(deliverable.labor_hours_spent_total(validated_period))) %>/<%= h(format_value_field_for_contracts(deliverable.labor_budget_hours(validated_period))) %> <%= l(:text_short_hours) %> + Overhead <%= h(format_value_field_for_contracts(deliverable.overhead_spent(validated_period))) %> <%= h(format_value_field_for_contracts(deliverable.overhead_budget_total(validated_period))) %> - TODO: Release 2 / TODO hrs + TODO: Release 2 / TODO hrs <% deliverable.fixed_budgets.by_period(validated_period).each do |fixed_budget| %> diff --git a/test/integration/contracts_show_test.rb b/test/integration/contracts_show_test.rb index 4a3b780..101c422 100644 --- a/test/integration/contracts_show_test.rb +++ b/test/integration/contracts_show_test.rb @@ -281,6 +281,40 @@ class ContractsShowTest < ActionController::IntegrationTest end end + should "show the labor hours for the deliverable" do + configure_overhead_plugin + + @manager = User.generate! + + @deliverable1 = FixedDeliverable.generate!(:contract => @contract, :manager => @manager) + LaborBudget.generate!(:deliverable => @deliverable1, + :hours => 100, + :budget => 4000.5) + + @issue1 = Issue.generate_for_project!(@project) + @time_entry1 = TimeEntry.generate!(:issue => @issue1, + :project => @project, + :activity => @billable_activity, + :spent_on => Date.today, + :hours => 10, + :user => @manager) + + @rate = Rate.generate!(:project => @project, + :user => @manager, + :date_in_effect => Date.yesterday, + :amount => 100) + + @deliverable1.issues << @issue1 + + assert_equal 1, @deliverable1.issues.count + + visit_contract_page(@contract) + assert_select "table#deliverables" do + assert_select "td.labor_hours", :text => /10\/100/ + end + + end + should "show the current period for a Retainer" do today_mock = Date.new(2010,2,15) Date.stubs(:today).returns(today_mock) diff --git a/test/unit/retainer_deliverable_test.rb b/test/unit/retainer_deliverable_test.rb index 8460e72..860b0da 100644 --- a/test/unit/retainer_deliverable_test.rb +++ b/test/unit/retainer_deliverable_test.rb @@ -176,6 +176,66 @@ class RetainerDeliverableTest < ActiveSupport::TestCase end end + context "#labor_hours_spent_total" do + setup do + @project = Project.generate! + @contract = Contract.generate!(:billable_rate => 100, :project => @project) + @deliverable = RetainerDeliverable.generate!(:start_date => '2010-01-01', :end_date => '2010-03-31', :contract => @contract) + + @manager = User.generate! + @role = Role.generate! + User.add_to_project(@manager, @project, @role) + + configure_overhead_plugin + + @issue1 = Issue.generate_for_project!(@project) + @time_entry1 = TimeEntry.generate!(:issue => @issue1, + :project => @project, + :activity => @billable_activity, + :spent_on => Date.new(2010,1,2), + :hours => 10, + :user => @manager) + @time_entry2 = TimeEntry.generate!(:issue => @issue1, + :project => @project, + :activity => @billable_activity, + :spent_on => Date.new(2010,2,1), + :hours => 20, + :user => @manager) + + @rate = Rate.generate!(:project => @project, + :user => @manager, + :date_in_effect => Date.new(2010,1,1), + :amount => 100) + + @deliverable.issues << @issue1 + assert_equal 30, @deliverable.labor_hours_spent_total + end + + context "with a empty period" do + should "use all periods" do + assert_equal 30.0, @deliverable.labor_hours_spent_total(nil) + end + end + + context "with a period out of the retainer range" do + should "filter the records" do + assert_equal 0, @deliverable.labor_hours_spent_total(Date.new(2011,1,1)) + end + end + + context "with an invalid period" do + should "return 0" do + assert_equal 0, @deliverable.labor_hours_spent_total('1') + end + end + + context "with a period in the retainer range" do + should "filter the records" do + assert_equal 20.0, @deliverable.labor_hours_spent_total(Date.new(2010,2,1)) + end + end + end + context "#overhead_spent" do setup do @project = Project.generate!