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

Add tests and coverage information for the project #8

Closed
omaciel opened this issue Dec 8, 2015 · 7 comments
Closed

Add tests and coverage information for the project #8

omaciel opened this issue Dec 8, 2015 · 7 comments
Assignees
Milestone

Comments

@omaciel
Copy link
Member

omaciel commented Dec 8, 2015

We need to write tests for Betelgeuse.

@omaciel omaciel added this to the 0.0.1 milestone Dec 8, 2015
@elyezer
Copy link
Collaborator

elyezer commented Dec 10, 2015

👍

@elyezer elyezer self-assigned this Dec 10, 2015
@elyezer
Copy link
Collaborator

elyezer commented Dec 10, 2015

Testing betelgeuse on travis will be hard since pylarion is not public available. For more info check: https://travis-ci.org/SatelliteQE/betelgeuse/builds/96028707

@elyezer
Copy link
Collaborator

elyezer commented Dec 10, 2015

I was able to work around this issue by creating https://github.com/elyezer/fake_pylarion which is a fake package that offers all required imports.

@sthirugn
Copy link
Contributor

@elyezer how many imports/functions we talking? Can we use mock for testing?

@elyezer
Copy link
Collaborator

elyezer commented Dec 10, 2015

@sthirugn even though we will be using mock, we need to import the betelgeuse module in order to test its functions. There is no other way to handle that since we need to call the real betelgeuse functions.

@elyezer
Copy link
Collaborator

elyezer commented Dec 11, 2015

Even though the test coverage status is being reported on README I will close this when I get a decent coverage number.

@sthirugn
Copy link
Contributor

@elyezer ty for explanation.

elyezer added a commit to elyezer/betelgeuse that referenced this issue Apr 17, 2016
Add tests for every single function of Betelgeuse.

Closes SatelliteQE#8.
elyezer added a commit to elyezer/betelgeuse that referenced this issue Jun 27, 2016
Add tests for every single function of Betelgeuse.

Closes SatelliteQE#8.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants