Breaking Role Name changes. CanGetCustomersAtAnyBank ->

CanGetCustomersAtAllBanks and CanGetCustomers ->
CanGetCustomersAtOneBank
This commit is contained in:
simonredfern 2025-11-18 00:23:10 +01:00
parent dda1432132
commit 0dec283247
6 changed files with 11 additions and 11 deletions

View File

@ -4656,7 +4656,7 @@ object SwaggerDefinitionsJSON {
bank_id = bankIdExample.value
)
lazy val canGetCustomersJson = ApiRole.canGetCustomers
lazy val canGetCustomersJson = ApiRole.canGetCustomersAtOneBank
lazy val cardAttributeCommons = CardAttributeCommons(
bankId = Some(BankId(bankIdExample.value)),

View File

@ -84,14 +84,14 @@ object ApiRole extends MdcLoggable{
case class CanSearchMetrics(requiresBankId: Boolean = false) extends ApiRole
lazy val canSearchMetrics = CanSearchMetrics()
case class CanGetCustomersAtAnyBank(requiresBankId: Boolean = false) extends ApiRole
lazy val canGetCustomersAtAnyBank = CanGetCustomersAtAnyBank()
case class CanGetCustomersAtAllBanks(requiresBankId: Boolean = false) extends ApiRole
lazy val canGetCustomersAtAllBanks = CanGetCustomersAtAllBanks()
case class CanGetCustomersMinimalAtAnyBank(requiresBankId: Boolean = false) extends ApiRole
lazy val canGetCustomersMinimalAtAnyBank = CanGetCustomersMinimalAtAnyBank()
case class CanGetCustomers(requiresBankId: Boolean = true) extends ApiRole
lazy val canGetCustomers = CanGetCustomers()
case class CanGetCustomersAtOneBank(requiresBankId: Boolean = true) extends ApiRole
lazy val canGetCustomersAtOneBank = CanGetCustomersAtOneBank()
case class CanGetCustomersMinimal(requiresBankId: Boolean = true) extends ApiRole
lazy val canGetCustomersMinimal = CanGetCustomersMinimal()

View File

@ -6857,7 +6857,7 @@ trait APIMethods400 extends MdcLoggable {
UnknownError
),
List(apiTagCustomer, apiTagUser),
Some(List(canGetCustomersAtAnyBank))
Some(List(canGetCustomersAtAllBanks))
)
lazy val getCustomersAtAnyBank: OBPEndpoint = {
case "customers" :: Nil JsonGet _ => { cc =>

View File

@ -1609,7 +1609,7 @@ trait APIMethods500 {
UnknownError
),
List(apiTagCustomer, apiTagUser),
Some(List(canGetCustomers))
Some(List(canGetCustomersAtOneBank))
)
lazy val getCustomersAtOneBank : OBPEndpoint = {

View File

@ -1129,7 +1129,7 @@ trait APIMethods600 {
UnknownError
),
List(apiTagCustomer, apiTagUser),
Some(List(canGetCustomers))
Some(List(canGetCustomersAtOneBank))
)
lazy val getCustomersAtOneBank : OBPEndpoint = {

View File

@ -91,14 +91,14 @@ class CustomerTest extends V400ServerSetup with PropsReset{
val response = makeGetRequest(request)
Then("We should get a 403")
response.code should equal(403)
val errorMsg = UserHasMissingRoles + canGetCustomersAtAnyBank
val errorMsg = UserHasMissingRoles + canGetCustomersAtAllBanks
And("error should be " + errorMsg)
val errorMessage = response.body.extract[ErrorMessage].message
errorMessage contains (UserHasMissingRoles) should be (true)
errorMessage contains (canGetCustomersAtAnyBank.toString()) should be (true)
errorMessage contains (canGetCustomersAtAllBanks.toString()) should be (true)
}
scenario("We will call the endpoint with a user credentials and a proper role", ApiEndpoint1, VersionOfApi) {
Entitlement.entitlement.vend.addEntitlement("", resourceUser1.userId, CanGetCustomersAtAnyBank.toString)
Entitlement.entitlement.vend.addEntitlement("", resourceUser1.userId, CanGetCustomersAtAllBanks.toString)
When(s"We make a request $VersionOfApi")
val request = (v4_0_0_Request / "customers").GET <@(user1)
val response = makeGetRequest(request)