Add NINO to User admin page

DFE-Digital · Jan 9, 2024 · 23e4d27 · 23e4d27
1 parent a6677c9
commit 23e4d27
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 2 deletions.
diff --git a/dotnet-authserver/src/TeacherIdentity.AuthServer/Pages/Admin/User.cshtml b/dotnet-authserver/src/TeacherIdentity.AuthServer/Pages/Admin/User.cshtml
@@ -73,6 +73,10 @@
                     <govuk-summary-list-row-action asp-page="EditUserMobileNumber" asp-route-userId="@Model.UserId" visually-hidden-text="mobile number">Change</govuk-summary-list-row-action>
                 </govuk-summary-list-row-actions>
             </govuk-summary-list-row>
+            <govuk-summary-list-row>
+                <govuk-summary-list-row-key>National insurance number</govuk-summary-list-row-key>
+                <govuk-summary-list-row-value>@Model.NationalInsuranceNumber</govuk-summary-list-row-value>
+            </govuk-summary-list-row>
             <govuk-summary-list-row>
                 <govuk-summary-list-row-key>Date of birth</govuk-summary-list-row-key>
                 <govuk-summary-list-row-value>@(Model.DateOfBirth.HasValue ? Model.DateOfBirth.Value.ToString("d MMMM yyyy") : "")</govuk-summary-list-row-value>
@@ -131,7 +135,7 @@
                 </govuk-summary-list-row>
                 <govuk-summary-list-row>
                     <govuk-summary-list-row-key>National insurance number</govuk-summary-list-row-key>
-                    <govuk-summary-list-row-value>@(!string.IsNullOrEmpty(Model.DqtNationalInsuranceNumber) ? "Provided" : "Not provided")</govuk-summary-list-row-value>
+                    <govuk-summary-list-row-value>@Model.DqtNationalInsuranceNumber</govuk-summary-list-row-value>
                 </govuk-summary-list-row>
                 <govuk-summary-list-row>
                     <govuk-summary-list-row-key>TRN</govuk-summary-list-row-key>

diff --git a/dotnet-authserver/src/TeacherIdentity.AuthServer/Pages/Admin/User.cshtml.cs b/dotnet-authserver/src/TeacherIdentity.AuthServer/Pages/Admin/User.cshtml.cs
@@ -41,6 +41,8 @@ public UserModel(TeacherIdentityServerDbContext dbContext, IDqtApiClient dqtApiC
 
     public DateOnly? DateOfBirth { get; set; }
 
+    public string? NationalInsuranceNumber { get; set; }
+
     public string? DqtNationalInsuranceNumber { get; set; }
 
     public string? Trn { get; set; }
@@ -70,6 +72,7 @@ public async Task<IActionResult> OnGet()
                 u.LastName,
                 u.PreferredName,
                 u.DateOfBirth,
+                u.NationalInsuranceNumber,
                 u.Trn,
                 u.Created,
                 u.UserType,
@@ -100,6 +103,7 @@ public async Task<IActionResult> OnGet()
         CanChangeDqtRecord = !HaveDqtRecord;
         MergedUserIds = user.MergedUserIds;
         DateOfBirth = user.DateOfBirth;
+        NationalInsuranceNumber = user.NationalInsuranceNumber;
         EffectiveVerificationLevel = user.EffectiveVerificationLevel;
 
         if (user.Trn is not null)

diff --git a/dotnet-authserver/tests/TeacherIdentity.AuthServer.Tests/EndpointTests/Admin/UserTests.cs b/dotnet-authserver/tests/TeacherIdentity.AuthServer.Tests/EndpointTests/Admin/UserTests.cs
@@ -120,7 +120,7 @@ public async Task Get_ValidRequestForUserWithTrn_RendersExpectedContent()
         Assert.NotNull(doc.GetElementByTestId("DqtSection"));
         Assert.Equal($"{dqtFirstName} {dqtLastName}", doc.GetSummaryListValueForKey("DQT name"));
         Assert.Equal(dqtDateOfBirth.ToString("d MMMM yyyy"), doc.GetElementByTestId("DqtSection")?.GetSummaryListValueForKey("Date of birth"));
-        Assert.Equal("Provided", doc.GetSummaryListValueForKey("National insurance number"));
+        Assert.Equal(dqtNino, doc.GetSummaryListValueForKey("National insurance number"));
         Assert.Equal(user.Trn, doc.GetSummaryListValueForKey("TRN"));
     }