[#4178] Fixed some more tests from the UI changes.
This commit is contained in:
1 parent
cb8fd95f29
commit
61270c60d5
3 files changed
+9
-7
No files matched your search
@@ -6,12 +6,14 @@ module ContractsHelper
|
|||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
def format_budget_for_deliverable(deliverable, spent, total)
|
def format_budget_for_deliverable(deliverable, spent, total, options={})
|
||||||
|
extra_css_class = options[:class] || ''
|
||||||
|
|
||||||
if total > 0 || spent > 0
|
if total > 0 || spent > 0
|
||||||
content_tag(:td, h(number_to_currency(spent, :unit => '')), :class => 'spent-amount') +
|
content_tag(:td, h(number_to_currency(spent, :unit => '')), :class => 'spent-amount ' + extra_css_class) +
|
||||||
content_tag(:td, h(number_to_currency(total, :unit => '')), :class => 'total-amount white')
|
content_tag(:td, h(number_to_currency(total, :unit => '')), :class => 'total-amount white ' + extra_css_class)
|
||||||
else
|
else
|
||||||
content_tag(:td, '----', :colspan => '2', :class => 'no-value')
|
content_tag(:td, '----', :colspan => '2', :class => 'no-value ' + extra_css_class)
|
||||||
end
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
|
|||||||
@@ -102,8 +102,8 @@
|
|||||||
<td width="25%" class="title"><%= h deliverable.title %></td>
|
<td width="25%" class="title"><%= h deliverable.title %></td>
|
||||||
<td width="15%">TODO: Release 5</td>
|
<td width="15%">TODO: Release 5</td>
|
||||||
<td width="15%" class="manager"><%= h deliverable.manager.try(:name) %></td>
|
<td width="15%" class="manager"><%= h deliverable.manager.try(:name) %></td>
|
||||||
<%= format_budget_for_deliverable(deliverable, deliverable.labor_budget_spent, deliverable.labor_budget_total) %>
|
<%= format_budget_for_deliverable(deliverable, deliverable.labor_budget_spent, deliverable.labor_budget_total, :class => 'labor') %>
|
||||||
<%= format_budget_for_deliverable(deliverable, deliverable.overhead_spent, deliverable.overhead_budget_total) %>
|
<%= format_budget_for_deliverable(deliverable, deliverable.overhead_spent, deliverable.overhead_budget_total, :class => 'overhead') %>
|
||||||
<%= format_budget_for_deliverable(deliverable, 0, 0) %><%# TODO: Release 2, Fixed Budgets %>
|
<%= format_budget_for_deliverable(deliverable, 0, 0) %><%# TODO: Release 2, Fixed Budgets %>
|
||||||
<% end %>
|
<% end %>
|
||||||
|
|
||||||
|
|||||||
@@ -45,7 +45,7 @@ class DeliverablesEditTest < ActionController::IntegrationTest
|
|||||||
|
|
||||||
should "allow any user to edit the Hourly deliverable" do
|
should "allow any user to edit the Hourly deliverable" do
|
||||||
visit_contract_page(@contract)
|
visit_contract_page(@contract)
|
||||||
click_link_within "#hourly_deliverable_#{@hourly_deliverable.id}", 'Edit'
|
click_link_within "#deliverable_details_#{@hourly_deliverable.id}", 'Edit'
|
||||||
assert_response :success
|
assert_response :success
|
||||||
assert_template 'deliverables/edit'
|
assert_template 'deliverables/edit'
|
||||||
|
|
||||||
|
|||||||
Reference in new issue
Block a user