Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update tests for proseco p2 overlap change #209

Merged
merged 1 commit into from
Mar 29, 2024
Merged

Update tests for proseco p2 overlap change #209

merged 1 commit into from
Mar 29, 2024

Conversation

jeanconn
Copy link
Contributor

@jeanconn jeanconn commented Mar 28, 2024

Description

Update tests for proseco p2 overlap change (in sot/proseco#394)

Fixes the issue that environments with that updated overlap-p2 proseco version see this in the tests

(ska3-masters) jeanconn-fido> pytest 
================================================================================= test session starts =================================================================================
platform linux -- Python 3.11.8, pytest-8.0.2, pluggy-1.4.0
rootdir: /proj/sot/ska/jeanproj/git
configfile: pytest.ini
plugins: timeout-2.2.0, anyio-4.3.0
collected 103 items                                                                                                                                                                   

sparkles/tests/test_checks.py .......................................................................FF...                                                                      [ 73%]
sparkles/tests/test_find_er_catalog.py .....                                                                                                                                    [ 78%]
sparkles/tests/test_review.py ..................                                                                                                                                [ 96%]
sparkles/tests/test_yoshi.py F...                                                                                                                                               [100%]

====================================================================================== FAILURES
...

Interface impacts

Testing

Unit tests

  • Linux
(ska3-masters) jeanconn-fido> pytest 
================================================================================= test session starts =================================================================================
platform linux -- Python 3.11.8, pytest-8.0.2, pluggy-1.4.0
rootdir: /proj/sot/ska/jeanproj/git
configfile: pytest.ini
plugins: timeout-2.2.0, anyio-4.3.0
collected 103 items                                                                                                                                                                   

sparkles/tests/test_checks.py ............................................................................                                                                      [ 73%]
sparkles/tests/test_find_er_catalog.py .....                                                                                                                                    [ 78%]
sparkles/tests/test_review.py ..................                                                                                                                                [ 96%]
sparkles/tests/test_yoshi.py ....                                                                                                                                               [100%]

=========================================================================== 103 passed in 85.72s (0:01:25) ============================================================================
(ska3-masters) jeanconn-fido> git rev-parse HEAD
ceccd8e712cbad4dfffede231222efdcebafe8b3

Independent check of unit tests by [REVIEWER NAME]

  • [PLATFORM]:

Functional tests

No functional testing.

@@ -896,7 +896,7 @@ def test_check_guide_geometry(aca_review_table):
assert len(acar.messages) == 1
msg = acar.messages[0]
assert msg["category"] == "critical"
assert 'Guide indexes [4, 5, 6] clustered within 500" radius' in msg["text"]
assert 'Guide indexes [4, 7, 8] clustered within 500" radius' in msg["text"]
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I was not immediately anticipating that guide-related test would change, but these guide indexes are the overall catalog idx values, and acquisition-related changes fiddle with order of BOT stars in the list. I confirmed that these are the same 3 stars.

@jeanconn jeanconn requested a review from taldcroft March 28, 2024 20:32
@taldcroft taldcroft merged commit 4bae4ba into master Mar 29, 2024
1 of 2 checks passed
@taldcroft taldcroft deleted the p2-change branch March 29, 2024 12:33
@javierggt javierggt mentioned this pull request May 1, 2024
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