Fix tests to use new object.

Author: ryuyu

tests/NuGetGallery.Facts/Controllers/JsonApiControllerFacts.cs

@@ -355,9 +355,15 @@ public async Task FailsIfUserInputIsEmailAddress(Func<Fakes, User> getCurrentUse
                             var package = fakes.Package;
                             var controller = GetController<JsonApiController>();
                             controller.SetCurrentUser(currentUser);
+                            AddPackageOwnerViewModel testData = new AddPackageOwnerViewModel
+                            {
+                                Id = package.Id,
+                                Username = usernameToAdd,
+                                Message = "a message"
+                            };
 
                             // Act
-                            var result = await controller.AddPackageOwner(package.Id, usernameToAdd, "a message");
+                            var result = await controller.AddPackageOwner(testData);
                             dynamic data = result.Data;
 
                             // Assert
@@ -444,8 +450,15 @@ public async Task CreatesPackageOwnerRequestSendsEmailAndReturnsPendingState(Fun
                                         .Verifiable();
                                 }
                             }
+                            AddPackageOwnerViewModel testData = new AddPackageOwnerViewModel
+                            {
+                                Id = fakes.Package.Id,
+                                Username = userToAdd.Username,
+                                Message = "Hello World! Html Encoded <3"
+                            };
 
-                            JsonResult result = await controller.AddPackageOwner(fakes.Package.Id, userToAdd.Username, "Hello World! Html Encoded <3");
+
+                            JsonResult result = await controller.AddPackageOwner(testData);
                             dynamic data = result.Data;
                             PackageOwnersResultViewModel model = data.model;
 
@@ -679,7 +692,14 @@ public async Task RemovesExistingOwner(Func<Fakes, User> getCurrentUser, Func<Fa
 
                 private static async Task<ActionResult> AddPackageOwner(JsonApiController jsonApiController, string packageId, string username)
                 {
-                    return await jsonApiController.AddPackageOwner(packageId, username, "message");
+                    AddPackageOwnerViewModel testData = new AddPackageOwnerViewModel
+                    {
+                        Id = packageId,
+                        Username = username,
+                        Message = "message"
+                    };
+
+                    return await jsonApiController.AddPackageOwner(testData);
                 }
 
                 private static async Task<ActionResult> RemovePackageOwner(JsonApiController jsonApiController, string packageId, string username)