Test fixes

laugel · Jun 4, 2018 · 3073e12 · 3073e12
1 parent 20675c1
commit 3073e12
Show file tree

Hide file tree

Showing 4 changed files with 9 additions and 4 deletions.
diff --git a/src/Certify.Core/Management/Servers/ServerProviderIIS.cs b/src/Certify.Core/Management/Servers/ServerProviderIIS.cs
@@ -248,6 +248,8 @@ public async Task<bool> IsSNISupported()
 
         public async Task<ActionStep> AddOrUpdateSiteBinding(BindingInfo bindingSpec, bool addNew)
         {
+            if (string.IsNullOrEmpty(bindingSpec.SiteId)) throw new Exception("IIS.AddOrUpdateSiteBinding: SiteId not specified");
+
             var result = new ActionStep { };
 
             using (var iisManager = await GetDefaultServerManager())

diff --git a/src/Certify.Tests/Certify.Core.Tests.Integration/CertRequestTests.cs b/src/Certify.Tests/Certify.Core.Tests.Integration/CertRequestTests.cs
@@ -353,7 +353,8 @@ public async Task TestChallengeRequestDNS()
                             ChallengeProvider= "DNS01.API.Route53",
                             ChallengeCredentialKey=_awsCredStorageKey
                         }
-                    }
+                    },
+                    DeploymentSiteOption = DeploymentOption.SingleSite
                 },
                 ItemType = ManagedCertificateType.SSL_LetsEncrypt_LocalIIS
             };
@@ -564,4 +565,4 @@ private string GetPreviewSummary(List<ActionStep> steps)
             return output;
         }
     }
-}
+}
diff --git a/src/Certify.Tests/Certify.Core.Tests.Integration/IISManagerTests.cs b/src/Certify.Tests/Certify.Core.Tests.Integration/IISManagerTests.cs
@@ -162,7 +162,8 @@ public async Task TestLongBinding()
                     CertificateHashBytes = cert.GetCertHash(),
                     CertificateStore = certStoreName,
                     Port = 443,
-                    Protocol = "https"
+                    Protocol = "https",
+                    SiteId= site.Id.ToString()
                 }
                );
 

diff --git a/src/Certify.Tests/Certify.Core.Tests.Unit/BindingMatchTests.cs b/src/Certify.Tests/Certify.Core.Tests.Unit/BindingMatchTests.cs
@@ -76,7 +76,8 @@ All Sites(*.test.co.uk)
                             }
                         }),
                     PerformAutomatedCertBinding = true,
-                    WebsiteRootPath = "c:\\inetpub\\wwwroot"
+                    WebsiteRootPath = "c:\\inetpub\\wwwroot",
+                    DeploymentSiteOption = DeploymentOption.SingleSite
                 },
                 ItemType = ManagedCertificateType.SSL_LetsEncrypt_LocalIIS
             };