Skip to content

Commit

Permalink
CSHARP-4041: Remove use of example.com in Client Side Encryption pros…
Browse files Browse the repository at this point in the history
…e tests. (#746)
  • Loading branch information
DmitryLukyanov authored Mar 8, 2022
1 parent 6e8c224 commit 5d8edc1
Showing 1 changed file with 6 additions and 6 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -556,14 +556,14 @@ public void CreateDataKeyAndDoubleEncryptionTest(
[InlineData("aws", "kms.us-east-1.amazonaws.com:443", null, null)]
[InlineData("aws", "kms.us-east-1.amazonaws.com:12345", "$ConnectionRefused$", null)]
[InlineData("aws", "kms.us-east-2.amazonaws.com", "us-east-1", null)]
[InlineData("aws", "example.com", "parse error", null)]
[InlineData("aws", "doesnotexist.invalid", "$HostNotFound$", null)]
// additional not spec tests
[InlineData("aws", "$test$", "Invalid endpoint, expected dot separator in host, but got: $test$", null)]
// azure
[InlineData("azure", "key-vault-csfle.vault.azure.net", null, "parse error")]
[InlineData("azure", "key-vault-csfle.vault.azure.net", null, "$HostNotFound$")]
// gcp
[InlineData("gcp", "cloudkms.googleapis.com:443", null, "parse error")]
[InlineData("gcp", "example.com:443", "Invalid KMS response", null)]
[InlineData("gcp", "cloudkms.googleapis.com:443", null, "$HostNotFound$")]
[InlineData("gcp", "doesnotexist.invalid:443", "Invalid KMS response", null)]
// kmip
[InlineData("kmip", null, null, "$HostNotFound$")]
[InlineData("kmip", "localhost:5698", null, null)]
Expand Down Expand Up @@ -665,10 +665,10 @@ void InvalidKmsEndpointConfigurator(string kt, Dictionary<string, object> ko)
switch (kt)
{
case "azure":
ko.Add("identityPlatformEndpoint", "example.com:443");
ko.Add("identityPlatformEndpoint", "doesnotexist.invalid:443");
break;
case "gcp":
ko.Add("endpoint", "example.com:443");
ko.Add("endpoint", "doesnotexist.invalid:443");
break;
case "kmip":
AddOrReplace(ko, "endpoint", "doesnotexist.local:5698");
Expand Down

0 comments on commit 5d8edc1

Please sign in to comment.