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 17 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. |
Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
Signed-off-by: Trevor James Smith <10819524+Zeitsperre@users.noreply.github.com>
|
@fmigneault I figured out why the coverage was much higher: Previously, the values were reflective of both the source code ( The value currently is also impacted by the removal of a handful of smoke tests, like I mentioned, but on the order of ~1%. Getting that back would require request mocking and other tricks. Are you familiar with those approaches? |
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)