Better handling of ssl_context option. Fixes #70.

Author: ioquatix

lib/async/redis/endpoint.rb

@@ -80,7 +80,7 @@ def self.for(scheme, host, port: nil, database: nil, **options)
 					scheme ||= default_scheme
 				end
 
-				scheme ||= options.key?(:ssl_context) ? "rediss" : "redis"
+				scheme ||= options[:ssl_context] ? "rediss" : "redis"
 
 				uri_klass = SCHEMES.fetch(scheme.downcase) do
 					raise ArgumentError, "Unsupported scheme: #{scheme.inspect}"

test/async/redis/endpoint.rb

@@ -163,6 +163,13 @@
 			expect(endpoint.scheme).to be == "redis"
 			expect(endpoint).not.to be(:secure?)  # scheme takes precedence
 		end
+		
+		it "does not treat ssl_context: nil as requesting SSL" do
+			ssl_context = nil
+			endpoint = Async::Redis::Endpoint.for(nil, "localhost", port: 6379, ssl_context: ssl_context)
+			expect(endpoint.scheme).to be == "redis"
+			expect(endpoint).not.to be(:secure?)
+		end
 	end
 
 	with ".unix" do