chore: Add SecretManager service regional code samples

YashSaraf11 · YashSaraf11 · commit 56c7795814ea · 2025-03-11T14:27:38.000Z
diff --git a/secretmanager/api/SecretManager.Samples.Tests/AddRegionalSecretVersionTests.cs b/secretmanager/api/SecretManager.Samples.Tests/AddRegionalSecretVersionTests.cs
@@ -45,12 +45,12 @@ public void AddsRegionalSecretVersions()
         );
 
         // Access the secret version.
-        AccessSecretVersionResponse result = _fixture.client.AccessSecretVersion(secretVersion.SecretVersionName);
+        AccessSecretVersionResponse result = _fixture.Client.AccessSecretVersion(secretVersion.SecretVersionName);
 
         // Assert that the secret version was added with the correct data.
         Assert.Equal(data, result.Payload.Data.ToStringUtf8());
 
-        // Clean the created resources.
+        // Clean the created secret.
         _fixture.DeleteSecret(secret.SecretName);
     }
 }
diff --git a/secretmanager/api/SecretManager.Samples.Tests/CreateRegionalSecretWithAnnotationsTests.cs b/secretmanager/api/SecretManager.Samples.Tests/CreateRegionalSecretWithAnnotationsTests.cs
@@ -51,7 +51,7 @@ public void CreatesRegionalSecretsWithAnnotations()
         // Assert that the annotation values matches with the expected value.
         Assert.Equal(result.Annotations[annotationKey], annotationValue);
 
-        // Clean the created resources.
+        // Clean the created secret.
         _fixture.DeleteSecret(secretName);
     }
 }
diff --git a/secretmanager/api/SecretManager.Samples.Tests/CreateRegionalSecretWithLabelsTests.cs b/secretmanager/api/SecretManager.Samples.Tests/CreateRegionalSecretWithLabelsTests.cs
@@ -51,7 +51,7 @@ public void CreatesRegionalSecretsWithLabels()
         // Assert that the label value matches with the expected value.
         Assert.Equal(result.Labels[labelKey], labelValue);
 
-        // Clean the created resources.
+        // Clean the created secret.
         _fixture.DeleteSecret(secretName);
     }
 }
diff --git a/secretmanager/api/SecretManager.Samples.Tests/DeleteRegionalSecretTests.cs b/secretmanager/api/SecretManager.Samples.Tests/DeleteRegionalSecretTests.cs
@@ -46,6 +46,6 @@ public void DeletesRegionalSecrets()
         );
 
         // Assert that the secret was deleted.
-        Assert.Throws<Grpc.Core.RpcException>(() => _fixture.client.GetSecret(secretName));
+        Assert.Throws<Grpc.Core.RpcException>(() => _fixture.Client.GetSecret(secretName));
     }
 }
diff --git a/secretmanager/api/SecretManager.Samples.Tests/DeleteRegionalSecretWithEtagTests.cs b/secretmanager/api/SecretManager.Samples.Tests/DeleteRegionalSecretWithEtagTests.cs
@@ -39,7 +39,7 @@ public void DeletesRegionalSecretsWithEtag()
         SecretName secretName = secret.SecretName;
 
         // Get the secret from the cloud.
-        Secret secretFetched = _fixture.client.GetSecret(secretName);
+        Secret secretFetched = _fixture.Client.GetSecret(secretName);
 
         // Set the updated etag fetched.
         string updatedEtag = secretFetched.Etag;
@@ -53,6 +53,6 @@ public void DeletesRegionalSecretsWithEtag()
         );
 
         // Assert that the secret was deleted and not found.
-        Assert.Throws<Grpc.Core.RpcException>(() => _fixture.client.GetSecret(secretName));
+        Assert.Throws<Grpc.Core.RpcException>(() => _fixture.Client.GetSecret(secretName));
     }
 }
diff --git a/secretmanager/api/SecretManager.Samples.Tests/DestroyRegionalSecretVersionTests.cs b/secretmanager/api/SecretManager.Samples.Tests/DestroyRegionalSecretVersionTests.cs
@@ -48,7 +48,7 @@ public void DestroysRegionalSecretVersions()
         // Assert that the secret version state is set to destroyed.
         Assert.Equal(SecretVersion.Types.State.Destroyed, secretVersion.State);
 
-        // Clean the created resources.
+        // Clean the created secret.
         _fixture.DeleteSecret(secret.SecretName);
     }
 }
