Skip to content
Browse files

Merge pull request #5692 from avakhov/force-ssl-if-test

Tests :if option of force_ssl method
  • Loading branch information...
2 parents 0244c0d + 7f6bb2d commit e357d5b99343f6470dd3fe8bfc410a4227f3ba93 @josevalim josevalim committed Apr 1, 2012
Showing with 23 additions and 0 deletions.
  1. +23 −0 actionpack/test/controller/force_ssl_test.rb
View
23 actionpack/test/controller/force_ssl_test.rb
@@ -26,6 +26,14 @@ class ForceSSLExceptAction < ForceSSLController
force_ssl :except => :banana
end
+class ForceSSLIfCondition < ForceSSLController
+ force_ssl :if => :use_force_ssl?
+
+ def use_force_ssl?
+ action_name == 'cheeseburger'
+ end
+end
+
class ForceSSLFlash < ForceSSLController
force_ssl :except => [:banana, :set_flash, :use_flash]
@@ -109,6 +117,21 @@ def test_cheeseburger_redirects_to_https
end
end
+class ForceSSLIfConditionTest < ActionController::TestCase
+ tests ForceSSLIfCondition
+
+ def test_banana_not_redirects_to_https
+ get :banana
+ assert_response 200
+ end
+
+ def test_cheeseburger_redirects_to_https
+ get :cheeseburger
+ assert_response 301
+ assert_equal "https://test.host/force_ssl_if_condition/cheeseburger", redirect_to_url
+ end
+end
+
class ForceSSLFlashTest < ActionController::TestCase
tests ForceSSLFlash

0 comments on commit e357d5b

Please sign in to comment.
Something went wrong with that request. Please try again.