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

(chaos-exporter): Add Unit Test Cases for file chaos-exporter #781

Closed
Sanjay1611 opened this issue Sep 30, 2019 · 5 comments
Closed

(chaos-exporter): Add Unit Test Cases for file chaos-exporter #781

Sanjay1611 opened this issue Sep 30, 2019 · 5 comments

Comments

@Sanjay1611
Copy link
Contributor

Sanjay1611 commented Sep 30, 2019

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?:

@chandankumar4 chandankumar4 changed the title Add Unit Test Cases (chaos-exporter): Add Unit Test Cases for file pkg/chaosmetrics/scrapecr.go Sep 30, 2019
@dedifferentiator
Copy link

Does this file exist? I don't see it in chaos-exporter/pkg

@chandankumar4
Copy link
Member

@debojitkakoti It's moved to controller/scrape.go

@ispeakc0de ispeakc0de changed the title (chaos-exporter): Add Unit Test Cases for file pkg/chaosmetrics/scrapecr.go (chaos-exporter): Add Unit Test Cases for file chaos-exporter Sep 30, 2021
@jignyasamishra
Copy link

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?:

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

@prithvi1307
Copy link
Contributor

@ispeakc0de have we resolved this issue?

@ksatchit
Copy link
Member

ksatchit commented Mar 7, 2024

Exporter tests have been added by @Nageshbansal as part of this PR: litmuschaos/chaos-exporter#138. More tests will be added as we go/based on changes made.

@ksatchit ksatchit closed this as completed Mar 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

6 participants