test(storage): Add integration tests for Storage Control RapidCache CRUD APIs by nidhiii-27 · Pull Request #14529 · googleapis/google-cloud-java

nidhiii-27 · 2026-09-28T09:50:18Z

Description

Adds integration tests for Rapid Cache Ultra (RCU) control plane operations (ITRapidCacheTest.java) using the StorageControlClient.

Scope & Operations Tested

Create: Creates a Rapid Cache instance on an HNS-enabled regional bucket.
Duplicate Prevention: Verifies that creating a duplicate cache with the same name fails appropriately (ALREADY_EXISTS or expected service error).
Get: Retrieves active cache metadata and verifies its state (running).
List: Confirms the created cache is returned when listing caches for the bucket.
Update: Updates cache TTL configuration and validates field mask application.
Disable & Teardown: Disables the Rapid Cache instance and defensively cleans up resources in @AfterClass.
Negative Testing: Validates handling for non-existent cache retrieval and invalid zone/cache configurations.

Verification

Formatted with google-java-format.
Clean compilation via mvn test-compile -Dtest=ITRapidCacheTest.
Modernized assertion patterns using assertThrows and Google Truth.

[Generated-by: AI]

…RUD APIs Adds integration tests for Storage Control RapidCache CRUD operations in ITRapidCacheTest.java. [Generated-by: AI]

gemini-code-assist

Code Review

This pull request introduces a new integration test class, ITRapidCacheTest, to verify the CRUD lifecycle and negative scenarios for the Storage Control RapidCache API. The review feedback highlights an improvement opportunity in the creation retry logic of rapidCache_crudLifecycle; instead of repeatedly calling the asynchronous creation API, which can cause rate limiting or unexpected backend behavior, the test should call it once and poll the get API until the resource is active.

Resolve review feedback on RapidCache CRUD lifecycle test by invoking createRapidCacheAsync once and polling getRapidCache until active. [Generated-by: AI]

Remove temporary workarounds for b/564257939 in ITRapidCacheTest: - Remove retry and polling workaround on createRapidCacheAsync in rapidCache_crudLifecycle - Directly assert ExecutionException with cause ApiException(ALREADY_EXISTS) on duplicate cache creation - Directly assert ApiException(INVALID_ARGUMENT) on invalid cache configuration [Generated-by: AI]

@test

…ndle unimplemented disable - Split rapidCache_crudLifecycle into independent @test methods - Share bucket and cache created in setUpClass across read-only tests - Isolate createAndDisableRapidCache in a dedicated bucket with clean cleanup - Gracefully handle UNIMPLEMENTED for DisableRapidCache via logging and Assume - Add disableRapidCache_nonExistent negative test for full parity with C++ PR 16512 - Gracefully catch exceptions during tearDownClass cleanup [Generated-by: AI]

Dhriti07 · 2026-10-01T05:11:02Z

I noticed we are using a Thread.sleep(20000) here to wait for the bucket metadata to propagate. Out of curiosity, how was the 20-second value chosen?

The 20-second delay was chosen empirically during integration testing due to cross-service metadata propagation.

The bucket is created via the standard Storage client, while RapidCache is provisioned via the separate Storage Control plane service (StorageControlClient). Because it takes a few seconds for newly created bucket metadata and its HNS configuration to propagate to the Control plane backends, attempting cache creation too quickly caused the service to throw FAILED_PRECONDITION (the bucket is not yet recognized as HNS-enabled) or UNAVAILABLE. A 20-second wait reliably ensures the metadata has settled across services, preventing test flakiness.

Co-authored by AI Agent

…pidCacheTest Accommodate the current production DisableRapidCache LRO response where DisableRapidCacheOpTypeHandler::GetOperationResponse returns a default AnywhereCacheInstance (translating to name prefix 'projects/_/buckets/<bucket>/rapidCaches/' and state 'anywhere-cache-state-unspecified') until CL 990396263 rolls out. [Generated-by: AI]

Dhriti07

Nit: Use Locale.ROOT with toLowerCase()

…ation test Set the explicit cache name (ID = zone), cache_type rapid-cache-ultra and ttl 24h on create; wait for the disable LRO; tests assert running state, cache type, list membership and the updated ttl, and bound the disable wait. Aligns with googleapis/google-cloud-java#14529. b/482946642 [Generated-by: AI]

…ation test Set the explicit cache name (ID = zone), cache_type rapid-cache-ultra and ttl 24h on create; wait for the disable LRO; the auto-run asserts running state, cache type, list membership and the updated ttl, and bounds the disable wait. Aligns with googleapis/google-cloud-java#14529. b/482946642 [Generated-by: AI]

…ration test Set the explicit cache name (ID = zone), cache_type rapid-cache-ultra and ttl 24h on create; wait for the disable LRO; tests assert running state, cache type, list membership and the updated ttl, and bound the disable wait. Aligns with googleapis/google-cloud-java#14529. b/482946642 [Generated-by: AI]

Set the explicit cache name (ID = zone), cache_type rapid-cache-ultra and ttl 24h on create; wait for the disable LRO; tests assert running state, cache type, list membership and the updated ttl, and bound the disable wait. Aligns with googleapis/google-cloud-java#14529. b/482946642 [Generated-by: AI]

Set the explicit cache name (ID = zone), cache_type rapid-cache-ultra and ttl 24h on create; wait for the disable LRO; tests assert running state, cache type, list membership and the updated ttl, and bound the disable wait. Aligns with #14529. b/482946642 [Generated-by: AI]

