Skip to content

Conversation

vincenttran-msft
Copy link
Member

@vincenttran-msft vincenttran-msft commented Sep 16, 2025

.tsp: Azure/azure-rest-api-specs#37542

  • Implements exists() for BlobClient and BlobContainerClient
  • Introduces the StorageErrorCode enum from generated code changes.

@github-actions github-actions bot added the Storage Storage Service (Queues, Blobs, Files) label Sep 16, 2025
Copy link

github-actions bot commented Sep 16, 2025

API Change Check

APIView identified API level changes in this PR and created the following API reviews

azure_storage_blob

@vincenttran-msft vincenttran-msft marked this pull request as ready for review September 16, 2025 21:51
@Copilot Copilot AI review requested due to automatic review settings September 16, 2025 21:51
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds convenience exists() methods to both BlobClient and BlobContainerClient that return a boolean indicating whether the resource exists. The implementation uses the existing get_properties() method and returns true if the call succeeds or false if it fails.

  • Added exists() method to BlobClient and BlobContainerClient
  • Added comprehensive test coverage for the new methods
  • Updated test asset tag

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
sdk/storage/azure_storage_blob/src/clients/blob_client.rs Added exists() method that checks blob existence using get_properties()
sdk/storage/azure_storage_blob/src/clients/blob_container_client.rs Added exists() method that checks container existence using get_properties()
sdk/storage/azure_storage_blob/tests/blob_client.rs Added test assertions for blob existence checks in various scenarios
sdk/storage/azure_storage_blob/tests/blob_container_client.rs Added test assertions for container existence checks
sdk/storage/azure_storage_blob/assets.json Updated test asset tag

@vincenttran-msft vincenttran-msft marked this pull request as draft September 17, 2025 22:13
@vincenttran-msft vincenttran-msft marked this pull request as ready for review September 17, 2025 22:26
@vincenttran-msft vincenttran-msft enabled auto-merge (squash) September 17, 2025 23:18
@vincenttran-msft vincenttran-msft merged commit abb39d1 into Azure:main Sep 17, 2025
17 checks passed
@vincenttran-msft vincenttran-msft deleted the vincenttran/exists_api branch September 17, 2025 23:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Storage Storage Service (Queues, Blobs, Files)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants