Skip to content

Commit dac5b8e

Browse files
Renamed according to acronym
1 parent 28d0f1e commit dac5b8e

File tree

6 files changed

+27
-27
lines changed

6 files changed

+27
-27
lines changed

src/main/kotlin/uk/gov/justice/digital/hmpps/hmppsintegrationapi/gateways/ManagePOMCaseGateway.kt

+4-4
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@ import org.springframework.beans.factory.annotation.Value
55
import org.springframework.http.HttpMethod
66
import org.springframework.stereotype.Component
77
import uk.gov.justice.digital.hmpps.hmppsintegrationapi.extensions.WebClientWrapper
8-
import uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.hmpps.PrisonOfficerManager
8+
import uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.hmpps.PrisonOffenderManager
99
import uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.hmpps.Response
1010
import uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.hmpps.UpstreamApi
1111
import uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.managePOMCase.AllocationPrimaryPOM
@@ -19,7 +19,7 @@ class ManagePOMCaseGateway(
1919
@Autowired
2020
lateinit var hmppsAuthGateway: HmppsAuthGateway
2121

22-
fun getPrimaryPOMForNomisNumber(id: String): Response<PrisonOfficerManager> {
22+
fun getPrimaryPOMForNomisNumber(id: String): Response<PrisonOffenderManager> {
2323
val result =
2424
webClient.request<AllocationPrimaryPOM>(
2525
HttpMethod.GET,
@@ -30,12 +30,12 @@ class ManagePOMCaseGateway(
3030

3131
return when (result) {
3232
is WebClientWrapper.WebClientWrapperResponse.Success -> {
33-
Response(data = result.data.toPrisonOfficerManager())
33+
Response(data = result.data.toPrisonOffenderManager())
3434
}
3535

3636
is WebClientWrapper.WebClientWrapperResponse.Error -> {
3737
Response(
38-
data = PrisonOfficerManager(),
38+
data = PrisonOffenderManager(),
3939
errors = result.errors,
4040
)
4141
}
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
package uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.hmpps
22

33
data class PersonResponsibleOfficer(
4-
val prisonOfficerManager: PrisonOfficerManager = PrisonOfficerManager(),
4+
val prisonOffenderManager: PrisonOffenderManager = PrisonOffenderManager(),
55
val communityOffenderManager: CommunityOffenderManager = CommunityOffenderManager(),
66
)

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

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,6 @@
11
package uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.hmpps
22

3-
data class PrisonOfficerManager(
3+
data class PrisonOffenderManager(
44
val forename: String? = null,
55
val surname: String? = null,
66
val prison: Prison = Prison(),

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

+3-3
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,14 @@
11
package uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.managePOMCase
22

33
import uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.hmpps.Prison
4-
import uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.hmpps.PrisonOfficerManager
4+
import uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.hmpps.PrisonOffenderManager
55

66
data class AllocationPrimaryPOM(
77
val manager: AllocationManager = AllocationManager(),
88
val prison: AllocationPrison = AllocationPrison(),
99
) {
10-
fun toPrisonOfficerManager(): PrisonOfficerManager =
11-
PrisonOfficerManager(
10+
fun toPrisonOffenderManager(): PrisonOffenderManager =
11+
PrisonOffenderManager(
1212
forename = this.manager.forename,
1313
surname = this.manager.surname,
1414
prison =

src/main/kotlin/uk/gov/justice/digital/hmpps/hmppsintegrationapi/services/GetPrisonOfficerManagerForPersonService.kt src/main/kotlin/uk/gov/justice/digital/hmpps/hmppsintegrationapi/services/GetPrisonOffenderManagerForPersonService.kt

+7-7
Original file line numberDiff line numberDiff line change
@@ -3,26 +3,26 @@ package uk.gov.justice.digital.hmpps.hmppsintegrationapi.services
33
import org.springframework.beans.factory.annotation.Autowired
44
import org.springframework.stereotype.Service
55
import uk.gov.justice.digital.hmpps.hmppsintegrationapi.gateways.ManagePOMCaseGateway
6-
import uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.hmpps.PrisonOfficerManager
6+
import uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.hmpps.PrisonOffenderManager
77
import uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.hmpps.Response
88

99
@Service
10-
class GetPrisonOfficerManagerForPersonService(
10+
class GetPrisonOffenderManagerForPersonService(
1111
@Autowired val getPersonService: GetPersonService,
1212
@Autowired val managePOMCaseGateway: ManagePOMCaseGateway,
1313
) {
14-
fun execute(hmppsId: String): Response<PrisonOfficerManager> {
14+
fun execute(hmppsId: String): Response<PrisonOffenderManager> {
1515
val personResponse = getPersonService.execute(hmppsId = hmppsId)
1616
val nomisNumber = personResponse.data?.identifiers?.nomisNumber
17-
var prisonOfficeManager: Response<PrisonOfficerManager> = Response(data = PrisonOfficerManager())
17+
var prisonOffenderManager: Response<PrisonOffenderManager> = Response(data = PrisonOffenderManager())
1818

1919
if (nomisNumber != null) {
20-
prisonOfficeManager = managePOMCaseGateway.getPrimaryPOMForNomisNumber(nomisNumber)
20+
prisonOffenderManager = managePOMCaseGateway.getPrimaryPOMForNomisNumber(nomisNumber)
2121
}
2222

2323
return Response(
24-
data = prisonOfficeManager.data,
25-
errors = personResponse.errors + prisonOfficeManager.errors,
24+
data = prisonOffenderManager.data,
25+
errors = personResponse.errors + prisonOffenderManager.errors,
2626
)
2727
}
2828
}

src/test/kotlin/uk/gov/justice/digital/hmpps/hmppsintegrationapi/services/GetPrisonOfficerManagerForPersonServiceTest.kt src/test/kotlin/uk/gov/justice/digital/hmpps/hmppsintegrationapi/services/GetPrisonOffenderManagerForPersonServiceTest.kt

+11-11
Original file line numberDiff line numberDiff line change
@@ -13,37 +13,37 @@ import uk.gov.justice.digital.hmpps.hmppsintegrationapi.gateways.ManagePOMCaseGa
1313
import uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.hmpps.Identifiers
1414
import uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.hmpps.Person
1515
import uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.hmpps.Prison
16-
import uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.hmpps.PrisonOfficerManager
16+
import uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.hmpps.PrisonOffenderManager
1717
import uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.hmpps.Response
1818
import uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.hmpps.UpstreamApi
1919
import uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.hmpps.UpstreamApiError
2020

2121
@ContextConfiguration(
2222
initializers = [ConfigDataApplicationContextInitializer::class],
23-
classes = [GetPrisonOfficerManagerForPersonService::class],
23+
classes = [GetPrisonOffenderManagerForPersonService::class],
2424
)
25-
class GetPrisonOfficerManagerForPersonServiceTest(
25+
class GetPrisonOffenderManagerForPersonServiceTest(
2626
@MockBean val managePOMCaseGateway: ManagePOMCaseGateway,
2727
@MockBean val getPersonService: GetPersonService,
28-
private val getPrisonOfficerManagerForPersonService: GetPrisonOfficerManagerForPersonService,
28+
private val getPrisonOffenderManagerForPersonService: GetPrisonOffenderManagerForPersonService,
2929
) : DescribeSpec(
3030
{
3131
val hmppsId = "1234/56789B"
3232
val nomisNumber = "Z99999ZZ"
3333
val person = Person(firstName = "Julianna", lastName = "Blake", identifiers = Identifiers(nomisNumber = nomisNumber))
3434

35-
val prisonOfficerManager = PrisonOfficerManager(forename = "Paul", surname = "Smith", prison = Prison(code = "RED"))
35+
val prisonOffenderManager = PrisonOffenderManager(forename = "Paul", surname = "Smith", prison = Prison(code = "RED"))
3636

3737
beforeEach {
3838
Mockito.reset(getPersonService)
3939
Mockito.reset(managePOMCaseGateway)
4040

4141
whenever(getPersonService.execute(hmppsId = hmppsId)).thenReturn(Response(person))
42-
whenever(managePOMCaseGateway.getPrimaryPOMForNomisNumber(id = nomisNumber)).thenReturn(Response(prisonOfficerManager))
42+
whenever(managePOMCaseGateway.getPrimaryPOMForNomisNumber(id = nomisNumber)).thenReturn(Response(prisonOffenderManager))
4343
}
4444

4545
it("performs a search according to hmpps Id") {
46-
getPrisonOfficerManagerForPersonService.execute(hmppsId)
46+
getPrisonOffenderManagerForPersonService.execute(hmppsId)
4747
verify(getPersonService, VerificationModeFactory.times(1)).execute(hmppsId = hmppsId)
4848
}
4949

@@ -53,8 +53,8 @@ class GetPrisonOfficerManagerForPersonServiceTest(
5353
data = person,
5454
),
5555
)
56-
val result = getPrisonOfficerManagerForPersonService.execute(hmppsId)
57-
result.shouldBe(Response(data = prisonOfficerManager))
56+
val result = getPrisonOffenderManagerForPersonService.execute(hmppsId)
57+
result.shouldBe(Response(data = prisonOffenderManager))
5858
}
5959

6060
it("should return a list of errors if person not found") {
@@ -70,8 +70,8 @@ class GetPrisonOfficerManagerForPersonServiceTest(
7070
),
7171
),
7272
)
73-
val result = getPrisonOfficerManagerForPersonService.execute("NOT_FOUND")
74-
result.data.shouldBe(PrisonOfficerManager())
73+
val result = getPrisonOffenderManagerForPersonService.execute("NOT_FOUND")
74+
result.data.shouldBe(PrisonOffenderManager())
7575
result.errors.first().type.shouldBe(UpstreamApiError.Type.ENTITY_NOT_FOUND)
7676
}
7777
},

0 commit comments

Comments
 (0)