-
Notifications
You must be signed in to change notification settings - Fork 9
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
✨Feature Request - support for generating coverage reports #17
Comments
Hi, What do you have in mind exactly? |
Sure that's a good point, and I'm not sure I have a lot of specifics in mind. But for example on a given run, skipping a task would probably be the equivalent of a line not being tested. Then ideally the combination of scenarios we test does end up covering all paths. I suppose conditionals get a bit complex. If a task has a conditional, we want to test it with both
I don't think we'd need to dig into the particulars of complicated conditionals ( |
And so Monkeyble already answer to that. You can test both scenario with |
Right exactly, the idea is to generate some standard coverage report that we use in coverage tools like https://codecov.io , that makes it easier to see if we missed some pathways or something and want to test more scenarios |
The problem is that Monkeyble works as a callback plugin. It's not aware of a side executions. |
That makes sense, I'm not suggesting that a single report needs to be created that takes into account all of the other scenarios, rather each run would only have to be concerned with what that run covered, multiple scenarios results in multiple reports, and the coverage tool takes all of those into account. I think that's how it usually works with other tools, though I am a bit out of my depth on the particulars. |
I can relate to the limitations of running within a callback interface, even though what we can do is sometimes surprising :p @briantist it's probably not what you are looking for but once playbook task results are recorded in ara the data is available to query over its API. If the data you need is in the results, you could find them and use them that way.
|
It would be really cool if we cool if we could generate coverage reports with monkeyble so we could see coverage in our CI/CD
The text was updated successfully, but these errors were encountered: