Skip to content

Commit

Permalink
Enable fileshare crc (#45124)
Browse files Browse the repository at this point in the history
* impl

* testproxy

* shares fix

* testproxy
  • Loading branch information
jaschrep-msft authored Jul 23, 2024
1 parent 6231eb5 commit 1b7ed3c
Show file tree
Hide file tree
Showing 15 changed files with 158 additions and 94 deletions.
2 changes: 1 addition & 1 deletion sdk/storage/Azure.Storage.Blobs/assets.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,5 +2,5 @@
"AssetsRepo": "Azure/azure-sdk-assets",
"AssetsRepoPrefixPath": "net",
"TagPrefix": "net/storage/Azure.Storage.Blobs",
"Tag": "net/storage/Azure.Storage.Blobs_8c93ff57db"
"Tag": "net/storage/Azure.Storage.Blobs_d0c9efbfaf"
}
Original file line number Diff line number Diff line change
Expand Up @@ -1085,7 +1085,7 @@ public virtual async Task ParallelUploadPrecalculatedComposableHashAccepted(Stor
PrecalculatedChecksum = hash
};

var client = await GetResourceClientAsync(disposingContainer.Container, dataLength);
var client = await GetResourceClientAsync(disposingContainer.Container, dataLength, createResource: true);

// Act
await DoesNotThrowOrInconclusiveAsync(
Expand Down Expand Up @@ -1733,7 +1733,7 @@ public virtual async Task DownloadSuccessfulHashVerification(StorageChecksumAlgo
var validationOptions = new DownloadTransferValidationOptions { ChecksumAlgorithm = algorithm };

// Act
var dest = new MemoryStream();
using var dest = new MemoryStream();
var response = await DownloadPartitionAsync(client, dest, validationOptions, new HttpRange(length: data.Length));

// Assert
Expand All @@ -1750,7 +1750,8 @@ public virtual async Task DownloadSuccessfulHashVerification(StorageChecksumAlgo
Assert.Fail("Test can't validate given algorithm type.");
break;
}
Assert.IsTrue(dest.ToArray().SequenceEqual(data));
var result = dest.ToArray();
Assert.IsTrue(result.SequenceEqual(data));
}

#if BlobSDK || DataLakeSDK
Expand Down
2 changes: 1 addition & 1 deletion sdk/storage/Azure.Storage.Files.DataLake/assets.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,5 +2,5 @@
"AssetsRepo": "Azure/azure-sdk-assets",
"AssetsRepoPrefixPath": "net",
"TagPrefix": "net/storage/Azure.Storage.Files.DataLake",
"Tag": "net/storage/Azure.Storage.Files.DataLake_6eeb51b62e"
"Tag": "net/storage/Azure.Storage.Files.DataLake_8406b24e8b"
}
2 changes: 1 addition & 1 deletion sdk/storage/Azure.Storage.Files.Shares/assets.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,5 +2,5 @@
"AssetsRepo": "Azure/azure-sdk-assets",
"AssetsRepoPrefixPath": "net",
"TagPrefix": "net/storage/Azure.Storage.Files.Shares",
"Tag": "net/storage/Azure.Storage.Files.Shares_01d63d33e8"
"Tag": "net/storage/Azure.Storage.Files.Shares_e12d252b83"
}
Original file line number Diff line number Diff line change
Expand Up @@ -84,6 +84,10 @@
<Compile Include="$(AzureStorageSharedSources)StorageTelemetryPolicy.cs" LinkBase="Shared" />
<Compile Include="$(AzureStorageSharedSources)StorageVersionExtensions.cs" LinkBase="Shared" />
<Compile Include="$(AzureStorageSharedSources)StreamPartition.cs" LinkBase="Shared" />
<Compile Include="$(AzureStorageSharedSources)StructuredMessageDecodingRetriableStream.cs" LinkBase="Shared" />
<Compile Include="$(AzureStorageSharedSources)StructuredMessageDecodingStream.cs" LinkBase="Shared" />
<Compile Include="$(AzureStorageSharedSources)StructuredMessageEncodingStream.cs" LinkBase="Shared" />
<Compile Include="$(AzureStorageSharedSources)StructuredMessagePrecalculatedCrcWrapperStream.cs" LinkBase="Shared" />
<Compile Include="$(AzureStorageSharedSources)StructuredMessage.cs" LinkBase="Shared" />
<Compile Include="$(AzureStorageSharedSources)TransferValidationOptionsExtensions.cs" LinkBase="Shared" />
<Compile Include="$(AzureStorageSharedSources)UriExtensions.cs" LinkBase="Shared" />
Expand Down

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit 1b7ed3c

Please sign in to comment.