From 5bea316db47714ed54d52671f7ab0ed07daea7ac Mon Sep 17 00:00:00 2001
From: Andrew Omondi <anomondi@microsoft.com>
Date: Mon, 22 Apr 2024 17:47:52 +0300
Subject: [PATCH] Adds comment

---
 .../Extensions/OpenApiUrlTreeNodeExtensionsTests.cs            | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/tests/Kiota.Builder.Tests/Extensions/OpenApiUrlTreeNodeExtensionsTests.cs b/tests/Kiota.Builder.Tests/Extensions/OpenApiUrlTreeNodeExtensionsTests.cs
index 6e36e0f2a3..99fa6e6662 100644
--- 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)
     {