Refactored buildUrl

Author: darrelmiller

src/Microsoft.OpenApi.Readers/V2/OpenApiDocumentDeserializer.cs

@@ -149,50 +149,19 @@ private static void MakeServers(IList<OpenApiServer> servers, ParsingContext con
             {
                 foreach (var scheme in schemes)
                 {
-                    if (String.IsNullOrEmpty(scheme))
-                    {
-                        host = "//" + host;  // The double slash prefix creates a relative url where the scheme is defined by the BaseUrl
-                    }
-                    int? port = null;
-                    if (host.Contains(":"))
-                    {
-                        var pieces = host.Split(':');
-                        host = pieces.First();
-                        port = int.Parse(pieces.Last());
-                    }
-                    var uriBuilder = new UriBuilder(scheme, host)
-                    {
-                        Path = basePath
-                    };
-
-                    if (port != null)
-                    {
-                        uriBuilder.Port = port.Value;
-                    }
-
                     var server = new OpenApiServer
                     {
-                        Url = uriBuilder.ToString()
+                        Url = BuildUrl(scheme, host, basePath)
                     };
 
                     servers.Add(server);
                 }
             }
             else
             {
-                if (!String.IsNullOrEmpty(host))
-                {
-                    host = "//" + host;  // The double slash prefix creates a relative url where the scheme is defined by the BaseUrl
-                }
-                var uriBuilder = new UriBuilder()
-                {
-                    Scheme = null,
-                    Host = host,
-                    Path = basePath
-                };
                 var server = new OpenApiServer
                 {
-                    Url = uriBuilder.ToString()
+                    Url = BuildUrl(null, host, basePath)
                 };
 
                 servers.Add(server);
@@ -209,6 +178,37 @@ private static void MakeServers(IList<OpenApiServer> servers, ParsingContext con
             }
         }
 
+        private static string BuildUrl(string scheme, string host, string basePath)
+        {
+            if (String.IsNullOrEmpty(scheme) && !String.IsNullOrEmpty(host))
+            {
+                host = "//" + host;  // The double slash prefix creates a relative url where the scheme is defined by the BaseUrl
+            }
+
+            int? port = null;
+
+            if (!String.IsNullOrEmpty(host) && host.Contains(":"))
+            {
+                var pieces = host.Split(':');
+                host = pieces.First();
+                port = int.Parse(pieces.Last());
+            }
+
+            var uriBuilder = new UriBuilder()
+            {
+                Scheme = scheme,
+                Host = host,
+                Path = basePath
+            };
+
+            if (port != null)
+            {
+                uriBuilder.Port = port.Value;
+            }
+
+            return uriBuilder.ToString();
+        }
+
         public static OpenApiDocument LoadOpenApi(RootNode rootNode)
         {
             var openApidoc = new OpenApiDocument();