diff options
| author | Azul <azul@leap.se> | 2012-11-19 14:46:11 +0100 |
|---|---|---|
| committer | Azul <azul@leap.se> | 2012-11-19 14:46:11 +0100 |
| commit | ac4b60d13162c46ec72112f0ef3d315734868d1e (patch) | |
| tree | 0921adcba7a56e8709dcec923851ccf273437feb | |
| parent | 5a5e73f2fefa8e1a06f18063a6d644a6d1c2a869 (diff) | |
require authentication for cert download w/ test
| -rw-r--r-- | certs/app/controllers/certs_controller.rb | 2 | ||||
| -rw-r--r-- | certs/test/functional/certs_controller_test.rb | 7 | ||||
| -rw-r--r-- | certs/test/test_helper.rb | 2 |
3 files changed, 10 insertions, 1 deletions
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! |