Set the explicit cache name (ID = zone), cache_type rapid-cache-ultra and ttl 24h on create; wait for the disable LRO; tests assert running state, cache type, list membership and the updated ttl, and bound the disable wait. Aligns with googleapis/google-cloud-java#14529. b/482946642 [Generated-by: AI]

…CU is in prod Co-authored-by: Gemini CLI <218195315+gemini-cli@users.noreply.github.com> [Generated-by: AI]

nidhiii-27 · 2026-11-03T10:03:36Z

Updated all toLowerCase() calls in ITRapidCacheTest to use Locale.ROOT. Also added @Ignore("TODO: Enable tests once RCU is available in prod") to skip the test suite until RCU is available in prod.

Co-authored by AI Agent

test(storage): Add integration tests for Storage Control RapidCache C…

c18fc16

…RUD APIs Adds integration tests for Storage Control RapidCache CRUD operations in ITRapidCacheTest.java. [Generated-by: AI]

nidhiii-27 added the ai-generated Pull Request generated by AI label Sep 28, 2026

gemini-code-assist Bot reviewed Sep 28, 2026

View reviewed changes

Comment thread ...storage/google-cloud-storage/src/test/java/com/google/cloud/storage/it/ITRapidCacheTest.java Resolved

nidhiii-27 added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 29, 2026

yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 29, 2026

nidhiii-27 marked this pull request as ready for review September 29, 2026 10:09

nidhiii-27 requested review from a team as code owners September 29, 2026 10:09

nidhiii-27 added 3 commits September 29, 2026 10:31

fix(storage): poll getRapidCache on UNAVAILABLE or ALREADY_EXISTS

1b52b46

Resolve review feedback on RapidCache CRUD lifecycle test by invoking createRapidCacheAsync once and polling getRapidCache until active. [Generated-by: AI]

Dhriti07 reviewed Oct 6, 2026

View reviewed changes

nidhiii-27 requested a review from Dhriti07 October 6, 2026 06:54

Dhriti07 and others added 2 commits October 27, 2026 12:39

Merge branch 'main' into feat/storage-control-rapidcache-crud

097966f

Dhriti07 approved these changes Oct 29, 2026

View reviewed changes

Dhriti07 reviewed Oct 29, 2026

View reviewed changes

sahusneha2004 mentioned this pull request Nov 2, 2026

feat(storagecontrol): add Rapid Cache samples GoogleCloudPlatform/php-docs-samples#2343

Draft

sahusneha2004 mentioned this pull request Nov 2, 2026

docs(storagecontrol): add Rapid Cache samples googleapis/google-cloud-cpp#16611

Draft

This was referenced Nov 2, 2026

samples(storage-control): add Rapid Cache samples googleapis/google-cloud-ruby#36687

Draft

samples(Storage): add Storage Control Rapid Cache samples GoogleCloudPlatform/dotnet-docs-samples#3353

Draft

sahusneha2004 mentioned this pull request Nov 2, 2026

feat(storagecontrol): add Rapid Cache samples GoogleCloudPlatform/python-docs-samples#14724

Draft

5 tasks done

sahusneha2004 mentioned this pull request Nov 2, 2026

docs(storage): add Rapid Cache samples googleapis/google-cloud-rust#7346

Draft

sahusneha2004 mentioned this pull request Nov 2, 2026

docs(storage): add Rapid Cache samples #14886

Draft

sahusneha2004 mentioned this pull request Nov 2, 2026

feat(storage): add Storage Control Rapid Cache samples GoogleCloudPlatform/golang-samples#5891

Draft

6 of 9 tasks

test(storage): use Locale.ROOT in ITRapidCacheTest and ignore until R…

9910027

…CU is in prod Co-authored-by: Gemini CLI <218195315+gemini-cli@users.noreply.github.com> [Generated-by: AI]

nidhiii-27 merged commit aa38063 into main Nov 3, 2026
203 checks passed

nidhiii-27 deleted the feat/storage-control-rapidcache-crud branch November 3, 2026 11:06

Navigation Menu

Repository navigation

test(storage): Add integration tests for Storage Control RapidCache CRUD APIs - #14529

test(storage): Add integration tests for Storage Control RapidCache CRUD APIs#14529
nidhiii-27 merged 7 commits into
mainfrom
feat/storage-control-rapidcache-crud

nidhiii-27 commented Sep 28, 2026 •

edited

Loading

Uh oh!

gemini-code-assist Bot left a comment

Uh oh!

Uh oh!

Dhriti07 Oct 1, 2026

Uh oh!

nidhiii-27 Oct 6, 2026

Uh oh!

Dhriti07 left a comment

Uh oh!

nidhiii-27 commented Nov 3, 2026

Uh oh!

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

nidhiii-27 commented Sep 28, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Scope & Operations Tested

Verification

Uh oh!

gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

Code Review

Uh oh!

Uh oh!

Dhriti07 Oct 1, 2026

Choose a reason for hiding this comment

Uh oh!

nidhiii-27 Oct 6, 2026

Choose a reason for hiding this comment

Uh oh!

Dhriti07 left a comment

Choose a reason for hiding this comment

Uh oh!

nidhiii-27 commented Nov 3, 2026

Uh oh!

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

nidhiii-27 commented Sep 28, 2026 •

edited

Loading