update for CredentialModel having a Uri type

Author: spikecurtis

App/Services/UriHandler.cs

@@ -140,9 +140,9 @@ private void CheckAuthority(Uri uri, string errTitle)
                 $"Failed to open '{uri.AbsolutePath}' because you are not signed in.");
         }
 
-        // here we assume that the URL is valid since the credentials are marked valid. If not it's an internal error
+        // here we assume that the URL is non-null since the credentials are marked valid. If not it's an internal error
         // and the App will handle catching the exception and logging it.
-        var coderUri = new Uri(credentialModel.CoderUrl!);
+        var coderUri = credentialModel.CoderUrl!;
         if (uri.Authority != coderUri.Authority)
         {
             logger.LogWarning(

Tests.App/Services/UriHandlerTest.cs

@@ -64,7 +64,7 @@ public void AgentWorkspaceAndCredentialFixtures()
         credentialModel1 = new CredentialModel
         {
             State = CredentialState.Valid,
-            CoderUrl = "https://coder.test",
+            CoderUrl = new Uri("https://coder.test"),
         };
     }