diff options
author | Kostiantyn Kahanskyi <kostiantyn.kahanskyi@googlemail.com> | 2014-09-12 14:58:04 +0200 |
---|---|---|
committer | Kostiantyn Kahanskyi <kostiantyn.kahanskyi@googlemail.com> | 2014-09-12 15:09:00 +0200 |
commit | 4bf9d1938bb144ef2a6c3d03a2d01add62e90114 (patch) | |
tree | c881ff8cf675795aa9057957dd3f6615820b4eb1 /activesupport/test | |
parent | b259b06e73a44f87654d26759b0a679e1ecc069c (diff) | |
download | rails-4bf9d1938bb144ef2a6c3d03a2d01add62e90114.tar.gz rails-4bf9d1938bb144ef2a6c3d03a2d01add62e90114.tar.bz2 rails-4bf9d1938bb144ef2a6c3d03a2d01add62e90114.zip |
MessageVerifier raises an appropriate exception if the secret is nil
Otherwise this will lead to another error later on
when generating a signature:
TypeError (no implicit conversion of nil into String).
Diffstat (limited to 'activesupport/test')
-rw-r--r-- | activesupport/test/message_verifier_test.rb | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/activesupport/test/message_verifier_test.rb b/activesupport/test/message_verifier_test.rb index a5748d28ba..28035bc428 100644 --- a/activesupport/test/message_verifier_test.rb +++ b/activesupport/test/message_verifier_test.rb @@ -69,6 +69,13 @@ class MessageVerifierTest < ActiveSupport::TestCase "undefined class/module MessageVerifierTest::AutoloadClass"], exception.message end + def test_raise_error_when_secret_is_nil + exception = assert_raise(ArgumentError) do + ActiveSupport::MessageVerifier.new(nil) + end + assert_equal exception.message, 'Secret should not be nil.' + end + def assert_not_verified(message) assert_raise(ActiveSupport::MessageVerifier::InvalidSignature) do @verifier.verify(message) |