date test - needs main merge

DFE-Digital · Dec 9, 2024 · cef6596 · cef6596
1 parent db52644
commit cef6596
Showing 1 changed file with 4 additions and 6 deletions.
diff --git a/...sts/TeachingRecordSystem.SupportUi.Tests/PageTests/Persons/PersonDetail/InductionTests.cs b/...sts/TeachingRecordSystem.SupportUi.Tests/PageTests/Persons/PersonDetail/InductionTests.cs
@@ -163,7 +163,6 @@ public async Task Get_WithPersonIdForPersonWithInductionStatusRequiringStartDate
 
         // Assert
         var doc = await AssertEx.HtmlResponseAsync(response);
-        //Assert.Contains(expectedWarning, doc.GetElementByTestId("induction-status-warning")!.TextContent); // to be covered by other test (below)
         var inductionStatus = doc.GetElementByTestId("induction-status");
         Assert.Contains(StatusStrings[setInductionStatus], inductionStatus!.TextContent);
         var startDate = doc.GetElementByTestId("induction-start-date")!.Children[1].TextContent;
@@ -172,15 +171,14 @@ public async Task Get_WithPersonIdForPersonWithInductionStatusRequiringStartDate
         Assert.NotNull(doc.GetAllElementsByTestId("induction-backlink"));
     }
 
-    [Fact(Skip ="TestData setup doesn't allow null start date")]
+    [Fact]
     public async Task Get_WithPersonIdForPersonWithInductionStatusRequiringStartDateButStartDateIsNull_DisplaysExpectedContent()
     {
         // Arrange
-        //var expectedWarning = "To change a teacher's induction status ";
         var person = await TestData.CreatePersonAsync(
-                x => x
-                .WithQts()
-                .WithInductionStatus(InductionStatus.InProgress)
+                builder => builder
+                    .WithQts()
+                    .WithInductionStatus(InductionStatus.InProgress)
                 );
 
         var request = new HttpRequestMessage(HttpMethod.Get, $"/persons/{person.ContactId}/induction");