Skip to content

Commit

Permalink
Merge pull request #2297 from hongwei1/develop
Browse files Browse the repository at this point in the history
refactor/added the checkShortString to createBank endpoint-fixed test
  • Loading branch information
simonredfern committed Oct 19, 2023
2 parents 66ff691 + a795b51 commit ec0d27b
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -1020,7 +1020,7 @@ object SwaggerDefinitionsJSON {
)

val postBankJson500 = PostBankJson500(
id = Some(idExample.value),
id = Some(bankIdExample.value),
bank_code = bankCodeExample.value,
full_name = Some(fullNameExample.value),
logo = Some(logoExample.value),
Expand Down
2 changes: 1 addition & 1 deletion obp-api/src/test/scala/code/api/v5_0_0/BankTests.scala
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,7 @@ class BankTests extends V500ServerSetupAsync with DefaultUsers {
And("We make the request")
val firstFullName = "A new full name"
val secondFullName = "A second new full name"
val postBank = postBankJson500.copy(id = Some(APIUtil.generateUUID()))
val postBank = postBankJson500.copy(id = Some(APIUtil.generateUUID().substring(25)))
val bankId = postBank.id.getOrElse("some_bank_id")
val request = (v5_0_0_Request / "banks").POST <@ (user1)
val requestPut = (v5_0_0_Request / "banks").PUT <@ (user1)
Expand Down

0 comments on commit ec0d27b

Please sign in to comment.