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

[UNIT TESTING]: Consolidated branch for unit test coverage of across the observability plugin. #112

Open
DipraAich opened this issue Nov 23, 2022 · 1 comment
Labels
enhancement New feature or request

Comments

@DipraAich
Copy link

DipraAich commented Nov 23, 2022

Is your feature request related to a problem?
Insufficient code coverage for unit tests.

What solution would you like?
Write unit test cases and cover the uncovered code.

Do you have any additional context?
Following will be the coverage report:
index.zip

@DipraAich DipraAich added the enhancement New feature or request label Nov 23, 2022
@DipraAich
Copy link
Author

Hi @anirudha & @mengweieric ,

We have covered approximately 70% from 32% coverage of the UTs. Following is the report. Most of them are now in green or yellow state. Please take a look. We have raised a PR against main for the same. This percentage excludes query manager files and trace analytics. For query manager Eric has written test cases and we have excluded trace analytics because we have no clue about what is happening in there.

File:
index.zip

Thank You,

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

No branches or pull requests

1 participant