[Fix]Fix the Breaking Recordings

Add a private filed in storageclient, so that only validate client request
id when the test is running in Playback mode.

Author: xiafu-msft

azure-storage-common/azure/storage/common/storageclient.py

@@ -150,6 +150,7 @@ def __init__(self, connection_params):
         self.retry_callback = None
         self._X_MS_VERSION = DEFAULT_X_MS_VERSION
         self._USER_AGENT_STRING = DEFAULT_USER_AGENT_STRING
+        self._is_validating_request_id = True
 
     def _update_user_agent_string(self, service_package_version):
         self._USER_AGENT_STRING = '{}{} {}'.format(USER_AGENT_STRING_PREFIX,
@@ -336,7 +337,8 @@ def _perform_request(self, request, parser=None, parser_args=None, operation_con
                         self.response_callback(response)
 
                     # Validate the client request ID
-                    self._validate_echoed_client_request_id(request, response)
+                    if self._is_validating_request_id:
+                        self._validate_echoed_client_request_id(request, response)
 
                     # Set the response context
                     retry_context.response = response

tests/testcase.py

@@ -168,6 +168,7 @@ def _create_storage_service(self, service_class, settings):
                 settings.STORAGE_ACCOUNT_KEY,
                 protocol=settings.PROTOCOL,
             )
+        service._is_validating_request_id = False if TestMode.is_playback(self.test_mode) else True
         self._set_test_proxy(service, settings)
         return service
 
@@ -186,6 +187,7 @@ def _create_storage_service_for_blob_storage_account(self, service_class, settin
         else:
             raise SkipTest('BLOB_CONNECTION_STRING or BLOB_STORAGE_ACCOUNT_NAME must be populated to run this test')
 
+        service._is_validating_request_id = False if TestMode.is_playback(self.test_mode) else True
         self._set_test_proxy(service, settings)
         return service
 
@@ -203,6 +205,7 @@ def _create_premium_storage_service(self, service_class, settings):
         else:
             raise SkipTest('PREMIUM_CONNECTION_STRING or PREMIUM_STORAGE_ACCOUNT_NAME must be populated to run this test')
 
+        service._is_validating_request_id = False if TestMode.is_playback(self.test_mode) else True
         self._set_test_proxy(service, settings)
         return service
 
@@ -215,6 +218,8 @@ def _create_remote_storage_service(self, service_class, settings):
             )
         else:
             print("REMOTE_STORAGE_ACCOUNT_NAME and REMOTE_STORAGE_ACCOUNT_KEY not set in test settings file.")
+
+        service._is_validating_request_id = False if TestMode.is_playback(self.test_mode) else True
         self._set_test_proxy(service, settings)
         return service