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

C unit tests should be incorporated into the CI #49

Open
jdavies-st opened this issue Dec 31, 2019 · 1 comment
Open

C unit tests should be incorporated into the CI #49

jdavies-st opened this issue Dec 31, 2019 · 1 comment

Comments

@jdavies-st
Copy link
Collaborator

There are C unit tests here

https://github.com/spacetelescope/drizzle/blob/master/src/tests/utest_cdrizzle.c

which use a vendorized framework fctx

https://github.com/imb/fctx

that is no longer maintained, plus some pandokia stuff. @jhunkeler any knowledge on either of these dependencies?

We should figure out if (and how) these can be compiled and get them into the CI, if they cannot be compiled, then duplicate these tests in python unit tests.

@jdavies-st
Copy link
Collaborator Author

And it goes without saying that we want to get rid of the pandokia-related code in the above tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant