[Storage][DataMovement] Record some tests that could not be previousl…

…y recorded (#46716)
Azure · Oct 18, 2024 · 7c7f12b · 7c7f12b
1 parent 14d9766
commit 7c7f12b
Show file tree

Hide file tree

Showing 5 changed files with 10 additions and 44 deletions.
diff --git a/sdk/storage/Azure.Storage.DataMovement.Blobs/assets.json b/sdk/storage/Azure.Storage.DataMovement.Blobs/assets.json
@@ -2,5 +2,5 @@
   "AssetsRepo": "Azure/azure-sdk-assets",
   "AssetsRepoPrefixPath": "net",
   "TagPrefix": "net/storage/Azure.Storage.DataMovement.Blobs",
-  "Tag": "net/storage/Azure.Storage.DataMovement.Blobs_e796f456ff"
+  "Tag": "net/storage/Azure.Storage.DataMovement.Blobs_c66f15b319"
 }
diff --git a/sdk/storage/Azure.Storage.DataMovement.Blobs/tests/ProgressHandlerTests.cs b/sdk/storage/Azure.Storage.DataMovement.Blobs/tests/ProgressHandlerTests.cs
@@ -8,15 +8,12 @@
 using System.Linq;
 using System.Threading;
 using System.Threading.Tasks;
-using Azure.Core.TestFramework;
-using Azure.Storage.Blobs.Tests;
-using Azure.Storage.DataMovement.Blobs.Tests;
+using Azure.Storage.DataMovement.Tests;
 using BaseBlobs::Azure.Storage.Blobs;
-using BaseBlobs::Azure.Storage.Blobs.Models;
 using DMBlobs::Azure.Storage.DataMovement.Blobs;
 using NUnit.Framework;
 