diff --git a/secretmanager/api/SecretManager.Samples.Tests/DestroyRegionalSecretVersionWithEtagTests.cs b/secretmanager/api/SecretManager.Samples.Tests/DestroyRegionalSecretVersionWithEtagTests.cs
@@ -52,7 +52,7 @@ public void DestroysRegionalSecretVersionsWithEtag()
         // Assert that the secret version state is set to destroyed.
         Assert.Equal(SecretVersion.Types.State.Destroyed, secretVersion.State);
 
-        // Clean the created resources.
+        // Clean the created secret.
         _fixture.DeleteSecret(secret.SecretName);
     }
 }
diff --git a/secretmanager/api/SecretManager.Samples.Tests/DisableRegionalSecretVersionTests.cs b/secretmanager/api/SecretManager.Samples.Tests/DisableRegionalSecretVersionTests.cs
@@ -48,7 +48,7 @@ public void DisablesRegionalSecretVersions()
         // Assert that the secret version is in disabled state.
         Assert.Equal(SecretVersion.Types.State.Disabled, secretVersion.State);
 
-        // Clean the created resources.
+        // Clean the created secret.
         _fixture.DeleteSecret(secret.SecretName);
     }
 }
diff --git a/secretmanager/api/SecretManager.Samples.Tests/DisableRegionalSecretVersionWithEtagTests.cs b/secretmanager/api/SecretManager.Samples.Tests/DisableRegionalSecretVersionWithEtagTests.cs
@@ -52,7 +52,7 @@ public void DisablesRegionalSecretVersionsWithEtag()
         // Assert that the secret version is in disabled state.
         Assert.Equal(SecretVersion.Types.State.Disabled, secretVersion.State);
 
-        // Clean the created resources.
+        // Clean the created secret.
         _fixture.DeleteSecret(secret.SecretName);
     }
 }
diff --git a/secretmanager/api/SecretManager.Samples.Tests/EnableRegionalSecretVersionTests.cs b/secretmanager/api/SecretManager.Samples.Tests/EnableRegionalSecretVersionTests.cs
@@ -48,7 +48,7 @@ public void EnablesRegionalSecretVersions()
         // Assert that the secret version is in enabled state.
         Assert.Equal(SecretVersion.Types.State.Enabled, secretVersion.State);
 
-        // Clean the created resources.
+        // Clean the created secret.
         _fixture.DeleteSecret(secret.SecretName);
     }
 }
diff --git a/secretmanager/api/SecretManager.Samples.Tests/EnableRegionalSecretVersionWithEtagTests.cs b/secretmanager/api/SecretManager.Samples.Tests/EnableRegionalSecretVersionWithEtagTests.cs
@@ -55,7 +55,7 @@ public void EnablesRegionalSecretVersionsWithEtag()
         // Assert that the secret version is in enabled state.
         Assert.Equal(SecretVersion.Types.State.Enabled, secretVersion.State);
 
-        // Clean the created resources.
+        // Clean the created secret.
         _fixture.DeleteSecret(secret.SecretName);
     }
 }
