Unit tests with GitHub Actions failing

[sapols]

We're going to attempt to automate running all core package unit tests with GitHub Actions. Ideally with multiple Python versions and OSs. The thinking being that if the unit tests pass, the latest versions of the core packages must all be compatible together (double check that reasoning to ensure we're thinking about this correctly...).

The tests currently fail. Fix them.

[sapols]

Note PlasmaPy removed its requirements.txt file so that part will have to be rewritten to install dependencies from their pyproject.toml file. Could be simple using poetry? pip install poetry poetry install

[sapols]

Test Summary for Core PyHC Packages

Package	Passing?	Notes
hapiclient	Yes ✅	Passes with all Python versions ✅
kamodo	No ❌	No unit tests exist ❌
plasmapy	Yes ✅	Ignoring 6 files that error and 18 failing tests
pysat	No ❌	Commented out due to "error in Sphinx setup command: use_2to3 is invalid."
pyspedas	Yes ✅	Ignoring 6 tests
spacepy	Yes ✅	Tests from 4 files fail in test_all.py; manually running all test files but those 4 instead
sunpy	No ❌	Commented out due to "pytest: error: unrecognized arguments: --doctest-rst --dist --arraydiff"

[sapols]

pysat made a release candidate with tests packaged along with the code.

Hi Shawn,

We’ve got a new pysat release candidate up at https://test.pypi.org/project/pysat/#history

The unit tests should be packaged along with the code if you want to use this as a test case for phyc-core. (We import standard tests throughout the ecosystem, so I am not sure if this is configured correctly for testing via pypi install.) Let me know if you run into issues or if there is a better way to package the tests.

Cheers,
-Jeff

[jtniehof]

@sapols which four fail? Probably astropy related stuff? I plan to have a spacepy[all] or spacepy[tests] to make it easier to bring in astropy and pandas if you don't need it.

(Is there consistency on whether tests_require should be replaced by [test] or [tests] or [testing]?)

[jtniehof]

Can you set the tests up to require in parallel instead of in serial? Right now hapiclient is blocking the runs for everybody else.