Fix tests

Author: aavasthy

Microsoft.Azure.Cosmos/tests/Microsoft.Azure.Cosmos.EmulatorTests/ReadConsistencyStrategyTests.cs

@@ -232,19 +232,14 @@ public async Task ReadManyWithReadConsistencyStrategy()
             Assert.AreEqual(itemsToRead.Count, response.Count, "ReadMany with ReadConsistencyStrategy should return all requested items");
         }
 
-        /// <summary>
-        /// Verifies that the Direct layer's ServerStoreModel rejects requests when both
-        /// x-ms-consistency-level and x-ms-cosmos-read-consistency-strategy headers are set.
-        /// This guards against SDK regressions that accidentally send both headers.
-        /// </summary>
         [TestMethod]
-        public async Task DualConsistencyHeadersRejectedWithBadRequest()
+        public async Task DualConsistencyHeadersAcceptedReadConsistencyStrategyTakesPrecedence()
         {
             ToDoActivity testItem = ToDoActivity.CreateRandomToDoActivity();
             await this.Container.CreateItemAsync(testItem);
 
             // Inject both headers manually via a custom handler to bypass SDK's logic
-            // that normally prevents this.
+            // that normally only sends one of them.
             RequestHandlerHelper headerInjector = new RequestHandlerHelper();
             headerInjector.UpdateRequestMessage = (request) =>
             {
@@ -253,7 +248,7 @@ public async Task DualConsistencyHeadersRejectedWithBadRequest()
                 {
                     request.Headers.Set(
                         HttpConstants.HttpHeaders.ConsistencyLevel,
-                        Cosmos.ConsistencyLevel.Session.ToString());
+                        Cosmos.ConsistencyLevel.Eventual.ToString());
                     request.Headers.Set(
                         HttpConstants.HttpHeaders.ReadConsistencyStrategy,
                         ReadConsistencyStrategy.Session.ToString());
@@ -269,20 +264,15 @@ public async Task DualConsistencyHeadersRejectedWithBadRequest()
             using CosmosClient customClient = TestCommon.CreateCosmosClient(clientOptions);
             Container container = customClient.GetContainer(this.database.Id, this.Container.Id);
 
-            try
-            {
-                await container.ReadItemAsync<ToDoActivity>(
-                    testItem.id,
-                    new Cosmos.PartitionKey(testItem.pk));
+            ItemResponse<ToDoActivity> response = await container.ReadItemAsync<ToDoActivity>(
+                testItem.id,
+                new Cosmos.PartitionKey(testItem.pk));
 
-                Assert.Fail("Expected BadRequest when both ConsistencyLevel and ReadConsistencyStrategy headers are set");
-            }
-            catch (CosmosException ex) when (ex.StatusCode == HttpStatusCode.BadRequest)
-            {
-                Assert.IsTrue(
-                    ex.Message.Contains("Cannot specify both"),
-                    $"Error should mention dual header conflict. Got: {ex.Message}");
-            }
+            Assert.AreEqual(
+                HttpStatusCode.OK,
+                response.StatusCode,
+                "Read with both ConsistencyLevel and ReadConsistencyStrategy headers should succeed (Direct package 3.43.1+ contract).");
+            Assert.AreEqual(testItem.id, response.Resource.id);
         }
     }
 }