Remove test for default port, as it's required by gRPC channel (#156)

* Remove test for default port, as it's required by gRPC channel

* Https is OK

Author: tmarkovski

dotnet/Trinsic.Tests/Tests.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.IO;
 using System.Runtime.CompilerServices;
 using System.Threading.Tasks;
@@ -35,9 +35,9 @@ public void TestParseURL()
         {
             Assert.Throws<UriFormatException>(() => ServiceBase.CreateChannelIfNeeded("localhost"));
             Assert.Throws<ArgumentException>(() => ServiceBase.CreateChannelIfNeeded("localhost:5000"));
-            Assert.Throws<ArgumentException>(() => ServiceBase.CreateChannelIfNeeded("http://localhost"));
+            // Assert.Throws<ArgumentException>(() => ServiceBase.CreateChannelIfNeeded("http://localhost"));
             // Throws because HTTPS is not yet supported.
-            Assert.Throws<ArgumentException>(() => ServiceBase.CreateChannelIfNeeded("https://localhost:5000"));
+            // Assert.Throws<ArgumentException>(() => ServiceBase.CreateChannelIfNeeded("https://localhost:5000"));
 
             Assert.NotNull(ServiceBase.CreateChannelIfNeeded("http://localhost:5000"));
 

dotnet/Trinsic/ServiceBase.cs

@@ -65,7 +65,7 @@ public void SetProfile(WalletProfile profile)
         public static GrpcChannel CreateChannelIfNeeded(string serviceAddress)
         {
             var url = new Uri(serviceAddress);
-            if (url.IsDefaultPort) throw new ArgumentException("GRPC Port and scheme required");
+            //if (url.IsDefaultPort) throw new ArgumentException("GRPC Port and scheme required");
             if ("https".Equals(url.Scheme)) throw new ArgumentException("HTTPS not yet supported");
             return GrpcChannel.ForAddress(serviceAddress, new GrpcChannelOptions());
         }