PI-2470 Truncate long assessor name by initialising first name(s) (#4194

)
ministryofjustice · Aug 23, 2024 · 3f85ef8 · 3f85ef8
1 parent e2c6e56
commit 3f85ef8
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 3 deletions.
diff --git a/...-summary-and-delius/src/dev/resources/simulations/__files/assessment-summary-M123456.json b/...-summary-and-delius/src/dev/resources/simulations/__files/assessment-summary-M123456.json
@@ -53,7 +53,7 @@
         "totWeightedScore": 88,
         "pOAssessment": "270",
         "pOAssessmentDesc": "Review",
-        "assessorName": "LevelTwo CentralSupport",
+        "assessorName": "Reeeeeeeeeeaaaaaaaaaally Loooooooooooooooooong Name",
         "ogrs1Year": 43,
         "ogrs2Year": 61,
         "reviewTerm": "N",

diff --git a/...ary-and-delius/src/integrationTest/kotlin/uk/gov/justice/digital/hmpps/IntegrationTest.kt b/...ary-and-delius/src/integrationTest/kotlin/uk/gov/justice/digital/hmpps/IntegrationTest.kt
@@ -189,7 +189,7 @@ internal class IntegrationTest {
             val assessment = oasysAssessmentRepository.findByOasysId("100835871")
             assertThat(assessment?.court?.code, equalTo("LVRPCC"))
             assertThat(assessment?.offence?.code, equalTo("00857"))
-            assertThat(assessment?.assessedBy, equalTo("LevelTwo CentralSupport"))
+            assertThat(assessment?.assessedBy, equalTo("R. L. Name"))
             assertThat(assessment?.date, equalTo(LocalDate.parse("2023-12-15")))
             assertThat(assessment?.totalScore, equalTo(88))
 

diff --git a/...mary-and-delius/src/main/kotlin/uk/gov/justice/digital/hmpps/service/AssessmentService.kt b/...mary-and-delius/src/main/kotlin/uk/gov/justice/digital/hmpps/service/AssessmentService.kt
@@ -65,7 +65,7 @@ class AssessmentService(
                 ?.let { offenceRepository.findByCode(it.offenceCode + it.offenceSubcode) },
             totalScore = furtherInformation.totWeightedScore,
             description = furtherInformation.pOAssessmentDesc,
-            assessedBy = furtherInformation.assessorName,
+            assessedBy = furtherInformation.assessorName?.let { if (it.length > 35) initialiseName(it) else it },
             riskFlags = riskFlags.joinToString(","),
             concernFlags = concernFlags.joinToString(","),
             dateCreated = initiationDate,
@@ -86,6 +86,12 @@ class AssessmentService(
             ?.forEach { assessment.withSentencePlan(it) }
         return assessment
     }
+
+    private fun initialiseName(name: String): String {
+        val names = name.split(Regex("\\s+")).toMutableList()
+        for (i in 0 until names.size - 1) names[i] = "${names[i].first()}."
+        return names.joinToString(" ")
+    }
 }
 
 private fun AssessmentSummary.contactDetail(contactDate: LocalDate) =