🔥 Remove the ToHttps method

We no longer need this.

Author: haacked

src/GitHub.Extensions/UriExtensions.cs

@@ -38,30 +38,6 @@ public static Uri WithAbsolutePath(this Uri uri, string absolutePath)
             return new Uri(uri, new Uri(absolutePath, UriKind.Relative));
         }
 
-        [return: AllowNull]
-        public static Uri ToHttps([AllowNull] this Uri uri)
-        {
-            if (uri == null)
-                return null;
-
-            var str = uri.ToString();
-            if (str.EndsWith(".git", StringComparison.Ordinal))
-                str = str.Remove(str.Length - 4);
-
-            if (str.StartsWith("[email protected]:", StringComparison.Ordinal))
-                str = str.Replace("[email protected]:", "https://github.com/");
-
-            if (!Uri.TryCreate(str, UriKind.Absolute, out uri))
-                return null;
-
-            var uriBuilder = new UriBuilder(uri);
-
-            uriBuilder.Scheme = Uri.UriSchemeHttps;
-            // trick to keep uriBuilder from explicitly appending :80 to the HTTPS URI
-            uriBuilder.Port = -1;
-            return uriBuilder.Uri;
-        }
-
         public static string ToUpperInvariantString(this Uri uri)
         {
             return uri == null ? "" : uri.ToString().ToUpperInvariant();

src/UnitTests/GitHub.Extensions/URITests.cs

@@ -1,7 +1,4 @@
 using System;
-using GitHub.Services;
-using Microsoft.TeamFoundation.Git.Controls.Extensibility;
-using NSubstitute;
 using Xunit;
 using GitHub.Extensions;
 
@@ -28,22 +25,4 @@ public void OnlyParsesUrlsWithThreeParts(string uri, string expected)
             Assert.Equal(new Uri(uri).GetRepo(), expected);
         }
     }
-
-    public class TheHttpsMethod : TestBaseClass
-    {
-        [Theory]
-        [InlineData(null, null)]
-        [InlineData("", null)]
-        [InlineData("[email protected]:bla/test.git", "https://github.com/bla/test")]
-        [InlineData("git://github.com/bla/test.git", "https://github.com/bla/test")]
-        public void CreatesHttpsUrl(string uri, string expected)
-        {
-            Uri value = null;
-            Uri.TryCreate(uri, UriKind.RelativeOrAbsolute, out value);
-
-            var ret = value.ToHttps();
-            var ret2 = ret != null ? ret.ToString() : null;
-            Assert.Equal(expected, ret2);
-        }
-    }
 }