Skip to content

Commit b3d1a8e

Browse files
brightonsboxbrightonsbox
committed
CDPS-1096: Fixing linting issues
1 parent 63f0bb7 commit b3d1a8e

File tree

9 files changed

+85
-96
lines changed

9 files changed

+85
-96
lines changed

‎src/main/kotlin/uk/gov/justice/digital/hmpps/personintegrationapi/common/config/HmppsPersonIntegrationApiExceptionHandler.kt

+50-55
Original file line numberDiff line numberDiff line change
@@ -46,69 +46,64 @@ class HmppsPersonIntegrationApiExceptionHandler {
4646
}
4747

4848
@ExceptionHandler(HttpMessageNotReadableException::class)
49-
fun handleValidationException(ex: HttpMessageNotReadableException): ResponseEntity<ErrorResponse> =
50-
ResponseEntity
51-
.status(BAD_REQUEST)
52-
.body(
53-
ErrorResponse(
54-
status = BAD_REQUEST,
55-
userMessage = "Unable to read the request",
56-
developerMessage = ex.message,
57-
),
58-
).also { log.info(ex.message) }
49+
fun handleValidationException(ex: HttpMessageNotReadableException): ResponseEntity<ErrorResponse> = ResponseEntity
50+
.status(BAD_REQUEST)
51+
.body(
52+
ErrorResponse(
53+
status = BAD_REQUEST,
54+
userMessage = "Unable to read the request",
55+
developerMessage = ex.message,
56+
),
57+
).also { log.info(ex.message) }
5958

6059
@ExceptionHandler(HandlerMethodValidationException::class)
61-
fun handleHandlerMethodValidationException(e: HandlerMethodValidationException): ResponseEntity<ErrorResponse> =
62-
e.allErrors.map { it.toString() }.distinct().sorted().joinToString("\n")
63-
.let { validationErrors ->
64-
ResponseEntity
65-
.status(BAD_REQUEST)
66-
.body(
67-
ErrorResponse(
68-
status = BAD_REQUEST,
69-
userMessage = "Validation failure(s): ${
70-
e.allErrors.map { it.defaultMessage }.distinct().sorted().joinToString("\n")
71-
}",
72-
developerMessage = "${e.message} $validationErrors",
73-
),
74-
).also { log.info("Validation exception: $validationErrors\n {}", e.message) }
75-
}
60+
fun handleHandlerMethodValidationException(e: HandlerMethodValidationException): ResponseEntity<ErrorResponse> = e.allErrors.map { it.toString() }.distinct().sorted().joinToString("\n")
61+
.let { validationErrors ->
62+
ResponseEntity
63+
.status(BAD_REQUEST)
64+
.body(
65+
ErrorResponse(
66+
status = BAD_REQUEST,
67+
userMessage = "Validation failure(s): ${
68+
e.allErrors.map { it.defaultMessage }.distinct().sorted().joinToString("\n")
69+
}",
70+
developerMessage = "${e.message} $validationErrors",
71+
),
72+
).also { log.info("Validation exception: $validationErrors\n {}", e.message) }
73+
}
7674

7775
@ExceptionHandler(ValidationException::class)
78-
fun handleValidationException(e: ValidationException): ResponseEntity<ErrorResponse> =
79-
ResponseEntity
80-
.status(BAD_REQUEST)
81-
.body(
82-
ErrorResponse(
83-
status = BAD_REQUEST,
84-
userMessage = "Validation failure: ${e.message}",
85-
developerMessage = e.message,
86-
),
87-
).also { log.info("Validation exception: {}", e.message) }
76+
fun handleValidationException(e: ValidationException): ResponseEntity<ErrorResponse> = ResponseEntity
77+
.status(BAD_REQUEST)
78+
.body(
79+
ErrorResponse(
80+
status = BAD_REQUEST,
81+
userMessage = "Validation failure: ${e.message}",
82+
developerMessage = e.message,
83+
),
84+
).also { log.info("Validation exception: {}", e.message) }
8885

8986
@ExceptionHandler(NoResourceFoundException::class)
90-
fun handleNoResourceFoundException(e: NoResourceFoundException): ResponseEntity<ErrorResponse> =
91-
ResponseEntity
92-
.status(NOT_FOUND)
93-
.body(
94-
ErrorResponse(
95-
status = NOT_FOUND,
96-
userMessage = "No resource found failure: ${e.message}",
97-
developerMessage = e.message,
98-
),
99-
).also { log.info("No resource found exception: {}", e.message) }
87+
fun handleNoResourceFoundException(e: NoResourceFoundException): ResponseEntity<ErrorResponse> = ResponseEntity
88+
.status(NOT_FOUND)
89+
.body(
90+
ErrorResponse(
91+
status = NOT_FOUND,
92+
userMessage = "No resource found failure: ${e.message}",
93+
developerMessage = e.message,
94+
),
95+
).also { log.info("No resource found exception: {}", e.message) }
10096

