GODRIVER-3452 MergeClientOptions returns object when given nil arguments (#1917) [release/2.0] (#1948)

Co-authored-by: Preston Vasquez <[email protected]>

Author: matthewdale

mongo/options/clientoptions.go

@@ -1265,6 +1265,10 @@ func extractX509UsernameFromSubject(subject string) string {
 // precedence.
 func MergeClientOptions(opts ...*ClientOptions) *ClientOptions {
 	if len(opts) == 1 {
+		if opts[0] == nil {
+			return Client()
+		}
+
 		return opts[0]
 	}
 

mongo/options/clientoptions_test.go

@@ -175,6 +175,16 @@ func TestClientOptions(t *testing.T) {
 				t.Errorf("Merged client options do not match. got %v; want %v", got.err.Error(), opt1.err.Error())
 			}
 		})
+
+		t.Run("MergeClientOptions single nil option", func(t *testing.T) {
+			got := MergeClientOptions(nil)
+			assert.Equal(t, Client(), got)
+		})
+
+		t.Run("MergeClientOptions multiple nil options", func(t *testing.T) {
+			got := MergeClientOptions(nil, nil)
+			assert.Equal(t, Client(), got)
+		})
 	})
 	t.Run("direct connection validation", func(t *testing.T) {
 		t.Run("multiple hosts", func(t *testing.T) {