Fix functional test failure SearchMicrosoftDotNetCuratedFeed #3941 (#…

…3942)
NuGet · May 15, 2017 · 48fa837 · 48fa837
1 parent 50c753d
commit 48fa837
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/src/NuGetGallery/OData/Serializers/V2FeedPackageAnnotationStrategy.cs b/src/NuGetGallery/OData/Serializers/V2FeedPackageAnnotationStrategy.cs
@@ -85,7 +85,11 @@ private static void NormalizeNavigationLinks(ODataEntry entry, HttpRequestMessag
         private static Uri BuildIdLink(string id, string version, HttpRequestMessage request)
         {
             var packageIdentityQuery = $"(Id='{id}',Version='{version}')";
-            var localPath = request.RequestUri.LocalPath.Replace(packageIdentityQuery, string.Empty);
+            var localPath = request.RequestUri.LocalPath
+                // Remove closing brackets from Packages collection
+                .Replace("/Packages()", "/Packages")
+                // Remove package identity query
+                .Replace(packageIdentityQuery, string.Empty);
 
             return new Uri($"{request.RequestUri.Scheme}://{request.RequestUri.Host}{localPath}{packageIdentityQuery}");
         }

diff --git a/tests/NuGetGallery.Facts/OData/Serializers/V2FeedPackageAnnotationStrategyFacts.cs b/tests/NuGetGallery.Facts/OData/Serializers/V2FeedPackageAnnotationStrategyFacts.cs
@@ -83,6 +83,7 @@ public void SetsAtomEntryMetadataAnnotation()
 
         [Theory]
         [InlineData("https://localhost/api/v2/Packages")]
+        [InlineData("https://localhost/api/v2/Packages()")]
         [InlineData("https://localhost/api/v2/Packages(Id='SomePackageId',Version='1.0.0')")]
         public void NormalizesNavigationLinksWhenSet(string requestUri)
         {