Skip to content
Browse files

Fix test failures

  • Loading branch information...
1 parent aa724ff commit 563cf1074ba8a351e0240cb85255a0824cf10dbf @anurse anurse committed Feb 3, 2014
Showing with 2 additions and 4 deletions.
  1. +2 −4 tests/NuGetGallery.Facts/Controllers/ApiControllerFacts.cs
View
6 tests/NuGetGallery.Facts/Controllers/ApiControllerFacts.cs
@@ -336,8 +336,7 @@ public class TheGetPackageAction
var actionResult = new EmptyResult();
var controller = new TestableApiController(MockBehavior.Strict);
controller.MockPackageService.Setup(x => x.FindPackageByIdAndVersion(PackageId, "1.0.1", false)).Returns(package);
- //Disable it temporarily as AddDownloadStats is not being invoked currently
- // controller.MockPackageService.Setup(x => x.AddDownloadStatistics(It.IsAny<PackageStatistics>())).Verifiable();
+ controller.MockPackageService.Setup(x => x.AddDownloadStatistics(It.IsAny<PackageStatistics>())).Verifiable();
controller.MockPackageFileService.Setup(s => s.CreateDownloadPackageActionResultAsync(HttpRequestUrl, PackageId, package.NormalizedVersion))
.Returns(Task.FromResult<ActionResult>(actionResult))
.Verifiable();
@@ -411,8 +410,7 @@ public class TheGetPackageAction
var actionResult = new EmptyResult();
var controller = new TestableApiController(MockBehavior.Strict);
controller.MockPackageService.Setup(x => x.FindPackageByIdAndVersion(PackageId, "", false)).Returns(package);
- //Disable it temporarily as AddDownloadStats is not being invoked currently
- // controller.MockPackageService.Setup(x => x.AddDownloadStatistics(It.IsAny<PackageStatistics>())).Verifiable();
+ controller.MockPackageService.Setup(x => x.AddDownloadStatistics(It.IsAny<PackageStatistics>())).Verifiable();
controller.MockPackageFileService.Setup(s => s.CreateDownloadPackageActionResultAsync(HttpRequestUrl, PackageId, package.NormalizedVersion))
.Returns(Task.FromResult<ActionResult>(actionResult))

0 comments on commit 563cf10

Please sign in to comment.
Something went wrong with that request. Please try again.