MrPowers / chispa

PySpark test helper methods with beautiful error messages
https://mrpowers.github.io/chispa/
MIT License
595 stars 65 forks source link

chore: add test coverage commands #112

Closed fpgmaas closed 2 months ago

fpgmaas commented 2 months ago

I do not fully understand the idea. Why do we need both html and xml? And why do we generate html by default. Why cannot we just use stdout? With a simple --cov=chispa it will print a nice and readable message with summary about coverage.

I would like to see the stdout message on a make test and html/xml reports should be generated by test-coverage or something like this.

Also, for example, I do not have an open command (xdg-open should be used instead). What do you think about removing open and generate only reports?

@SemyonSinchenko Sorry, I could have elaborated a bit more in the PR description. The idea is to have:

Hope this clarifies! I modified the commands and removed the open command for the HTML report.