Add overload to deserialize GetBulkStateAsync item values

[WhitWaldo]

Adds overload to BulkStateItem and GetBulkStateAsync to perform SDK-based deserialization of returned values instead of strictly returning serialized strings.

Signed-off-by: Whit Waldo [email protected]

Description

Explained in more detail at #1172 , but GetBulkItemAsync returns a BulkStateItem with a Key, ETag and Value wherein each is typed as a string and the user is expected to deserialize the Value. Because the state might have been originally added one by one, relying on Dapr and its serialization options to serialize everything, this can cause downstream issues deserializing values not originally serialized by the developer.

As such, just as GetStateAsync allows a type parameter to deserialize each returned value, I've added the same overload to GetBulkItemAsync to deserialize using the same mechanism as GetStateAsync to deserialize each of the returned items as part of an overloaded and generically-typed BulkItemState.

Issue reference

As it's a simple addition, I didn't think it really merited a lot of discussion as the functionality is already available for single items (and this just adds the same for multiple items).

Please reference the issue this PR will close: #1172

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

• Code compiles correctly
• Created/updated tests
• Extended the documentation - PR created at Added example for retrieving bulk deserialized items in dotnet-sdk repo docs#3843

I'll work on finding relevant documentation and updating that right now.

[codecov]

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (034de3e) 67.28% compared to head (fada655) 67.36%.

Files	Patch %	Lines
src/Dapr.Client/BulkStateItem.cs	71.42%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1173      +/-   ##
==========================================
+ Coverage   67.28%   67.36%   +0.08%     
==========================================
  Files         174      174              
  Lines        6006     6028      +22     
  Branches      670      672       +2     
==========================================
+ Hits         4041     4061      +20     
- Misses       1798     1800       +2     
  Partials      167      167              

Flag	Coverage Δ
net6	67.34% <92.30%> (+0.08%)	⬆️
net7	67.34% <92.30%> (+0.08%)	⬆️
net8	67.35% <92.30%> (+0.08%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[halspang]

Looks good overall, just some formatting nitpicks.

[hhunter-ms]

@halspang @philliphoff is this ready for merge? just checking since there's a corresponding docs PR waiting in the wings for this :)

[philliphoff]

@hhunter-ms @halspang If Hal is ok with the last set to changes he requested (and the required merge is clean), I say merge it.