Ouranosinc / xclim / 15471792852

92%

push

github

Allow pooch to be mocked (#2188)

### What kind of change does this PR introduce?

* Catches a really specific corner case that can affect downstream
projects that rely on mocking of `pooch`.

### Does this PR introduce a breaking change?

No.

### Other information:

OK, this is wild. Let's say we want to install a very minimal amount of
dependencies in order to reduce the memory load on ReadTheDocs; We can
very safely mock most libraries and not need to worry unless one of the
libraries expects to raise a specific error in the normal operation.

If we mock `pooch` here, then the `pooch` call resolves but doesn't
provide the expected data structure needed for `pathlib.Path` and
instead of raising an `AttributeError` (`None` has no attributes), it
raises instead a `TypeError` (the mocked `pooch.os_cache._type` is not a
parsable string or Path).

Relevant Pull Request (`miranda`):
https://github.com/Ouranosinc/miranda/pull/241

0 of 1 new or added line in 1 file covered. (0.0%)

7600 of 8232 relevant lines covered (92.32%)

8.09 hits per line