removed null check on 2 arguments because it was breaking other tests. I will maybe come back to it later.

Author: philipthomas-MSFT

Microsoft.Azure.Cosmos/src/GatewayStoreClient.cs

@@ -140,16 +140,6 @@ internal static async Task<DocumentClientException> CreateDocumentClientExceptio
             HttpResponseMessage responseMessage,
             IClientSideRequestStatistics requestStatistics)
         {
-            if (responseMessage is null)
-            {
-                throw new ArgumentNullException(nameof(responseMessage));
-            }
-
-            if (requestStatistics is null)
-            {
-                throw new ArgumentNullException(nameof(requestStatistics));
-            }
-
             if (!PathsHelper.TryParsePathSegments(
                 resourceUrl: responseMessage.RequestMessage.RequestUri.LocalPath,
                 isFeed: out _,

Microsoft.Azure.Cosmos/tests/Microsoft.Azure.Cosmos.Tests/GatewayStoreClientTests.cs

@@ -246,48 +246,6 @@ public async Task TestCreateDocumentClientExceptionWhenMediaTypeIsApplicationJso
             Assert.IsNotNull(value: documentClientException.Error.Message);
         }
 
-        /// <summary>
-        /// Testing CreateDocumentClientExceptionAsync when response message argument is null, then expects an argumentNullException.
-        /// </summary>
-        [TestMethod]
-        [Owner("philipthomas-MSFT")]
-        public async Task TestCreateDocumentClientExceptionWhenResponseMessageIsNullExpectsArgumentNullException()
-        {
-            ArgumentNullException argumentNullException = await Assert.ThrowsExceptionAsync<ArgumentNullException>(async () => await GatewayStoreClient.CreateDocumentClientExceptionAsync(
-                responseMessage: default,
-                requestStatistics: GatewayStoreClientTests.CreateClientSideRequestStatistics())
-            );
-
-            Assert.IsNotNull(argumentNullException);
-            Assert.AreEqual(expected: "Value cannot be null. (Parameter 'responseMessage')", actual: argumentNullException.Message);
-        }
-
-        /// <summary>
-        /// Testing CreateDocumentClientExceptionAsync when request statistics argument is null, then expects an argumentNullException.
-        /// </summary>
-        [TestMethod]
-        [Owner("philipthomas-MSFT")]
-        public async Task TestCreateDocumentClientExceptionWhenRequestStatisticsIsNullExpectsArgumentNullException()
-        {
-            HttpResponseMessage responseMessage = new(statusCode: HttpStatusCode.NotFound)
-            {
-                RequestMessage = new HttpRequestMessage(
-                    method: HttpMethod.Get,
-                    requestUri: @"https://pt_ac_test_uri.com/"),
-                Content = new StringContent(
-                    content: JsonConvert.SerializeObject(
-                        value: new Error() { Code = HttpStatusCode.NotFound.ToString(), Message = "" })),
-            };
-
-            ArgumentNullException argumentNullException = await Assert.ThrowsExceptionAsync<ArgumentNullException>(async () => await GatewayStoreClient.CreateDocumentClientExceptionAsync(
-                responseMessage: responseMessage,
-                requestStatistics: default)
-            );
-
-            Assert.IsNotNull(argumentNullException);
-            Assert.AreEqual(expected: "Value cannot be null. (Parameter 'requestStatistics')", actual: argumentNullException.Message);
-        }
-
         private static IClientSideRequestStatistics CreateClientSideRequestStatistics()
         {
             return new ClientSideRequestStatisticsTraceDatum(