kammerje / spaceKLIP

Pipeline for reducing JWST high-contrast imaging data. Published in Kammerer et al. 2022 and Carter et al. 2022.
https://ui.adsabs.harvard.edu/abs/2022SPIE12180E..3NK/abstract
MIT License
16 stars 10 forks source link

add initial test functions using pytest #73

Closed mperrin closed 1 year ago

mperrin commented 1 year ago

Replacement of PR #71, recreated cleanly after the 1.0/2.0 swap

kammerje commented 1 year ago

Shall I merge this? All checks are currently failing for me.

mperrin commented 1 year ago

No, we shouldn’t merge until it’s working here. More debugging to do next week!

mperrin commented 1 year ago

I’m not practiced enough with CI setup, every time I do this in a different package it seems to inevitably take a moderate amount of messing around with little syntax and config tweaks to get things working on GitHub actions… 😂🥲😅

mperrin commented 1 year ago

Tests passing in CI now, both for building the docs and for running the (small, limited) set of tests.

This PR also relies on a ZIP file of some example test data (a few files from the ERS 1386 program) which are used the test suite. This zip file is retrieved from a Box folder and installed on the Github actions CI server as part of the CI workflow.

This file is owned/maintained by me, in Box, and I can coordinate updating this as needed for future CI testing when/if we want to include more comprehensive testing here.