136 Validating and testing new argument - Githubissues

lsst-uk / lsst-adler

Apache License 2.0

0 stars 0 forks source link

136 Validating and testing new argument #141

Closed astronomerritt closed 2 months ago

astronomerritt commented 2 months ago

Fixes #136 .

Added validation for the --sql_filename command-line argument that makes sure the filename exists.
Fixed a unit test that did not do what I thought it did.
Wrote a unit test to cover this.

Review Checklist for Source Code Changes

[x] Does pip install still work?
[x] Have you written a unit test for any new functions?
[x] Do all the units tests run successfully?
[x] Does adler run successfully on a test set of input files/databases?
[x] Have you used black on the files you have updated to confirm python programming style guide enforcement?