ExposuresProvider / cam-pipeline

Data loading pipeline for CAM database
https://exposuresprovider.github.io/cam-pipeline/
MIT License
2 stars 4 forks source link

Add Python tests for Automat-CAM-KP #111

Closed gaurav closed 4 months ago

gaurav commented 8 months ago

This PR adds a test suite in Python for CAM-KP-API to cam-pipeline. Some of this code has been moved over from https://github.com/ExposuresProvider/cam-kp-api and the rest has been newly written for this.

There are three tests here:

I tried to move over the integration tests from CAM-KP-API, but I couldn't work out the easiest way to figure out how to move them over as a Scala project. I think these Python tests are easier to read and maintain, but I'm happy to be proved wrong.

Closes #94

Missing tests that will need to be handled separately (#123):

gaurav commented 5 months ago

Some of the tests need to be updated for the new release of Automat-CAM-KP that happened in December. I'll update these tests and resubmit them for review ASAP.

gaurav commented 5 months ago

Tests updated, ready for review again! There are some failing tests in tests/test_curie.py, but they've all been documented in https://github.com/ExposuresProvider/cam-pipeline/issues/101#issuecomment-1793934634 and will be dealt with there.