Address more review comments

Author: sethvargo

kms/api/Kms.Samples.Tests/DestroyKeyVersionTests.cs

@@ -44,8 +44,8 @@ public void DestroysKeyVersions()
         // Verify marked for deletion.
         var acceptableStates = new CryptoKeyVersion.Types.CryptoKeyVersionState[]
         {
-          CryptoKeyVersion.Types.CryptoKeyVersionState.DestroyScheduled,
-          CryptoKeyVersion.Types.CryptoKeyVersionState.Destroyed,
+            CryptoKeyVersion.Types.CryptoKeyVersionState.DestroyScheduled,
+            CryptoKeyVersion.Types.CryptoKeyVersionState.Destroyed,
         };
         Assert.Contains(response.State, acceptableStates);
     }

kms/api/Kms.Samples.Tests/IamAddMemberTests.cs

@@ -14,7 +14,6 @@
  * limitations under the License.
  */
 
-using Google.Cloud.Iam.V1;
 using Xunit;
 
 [Collection(nameof(KmsFixture))]
@@ -37,18 +36,7 @@ public void AddsMember()
             projectId: _fixture.ProjectId, locationId: _fixture.LocationId, keyRingId: _fixture.KeyRingId, keyId: _fixture.AsymmetricDecryptKeyId,
             member: "group:test@google.com");
 
-        // Verify result.
-        Binding role = null;
-        foreach (var b in result.Bindings)
-        {
-            if (b.Role == "roles/cloudkms.cryptoKeyEncrypterDecrypter")
-            {
-                role = b;
-                break;
-            }
-        }
-
-        Assert.NotNull(role);
-        Assert.Contains("group:test@google.com", role.Members);
+        Assert.Contains(result.Bindings,
+            binding => binding.Role == "roles/cloudkms.cryptoKeyEncrypterDecrypter" && binding.Members.Contains("group:test@google.com"));
     }
 }

kms/api/Kms.Samples.Tests/IamRemoveMemberTests.cs

@@ -14,7 +14,6 @@
  * limitations under the License.
  */
 
-using Google.Cloud.Iam.V1;
 using Xunit;
 
 [Collection(nameof(KmsFixture))]
@@ -37,20 +36,7 @@ public void RemovesMembers()
             projectId: _fixture.ProjectId, locationId: _fixture.LocationId, keyRingId: _fixture.KeyRingId, keyId: _fixture.AsymmetricSignEcKeyId,
             member: "group:test@google.com");
 
-        // Verify result.
-        Binding role = null;
-        foreach (var b in result.Bindings)
-        {
-            if (b.Role == "roles/cloudkms.cryptoKeyEncrypterDecrypter")
-            {
-                role = b;
-                break;
-            }
-        }
-
-        if (role != null)
-        {
-            Assert.DoesNotContain("group:test@google.com", role.Members);
-        }
+        Assert.DoesNotContain(result.Bindings,
+            binding => binding.Role == "roles/cloudkms.cryptoKeyEncrypterDecrypter" && binding.Members.Contains("group:test@google.com"));
     }
 }

kms/api/Kms.Samples/CreateKeyAsymmetricDecrypt.cs

@@ -15,6 +15,7 @@
  */
 
 // [START kms_create_key_asymmetric_decrypt]
+
 using Google.Cloud.Kms.V1;
 
 public class CreateKeyAsymmetricDecryptSample

kms/api/Kms.Samples/CreateKeyAsymmetricSign.cs

@@ -15,6 +15,7 @@
  */
 
 // [START kms_create_key_asymmetric_sign]
+
 using Google.Cloud.Kms.V1;
 
 public class CreateKeyAsymmetricSignSample

kms/api/Kms.Samples/CreateKeyHsm.cs

@@ -15,6 +15,7 @@
  */
 
 // [START kms_create_key_hsm]
+
 using Google.Cloud.Kms.V1;
 
 public class CreateKeyHsmSample

kms/api/Kms.Samples/CreateKeyLabels.cs

@@ -15,6 +15,7 @@
  */
 
 // [START kms_create_key_labels]
+
 using Google.Cloud.Kms.V1;
 
 public class CreateKeyLabelsSample

kms/api/Kms.Samples/CreateKeyRing.cs

@@ -16,6 +16,7 @@
 
 // [START kms_create_key_ring]
 
+
 using Google.Api.Gax.ResourceNames;
 using Google.Cloud.Kms.V1;
 

kms/api/Kms.Samples/CreateKeyRotationSchedule.cs

@@ -15,6 +15,7 @@
  */
 
 // [START kms_create_key_rotation_schedule]
+
 using Google.Cloud.Kms.V1;
 using Google.Protobuf.WellKnownTypes;
 using System;

kms/api/Kms.Samples/CreateKeySymmetricEncryptDecrypt.cs

@@ -16,6 +16,7 @@
 
 // [START kms_create_key_symmetric_encrypt_decrypt]
 
+
 using Google.Cloud.Kms.V1;
 
 public class CreateKeySymmetricEncryptDecryptSample