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

ML OSS Account Lookup Service (ALS) Implementation -Testing #656

Closed
10 of 16 tasks
elnyry-sam-k opened this issue Feb 26, 2019 · 1 comment
Closed
10 of 16 tasks

ML OSS Account Lookup Service (ALS) Implementation -Testing #656

elnyry-sam-k opened this issue Feb 26, 2019 · 1 comment
Assignees
Labels
Milestone

Comments

@elnyry-sam-k
Copy link
Member

elnyry-sam-k commented Feb 26, 2019

Goal:

As a DFSP
I would like to have an implementation that is fully tested that meets the minimum requirements for code coverage. I would also want integration tests to fully test the ALS integration with Oracles and central-ledger.

Tasks:

  • Refactor the ALS code-base according to the design
    • Unit tests
      • Oracle
      • Parties
      • participant
    • Code Coverage
    • Integration tests
  • Update all dependencies
  • Fix knex pooling issue (@rmothilal may have more info on this)
    - [ ] Update the helm charts for updated default.json config:
    - [ ] DISPLAY_ROUTES
    - [ ] DB_CONNECTION_POOL_MIN and DB_CONNECTION_POOL_MAX
    Helm config will be covered as a part of Refactor central-services-database to use config instead of connection string #1008 helm changes

Acceptance Criteria:

  • Unit tests [Now using JEST testing framework]
  • Code coverage [Coverage is by the built in coverage tool in JEST i.e. nyc]
  • Integration tests

Notes:
Please check code in quoting service to validate how to configure JEST in the package.json

Pull Requests:

Follow-up:

  • TBD

Dependencies:

Accountability:

@elnyry-sam-k elnyry-sam-k changed the title ALS Implementation for Generic OSS ML OSS Account Lookup Service (ALS) Implementation Feb 27, 2019
@elnyry-sam-k elnyry-sam-k changed the title ML OSS Account Lookup Service (ALS) Implementation ML OSS Account Lookup Service (ALS) Implementation - Testing May 14, 2019
@elnyry-sam-k elnyry-sam-k added this to the Sprint 6.3 milestone May 24, 2019
@mdebarros mdebarros changed the title ML OSS Account Lookup Service (ALS) Implementation - Testing ML OSS Account Lookup Service (ALS) Implementation - Documentation/Testing May 28, 2019
@mdebarros mdebarros changed the title ML OSS Account Lookup Service (ALS) Implementation - Documentation/Testing ML OSS Account Lookup Service (ALS) Implementation -Testing May 28, 2019
@HenkKodde HenkKodde removed their assignment Jun 4, 2019
@elnyry-sam-k elnyry-sam-k removed this from the Sprint 6.3 milestone Jun 10, 2019
@lewisdaly lewisdaly self-assigned this Sep 26, 2019
@elnyry-sam-k elnyry-sam-k added this to the Sprint 8.2 milestone Oct 3, 2019
@lewisdaly
Copy link
Contributor

I'm currently waiting for review on the major unit test PR before adding the integration test config.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants