Fix non-canonical crate name pages

Before the Ember Data 5 update it was possible to visit `/crates/foo-bar` when the crate was actually called `foo_bar` and vice-versa. The Ember Data 5 update broke this behavior, and this commit is restoring it, by using `queryRecord()` instead of `findRecord()`, which can no longer deal with requesting a certain `id` and getting a different one returned from the server.

Author: Turbo87

app/adapters/crate.js

@@ -14,6 +14,26 @@ export default class CrateAdapter extends ApplicationAdapter {
     return super.findRecord(store, type, id, snapshot);
   }
 
+  queryRecord(store, type, query, adapterOptions) {
+    let { include } = query;
+    // This ensures `crate.versions` are always fetched from another request.
+    if (include === undefined) {
+      query.include = 'keywords,categories,downloads';
+    }
+    return super.queryRecord(store, type, query, adapterOptions);
+  }
+
+  urlForQueryRecord(query) {
+    let baseUrl = super.urlForQueryRecord(...arguments);
+    if (!query.name) {
+      return baseUrl;
+    }
+
+    let crateName = query.name;
+    delete query.name;
+    return `${baseUrl}/${crateName}`;
+  }
+
   groupRecordsForFindMany(store, snapshots) {
     let result = [];
     for (let i = 0; i < snapshots.length; i += BULK_REQUEST_GROUP_SIZE) {

app/routes/crate.js

@@ -12,7 +12,7 @@ export default class CrateRoute extends Route {
     let crateName = params.crate_id;
 
     try {
-      return await this.store.findRecord('crate', crateName);
+      return await this.store.queryRecord('crate', { name: crateName });
     } catch (error) {
       if (error instanceof NotFoundError) {
         let title = `${crateName}: Crate not found`;

tests/acceptance/crate-test.js

@@ -131,6 +131,19 @@ module('Acceptance | crate page', function (hooks) {
     assert.dom('[data-test-try-again]').exists();
   });
 
+  test('treats dash and underscore as equal', async function (assert) {
+    let crate = this.server.create('crate', { name: 'foo-bar' });
+    this.server.create('version', { crate });
+
+    await visit('/crates/foo_bar');
+
+    assert.strictEqual(currentURL(), '/crates/foo_bar');
+    assert.strictEqual(currentRouteName(), 'crate.index');
+    assert.strictEqual(getPageTitle(), 'foo-bar - crates.io: Rust Package Registry');
+
+    assert.dom('[data-test-heading] [data-test-crate-name]').hasText('foo-bar');
+  });
+
   test('navigating to the all versions page', async function (assert) {
     this.server.loadFixtures();