[ReceveryServices] Delay in long running operations

[VeryEarly]

Description

Some code changes introduced in PR: https://github.com/Azure/azure-powershell/pull/18154/files#diff-1aac40bedaa2fc2c2abbee9d763b1eb0e96c67a7817dc28d5f1212505f0803bcR48
for file src/RecoveryServices/RecoveryServices.Backup.Helpers/TrackingHelpers.cs

This change make the logic from:

• if not playback mode, sleep for 5 seconds (redundant check because https://github.com/Azure/azure-powershell-common/blob/main/src/Common/TestMockSupport.cs#L28)
• if record mode, sleep for another 5 seconds

to

• if not playback mode, sleep for 5 seconds
• if record or playback mode, sleep for another 5 seconds
  which cause test cases in playback mode timeout

From the history I have to guess what you want is to remove the redundant check for playback mode which should look like

if (String.Compare(testMode, "Record", StringComparison.OrdinalIgnoreCase) == 0)
{
    Thread.Sleep(10000);
}
else
{
  TestMockSupport.Delay(_defaultSleepForOperationTracking * 1000);
}

which is clearer:

• if record mode, sleep for 10 seconds
• if not test, sleep for 5 seconds

Issue script & Debug output

no debug output

Environment data

no environment data

Module versions

no module version

Error output

No response