Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

GP tests are failing when test currencies are used #2740

Closed
vijayg10 opened this issue Apr 13, 2022 · 5 comments
Closed

GP tests are failing when test currencies are used #2740

vijayg10 opened this issue Apr 13, 2022 · 5 comments
Assignees
Labels
bug Something isn't working or it has wrong behavior on a Mojaloop Core service core-dev-squad oss-core This is an issue - story or epic related to a feature on a Mojaloop core service or related to it

Comments

@vijayg10
Copy link

vijayg10 commented Apr 13, 2022

Summary:
If I use test currencies such as XXX, XTS in TTK environment values, the GP test collection fails.
I think account lookup service doesn't have these currencies in the enum of the swagger file. I am not sure about the other places in mojaloop services.

Severity:
Medium

Priority:
Medium

Expected Behavior
TTK GP should pass with test currencies

Steps to Reproduce

  1. Deploy mojaloop 14.1.0 (13.1.1) version
  2. Change the env variable in TTK 'currency' to 'XXX'
  3. Change the env variable in TTK 'currency2' to 'XTS'
  4. Run TTK GP

Specifications

  • Component (if known): account-lookup-service (There may be other services where the test currencies are not listed)
  • Version: 13.1.1
  • Bug found/raised by: @vijayg10

Pull Requests

@vijayg10 vijayg10 added oss-core This is an issue - story or epic related to a feature on a Mojaloop core service or related to it bug Something isn't working or it has wrong behavior on a Mojaloop Core service labels Apr 13, 2022
@mdebarros
Copy link
Member

The migration-seeds would also need to be updated to include the test currencies --> https://github.com/mojaloop/account-lookup-service/blob/master/seeds/currency.js

@elnyry-sam-k
Copy link
Member

Thanks Vijay.. Can confirm that ALS swagger doesn't have the test currencies listed in the swagger enum: https://github.com/mojaloop/account-lookup-service/blob/b2e8cf5ef7e4ebe72441f24045be5b0d890c764a/src/interface/api-swagger.yaml#L1061 , so this has to be updated as an immediate fix..

Long term - we also need to look at using api-snippets so that any changes made in one place to the API can be used in all the services..

@elnyry-sam-k
Copy link
Member

Marking this as high-priority to be picked up next

@kleyow kleyow self-assigned this Nov 25, 2022
@kleyow
Copy link

kleyow commented Nov 27, 2022

image.png
Updated the ALS on moja2 and ran GP.

Only tests that are failing are transaction-requests-service ones. I've updated the currencies here. https://github.com/mojaloop/transaction-requests-service and will close this once that is merged.

@elnyry-sam-k
Copy link
Member

Thanks Kevin, good find..

@kleyow kleyow closed this as completed Nov 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working or it has wrong behavior on a Mojaloop Core service core-dev-squad oss-core This is an issue - story or epic related to a feature on a Mojaloop core service or related to it
Projects
None yet
Development

No branches or pull requests

4 participants