10197
@ExceptionHandler(AccessDeniedException::class)
102-
fun handleAccessDeniedException(e: AccessDeniedException): ResponseEntity<ErrorResponse> =
103-
ResponseEntity
104-
.status(FORBIDDEN)
105-
.body(
106-
ErrorResponse(
107-
status = FORBIDDEN,
108-
userMessage = "Forbidden: ${e.message}",
109-
developerMessage = e.message,
110-
),
111-
).also { log.debug("Forbidden (403) returned: {}", e.message) }
98+
fun handleAccessDeniedException(e: AccessDeniedException): ResponseEntity<ErrorResponse> = ResponseEntity
99+
.status(FORBIDDEN)
100+
.body(
101+
ErrorResponse(
102+
status = FORBIDDEN,
103+
userMessage = "Forbidden: ${e.message}",
104+
developerMessage = e.message,
105+
),
106+
).also { log.debug("Forbidden (403) returned: {}", e.message) }
112107

113108
@ExceptionHandler(Exception::class)
114109
fun handleException(e: Exception): ResponseEntity<ErrorResponse> = ResponseEntity

‎src/main/kotlin/uk/gov/justice/digital/hmpps/personintegrationapi/common/config/OpenApiConfiguration.kt

+4-6
Original file line numberDiff line numberDiff line change
@@ -72,10 +72,8 @@ class OpenApiConfiguration(
7272
}
7373

7474
@Bean
75-
fun v1Api(): GroupedOpenApi {
76-
return GroupedOpenApi.builder()
77-
.group("v1")
78-
.pathsToMatch("/v1/**")
79-
.build()
80-
}
75+
fun v1Api(): GroupedOpenApi = GroupedOpenApi.builder()
76+
.group("v1")
77+
.pathsToMatch("/v1/**")
78+
.build()
8179
}

‎src/main/kotlin/uk/gov/justice/digital/hmpps/personintegrationapi/common/config/UserEnhancedOAuth2ClientCredentialGrantRequestConverter.kt

+1-2
Original file line numberDiff line numberDiff line change
@@ -8,8 +8,7 @@ import org.springframework.util.MultiValueMap
88
import java.util.Objects
99

