mysociety / fixmystreet

This is mySociety's popular map-based reporting platform: easy to install in new countries and regions
http://fixmystreet.org/
Other
501 stars 232 forks source link

Add order_by helper / fix BST tests #5004

Closed dracos closed 3 weeks ago

dracos commented 3 weeks ago

Lets you switch any use of search(undef, { order_by => 'title' }) to order_by('title') or search(undef, { order_by => { -desc => 'id' } }) to order_by('-id').

Also fix some test failures when you have a BST database rather than UTC. [skip changelog]

codecov[bot] commented 3 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 82.60%. Comparing base (72ea05a) to head (5c8a7bd). Report is 3 commits behind head on master.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #5004 +/- ## ======================================= Coverage 82.60% 82.60% ======================================= Files 393 395 +2 Lines 30766 30775 +9 Branches 4880 4881 +1 ======================================= + Hits 25414 25423 +9 Misses 3901 3901 Partials 1451 1451 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.