| <%= milestone.id %> |
diff --git a/app/views/budgets/investments/_milestones.html.erb b/app/views/budgets/investments/_milestones.html.erb
index 9e0f215c3..a9de80429 100644
--- a/app/views/budgets/investments/_milestones.html.erb
+++ b/app/views/budgets/investments/_milestones.html.erb
@@ -8,7 +8,7 @@
<% end %>
- <% @investment.milestones.each do |milestone| %>
+ <% @investment.milestones.order_by_publication_date.each do |milestone| %>
-
<% if milestone.publication_date.present? %>
diff --git a/spec/features/budgets/investments_spec.rb b/spec/features/budgets/investments_spec.rb
index d6fa36318..b41d8a712 100644
--- a/spec/features/budgets/investments_spec.rb
+++ b/spec/features/budgets/investments_spec.rb
@@ -910,9 +910,14 @@ feature 'Budget Investments' do
scenario "Show milestones", :js do
user = create(:user)
investment = create(:budget_investment)
- milestone = create(:budget_investment_milestone, investment: investment, title: "New text to show")
- image = create(:image, imageable: milestone)
- document = create(:document, documentable: milestone)
+ create(:budget_investment_milestone, investment: investment,
+ description: "Last milestone",
+ publication_date: Date.tomorrow)
+ first_milestone = create(:budget_investment_milestone, investment: investment,
+ description: "First milestone",
+ publication_date: Date.yesterday)
+ image = create(:image, imageable: first_milestone)
+ document = create(:document, documentable: first_milestone)
login_as(user)
visit budget_investment_path(budget_id: investment.budget.id, id: investment.id)
@@ -920,10 +925,12 @@ feature 'Budget Investments' do
find("#tab-milestones-label").trigger('click')
within("#tab-milestones") do
- expect(page).to have_content(milestone.description)
- expect(page).to have_content(Date.current)
- expect(page.find("#image_#{milestone.id}")['alt']).to have_content image.title
- expect(page).to have_link document.title
+ expect(first_milestone.description).to appear_before('Last milestone')
+ expect(page).to have_content(Date.tomorrow)
+ expect(page).to have_content(Date.yesterday)
+ expect(page).not_to have_content(Date.current)
+ expect(page.find("#image_#{first_milestone.id}")['alt']).to have_content(image.title)
+ expect(page).to have_link(document.title)
end
end
|