From ff6102b44a3b4efe21497fdf665abbda95f2540a Mon Sep 17 00:00:00 2001
From: Biao Li <biaol@microsoft.com>
Date: Thu, 11 Jan 2018 16:18:08 -0800
Subject: [PATCH] Restricting BatchOperationRequestMessage.dependsOnIds from
 List<string> to IEnumerable<string>, not allowing element updates.

---
 src/Microsoft.OData.Core/ODataBatchOperationRequestMessage.cs | 2 +-
 .../JsonLight/MultipartMixedBatchDependsOnIdsTests.cs         | 4 ++--
 .../Tests/OData.Common.Tests/PublicApi/PublicApi.bsl          | 2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/Microsoft.OData.Core/ODataBatchOperationRequestMessage.cs b/src/Microsoft.OData.Core/ODataBatchOperationRequestMessage.cs
index 9f0416b1c6..a09c238b63 100644
--- a/src/Microsoft.OData.Core/ODataBatchOperationRequestMessage.cs
+++ b/src/Microsoft.OData.Core/ODataBatchOperationRequestMessage.cs
@@ -137,7 +137,7 @@ public string GroupId
         /// <summary>
         /// Gets the prerequisite request or group ids.
         /// </summary>
-        public List<string> DependsOnIds
+        public IEnumerable<string> DependsOnIds
         {
             get
             {
diff --git a/test/FunctionalTests/Microsoft.OData.Core.Tests/ScenarioTests/Roundtrip/JsonLight/MultipartMixedBatchDependsOnIdsTests.cs b/test/FunctionalTests/Microsoft.OData.Core.Tests/ScenarioTests/Roundtrip/JsonLight/MultipartMixedBatchDependsOnIdsTests.cs
index 79bc34acad..8a51df6baf 100644
--- a/test/FunctionalTests/Microsoft.OData.Core.Tests/ScenarioTests/Roundtrip/JsonLight/MultipartMixedBatchDependsOnIdsTests.cs
+++ b/test/FunctionalTests/Microsoft.OData.Core.Tests/ScenarioTests/Roundtrip/JsonLight/MultipartMixedBatchDependsOnIdsTests.cs
@@ -246,12 +246,12 @@ private byte[] ServiceReadRequestAndWriterResponseForMultipartBatchVerifyDepends
                                     batchReader.CreateOperationRequestMessage();
 
                                 // Verify DependsOnIds are set correctly
-                                List<string> dependsOnIds = operationMessage.DependsOnIds;
+                                IEnumerable<string> dependsOnIds = operationMessage.DependsOnIds;
                                 switch (operationMessage.ContentId)
                                 {
                                     case "1":
                                     case "2A":
-                                        Assert.True(dependsOnIds.Count == 0);
+                                        Assert.True(dependsOnIds.Count() == 0);
                                         break;
 
                                     case "2B":
diff --git a/test/FunctionalTests/Tests/DataOData/Tests/OData.Common.Tests/PublicApi/PublicApi.bsl b/test/FunctionalTests/Tests/DataOData/Tests/OData.Common.Tests/PublicApi/PublicApi.bsl
index 20f30c25a0..c91028bba3 100644
--- a/test/FunctionalTests/Tests/DataOData/Tests/OData.Common.Tests/PublicApi/PublicApi.bsl
+++ b/test/FunctionalTests/Tests/DataOData/Tests/OData.Common.Tests/PublicApi/PublicApi.bsl
@@ -4703,7 +4703,7 @@ public sealed class Microsoft.OData.ODataBatchOperationRequestMessage : IContain
 	public const readonly string ContentId = 
 
 	System.IServiceProvider Container  { public virtual get; }
-	System.Collections.Generic.List`1[[System.String]] DependsOnIds  { public get; }
+	System.Collections.Generic.IEnumerable`1[[System.String]] DependsOnIds  { public get; }
 	string GroupId  { public get; }
 	System.Collections.Generic.IEnumerable`1[[System.Collections.Generic.KeyValuePair`2[[System.String],[System.String]]]] Headers  { public virtual get; }
 	string Method  { public virtual get; public virtual set; }