litmus
litmus copied to clipboard
(chaos-exporter): Add Unit Test Cases for file chaos-exporter
UNIT TEST
What happened:
- Write the unit-test
What you expected to happen:
- Write the unit test cases for the chaos-exporter repo
Anything else we need to know?:
- Issue related to Repository: https://github.com/litmuschaos/chaos-exporter
- Reference:- https://github.com/litmuschaos/chaos-operator/blob/master/pkg/controller/chaosengine/chaosengine_controller_test.go
Does this file exist? I don't see it in chaos-exporter/pkg
@debojitkakoti It's moved to controller/scrape.go
UNIT TEST
What happened:
- Write the unit-test
What you expected to happen:
- Write the unit test cases for the chaos-exporter repo
Anything else we need to know?:
- Issue related to Repository: https://github.com/litmuschaos/chaos-exporter
- Reference:- https://github.com/litmuschaos/chaos-operator/blob/master/pkg/controller/chaosengine/chaosengine_controller_test.go
Hello @Sanjay1611 I was going through this issue can you tell me which packages need unit test coverage here or do I need to add unit tests in all packages that have no unit tests present in -https://github.com/litmuschaos/chaos-exporter
@ispeakc0de have we resolved this issue?
Exporter tests have been added by @Nageshbansal as part of this PR: https://github.com/litmuschaos/chaos-exporter/pull/138. More tests will be added as we go/based on changes made.