casangi / xradio

Xarray Radio Astronomy Data IO
Other
9 stars 5 forks source link

Add unit tests for vis/_vis_utils #176

Closed FedeMPouzols closed 1 month ago

FedeMPouzols commented 1 month ago

This adds unit tests for vis/_vis_utils. So far in tests/unit/vis we only had a few unit tests for optimised_functions.py This PR adds a set of unit tests (230+) to cover the code under src/xradio/vis/_vis_utils. It adds a conftest with pytest fixtures, and an on-the-fly test MS geneator used in these tests. It also includes a few fixes for corner cases in the _tables read/write functions.

All pytests tests runtime before: ~55s, after: 1m25s Coverage (all repo) before: 65%, after: 87%


Note well:

This contribution is made under the current ALMA software agreements.
(c) European Southern Observatory, 2024
Copyright by ESO (in the framework of the ALMA collaboration)
CLAassistant commented 1 month ago

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.