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

Increase test coverage for jeeves #202

Open
ciecierski opened this issue Mar 29, 2021 · 3 comments
Open

Increase test coverage for jeeves #202

ciecierski opened this issue Mar 29, 2021 · 3 comments
Labels
discussion Further discussion is requested test Pertains to test suite

Comments

@ciecierski
Copy link
Collaborator

I would like to start a discussion on how we can add more tests to jeeves. There are already some simple tests implemented in tests folder, however they don't cover much of the code base. Tests can be helpful both for developer and reviewer. (for an example as a reviewer I would like to see the resulting report page in the CI and check how it differ versus before change).
My first idea is to look how tests are implemented in python-jenkins: https://opendev.org/jjb/python-jenkins/src/branch/master/tests and re-use some of them and add more jeeves specific tests. Any idea is more than welcome.

@ciecierski ciecierski added the discussion Further discussion is requested label Mar 29, 2021
@nathan-weinberg
Copy link
Owner

Very much in favor of this!

@nathan-weinberg nathan-weinberg added the test Pertains to test suite label Mar 29, 2021
@damani42
Copy link
Contributor

damani42 commented Feb 2, 2022

Yes it will be very great.

@EliadCohen
Copy link
Collaborator

+1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion Further discussion is requested test Pertains to test suite
Projects
None yet
Development

No branches or pull requests

4 participants