diff --git a/secretmanager/api/SecretManager.Samples.Tests/IamRegionalGrantAccessTests.cs b/secretmanager/api/SecretManager.Samples.Tests/IamRegionalGrantAccessTests.cs
@@ -34,7 +34,7 @@ public IamRegionalGrantAccessTests(RegionalSecretManagerFixture fixture)
     public void GrantsAccess()
     {
         // Get the secret name.
-        SecretName secretName = _fixture.SecretToCreateName;
+        SecretName secretName = _fixture.Secret.SecretName;
 
         // Run the code sample.
         Policy result = _sample.IamRegionalGrantAccess(
diff --git a/secretmanager/api/SecretManager.Samples.Tests/IamRegionalRevokeAccessTests.cs b/secretmanager/api/SecretManager.Samples.Tests/IamRegionalRevokeAccessTests.cs
@@ -35,15 +35,15 @@ public void GrantsAccess()
     {
         string member = "group:test@google.com";
         // Get the secret name.
-        SecretName secretName = _fixture.SecretToCreateName;
+        SecretName secretName = _fixture.Secret.SecretName;
 
         // Update the policy to add the user to the binding's list.
-        Policy policy = _fixture.client.GetIamPolicy(new GetIamPolicyRequest
+        Policy policy = _fixture.Client.GetIamPolicy(new GetIamPolicyRequest
         {
             ResourceAsResourceName = secretName,
         });
         policy.AddRoleMember("roles/secretmanager.secretAccessor", member);
-        policy = _fixture.client.SetIamPolicy(new SetIamPolicyRequest
+        policy = _fixture.Client.SetIamPolicy(new SetIamPolicyRequest
         {
             ResourceAsResourceName = secretName,
             Policy = policy,
diff --git a/secretmanager/api/SecretManager.Samples.Tests/RegionalQuickstartTests.cs b/secretmanager/api/SecretManager.Samples.Tests/RegionalQuickstartTests.cs
@@ -38,7 +38,7 @@ public void Runs()
         // Run the code sample.
         _sample.RegionalQuickstart(projectId: secretName.ProjectId, locationId: secretName.LocationId, secretId: secretName.SecretId);
 
-        // Clean the created resources.
+        // Clean the created secret.
         _fixture.DeleteSecret(secretName);
     }
 }
diff --git a/secretmanager/api/SecretManager.Samples.Tests/RegionalSecretManagerFixture.cs b/secretmanager/api/SecretManager.Samples.Tests/RegionalSecretManagerFixture.cs
@@ -22,7 +22,7 @@
 [CollectionDefinition(nameof(RegionalSecretManagerFixture))]
 public class RegionalSecretManagerFixture : IDisposable, ICollectionFixture<RegionalSecretManagerFixture>
 {
-    public readonly SecretManagerServiceClient client;
+    public SecretManagerServiceClient Client { get; }
     public string ProjectId { get; }
     public string LocationId { get; }
     public string AnnotationKey { get; }
@@ -32,8 +32,6 @@ public class RegionalSecretManagerFixture : IDisposable, ICollectionFixture<Regi
     public Secret Secret { get; }
     public SecretName SecretToCreateName { get; }
     public SecretVersion SecretVersion { get; }
-
-
     public RegionalSecretManagerFixture()
     {
         // Get the Google Cloud ProjectId
@@ -47,7 +45,7 @@ public RegionalSecretManagerFixture()
         LocationId = Environment.GetEnvironmentVariable("GOOGLE_CLOUD_LOCATION") ?? "us-west1";
 
         // Create the Regional Secret Manager Client
-        client = new SecretManagerServiceClientBuilder
+        Client = new SecretManagerServiceClientBuilder
         {
             Endpoint = $"secretmanager.{LocationId}.rep.googleapis.com"
         }.Build();
@@ -84,14 +82,15 @@ public Secret CreateSecret(string secretId)
         Secret secret = new Secret
         {
             Labels =
-          {
-              { LabelKey, LabelValue }
-          },
-            Annotations = {
-            { AnnotationKey, AnnotationValue }
-          }
+            {
+                { LabelKey, LabelValue }
+            },
+            Annotations =
+            {
+              { AnnotationKey, AnnotationValue }
+            }
         };
-        return client.CreateSecret(locationName, secretId, secret);
+        return Client.CreateSecret(locationName, secretId, secret);
     }
 
     public SecretVersion AddSecretVersion(Secret secret)
@@ -101,14 +100,14 @@ public SecretVersion AddSecretVersion(Secret secret)
             Data = ByteString.CopyFrom("my super secret data", Encoding.UTF8),
         };
 
-        return client.AddSecretVersion(secret.SecretName, payload);
+        return Client.AddSecretVersion(secret.SecretName, payload);
     }
 
     public void DeleteSecret(SecretName name)
     {
         try
         {
-            client.DeleteSecret(name);
+            Client.DeleteSecret(name);
         }
         catch (Grpc.Core.RpcException e) when (e.StatusCode == Grpc.Core.StatusCode.NotFound)
         {
@@ -119,6 +118,6 @@ public void DeleteSecret(SecretName name)
 
     public SecretVersion DisableSecretVersion(SecretVersion version)
     {
-        return client.DisableSecretVersion(version.SecretVersionName);
+        return Client.DisableSecretVersion(version.SecretVersionName);
     }
 }
diff --git a/secretmanager/api/SecretManager.Samples.Tests/UpdateRegionalSecretTests.cs b/secretmanager/api/SecretManager.Samples.Tests/UpdateRegionalSecretTests.cs
@@ -45,7 +45,7 @@ public void UpdatesRegionalSecrets()
         // Assert that the secret labels was correctly updated.
         Assert.Equal("rocks", result.Labels["secretmanager"]);
 
-        // Clean the created resource.
+        // Clean the created secret.
         _fixture.DeleteSecret(secretName);
     }
 }
diff --git a/secretmanager/api/SecretManager.Samples.Tests/UpdateRegionalSecretWithEtagTests.cs b/secretmanager/api/SecretManager.Samples.Tests/UpdateRegionalSecretWithEtagTests.cs
@@ -38,7 +38,7 @@ public void UpdatesRegionalSecretsWithEtag()
         SecretVersion secretVersion = _fixture.AddSecretVersion(secret);
 
         // Get the etag associated with secret version.
-        string etag = _fixture.client.GetSecret(secretName).Etag;
+        string etag = _fixture.Client.GetSecret(secretName).Etag;
 
         // Run the code sample.
         Secret result = _sample.UpdateRegionalSecretWithEtag(
@@ -51,7 +51,7 @@ public void UpdatesRegionalSecretsWithEtag()
         // Assert that the secret label was correctly updated.
         Assert.Equal("stones", result.Labels["secretmanager"]);
 
-        // Clean the created resources
+        // Clean the created secret
         _fixture.DeleteSecret(secretName);
     }
 }
diff --git a/secretmanager/api/SecretManager.Samples/AccessRegionalSecretVersion.cs b/secretmanager/api/SecretManager.Samples/AccessRegionalSecretVersion.cs
@@ -25,8 +25,7 @@ public String AccessRegionalSecretVersion(
       string projectId = "my-project",
       string locationId = "my-location",
       string secretId = "my-secret",
-      string secretVersionId = "123"
-    )
+      string secretVersionId = "123")
     {
         // Create the Regional Secret Manager Client.
         SecretManagerServiceClient client = new SecretManagerServiceClientBuilder
diff --git a/secretmanager/api/SecretManager.Samples/CreateRegionalSecret.cs b/secretmanager/api/SecretManager.Samples/CreateRegionalSecret.cs
@@ -37,7 +37,7 @@ public Secret CreateRegionalSecret(
         LocationName location = new LocationName(projectId, locationId);
 
         // Build the secret.
-        Secret secret = new Secret { };
+        Secret secret = new Secret();
 
         // Call the API.
         Secret createdSecret = client.CreateSecret(location, secretId, secret);
diff --git a/secretmanager/api/SecretManager.Samples/EditRegionalSecretAnnotations.cs b/secretmanager/api/SecretManager.Samples/EditRegionalSecretAnnotations.cs
@@ -44,26 +44,15 @@ public Secret EditRegionalSecretAnnotations(
         // Get the exisitng secret.
         Secret existingSecret = client.GetSecret(secretName);
 
-        // Get the secret annotations.
-        MapField<string, string> secretAnnotations = existingSecret.Annotations;
-
         // Add the mapping to the secret's annotations
-        secretAnnotations[annotationKey] = annotationValue;
-
-        // Build the secret with updated fields.
-        Secret secret = new Secret
-        {
-            SecretName = SecretName.FromProjectLocationSecret(projectId, locationId, secretId),
-            Annotations = {
-              secretAnnotations
-            }
-        };
+        existingSecret.Annotations[annotationKey] = annotationValue;
 
         // Build the field mask.
         FieldMask fieldMask = FieldMask.FromString("annotations");
 
         // Call the update secret API.
-        Secret updatedSecret = client.UpdateSecret(secret, fieldMask);
+        Secret updatedSecret = client.UpdateSecret(existingSecret, fieldMask);
+
         return updatedSecret;
 
     }

Original file line number	Diff line number	Diff line change
`@@ -45,12 +45,12 @@ public void AddsRegionalSecretVersions()`
`45`	`45`	`);`
`46`	`46`
`47`	`47`	`// Access the secret version.`
`48`		`- AccessSecretVersionResponse result = _fixture.client.AccessSecretVersion(secretVersion.SecretVersionName);`
	`48`	`+ AccessSecretVersionResponse result = _fixture.Client.AccessSecretVersion(secretVersion.SecretVersionName);`
`49`	`49`
`50`	`50`	`// Assert that the secret version was added with the correct data.`
`51`	`51`	`Assert.Equal(data, result.Payload.Data.ToStringUtf8());`
`52`	`52`
`53`		`- // Clean the created resources.`
	`53`	`+ // Clean the created secret.`
`54`	`54`	`_fixture.DeleteSecret(secret.SecretName);`
`55`	`55`	`}`
`56`	`56`	`}`
Original file line number	Diff line number	Diff line change
`@@ -51,7 +51,7 @@ public void CreatesRegionalSecretsWithAnnotations()`
`51`	`51`	`// Assert that the annotation values matches with the expected value.`
`52`	`52`	`Assert.Equal(result.Annotations[annotationKey], annotationValue);`
`53`	`53`
`54`		`- // Clean the created resources.`
	`54`	`+ // Clean the created secret.`
`55`	`55`	`_fixture.DeleteSecret(secretName);`
`56`	`56`	`}`
`57`	`57`	`}`
Original file line number	Diff line number	Diff line change
`@@ -46,6 +46,6 @@ public void DeletesRegionalSecrets()`
`46`	`46`	`);`
`47`	`47`
`48`	`48`	`// Assert that the secret was deleted.`
`49`		`- Assert.Throws<Grpc.Core.RpcException>(() => _fixture.client.GetSecret(secretName));`
	`49`	`+ Assert.Throws<Grpc.Core.RpcException>(() => _fixture.Client.GetSecret(secretName));`
`50`	`50`	`}`
`51`	`51`	`}`
Original file line number	Diff line number	Diff line change
`@@ -39,7 +39,7 @@ public void DeletesRegionalSecretsWithEtag()`
`39`	`39`	`SecretName secretName = secret.SecretName;`
`40`	`40`
`41`	`41`	`// Get the secret from the cloud.`
`42`		`- Secret secretFetched = _fixture.client.GetSecret(secretName);`
	`42`	`+ Secret secretFetched = _fixture.Client.GetSecret(secretName);`
`43`	`43`
`44`	`44`	`// Set the updated etag fetched.`
`45`	`45`	`string updatedEtag = secretFetched.Etag;`
`@@ -53,6 +53,6 @@ public void DeletesRegionalSecretsWithEtag()`
`53`	`53`	`);`
`54`	`54`
`55`	`55`	`// Assert that the secret was deleted and not found.`
`56`		`- Assert.Throws<Grpc.Core.RpcException>(() => _fixture.client.GetSecret(secretName));`
	`56`	`+ Assert.Throws<Grpc.Core.RpcException>(() => _fixture.Client.GetSecret(secretName));`
`57`	`57`	`}`
`58`	`58`	`}`
Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,7 @@ public void DestroysRegionalSecretVersions()`
`48`	`48`	`// Assert that the secret version state is set to destroyed.`
`49`	`49`	`Assert.Equal(SecretVersion.Types.State.Destroyed, secretVersion.State);`
`50`	`50`
`51`		`- // Clean the created resources.`
	`51`	`+ // Clean the created secret.`
`52`	`52`	`_fixture.DeleteSecret(secret.SecretName);`
`53`	`53`	`}`
`54`	`54`	`}`
Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,7 @@ public void DestroysRegionalSecretVersionsWithEtag()`
`52`	`52`	`// Assert that the secret version state is set to destroyed.`
`53`	`53`	`Assert.Equal(SecretVersion.Types.State.Destroyed, secretVersion.State);`
`54`	`54`
`55`		`- // Clean the created resources.`
	`55`	`+ // Clean the created secret.`
`56`	`56`	`_fixture.DeleteSecret(secret.SecretName);`
`57`	`57`	`}`
`58`	`58`	`}`
Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,7 @@ public void DisablesRegionalSecretVersions()`
`48`	`48`	`// Assert that the secret version is in disabled state.`
`49`	`49`	`Assert.Equal(SecretVersion.Types.State.Disabled, secretVersion.State);`
`50`	`50`
`51`		`- // Clean the created resources.`
	`51`	`+ // Clean the created secret.`
`52`	`52`	`_fixture.DeleteSecret(secret.SecretName);`
`53`	`53`	`}`
`54`	`54`	`}`
Original file line number	Diff line number	Diff line change
`@@ -52,7 +52,7 @@ public void DisablesRegionalSecretVersionsWithEtag()`
`52`	`52`	`// Assert that the secret version is in disabled state.`
`53`	`53`	`Assert.Equal(SecretVersion.Types.State.Disabled, secretVersion.State);`
`54`	`54`
`55`		`- // Clean the created resources.`
	`55`	`+ // Clean the created secret.`
`56`	`56`	`_fixture.DeleteSecret(secret.SecretName);`
`57`	`57`	`}`
`58`	`58`	`}`
Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,7 @@ public void EnablesRegionalSecretVersions()`
`48`	`48`	`// Assert that the secret version is in enabled state.`
`49`	`49`	`Assert.Equal(SecretVersion.Types.State.Enabled, secretVersion.State);`
`50`	`50`
`51`		`- // Clean the created resources.`
	`51`	`+ // Clean the created secret.`
`52`	`52`	`_fixture.DeleteSecret(secret.SecretName);`
`53`	`53`	`}`
`54`	`54`	`}`