loris-imageserver / loris

Loris IIIF Image Server
Other
209 stars 87 forks source link

Tidy up logging config and tests #374

Closed alexwlchan closed 7 years ago

alexwlchan commented 7 years ago

I noticed that the log_to = 'file' arm of configure_logging in webapp.py wasn't registering as covered in the test coverage. Weird… thought I’d added a test for it?

Adding some extra assertions in config_t.py exposes the error – the config is preserved between different tests. It should be reset each time, so we can check Loris processes the config correctly.