added test to csv download and style corrections

app/controllers/admin/budget_investments_controller.rb

@@ -65,19 +65,19 @@ class Admin::BudgetInvestmentsController < Admin::BaseController
       csv_string = CSV.generate(headers: true) do |csv|
         csv << attributes
         @investments.each do |investment|
-          id    = investment.id.to_s
+          id = investment.id.to_s
           title = investment.title
           total_votes = investment.total_votes.to_s
-          if investment.administrator.present?
-            administrator = investment.administrator.name
-          else
-            administrator = t("admin.budget_investments.index.no_admin_assigned")
-          end
-          if investment.valuators.empty?
-            valuators = t("admin.budget_investments.index.no_valuators_assigned")
-          else
-            valuators = investment.valuators.collect(&:description_or_name).join(', ')
-          end
+          administrator = if investment.administrator.present?
+                            investment.administrator.name
+                          else
+                            t("admin.budget_investments.index.no_admin_assigned")
+                          end
+          valuators = if investment.valuators.empty?
+                        t("admin.budget_investments.index.no_valuators_assigned")
+                      else
+                        investment.valuators.collect(&:description_or_name).join(', ')
+                      end
           heading_name = investment.heading.name
           price = t("admin.budget_investments.index.feasibility.#{investment.feasibility}", price: investment.formatted_price)
           valuation_finished = investment.valuation_finished? ? t('shared.yes') : t('shared.no')

spec/features/admin/budget_investments_spec.rb

@@ -625,7 +625,23 @@ feature 'Admin budget investments' do
         expect(page).not_to have_link('Selected')
       end
     end
+  end
 
+  context "Selecting csv" do
+
+    scenario "Downloading CSV file" do
+      create(:budget_investment, :unfeasible, budget: @budget)
+      create(:budget_investment, :feasible, budget: @budget)
+
+      visit admin_budget_budget_investments_path(@budget, format: :csv)
+
+      header = page.response_headers['Content-Disposition']
+      header.should match /^attachment/
+      header.should match /filename="budget_investments.csv"$/
+
+      expect(page).to have_content "Budget Investment 1 title"
+      expect(page).to have_content "Budget Investment 2 title"
+    end
   end
 
 end