sot / kadi

Chandra commands and events
https://sot.github.io/kadi
BSD 3-Clause "New" or "Revised" License
5 stars 3 forks source link

Just test commands version 2 #327

Closed jeanconn closed 2 months ago

jeanconn commented 3 months ago

Description

Just test commands version 2.

I started to strip out the rest of the machinery (that is in the testing infrastructure to be able to test v1 or v2), and then figured for a first go maybe it makes sense to leave it in there and just do this super-tiny change to just stop running the v1 tests.

Interface impacts

Testing

Unit tests

kadi/commands/tests/test_commands.py ............................................................................... [ 44%] kadi/commands/tests/test_states.py .......................x....................... [ 70%] kadi/commands/tests/test_validate.py .................... [ 82%] kadi/tests/test_events.py .......... [ 87%] kadi/tests/test_occweb.py ...................... [100%]

================================================================================ warnings summary ================================================================================ kadi/kadi/commands/tests/test_commands.py::test_get_starcats_each_year[year0] /proj/sot/ska3/flight/lib/python3.11/site-packages/setuptools_scm/git.py:308: UserWarning: git archive did not support describe output warnings.warn("git archive did not support describe output")

-- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html ============================================================= 177 passed, 1 xfailed, 1 warning in 172.71s (0:02:52)



Independent check of unit tests by [REVIEWER NAME]
- [ ] [PLATFORM]:

### Functional tests
<!-- Describe and document results of any functional tests, otherwise leave the text below -->
No functional testing.
jeanconn commented 3 months ago

Though I suppose we still want to just skip these tests if no internet, so I'll add that back.

jeanconn commented 2 months ago

Superseded by #328