broadinstitute / agora / 892

89%

develop: 88%

push

travis-ci

Small bug fix to configuration validation

Rawls changed the configuration payload field "methodStoreMethod" to "methodRepoMethod"
We check for the existence of this field during validation, and we were checking by the old name.

Fixed the field name in the configuration validation
Added better failure descriptions to our require statements
Added a validation exception so that we can avoid catching IllegalArgumentException at perRequest layer
Changed perRequest to respond with BadRequest on validation errors, not IllegalArgumentException
Changed methodStoreMethod -> methodRepoMethod in swagger, static test data

14 of 14 new or added lines in 4 files covered. (100.0%)

786 of 888 relevant lines covered (88.51%)

0.89 hits per line