Test requirements not documented

[sebastic]

pandas is now required for test/test_utils.py but is not included in requirements-dev.txt like xarray is.

certifi is required for test/test_network.py but is only included in install_requires in setup.cfg.

[snowman2]

pandas is now required for test/test_utils.py but is not included in requirements-dev.txt like xarray is.

That sounds like a good idea to be more explicit.

certifi is required for test/test_network.py but is only included in install_requires in setup.cfg.

Where would be the most helpful location to put the requirements?

[jorisvandenbossche]

I would personally also recommend to use those optional test dependencies behind a pytest "skip if not installed" (you can use eg pytest.importorskip, to be read as "import or skip")

That helps to be able to run the basic tests even without having all optional dependencies installed

[sebastic]

Where would be the most helpful location to put the requirements?

tests_require or extras_require: {'test': []} in setup() and/or requirements-tests.txt are likely good options.

[snowman2]

@sebastic, what about certifi?

[sebastic]

As certifi is not only required for tests install_requires is probably sufficient.