CCI-MOC / hil

Hardware Isolation Layer, formerly Hardware as a Service
Apache License 2.0
24 stars 54 forks source link

Remove pytest-catchlog from dependences. #961

Closed zenhack closed 6 years ago

zenhack commented 6 years ago

We've been getting warnings about it having been merged into pytest core, e.g:

https://travis-ci.org/CCI-MOC/hil/jobs/341455006#L2725

This also bumps the minimum version for pytest in our setup.py, since the change was made in 3.3.0:

naved001 commented 6 years ago

I tried doing this in #927 and when I removed pytest-catchlog from the requirements, it suddenly couldn't import it. So don't know what's up with that.

From my description there:

One message that is still annoying is this:

=============================================================================================== warnings summary ===============================================================================================
None
  pytest-catchlog plugin has been merged into the core, please remove it from your requirements.

-- Docs: http://doc.pytest.org/en/latest/warnings.html

I removed pytest-catchlog from the requirements, but then I couldn't import pytest_catchlog. Their docs don't say anything about it being merged into the core, it still tells you to install it explicitly. Don't know what's up with that.

zenhack commented 6 years ago

@naved001, found the problem, see the last commit. D'oh.