Storage STG99 Features

[microzchang]

Pull Request Checklist

Please leverage this checklist as a reminder to address commonly occurring feedback when submitting a pull request to make sure your PR can be reviewed quickly:

See the detailed list in the contributing guide.

• C++ Guidelines
• Doxygen docs
• Unit tests
• No unwanted commits/changes
• Descriptive title/description
  • PR is single purpose
  • Related issue listed
• Comments in source
• No typos
• Update changelog
• Not work-in-progress
• External references or docs updated
• Self review of PR done
• Any breaking changes?

[Copilot]

Pull Request Overview

This pull request implements new storage features and related tests for Azure Storage services. Key changes include:

• Adding new tests for approximate message count behaviors in queues.
• Refactoring internal models to handle large message counts and deprecate the old field.
• Updating error handling for share snapshot scenarios and improving error messages.

Reviewed Changes

Copilot reviewed 19 out of 19 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
sdk/storage/azure-storage-queues/test/ut/queue_client_test.cpp	Added tests for ApproximateMessageCount and ApproximateMessageCountLong, including a playback-only scenario.
sdk/storage/azure-storage-queues/src/rest_client.cpp	Updated the GetProperties function to use std::stoll for improved numeric conversion.
sdk/storage/azure-storage-queues/src/queue_client.cpp	Adjusted GetProperties conversion with fallback logic for message counts exceeding INT32_MAX.
sdk/storage/azure-storage-files-shares/src/share_client.cpp	Extended error handling to cover both ShareNotFound and ShareSnapshotNotFound cases.
Other files	Updated documentation, changelogs, and internal model definitions to support the changes.

Comments suppressed due to low confidence (3)

sdk/storage/azure-storage-queues/test/ut/queue_client_test.cpp:206

• [nitpick] The test name includes the suffix 'PLAYBACKONLY' which may not be immediately clear. Consider renaming the test to more explicitly indicate that it is only applicable in playback scenarios.

TEST_F(QueueClientTest, ApproximateMessageCountLong_PLAYBACKONLY_)

sdk/storage/azure-storage-queues/src/queue_client.cpp:172

• Since ApproximateMessageCount is set to -1 when the internal count exceeds INT32_MAX, consider adding a comment explaining this fallback behavior to improve code clarity and maintainability.

if (ret.Value.ApproximateMessageCount > static_cast<int64_t>(INT32_MAX))

sdk/storage/azure-storage-files-shares/src/share_client.cpp:225

• [nitpick] Adding the ShareSnapshotNotFound condition improves error handling for snapshots; consider adding an inline comment explaining why this additional error code is handled for clarity.

if (e.ErrorCode == _detail::ShareNotFound || e.ErrorCode == _detail::ShareSnapshotNotFound)