Skip to content

Commit 4c4fcae

Browse files
committed
Revert change to mock authorisation service.
1 parent f9cc9da commit 4c4fcae

File tree

2 files changed

+4
-8
lines changed

2 files changed

+4
-8
lines changed

src/main/kotlin/uk/gov/justice/digital/hmpps/hmppsintegrationapi/extensions/AuthorisationFilter.kt

+1-1
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,6 @@ class AuthorisationFilter
2222
@Autowired
2323
constructor(
2424
var authorisationConfig: AuthorisationConfig,
25-
var authoriseConsumerService: AuthoriseConsumerService,
2625
) : Filter {
2726
@Throws(IOException::class, ServletException::class)
2827
override fun doFilter(
@@ -32,6 +31,7 @@ class AuthorisationFilter
3231
) {
3332
val req = request as HttpServletRequest
3433
val res = response as HttpServletResponse
34+
val authoriseConsumerService = AuthoriseConsumerService()
3535
val subjectDistinguishedName = req.getAttribute("clientName") as String?
3636
val requestedPath = req.requestURI
3737

src/test/kotlin/uk/gov/justice/digital/hmpps/hmppsintegrationapi/extensions/AuthorisationFilterTest.kt

+3-7
Original file line numberDiff line numberDiff line change
@@ -11,11 +11,10 @@ import org.mockito.kotlin.whenever
1111
import uk.gov.justice.digital.hmpps.hmppsintegrationapi.config.AuthorisationConfig
1212
import uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.roleconfig.ConsumerConfig
1313
import uk.gov.justice.digital.hmpps.hmppsintegrationapi.models.roleconfig.ConsumerFilters
14-
import uk.gov.justice.digital.hmpps.hmppsintegrationapi.services.internal.AuthoriseConsumerService
1514

1615
class AuthorisationFilterTest {
1716
private var authorisationConfig: AuthorisationConfig = AuthorisationConfig()
18-
private var authorisationFilter: AuthorisationFilter = AuthorisationFilter(authorisationConfig, AuthoriseConsumerService())
17+
private var authorisationFilter: AuthorisationFilter = AuthorisationFilter(authorisationConfig)
1918
private var examplePath: String = "/v1/persons"
2019
private var exampleConsumer: String = "consumer-name"
2120

@@ -42,15 +41,12 @@ class AuthorisationFilterTest {
4241
val mockRequest = mock(HttpServletRequest::class.java)
4342
whenever(mockRequest.requestURI).thenReturn(examplePath)
4443
whenever(mockRequest.getAttribute("clientName")).thenReturn(exampleConsumer)
45-
4644
val mockResponse = mock(HttpServletResponse::class.java)
4745
val mockChain = mock(FilterChain::class.java)
4846

49-
val mockService = mock(AuthoriseConsumerService::class.java)
50-
whenever(mockService.execute(exampleConsumer, authorisationConfig.consumers, examplePath))
51-
.thenReturn(false)
47+
authorisationConfig.consumers = mapOf(exampleConsumer to ConsumerConfig(include = emptyList(), filters = ConsumerFilters(emptyMap())))
5248

53-
val authorisationFilter = AuthorisationFilter(authorisationConfig, mockService)
49+
val authorisationFilter = AuthorisationFilter(authorisationConfig)
5450

5551
// Act
5652
authorisationFilter.doFilter(mockRequest, mockResponse, mockChain)

0 commit comments

Comments
 (0)