Fixed issue when mimicing long running operation for test in Playback mode. (Azure#18490)

vidai-msft · web-flow · commit a5e2a832e7ae · 2022-06-14T11:41:25.000+08:00
diff --git a/src/RecoveryServices/RecoveryServices.Backup.Helpers/TrackingHelpers.cs b/src/RecoveryServices/RecoveryServices.Backup.Helpers/TrackingHelpers.cs
@@ -48,7 +48,7 @@ public static T GetOperationStatus<T>(RestAzureNS.AzureOperationResponse respons
             while (opStatusResponse.Body.Status == ServiceClientModel.OperationStatusValues.InProgress)
             {
                 TestMockSupport.Delay(_defaultSleepForOperationTracking * 1000);
-                if (String.Compare(testMode, "Record", StringComparison.OrdinalIgnoreCase) == 0 || TestMockSupport.RunningMocked)
+                if (String.Compare(testMode, "Record", StringComparison.OrdinalIgnoreCase) == 0 || !TestMockSupport.RunningMocked)
                 {
                     Thread.Sleep(5000);
                 }
@@ -80,7 +80,7 @@ public static T GetOperationStatus<T, S>(RestAzureNS.AzureOperationResponse<S> r
                 ServiceClientModel.OperationStatusValues.InProgress)
             {
                 TestMockSupport.Delay(_defaultSleepForOperationTracking * 1000);
-                if (String.Compare(testMode, "Record", StringComparison.OrdinalIgnoreCase) == 0 || TestMockSupport.RunningMocked)
+                if (String.Compare(testMode, "Record", StringComparison.OrdinalIgnoreCase) == 0 || !TestMockSupport.RunningMocked)
                 {
                     Thread.Sleep(5000);
                 }
@@ -111,7 +111,7 @@ public static RestAzureNS.AzureOperationResponse GetOperationResult(
             while (opStatusResponse.Response.StatusCode == SystemNet.HttpStatusCode.Accepted)
             {
                 TestMockSupport.Delay(_defaultSleepForOperationTracking * 1000);
-                if (String.Compare(testMode, "Record", StringComparison.OrdinalIgnoreCase) == 0 || TestMockSupport.RunningMocked)
+                if (String.Compare(testMode, "Record", StringComparison.OrdinalIgnoreCase) == 0 || !TestMockSupport.RunningMocked)
                 {
                     Thread.Sleep(5000);
                 }
@@ -142,7 +142,7 @@ public static RestAzureNS.AzureOperationResponse<T> GetOperationStatusDataMove<T
             while (opStatusResponse.Body.Status == "InProgress")
             {
                 TestMockSupport.Delay(_defaultSleepForOperationTracking * 1000);
-                if (String.Compare(testMode, "Record", StringComparison.OrdinalIgnoreCase) == 0 || TestMockSupport.RunningMocked)
+                if (String.Compare(testMode, "Record", StringComparison.OrdinalIgnoreCase) == 0 || !TestMockSupport.RunningMocked)
                 {
                     Thread.Sleep(5000);
                 }
@@ -172,7 +172,7 @@ public static RestAzureNS.AzureOperationResponse<T> GetOperationResult<T>(
             while (opStatusResponse.Response.StatusCode == SystemNet.HttpStatusCode.Accepted)
             {
                 TestMockSupport.Delay(_defaultSleepForOperationTracking * 1000);
-                if (String.Compare(testMode, "Record", StringComparison.OrdinalIgnoreCase) == 0 || TestMockSupport.RunningMocked)
+                if (String.Compare(testMode, "Record", StringComparison.OrdinalIgnoreCase) == 0 || !TestMockSupport.RunningMocked)
                 {
                     Thread.Sleep(5000);
                 }
@@ -217,7 +217,7 @@ public static RestAzureNS.AzureOperationResponse<T> GetOperationResult<T>(
             while (opStatusResponse.Response.StatusCode == SystemNet.HttpStatusCode.Accepted)
             {
                 TestMockSupport.Delay(_defaultSleepForOperationTracking * 1000);
-                if (String.Compare(testMode, "Record", StringComparison.OrdinalIgnoreCase) == 0 || TestMockSupport.RunningMocked)
+                if (String.Compare(testMode, "Record", StringComparison.OrdinalIgnoreCase) == 0 || !TestMockSupport.RunningMocked)
                 {
                     Thread.Sleep(5000);
                 }

Original file line number	Diff line number	Diff line change
`@@ -48,7 +48,7 @@ public static T GetOperationStatus<T>(RestAzureNS.AzureOperationResponse respons`
`48`	`48`	`while (opStatusResponse.Body.Status == ServiceClientModel.OperationStatusValues.InProgress)`
`49`	`49`	`{`
`50`	`50`	`TestMockSupport.Delay(_defaultSleepForOperationTracking * 1000);`
`51`		`- if (String.Compare(testMode, "Record", StringComparison.OrdinalIgnoreCase) == 0 \|\| TestMockSupport.RunningMocked)`
	`51`	`+ if (String.Compare(testMode, "Record", StringComparison.OrdinalIgnoreCase) == 0 \|\| !TestMockSupport.RunningMocked)`
`52`	`52`	`{`
`53`	`53`	`Thread.Sleep(5000);`
`54`	`54`	`}`
`@@ -80,7 +80,7 @@ public static T GetOperationStatus<T, S>(RestAzureNS.AzureOperationResponse<S> r`
`80`	`80`	`ServiceClientModel.OperationStatusValues.InProgress)`
`81`	`81`	`{`
`82`	`82`	`TestMockSupport.Delay(_defaultSleepForOperationTracking * 1000);`
`83`		`- if (String.Compare(testMode, "Record", StringComparison.OrdinalIgnoreCase) == 0 \|\| TestMockSupport.RunningMocked)`
	`83`	`+ if (String.Compare(testMode, "Record", StringComparison.OrdinalIgnoreCase) == 0 \|\| !TestMockSupport.RunningMocked)`
`84`	`84`	`{`
`85`	`85`	`Thread.Sleep(5000);`
`86`	`86`	`}`
`@@ -111,7 +111,7 @@ public static RestAzureNS.AzureOperationResponse GetOperationResult(`
`111`	`111`	`while (opStatusResponse.Response.StatusCode == SystemNet.HttpStatusCode.Accepted)`
`112`	`112`	`{`
`113`	`113`	`TestMockSupport.Delay(_defaultSleepForOperationTracking * 1000);`
`114`		`- if (String.Compare(testMode, "Record", StringComparison.OrdinalIgnoreCase) == 0 \|\| TestMockSupport.RunningMocked)`
	`114`	`+ if (String.Compare(testMode, "Record", StringComparison.OrdinalIgnoreCase) == 0 \|\| !TestMockSupport.RunningMocked)`
`115`	`115`	`{`
`116`	`116`	`Thread.Sleep(5000);`
`117`	`117`	`}`
`@@ -142,7 +142,7 @@ public static RestAzureNS.AzureOperationResponse<T> GetOperationStatusDataMove<T`
`142`	`142`	`while (opStatusResponse.Body.Status == "InProgress")`
`143`	`143`	`{`
`144`	`144`	`TestMockSupport.Delay(_defaultSleepForOperationTracking * 1000);`
`145`		`- if (String.Compare(testMode, "Record", StringComparison.OrdinalIgnoreCase) == 0 \|\| TestMockSupport.RunningMocked)`
	`145`	`+ if (String.Compare(testMode, "Record", StringComparison.OrdinalIgnoreCase) == 0 \|\| !TestMockSupport.RunningMocked)`
`146`	`146`	`{`
`147`	`147`	`Thread.Sleep(5000);`
`148`	`148`	`}`
`@@ -172,7 +172,7 @@ public static RestAzureNS.AzureOperationResponse<T> GetOperationResult<T>(`
`172`	`172`	`while (opStatusResponse.Response.StatusCode == SystemNet.HttpStatusCode.Accepted)`
`173`	`173`	`{`
`174`	`174`	`TestMockSupport.Delay(_defaultSleepForOperationTracking * 1000);`
`175`		`- if (String.Compare(testMode, "Record", StringComparison.OrdinalIgnoreCase) == 0 \|\| TestMockSupport.RunningMocked)`
	`175`	`+ if (String.Compare(testMode, "Record", StringComparison.OrdinalIgnoreCase) == 0 \|\| !TestMockSupport.RunningMocked)`
`176`	`176`	`{`
`177`	`177`	`Thread.Sleep(5000);`
`178`	`178`	`}`
`@@ -217,7 +217,7 @@ public static RestAzureNS.AzureOperationResponse<T> GetOperationResult<T>(`
`217`	`217`	`while (opStatusResponse.Response.StatusCode == SystemNet.HttpStatusCode.Accepted)`
`218`	`218`	`{`
`219`	`219`	`TestMockSupport.Delay(_defaultSleepForOperationTracking * 1000);`
`220`		`- if (String.Compare(testMode, "Record", StringComparison.OrdinalIgnoreCase) == 0 \|\| TestMockSupport.RunningMocked)`
	`220`	`+ if (String.Compare(testMode, "Record", StringComparison.OrdinalIgnoreCase) == 0 \|\| !TestMockSupport.RunningMocked)`
`221`	`221`	`{`
`222`	`222`	`Thread.Sleep(5000);`
`223`	`223`	`}`