From 778d2324623acfc013d9b22855be036fbf503827 Mon Sep 17 00:00:00 2001
From: Markus Jung <markus.jung@vivavis.com>
Date: Wed, 7 Aug 2024 17:22:04 +0200
Subject: [PATCH] Refactoring to incorporate review feedback

---
 ext/openssl/ossl_ssl.c   |  3 ---
 test/openssl/test_ssl.rb | 38 +++++++++++++++++++-------------------
 2 files changed, 19 insertions(+), 22 deletions(-)

diff --git a/ext/openssl/ossl_ssl.c b/ext/openssl/ossl_ssl.c
index cd7f4878f..a77e038e1 100644
--- a/ext/openssl/ossl_ssl.c
+++ b/ext/openssl/ossl_ssl.c
@@ -1120,9 +1120,6 @@ ossl_sslctx_set_sigalgs(VALUE self, VALUE v)
 {
     SSL_CTX *ctx;
 
-    if (NIL_P(v))
-        return v;
-
     rb_check_frozen(self);
     GetSSLCTX(self, ctx);
 
diff --git a/test/openssl/test_ssl.rb b/test/openssl/test_ssl.rb
index b9d86f332..18a9fb726 100644
--- a/test/openssl/test_ssl.rb
+++ b/test/openssl/test_ssl.rb
@@ -1736,25 +1736,6 @@ def test_ciphersuites_method_bogus_csuite
     ) { ssl_ctx.ciphersuites = 'BOGUS' }
   end
 
-  def test_ciphers_method_tls_connection
-    csuite = ['ECDHE-RSA-AES256-GCM-SHA384', 'TLSv1.2', 256, 256]
-    inputs = [csuite[0], [csuite[0]], [csuite]]
-
-    start_server do |port|
-      inputs.each do |input|
-        cli_ctx = OpenSSL::SSL::SSLContext.new
-        cli_ctx.min_version = cli_ctx.max_version = OpenSSL::SSL::TLS1_2_VERSION
-        cli_ctx.ciphers = input
-
-        server_connect(port, cli_ctx) do |ssl|
-          assert_equal('TLSv1.2', ssl.ssl_version)
-          assert_equal(csuite[0], ssl.cipher[0])
-          ssl.puts('abc'); assert_equal("abc\n", ssl.gets)
-        end
-      end
-    end
-  end
-
   def test_sigalgs_method_nil_argument
     ssl_ctx = OpenSSL::SSL::SSLContext.new
     pend 'sigalgs= method is missing' unless ssl_ctx.respond_to?(:sigalgs=)
@@ -1785,6 +1766,25 @@ def test_sigalgs_method_bogus_sigalgs
     assert_raise(OpenSSL::SSL::SSLError) { ssl_ctx.sigalgs = 'BOGUS' }
   end
 
+  def test_ciphers_method_tls_connection
+    csuite = ['ECDHE-RSA-AES256-GCM-SHA384', 'TLSv1.2', 256, 256]
+    inputs = [csuite[0], [csuite[0]], [csuite]]
+
+    start_server do |port|
+      inputs.each do |input|
+        cli_ctx = OpenSSL::SSL::SSLContext.new
+        cli_ctx.min_version = cli_ctx.max_version = OpenSSL::SSL::TLS1_2_VERSION
+        cli_ctx.ciphers = input
+
+        server_connect(port, cli_ctx) do |ssl|
+          assert_equal('TLSv1.2', ssl.ssl_version)
+          assert_equal(csuite[0], ssl.cipher[0])
+          ssl.puts('abc'); assert_equal("abc\n", ssl.gets)
+        end
+      end
+    end
+  end
+
   def test_ciphers_method_nil_argument
     ssl_ctx = OpenSSL::SSL::SSLContext.new
     assert_nothing_raised { ssl_ctx.ciphers = nil }