Adds comment

microsoft · Apr 22, 2024 · 5bea316 · 5bea316
1 parent d29bd48
commit 5bea316
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/tests/Kiota.Builder.Tests/Extensions/OpenApiUrlTreeNodeExtensionsTests.cs b/tests/Kiota.Builder.Tests/Extensions/OpenApiUrlTreeNodeExtensionsTests.cs
@@ -1079,6 +1079,9 @@ public void repro4085()
     [InlineData("files{path}", "filesWithPath")]
     [InlineData("name(idParam='{id}')", "nameWithId")]
     [InlineData("name(idParam={id})", "nameWithId")]
+    [InlineData("name(idParamFoo={id})", "nameWithId")] // The current implementation only uses the placeholder i.e {id} for the naming to ignore `idParamFoo`
+                                                        // and thus generates the same identifier as the previous case. This collision risk is unlikely and constrained to an odata service scenario
+                                                        // which would be invalid for functions scenario(overloads with the same parameters))
     [InlineData("name(idParam='{id}',idParam2='{id2}')", "nameWithIdWithId2")]
     public void CleanupParametersFromPathGeneratesDifferentResultsWithPrefixPresent(string segmentName, string expectedIdentifer)
     {