Closed fmigneault closed 1 year ago
Merging #543 (a6befbe) into master (fd1e68e) will decrease coverage by
0.01%
. The diff coverage is84.21%
.:exclamation: Current head a6befbe differs from pull request most recent head f0d9aff. Consider uploading reports for the commit f0d9aff to get more accurate results
@@ Coverage Diff @@
## master #543 +/- ##
==========================================
- Coverage 84.39% 84.38% -0.01%
==========================================
Files 79 79
Lines 17596 17601 +5
Branches 2670 2670
==========================================
+ Hits 14850 14853 +3
- Misses 2017 2019 +2
Partials 729 729
Impacted Files | Coverage Δ | |
---|---|---|
weaver/database/mongodb.py | 89.53% <ø> (ø) |
|
weaver/formats.py | 88.44% <ø> (ø) |
|
weaver/xml_util.py | 100.00% <ø> (ø) |
|
weaver/utils.py | 82.86% <71.42%> (-0.08%) |
:arrow_down: |
weaver/owsexceptions.py | 75.14% <75.00%> (ø) |
|
weaver/datatype.py | 78.81% <100.00%> (ø) |
|
weaver/processes/utils.py | 79.70% <100.00%> (ø) |
|
weaver/processes/wps_process_base.py | 84.51% <100.00%> (ø) |
|
weaver/wps/service.py | 84.44% <100.00%> (ø) |
|
weaver/wps_restapi/colander_extras.py | 82.53% <100.00%> (ø) |
Docker build issue identified in https://github.com/crim-ca/weaver/actions/runs/5476798191/jobs/9974868828?pr=542
Fixes
Since the CI fails with either combination of those fixes individually (docker build fail or lint checks fail), combine them all in one.
weaver-worker
image due to unresolveddocker-ce-cli
package. Installation is updated according to the reference documentation (https://docs.docker.com/engine/install/debian/).bytes
instead ofstr
) for some handlers inopen_module_resource_file
.jsonschema.validators.RefResolver
reference injsonschema>=4.18.0
caused by refactor (see https://github.com/python-jsonschema/jsonschema/blob/main/CHANGELOG.rst#v4180 and https://github.com/python-jsonschema/jsonschema/pull/1049).