Add a test to ensure that the default configs are valid.

allenai / AMPT

Aquatic Mammal Photogrammetry Tool

Apache License 2.0

3 stars 0 forks source link

Closed APatrickJ closed 3 years ago

APatrickJ commented 3 years ago

This allows us to ensure that we are shipping valid configs with any new release of the AMPT plugin.

Refactor the code a bit so that it's easy to read the location of the default config directory.

Fixes #41

APatrickJ commented 3 years ago

Discovered an issue reading the default configs in Zip format. Going to fix that then re-mark as ready for review once fixed.

APatrickJ commented 3 years ago

Discovered an issue reading the default configs in Zip format. Going to fix that then re-mark as ready for review once fixed.

I think an email was probably sent when I marked this as ready to review, but just in case, it's ready for review now!

I've got a followup PR to this that will fix the startup regression with an empty config directory.