controllers/crates: default to sort by recent downloads

[punkeel]

"Browse All Crates" is the most prominent link in the menu bar, displayed on all pages. Before this change, this link would present the user with (mostly) irrelevant crates, as it was sorted alphabetically: crates like a, a-, a0, etc.

With this change, the list is now sorted to present the most downloaded crates first.

/keywords/:term already defaults to that same order.

[Turbo87]

@punkeel it looks like a couple of tests are failing though. could you take a look?

[eth3lbert]

Digging into the failure a bit further, to make the test pass, you need to:

1. Make the response reflect the sorting method: Add support for sorting by recent-downloads in the mirage/route-handlers/crates.js file.
2. Update the corresponding assert value: Since the default sorting method has now changed to recent-downloads, we need to update crates sort, All-Time and Recent values in the tests/acceptance/crates-test.js file accordingly.

[codecov[bot]]

Codecov Report

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

Project coverage is 89.06%. Comparing base (b6ddda3) to head (39a2ff0).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #7889      +/-   ##
==========================================
+ Coverage   89.04%   89.06%   +0.02%     
==========================================
  Files         278      278              
  Lines       27987    27987              
==========================================
+ Hits        24922    24928       +6     
+ Misses       3065     3059       -6     

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