locomotivemtl / charcoal-cms / 367

21%

push

travis-ci

Improve dynamic route handlers

Improved logic of route handlers to return 404 responses where the model or object route is missing, 500 responses when the model is invalid (such as incomplete templating details).

Added:
- Logs (debug/warning) to various route handlers to better track when issues arise
- Checks to validate rendered template view is not empty for matches the model's template identifier
- Catch to `*Route::load*FromPath()` methods to prevent nonexistent models from breaking the runtime
- Validation/Assertion methods to `GenericRoute` to determine if the context object or current object route are valid

Changed:
- Replaced 404 status codes with 500 when the failure does not stem from a missing model
- Refactored unit tests for route handlers

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

779 of 3793 relevant lines covered (20.54%)

5.34 hits per line