feat(storage): add dotnet deleteFolderRecursive sample by nidhiii-27 · Pull Request #3340 · GoogleCloudPlatform/dotnet-docs-samples

nidhiii-27 · 2026-07-02T06:44:01Z

Adds a .NET code sample demonstrating hierarchical namespace recursive folder delete.

Fixes: b/530059315

This adds a sample demonstrating how to recursively delete a folder in a hierarchical namespace bucket. Fixes: b/521168740 [Generated-by: AI]

- Update Google.Cloud.Storage.Control.V2 to 1.8.0 to resolve DeleteFolderRecursive method - Use fixture.GenerateName() instead of hardcoded folder name in tests - Assert with predicate instead of on the full Folder object to prevent flaky tests [Generated-by: AI]

gemini-code-assist · 2026-07-02T06:55:10Z

Warning

Gemini encountered an error creating the review. You can try again by commenting /gemini review.

…iveTest Add using System.Linq and replace invalid Assert.DoesNotContain predicate call with Assert.False(folders.Any(...)). [Generated-by: AI]

snippet-bot · 2026-07-02T09:39:37Z

Here is the summary of changes.

You are about to add 1 region tag.

storage/api/Storage.Samples/StorageControlDeleteFolderRecursive.cs:15, tag storage_control_delete_folder_recursive

This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:

Refresh this comment

krishnamd-jkp · 2026-07-06T04:52:49Z

+        StorageControlCreateFolderSample createSample = new StorageControlCreateFolderSample();
+        var folder = createSample.StorageControlCreateFolder(_fixture.BucketNameHns, folderName);
+
+        StorageControlDeleteFolderRecursiveSample deleteSample = new StorageControlDeleteFolderRecursiveSample();


Could you create a subfolder also and check when deleting the sample deletes both of them?

Done

Co-authored by AI Agent

Resolved PR review feedback on recursive folder delete test by adding subfolder check. [Generated-by: AI]

…ests Re-enable recursive delete folder integration tests and resolve GCS billing project limitations. [Generated-by: AI]

LAV-IDER33 · 2026-08-22T20:13:45Z

Private 🛠️⚡🧩👻U

[Generated-by: AI]

krishnamd-jkp · 2026-09-07T09:28:34Z

@@ -1,4 +1,4 @@
-// Copyright 2020 Google Inc.
+// Copyright 2020 Google Inc.


Might have added a space here. Check and remove it if not necessary.

Done

Co-authored by AI Agent

krishnamd-jkp · 2026-09-07T09:29:09Z

@@ -1,11 +1,11 @@
-<Project Sdk="Microsoft.NET.Sdk">
+<Project Sdk="Microsoft.NET.Sdk">


Same here. Check if it is unnecessary.

Done

Co-authored by AI Agent

krishnamd-jkp · 2026-09-07T09:35:28Z

+        StorageControlClientBuilder builder = new StorageControlClientBuilder
+        {
+            QuotaProject = ""
+        };
+        StorageControlClient storageControl = builder.Build();


Any reason we are using storage builder instead of simple StorageControlClient storageControl = StorageControlClient.Create();

Done

Co-authored by AI Agent

krishnamd-jkp · 2026-09-07T09:38:28Z

+        StorageControlClientBuilder builder = new StorageControlClientBuilder
+        {
+            QuotaProject = ""
+        };
+        StorageControlClient storageControl = builder.Build();


Is this needed and relevant?

Done

Co-authored by AI Agent

[Generated-by: AI]

nidhiii-27 added 2 commits July 2, 2026 06:42

feat(.NET): add deleteFolderRecursive sample

d00a500

This adds a sample demonstrating how to recursively delete a folder in a hierarchical namespace bucket. Fixes: b/521168740 [Generated-by: AI]

nidhiii-27 added ai-generated Generated by AI storage-sample-architect Generated by storage-sample-architect skill labels Jul 2, 2026

product-auto-label Bot added api: storage Issues related to the Cloud Storage API. samples Issues that are directly related to samples. labels Jul 2, 2026

fix: resolve test compile failure in StorageControlDeleteFolderRecurs…

b014d3b

…iveTest Add using System.Linq and replace invalid Assert.DoesNotContain predicate call with Assert.False(folders.Any(...)). [Generated-by: AI]

nidhiii-27 marked this pull request as ready for review July 2, 2026 09:39

nidhiii-27 requested review from a team as code owners July 2, 2026 09:39

blunderbuss-gcf Bot assigned vsharonlynn Jul 2, 2026

krishnamd-jkp reviewed Jul 6, 2026

View reviewed changes

fix(storage): resolve PR comments for deleteFolderRecursive

26c3f41

Resolved PR review feedback on recursive folder delete test by adding subfolder check. [Generated-by: AI]

krishnamd-jkp previously approved these changes Jul 6, 2026

View reviewed changes

feat(storagecontrol): re-enable recursive delete folder integration t…

1166b85

…ests Re-enable recursive delete folder integration tests and resolve GCS billing project limitations. [Generated-by: AI]

nidhiii-27 dismissed krishnamd-jkp’s stale review via 1166b85 August 17, 2026 08:30

This was referenced Aug 22, 2026

 #3345

Open

> [!WARNING] #3346

Open

fix(storage): resolve PR feedback and CI failures

1ae312d

[Generated-by: AI]

krishnamd-jkp reviewed Sep 7, 2026

View reviewed changes

fix(storage): resolve PR review comments

e7318a7

[Generated-by: AI]

krishnamd-jkp approved these changes Sep 7, 2026

View reviewed changes

nidhiii-27 merged commit 55b6f7a into main Sep 7, 2026
16 checks passed

nidhiii-27 deleted the add-delete-folder-recursive-sample branch September 7, 2026 17:02

		@@ -1,4 +1,4 @@
		// Copyright 2020 Google Inc.
		// Copyright 2020 Google Inc.

		@@ -1,11 +1,11 @@
		<Project Sdk="Microsoft.NET.Sdk">
		<Project Sdk="Microsoft.NET.Sdk">

Uh oh!

Conversation

nidhiii-27 commented Jul 2, 2026

Uh oh!

gemini-code-assist Bot commented Jul 2, 2026

Uh oh!

snippet-bot Bot commented Jul 2, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

LAV-IDER33 commented Aug 22, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

snippet-bot Bot commented Jul 2, 2026 •

edited

Loading