shows links to unfeasible/selected only after budget is balloting

app/views/budgets/groups/show.html.erb

@@ -45,24 +45,26 @@
   </div>
 </div>
 
-<% unless @current_filter == "unfeasible" %>
-  <div class="row">
-    <div class="small-12 column">
-      <small>
-        <%= link_to t("budgets.groups.show.unfeasible"),
-                    budget_path(@budget, filter: "unfeasible") %>
-      </small>
+<% if @budget.balloting_or_later? %>
+  <% unless @current_filter == "unfeasible" %>
+    <div class="row">
+      <div class="small-12 column">
+        <small>
+          <%= link_to t("budgets.groups.show.unfeasible"),
+                      budget_path(@budget, filter: "unfeasible") %>
+        </small>
+      </div>
     </div>
-  </div>
-<% end %>
+  <% end %>
 
-<% unless @current_filter == "unselected" %>
-  <div class="row">
-    <div class="small-12 column">
-      <small>
-        <%= link_to t("budgets.groups.show.unselected"),
-                    budget_path(@budget, filter: "unselected") %>
-      </small>
+  <% unless @current_filter == "unselected" %>
+    <div class="row">
+      <div class="small-12 column">
+        <small>
+          <%= link_to t("budgets.groups.show.unselected"),
+                      budget_path(@budget, filter: "unselected") %>
+        </small>
+      </div>
     </div>
-  </div>
+  <% end %>
 <% end %>

app/views/budgets/show.html.erb

@@ -70,24 +70,26 @@
   </div>
 </div>
 
-<% unless @current_filter == "unfeasible" %>
-  <div class="row">
-    <div class="small-12 column">
-      <small>
-        <%= link_to t("budgets.show.unfeasible"),
-                    budget_path(@budget, filter: "unfeasible") %>
-      </small>
+<% if @budget.balloting_or_later? %>
+  <% unless @current_filter == "unfeasible" %>
+    <div class="row">
+      <div class="small-12 column">
+        <small>
+          <%= link_to t("budgets.show.unfeasible"),
+                      budget_path(@budget, filter: "unfeasible") %>
+        </small>
+      </div>
     </div>
-  </div>
-<% end %>
+  <% end %>
 
-<% unless @current_filter == "unselected" %>
-  <div class="row">
-    <div class="small-12 column">
-      <small>
-        <%= link_to t("budgets.show.unselected"),
-                    budget_path(@budget, filter: "unselected") %>
-      </small>
+  <% unless @current_filter == "unselected" %>
+    <div class="row">
+      <div class="small-12 column">
+        <small>
+          <%= link_to t("budgets.show.unselected"),
+                      budget_path(@budget, filter: "unselected") %>
+        </small>
+      </div>
     </div>
-  </div>
+  <% end %>
 <% end %>

spec/features/budgets/budgets_spec.rb

@@ -8,14 +8,57 @@ feature 'Budgets' do
     budgets.each {|budget| expect(page).to have_link(budget.name)}
   end
 
-  scenario 'Show' do
-    budget = create(:budget)
-    group1 = create(:budget_group, budget: budget)
-    group2 = create(:budget_group, budget: budget)
+  context 'Show' do
 
-    visit budget_path(budget)
+    scenario "List all groups" do
+      budget = create(:budget)
+      group1 = create(:budget_group, budget: budget)
+      group2 = create(:budget_group, budget: budget)
+
+      visit budget_path(budget)
+
+      budget.groups.each {|group| expect(page).to have_link(group.name)}
+    end
+
+    scenario "Links to unfeasible and selected if balloting or later" do
+      budget = create(:budget, :selecting)
+      group = create(:budget_group, budget: budget)
+
+      visit budget_path(budget)
+
+      expect(page).to_not have_link "See unfeasible investments"
+      expect(page).to_not have_link "See investments not selected for balloting phase"
+
+      click_link group.name
+
+      expect(page).to_not have_link "See unfeasible investments"
+      expect(page).to_not have_link "See investments not selected for balloting phase"
+
+      budget.update(phase: :balloting)
+
+      visit budget_path(budget)
+
+      expect(page).to have_link "See unfeasible investments"
+      expect(page).to have_link "See investments not selected for balloting phase"
+
+      click_link group.name
+
+      expect(page).to have_link "See unfeasible investments"
+      expect(page).to have_link "See investments not selected for balloting phase"
+
+      budget.update(phase: :finished)
+
+      visit budget_path(budget)
+
+      expect(page).to have_link "See unfeasible investments"
+      expect(page).to have_link "See investments not selected for balloting phase"
+
+      click_link group.name
+
+      expect(page).to have_link "See unfeasible investments"
+      expect(page).to have_link "See investments not selected for balloting phase"
+    end
 
-    budget.groups.each {|group| expect(page).to have_link(group.name)}
   end
 
   context 'Accepting' do