From ac4b60d13162c46ec72112f0ef3d315734868d1e Mon Sep 17 00:00:00 2001
From: Azul <azul@leap.se>
Date: Mon, 19 Nov 2012 14:46:11 +0100
Subject: require authentication for cert download w/ test

---
 certs/app/controllers/certs_controller.rb      | 2 ++
 certs/test/functional/certs_controller_test.rb | 7 +++++++
 certs/test/test_helper.rb                      | 2 +-
 3 files changed, 10 insertions(+), 1 deletion(-)

diff --git a/certs/app/controllers/certs_controller.rb b/certs/app/controllers/certs_controller.rb
index 6988a38..402bef3 100644
--- a/certs/app/controllers/certs_controller.rb
+++ b/certs/app/controllers/certs_controller.rb
@@ -1,5 +1,7 @@
 class CertsController < ApplicationController
 
+  before_filter :authorize
+
   # GET /cert
   def show
     @cert = Cert.pick_from_pool
diff --git a/certs/test/functional/certs_controller_test.rb b/certs/test/functional/certs_controller_test.rb
index 295515b..9bba8c0 100644
--- a/certs/test/functional/certs_controller_test.rb
+++ b/certs/test/functional/certs_controller_test.rb
@@ -4,7 +4,14 @@ class CertsControllerTest < ActionController::TestCase
   setup do
   end
 
+  test "should require login" do
+    get :show
+    assert_response :redirect
+    assert_redirected_to login_url
+  end
+
   test "should send cert" do
+    login
     cert = stub :zipped => "adsf", :zipname => "cert_stub.zip"
     Cert.expects(:pick_from_pool).returns(cert)
     get :show
diff --git a/certs/test/test_helper.rb b/certs/test/test_helper.rb
index 08d4d41..f6b4eb8 100644
--- a/certs/test/test_helper.rb
+++ b/certs/test/test_helper.rb
@@ -1,7 +1,7 @@
 ENV["RAILS_ENV"] = "test"
 require File.expand_path('../../../test/dummy/config/environment', __FILE__)
 require 'rails/test_help'
-require 'mocha'
+require 'mocha/setup'
 
 Rails.backtrace_cleaner.remove_silencers!
 
-- 
cgit v1.2.3