Merge pull request #17903 from claudiob/remove-rescue-openssl

Remove "rescue" clause around "require 'openssl'"

actionpack/test/dispatch/cookies_test.rb

 require 'abstract_unit'
-
-begin
-  require 'openssl'
-  OpenSSL::PKCS5
-rescue LoadError, NameError
-  $stderr.puts "Skipping KeyGenerator test: broken OpenSSL install"
-else
-
+require 'openssl'
 require 'active_support/key_generator'
 require 'active_support/message_verifier'
 
@@ -1155,5 +1148,3 @@ def assert_not_cookie_header(expected)
       end
     end
 end
-
-end

activesupport/test/message_encryptor_test.rb

 require 'abstract_unit'
-
-begin
-  require 'openssl'
-  OpenSSL::Digest::SHA1
-rescue LoadError, NameError
-  $stderr.puts "Skipping MessageEncryptor test: broken OpenSSL install"
-else
-
+require 'openssl'
 require 'active_support/time'
 require 'active_support/json'
 
@@ -97,5 +90,3 @@ def munge(base64_string)
     ::Base64.strict_encode64(bits)
   end
 end
-
-end

activesupport/test/message_verifier_test.rb

 require 'abstract_unit'
-
-begin
-  require 'openssl'
-  OpenSSL::Digest::SHA1
-rescue LoadError, NameError
-  $stderr.puts "Skipping MessageVerifier test: broken OpenSSL install"
-else
-
+require 'openssl'
 require 'active_support/time'
 require 'active_support/json'
 
@@ -41,11 +34,11 @@ def test_simple_round_tripping
     assert_equal @data, @verifier.verified(message)
     assert_equal @data, @verifier.verify(message)
   end
-  
+
   def test_verified_returns_false_on_invalid_message
     assert !@verifier.verified("purejunk")
   end
-  
+
   def test_verify_exception_on_invalid_message
     assert_raise(ActiveSupport::MessageVerifier::InvalidSignature) do
       @verifier.verify("purejunk")
@@ -90,5 +83,3 @@ def test_raise_error_when_secret_is_nil
     assert_equal exception.message, 'Secret should not be nil.'
   end
 end
-
-end