Skip to content

WLNP-12375 : Investment service/increase code coverage#585

Open
apawan wants to merge 5 commits intomasterfrom
investment-service/increase-code-coverage
Open

WLNP-12375 : Investment service/increase code coverage#585
apawan wants to merge 5 commits intomasterfrom
investment-service/increase-code-coverage

Conversation

@apawan
Copy link
Contributor

@apawan apawan commented Mar 12, 2026

Description

The goal is to increase the code coverage for the Streams/investment package to over 80%. This is necessary to ensure that the code is properly tested and verified.

Checklist

  • I made sure, I read CONTRIBUTING.md to put right branch prefix as per my need.
  • I made sure to update CHANGELOG.md.
  • I made sure to update Stream Wiki(only valid in case of new stream module or architecture changes).
  • My changes are adequately tested.
  • I made sure all the SonarCloud Quality Gate are passed.

Copy link
Collaborator

@Aboruhen Aboruhen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.
Fix the sonar issues, please. move comments into a method doc

@apawan apawan marked this pull request as ready for review March 16, 2026 12:50
@apawan apawan requested a review from a team as a code owner March 16, 2026 12:50
@apawan apawan changed the title Investment service/increase code coverage WLNP-12375 : Investment service/increase code coverage Mar 16, 2026
@sonarqubecloud
Copy link

@apawan apawan requested a review from Aboruhen March 16, 2026 13:39
Copy link
Collaborator

@Aboruhen Aboruhen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Collaborator

@Aboruhen Aboruhen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants