Fix broken tests

ardalis · May 25, 2024 · 111c52b · 111c52b
1 parent 81c451a
commit 111c52b
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 4 deletions.
diff --git a/sample/Ardalis.Result.Sample.UnitTests/ServiceTests/PersonServiceCreate.cs b/sample/Ardalis.Result.Sample.UnitTests/ServiceTests/PersonServiceCreate.cs
@@ -30,7 +30,7 @@ public void ReturnsInvalidResultWith2ErrorsGivenSomeLongNameSurname()
     }
 
     [Fact]
-    public void ReturnsConflictResultGivenExistPerson()
+    public void ReturnsConflictResultGivenExistingPerson()
     {
         var service = new PersonService();
         string firstName = "John";
@@ -39,6 +39,6 @@ public void ReturnsConflictResultGivenExistPerson()
         var result = service.Create(firstName, lastName);
 
         result.Status.Should().Be(ResultStatus.Conflict);
-        result.Errors.Single().Should().Be($"Person ({firstName} {lastName}) is exist");
+        result.Errors.Single().Should().Be($"Person ({firstName} {lastName}) already exists in the system");
     }
 }
diff --git a/sample/Ardalis.Result.SampleWeb.FunctionalTests/PersonControllerCreate.cs b/sample/Ardalis.Result.SampleWeb.FunctionalTests/PersonControllerCreate.cs
@@ -43,7 +43,7 @@ public async Task ReturnsConflictGivenExistPerson(string route)
         var problemDetails = JsonConvert.DeserializeObject<ProblemDetails>(stringResponse);
 
         Assert.Contains("There was a conflict.", problemDetails.Title);
-        Assert.Contains("Next error(s) occurred:* Person (John Smith) is exist", problemDetails.Detail);
+        Assert.Contains("Next error(s) occurred:* Person (John Smith) already exists in the system", problemDetails.Detail);
         Assert.Equal(409, problemDetails.Status);
     }
 }
diff --git a/sample/Ardalis.Result.SampleWeb.FunctionalTests/PersonControllerDelete.cs b/sample/Ardalis.Result.SampleWeb.FunctionalTests/PersonControllerDelete.cs
@@ -51,7 +51,7 @@ public async Task ReturnsNotFoundGivenUnknownId(string route)
         var problemDetails = JsonConvert.DeserializeObject<ProblemDetails>(stringResponse);
 
         Assert.Contains("Resource not found.", problemDetails.Title);
-        Assert.Contains("Person Not Found", problemDetails.Detail);
+        Assert.Contains("Person with id 2 Not Found", problemDetails.Detail);
         Assert.Equal(404, problemDetails.Status);
     }