Skip to content

Commit 3ede0c2

Browse files
Merge pull request #384 from ministryofjustice/HIA-666
Golden record for assessments
2 parents 79a14ed + abc4423 commit 3ede0c2

File tree

6 files changed

+72
-7
lines changed

6 files changed

+72
-7
lines changed
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,13 @@
11
package uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.hmpps
22

33
data class RiskAssessment(
4-
val classificationCode: String?,
4+
val classificationCode: String? = null,
5+
val classification: String? = null,
6+
val assessmentCode: String? = null,
7+
val assessmentDescription: String? = null,
8+
val assessmentDate: String? = null,
9+
val nextReviewDate: String? = null,
10+
val assessmentAgencyId: String? = null,
11+
val assessmentStatus: String? = null,
12+
val assessmentComment: String? = null,
513
)

src/main/kotlin/uk/gov/justice/digital/hmpps/hmppsintegrationapi/models/hmpps/RiskCategory.kt

+2
Original file line numberDiff line numberDiff line change
@@ -3,4 +3,6 @@ package uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.hmpps
33
data class RiskCategory(
44
val offenderNo: String? = null,
55
val assessments: List<RiskAssessment> = emptyList(),
6+
val category: String? = null,
7+
val categoryCode: String? = null,
68
)

src/main/kotlin/uk/gov/justice/digital/hmpps/hmppsintegrationapi/models/nomis/NomisAssessment.kt

+16
Original file line numberDiff line numberDiff line change
@@ -4,8 +4,24 @@ import uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.hmpps.RiskAssessm
44

55
data class NomisAssessment(
66
val classificationCode: String? = null,
7+
val classification: String? = null,
8+
val assessmentCode: String? = null,
9+
val assessmentDescription: String? = null,
10+
val assessmentDate: String? = null,
11+
val nextReviewDate: String? = null,
12+
val assessmentAgencyId: String? = null,
13+
val assessmentStatus: String? = null,
14+
val assessmentComment: String? = null,
715
) {
816
fun toRiskAssessment() = RiskAssessment(
917
classificationCode = this.classificationCode,
18+
classification = this.classification,
19+
assessmentCode = this.assessmentCode,
20+
assessmentDescription = this.assessmentDescription,
21+
assessmentDate = this.assessmentDate,
22+
nextReviewDate = this.nextReviewDate,
23+
assessmentAgencyId = this.assessmentAgencyId,
24+
assessmentStatus = this.assessmentStatus,
25+
assessmentComment = this.assessmentComment,
1026
)
1127
}

src/main/kotlin/uk/gov/justice/digital/hmpps/hmppsintegrationapi/models/nomis/NomisInmateDetail.kt

+4
Original file line numberDiff line numberDiff line change
@@ -5,9 +5,13 @@ import uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.hmpps.RiskCategor
55
data class NomisInmateDetail(
66
val offenderNo: String? = null,
77
val assessments: List<NomisAssessment> = emptyList(),
8+
val category: String? = null,
9+
val categoryCode: String? = null,
810
) {
911
fun toRiskCategory(): RiskCategory = RiskCategory(
1012
offenderNo = this.offenderNo,
1113
assessments = this.assessments.map { it.toRiskAssessment() },
14+
category = this.category,
15+
categoryCode = this.categoryCode,
1216
)
1317
}

src/test/kotlin/uk/gov/justice/digital/hmpps/hmppsintegrationapi/controllers/v1/person/RiskCategoriesControllerTest.kt

+29-4
Original file line numberDiff line numberDiff line change
@@ -45,7 +45,22 @@ internal class RiskCategoriesControllerTest(
4545
Response(
4646
data = RiskCategory(
4747
offenderNo = "A1234AA",
48-
assessments = listOf(RiskAssessment(classificationCode = "C")),
48+
assessments = listOf(
49+
RiskAssessment(
50+
classificationCode = "C",
51+
classification = "Cat C",
52+
assessmentCode = "CATEGORY",
53+
assessmentDescription = "Categorisation",
54+
assessmentDate = "2018-02-11",
55+
nextReviewDate = "2018-02-11",
56+
assessmentAgencyId = "MDI",
57+
assessmentStatus = "P",
58+
assessmentComment = "Comment details",
59+
),
60+
61+
),
62+
category = "string",
63+
categoryCode = "string",
4964
),
5065
),
5166
)
@@ -79,10 +94,20 @@ internal class RiskCategoriesControllerTest(
7994
"offenderNo": "A1234AA",
8095
"assessments": [
8196
{
82-
"classificationCode": "C"
97+
"classificationCode": "C",
98+
"classification": "Cat C",
99+
"assessmentCode": "CATEGORY",
100+
"assessmentDescription": "Categorisation",
101+
"assessmentDate": "2018-02-11",
102+
"nextReviewDate": "2018-02-11",
103+
"assessmentAgencyId": "MDI",
104+
"assessmentStatus": "P",
105+
"assessmentComment": "Comment details"
83106
}
84-
]
85-
}
107+
],
108+
"category": "string",
109+
"categoryCode": "string"
110+
}
86111
""".removeWhitespaceAndNewlines(),
87112
)
88113
}

src/test/kotlin/uk/gov/justice/digital/hmpps/hmppsintegrationapi/smoke/person/RisksSmokeTest.kt

+12-2
Original file line numberDiff line numberDiff line change
@@ -70,9 +70,19 @@ class RisksSmokeTest : DescribeSpec(
7070
"offenderNo": "A1234AA",
7171
"assessments": [
7272
{
73-
"classificationCode": "C"
73+
"classificationCode": "C",
74+
"classification": "Cat C",
75+
"assessmentCode": "CATEGORY",
76+
"assessmentDescription": "Categorisation",
77+
"assessmentDate": "2018-02-11",
78+
"nextReviewDate": "2018-02-11",
79+
"assessmentAgencyId": "MDI",
80+
"assessmentStatus": "P",
81+
"assessmentComment": "Comment details"
7482
}
75-
]
83+
],
84+
"category": "string",
85+
"categoryCode": "string"
7686
}
7787
}
7888
""".removeWhitespaceAndNewlines(),

0 commit comments

Comments
 (0)