-namespace Azure.Storage.DataMovement.Tests
+namespace Azure.Storage.DataMovement.Blobs.Tests
 {
     public class ProgressHandlerTests : DataMovementBlobTestBase
     {
@@ -144,7 +141,6 @@ private async Task TransferAndAssertProgress(
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
         public async Task ProgressHandler_DownloadDirectory()
         {
             // Arrange
@@ -167,7 +163,6 @@ await TransferAndAssertProgress(
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
         public async Task ProgressHandler_DirectoryUpload()
         {
             // Arrange
@@ -190,7 +185,6 @@ await TransferAndAssertProgress(
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
         public async Task ProgressHandler_Copy()
         {
             // Arrange
@@ -214,7 +208,6 @@ await TransferAndAssertProgress(
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
         [TestCase(StorageResourceCreationPreference.SkipIfExists)]
         [TestCase(StorageResourceCreationPreference.FailIfExists)]
         public async Task ProgressHandler_Conflict(StorageResourceCreationPreference createMode)
@@ -246,7 +239,6 @@ await TransferAndAssertProgress(
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
         [TestCase(TransferDirection.Upload)]
         [TestCase(TransferDirection.Download)]
         [TestCase(TransferDirection.Copy)]
@@ -308,7 +300,6 @@ await TransferAndAssertProgress(
 
         [Ignore("https://github.com/Azure/azure-sdk-for-net/issues/35558")]
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
         [TestCase(0)]
         [TestCase(150)]
         public async Task ProgressHandler_PauseResume(int delayInMs)

diff --git a/sdk/storage/Azure.Storage.DataMovement.Files.Shares/assets.json b/sdk/storage/Azure.Storage.DataMovement.Files.Shares/assets.json
@@ -2,5 +2,5 @@
   "AssetsRepo": "Azure/azure-sdk-assets",
   "AssetsRepoPrefixPath": "net",
   "TagPrefix": "net/storage/Azure.Storage.DataMovement.Files.Shares",
-  "Tag": "net/storage/Azure.Storage.DataMovement.Files.Shares_4eaa88bc73"
+  "Tag": "net/storage/Azure.Storage.DataMovement.Files.Shares_44f04e1278"
 }
diff --git a/sdk/storage/Azure.Storage.DataMovement/tests/Shared/StartTransferDirectoryCopyTestBase.cs b/sdk/storage/Azure.Storage.DataMovement/tests/Shared/StartTransferDirectoryCopyTestBase.cs
@@ -8,7 +8,6 @@
 using System.Threading.Tasks;
 using Azure.Core;
 using Azure.Core.TestFramework;
-using Azure.Storage.Common;
 using Azure.Storage.Test.Shared;
 using NUnit.Framework;
 
@@ -276,7 +275,6 @@ await VerifyResultsAsync(
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
         [TestCase(0, 10)]
         [TestCase(DataMovementTestConstants.KB / 2, 10)]
         [TestCase(DataMovementTestConstants.KB, 10)]
@@ -357,7 +355,6 @@ await CopyDirectoryAndVerifyAsync(
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
         public async Task DirectoryToDirectory_EmptyFolder()
         {
             // Arrange
@@ -398,7 +395,6 @@ await TestTransferWithTimeout.WaitForCompletionAsync(
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
         public async Task DirectoryToDirectory_SingleFile()
         {
             // Arrange
@@ -423,7 +419,6 @@ await CopyDirectoryAndVerifyAsync(
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
         public async Task DirectoryToDirectory_ManySubDirectories()
         {
             // Arrange
@@ -458,7 +453,6 @@ await CopyDirectoryAndVerifyAsync(
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
         [TestCase(1)]
         [TestCase(2)]
         [TestCase(3)]
@@ -493,7 +487,6 @@ await CopyDirectoryAndVerifyAsync(
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
         public async Task DirectoryToDirectory_OverwriteExists()
         {
             // Arrange
@@ -543,7 +536,6 @@ await CopyDirectoryAndVerifyAsync(
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
         public async Task DirectoryToDirectory_OverwriteNotExists()
         {
             // Arrange
@@ -589,7 +581,6 @@ await CopyDirectoryAndVerifyAsync(
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
         public virtual async Task DirectoryToDirectory_OAuth()
         {
             // Arrange
@@ -698,7 +689,6 @@ private async Task<DataTransfer> CreateStartTransfer(
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
         public async Task StartTransfer_AwaitCompletion()
         {
             // Arrange
@@ -729,7 +719,7 @@ await TestTransferWithTimeout.WaitForCompletionAsync(
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
+        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/46717
         public async Task StartTransfer_AwaitCompletion_Failed()
         {
             // Arrange
@@ -767,7 +757,6 @@ await TestTransferWithTimeout.WaitForCompletionAsync(
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
         public async Task StartTransfer_AwaitCompletion_Skipped()
         {
             // Arrange
@@ -805,7 +794,6 @@ await TestTransferWithTimeout.WaitForCompletionAsync(
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
         public async Task StartTransfer_EnsureCompleted()
         {
             // Arrange
@@ -837,7 +825,7 @@ public async Task StartTransfer_EnsureCompleted()
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
+        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/46717
         public async Task StartTransfer_EnsureCompleted_Failed()
         {
             // Arrange
@@ -875,7 +863,6 @@ public async Task StartTransfer_EnsureCompleted_Failed()
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
         public async Task StartTransfer_EnsureCompleted_Skipped()
         {
             // Arrange
@@ -913,7 +900,7 @@ public async Task StartTransfer_EnsureCompleted_Skipped()
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
+        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/46717
         public async Task StartTransfer_EnsureCompleted_Failed_SmallChunks()
         {
             // Arrange
@@ -1025,7 +1012,6 @@ await VerifyResultsAsync(
         [TestCase((int) TransferPropertiesTestType.Preserve)]
         [TestCase((int) TransferPropertiesTestType.NoPreserve)]
         [TestCase((int) TransferPropertiesTestType.NewProperties)]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
         public async Task CopyRemoteObjects_VerifyProperties(int propertiesType)
         {
             // Arrange

diff --git a/...storage/Azure.Storage.DataMovement/tests/Shared/StartTransferDirectoryDownloadTestBase.cs b/...storage/Azure.Storage.DataMovement/tests/Shared/StartTransferDirectoryDownloadTestBase.cs
@@ -170,7 +170,6 @@ private async Task DownloadDirectoryAndVerifyAsync(
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
         [TestCase(0, 10)]
         [TestCase(100, 10)]
         [TestCase(Constants.KB, 10)]
@@ -228,7 +227,6 @@ await DownloadDirectoryAndVerifyAsync(
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
         public async Task DownloadDirectoryAsync_Empty()
         {
             // Arrange
@@ -266,7 +264,6 @@ await TestTransferWithTimeout.WaitForCompletionAsync(
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
         public async Task DownloadDirectoryAsync_SingleFile()
         {
             // Arrange
@@ -280,7 +277,6 @@ await DownloadDirectoryAndVerifyAsync(
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
         public async Task DownloadDirectoryAsync_ManySubDirectories()
         {
             // Arrange
@@ -304,7 +300,6 @@ await DownloadDirectoryAndVerifyAsync(
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
         [TestCase(1)]
         [TestCase(2)]
         [TestCase(3)]
@@ -332,7 +327,6 @@ await DownloadDirectoryAndVerifyAsync(
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
         public async Task DownloadDirectoryAsync_SmallChunks_ManyFiles()
         {
             // Arrange
@@ -379,7 +373,6 @@ await DownloadDirectoryAndVerifyAsync(
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
         public async Task DownloadDirectoryAsync_Root()
         {
             // Arrange
@@ -450,7 +443,6 @@ private async Task<DataTransfer> CreateStartTransfer(
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
         public async Task StartTransfer_AwaitCompletion()
         {
             // Arrange
@@ -483,7 +475,7 @@ await TestTransferWithTimeout.WaitForCompletionAsync(
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
+        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/46717
         public async Task StartTransfer_AwaitCompletion_Failed()
         {
             // Arrange
@@ -525,7 +517,6 @@ await TestTransferWithTimeout.WaitForCompletionAsync(
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
         public async Task StartTransfer_AwaitCompletion_Skipped()
         {
             // Arrange
@@ -567,7 +558,6 @@ await TestTransferWithTimeout.WaitForCompletionAsync(
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
         public async Task StartTransfer_EnsureCompleted()
         {
             // Arrange
@@ -601,7 +591,7 @@ public async Task StartTransfer_EnsureCompleted()
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
+        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/46717
         public async Task StartTransfer_EnsureCompleted_Failed()
         {
             // Arrange
@@ -643,7 +633,6 @@ public async Task StartTransfer_EnsureCompleted_Failed()
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
         public async Task StartTransfer_EnsureCompleted_Skipped()
         {
             // Arrange
@@ -685,7 +674,7 @@ public async Task StartTransfer_EnsureCompleted_Skipped()
         }
 
         [Test]
-        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/33082
+        [LiveOnly] // https://github.com/Azure/azure-sdk-for-net/issues/46717
         public async Task StartTransfer_EnsureCompleted_Failed_SmallChunks()
         {
             // Arrange