spedas / pyspedas

Python-based Space Physics Environment Data Analysis Software
https://pyspedas.readthedocs.io/
MIT License
149 stars 58 forks source link

SECS/EICS tests need more assertions #895

Closed jameswilburlewis closed 3 months ago

jameswilburlewis commented 3 months ago

Many of the SECS/EICS tests pass, despite the server not returning any data. For each test, we should check that the expected tplot variables are present in the list returned by the load routine, and that the variables actually exist.

jameswilburlewis commented 3 months ago

The SECS/EICS routines return dictionaries rather than producing tplot variables. So I've added assertions that the returned dictionaries are not empty.