Skip to content

Commit 005fde1

Browse files
committed
Fixing transaction post unit tests
1 parent e773b5c commit 005fde1

File tree

3 files changed

+3
-5
lines changed

3 files changed

+3
-5
lines changed

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

+1-1
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ class PostTransactionForPersonService(
5151
var response =
5252
nomisGateway.postTransactionForPerson(
5353
prisonId,
54-
hmppsId,
54+
nomisNumber,
5555
transactionRequest,
5656
)
5757

src/test/kotlin/uk/gov/justice/digital/hmpps/hmppsintegrationapi/gateways/nomis/PostTransactionForPersonTest.kt

-3
Original file line numberDiff line numberDiff line change
@@ -34,9 +34,6 @@ class PostTransactionForPersonTest(
3434
val nomisApiMockServer = NomisApiMockServer()
3535
val prisonId = "XYZ"
3636
val nomisNumber = "AA1234Z"
37-
val basePath = "/api/v1/prison/$prisonId/prisoners/$nomisNumber/transactions"
38-
val regexPath = "/api/v1/prison/.*/prisoners/.*/transactions"
39-
4037
val description = "Canteen Purchase of £16.34"
4138
val amount = 1634
4239
val clientTransactionId = "CL123212"

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

+2-1
Original file line numberDiff line numberDiff line change
@@ -12,6 +12,7 @@ import org.springframework.test.context.bean.override.mockito.MockitoBean
1212
import uk.gov.justice.digital.hmpps.hmppsintegrationapi.gateways.NomisGateway
1313
import uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.hmpps.NomisNumber
1414
import uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.hmpps.Response
15+
import uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.hmpps.TransactionCreateResponse
1516
import uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.hmpps.TransactionRequest
1617
import uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.hmpps.UpstreamApi
1718
import uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.hmpps.UpstreamApiError
@@ -103,7 +104,7 @@ internal class PostTransactionForPersonServiceTest(
103104
filters,
104105
)
105106

106-
result.data.shouldBe(exampleTransactionResponse)
107+
result.data.shouldBe(TransactionCreateResponse(transactionId = exampleTransactionResponse.id))
107108
}
108109

109110
it("records upstream API errors") {

0 commit comments

Comments
 (0)