1010
@Suppress("UNCHECKED_CAST")
11-
class UserEnhancedOAuth2ClientCredentialGrantRequestConverter :
12-
OAuth2ClientCredentialsGrantRequestEntityConverter() {
11+
class UserEnhancedOAuth2ClientCredentialGrantRequestConverter : OAuth2ClientCredentialsGrantRequestEntityConverter() {
1312

1413
fun enhanceWithUsername(
1514
grantRequest: OAuth2ClientCredentialsGrantRequest?,

‎src/main/kotlin/uk/gov/justice/digital/hmpps/personintegrationapi/common/config/WebClientConfiguration.kt

+9-13
Original file line numberDiff line numberDiff line change
@@ -36,27 +36,23 @@ class WebClientConfiguration(
3636
@Value("\${prison-api.timeout:30s}") private val prisonApiTimeout: Duration,
3737
) {
3838
@Bean
39-
fun authHealthWebClient(builder: WebClient.Builder): WebClient =
40-
builder.healthWebClient(authBaseUri, authHealthTimeout)
39+
fun authHealthWebClient(builder: WebClient.Builder): WebClient = builder.healthWebClient(authBaseUri, authHealthTimeout)
4140

4241
@Bean
43-
fun prisonApiHealthWebClient(builder: WebClient.Builder): WebClient =
44-
builder.healthWebClient(prisonApiBaseUri, prisonApiHealthTimeout)
42+
fun prisonApiHealthWebClient(builder: WebClient.Builder): WebClient = builder.healthWebClient(prisonApiBaseUri, prisonApiHealthTimeout)
4543

4644
@Bean
4745
@RequestScope
4846
fun prisonApiWebClient(
4947
clientRegistrationRepository: ClientRegistrationRepository,
5048
builder: WebClient.Builder,
51-
): WebClient {
52-
return getOAuthWebClient(
53-
authorizedClientManagerUserEnhanced(clientRegistrationRepository),
54-
builder,
55-
prisonApiBaseUri,
56-
"hmpps-person-integration-api",
57-
prisonApiTimeout,
58-
)
59-
}
49+
): WebClient = getOAuthWebClient(
50+
authorizedClientManagerUserEnhanced(clientRegistrationRepository),
51+
builder,
52+
prisonApiBaseUri,
53+
"hmpps-person-integration-api",
54+
prisonApiTimeout,
55+
)
6056

6157
@Bean
6258
@DependsOn("prisonApiWebClient")

‎src/main/kotlin/uk/gov/justice/digital/hmpps/personintegrationapi/corepersonrecord/resource/CorePersonRecordV1Resource.kt

+1-3
Original file line numberDiff line numberDiff line change
@@ -145,9 +145,7 @@ class CorePersonRecordV1Resource(
145145
fun putProfileImageByPrisonerNumber(
146146
@RequestParam(required = true) @Valid @ValidPrisonerNumber prisonerNumber: String,
147147
@RequestPart(name = "imageFile", required = true) profileImage: MultipartFile,
148-
): ResponseEntity<Void> {
149-
return noContent().build()
150-
}
148+
): ResponseEntity<Void> = noContent().build()
151149

152150
@GetMapping("reference-data/domain/{domain}/codes")
153151
@ResponseStatus(HttpStatus.OK)

‎src/main/kotlin/uk/gov/justice/digital/hmpps/personintegrationapi/personprotectercharacteristics/resource/PersonProtectedCharacteristicsV1Resource.kt

+6-8
Original file line numberDiff line numberDiff line change
@@ -84,14 +84,12 @@ class PersonProtectedCharacteristicsV1Resource {
8484
fun putReligionByPrisonerNumber(
8585
@RequestParam(required = true) @Valid @ValidPrisonerNumber prisonerNumber: String,
8686
@RequestBody(required = true) @Valid religionV1RequestDto: ReligionV1RequestDto,
87-
): ResponseEntity<PersonReligionInformationV1ResponseDto> {
88-
return ok().body(
89-
PersonReligionInformationV1ResponseDto(
90-
currentReligion = ReligionDto(),
91-
religionHistory = emptySet(),
92-
),
93-
)
94-
}
87+
): ResponseEntity<PersonReligionInformationV1ResponseDto> = ok().body(
88+
PersonReligionInformationV1ResponseDto(
89+
currentReligion = ReligionDto(),
90+
religionHistory = emptySet(),
91+
),
92+
)
9593

9694
@GetMapping("reference-data/domain/{domain}/codes")
9795
@ResponseStatus(HttpStatus.OK)

‎src/test/kotlin/uk/gov/justice/digital/hmpps/personintegrationapi/integration/ResourceSecurityTest.kt

+6-7
Original file line numberDiff line numberDiff line change
@@ -52,10 +52,9 @@ class ResourceSecurityTest : IntegrationTestBase() {
5252
}
5353
}
5454

55-
private fun RequestMappingInfo.getMappings() =
56-
methodsCondition.methods
57-
.map { it.name }
58-
.ifEmpty { listOf("") } // if no methods defined then match all rather than none
59-
.flatMap { method ->
60-
pathPatternsCondition?.patternValues?.map { "$method $it" } ?: emptyList()
61-
}
55+
private fun RequestMappingInfo.getMappings() = methodsCondition.methods
56+
.map { it.name }
57+
.ifEmpty { listOf("") } // if no methods defined then match all rather than none
58+
.flatMap { method ->
59+
pathPatternsCondition?.patternValues?.map { "$method $it" } ?: emptyList()
60+
}

‎src/test/kotlin/uk/gov/justice/digital/hmpps/personintegrationapi/integration/wiremock/HmppsAuthMockServer.kt

+4-1
Original file line numberDiff line numberDiff line change
@@ -14,7 +14,10 @@ import org.junit.jupiter.api.extension.ExtensionContext
1414
import java.time.LocalDateTime
1515
import java.time.ZoneOffset
1616

17-
class HmppsAuthApiExtension : BeforeAllCallback, AfterAllCallback, BeforeEachCallback {
17+
class HmppsAuthApiExtension :
18+
BeforeAllCallback,
19+
AfterAllCallback,
20+
BeforeEachCallback {
1821
companion object {
1922
@JvmField
2023
val hmppsAuth = HmppsAuthMockServer()

‎src/test/kotlin/uk/gov/justice/digital/hmpps/personintegrationapi/integration/wiremock/PrisonApiMockServer.kt

+4-1
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,10 @@ class PrisonApiMockServer : WireMockServer(8082) {
110110
}
111111
}
112112

113-
class PrisonApiExtension : BeforeAllCallback, AfterAllCallback, BeforeEachCallback {
113+
class PrisonApiExtension :
114+
BeforeAllCallback,
115+
AfterAllCallback,
116+
BeforeEachCallback {
114117
companion object {
115118
@JvmField
116119
val prisonApi = PrisonApiMockServer()

0 commit comments

Comments
 (0)