GODRIVER-1086 don't return the uri when parsing fails

Change-Id: Ia7ce0035da57defa49359cb1c134d32507505210

Author: iwysiu

mongo/options/clientoptions_test.go

@@ -28,7 +28,7 @@ func TestClientOptions(t *testing.T) {
 	t.Run("ApplyURI/doesn't overwrite previous errors", func(t *testing.T) {
 		uri := "not-mongo-db-uri://"
 		want := internal.WrapErrorf(
-			errors.New(`scheme must be "mongodb" or "mongodb+srv"`), "error parsing uri (%s)", "not-mongo-db-uri://",
+			errors.New(`scheme must be "mongodb" or "mongodb+srv"`), "error parsing uri",
 		)
 		co := Client().ApplyURI(uri).ApplyURI("mongodb://localhost/")
 		got := co.Validate()
@@ -177,7 +177,7 @@ func TestClientOptions(t *testing.T) {
 				"ParseError",
 				"not-mongo-db-uri://",
 				&ClientOptions{err: internal.WrapErrorf(
-					errors.New(`scheme must be "mongodb" or "mongodb+srv"`), "error parsing uri (%s)", "not-mongo-db-uri://",
+					errors.New(`scheme must be "mongodb" or "mongodb+srv"`), "error parsing uri",
 				)},
 			},
 			{

x/network/connstring/connstring.go

@@ -26,7 +26,7 @@ func Parse(s string) (ConnString, error) {
 	var p parser
 	err := p.parse(s)
 	if err != nil {
-		err = internal.WrapErrorf(err, "error parsing uri (%s)", s)
+		err = internal.WrapErrorf(err, "error parsing uri")
 	}
 	return p.ConnString, err
 }