Skip to content

Update unit tests expected answers for new angle calcs#457

Merged
jeanconn merged 1 commit intomasterfrom
angle-tests
Aug 27, 2025
Merged

Update unit tests expected answers for new angle calcs#457
jeanconn merged 1 commit intomasterfrom
angle-tests

Conversation

@jeanconn
Copy link
Copy Markdown
Contributor

Description

Update unit tests expected answers for new angle calcs

Fixes error I introduced in #455 when I forgot to rerun the unit tests after subtracting off the 10 seconds of NMM hold time.

Interface impacts

Testing

Unit tests

  • Mac
(ska3-latest) flame:starcheck jean$ git rev-parse HEAD
feb8073d87251d13fe2fb3d260fcd9180484893b
(ska3-latest) flame:starcheck jean$ pytest
====================================================================== test session starts =======================================================================
platform darwin -- Python 3.12.8, pytest-8.3.4, pluggy-1.5.0
rootdir: /Users/jean/git
configfile: pytest.ini
plugins: socket-0.7.0, anyio-4.7.0, timeout-2.3.1
collected 14 items                                                                                                                                               

starcheck/tests/test_state_checks.py .............                                                                                                         [ 92%]
starcheck/tests/test_utils.py .                                                                                                                            [100%]

======================================================================= 14 passed in 3.49s

Independent check of unit tests by [REVIEWER NAME]

  • [PLATFORM]:

Functional tests

No functional testing.

@jeanconn jeanconn merged commit c92a0bd into master Aug 27, 2025
2 checks passed
@jeanconn jeanconn deleted the angle-tests branch August 27, 2025 19:56
@javierggt javierggt mentioned this pull request Sep 22, 2025
@javierggt javierggt mentioned this pull request Oct 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants