Split extra libraries and add more granular control over tests#699
Split extra libraries and add more granular control over tests#699Zeitsperre wants to merge 7 commits intomainfrom
Conversation
… parallel coverage reporting
|
@cehbrecht I'm not certain why the coverage status is taking such a huge hit. I did remove a few useless imports, so it could be that the original values were too high? |
|
@Zeitsperre |
I have split the builds with There were a few tests that I disabled/skipped because there were no assertions made with the results: smoke tests, essentially. It's likely from those changes that the coverage is being impacted. In any case, I think the coverage score is fine. |
Overview
This Pull Request splits out the libraries that support some complex validators (geotiff, netcdf4, GDAL/OGR, zarr) so that basic data types are supported by default.
It also modifies some of the test controls so that
pytestmarkers can be used to explicitly select/deselect tests that rely on these extra libraries.Finally, coverage is now done across all builds in parallel, using
tox-ghand thecoverallsapp/github-actionto remove thecoveralls-pythonlibrary.Related Issue / Discussion
Changes here build on discussions found in #698
Closes #698
Additional Information
There are many more changes I'd like to implement, but I'll limit things here for now until discussion is had in #700.
Contribution Agreement
(as per https://github.com/geopython/pywps/blob/master/CONTRIBUTING.rst#contributions-and-licensing)