app: Include default_version by default

This commit would ensure that `default_version` is included in the
`crate` response by default. This eliminates the need to wait for the
`versions` request to complete when `default_version` is the only
version required.

Author: eth3lbert

app/adapters/crate.js

@@ -37,7 +37,7 @@ export default class CrateAdapter extends ApplicationAdapter {
 function setDefaultInclude(query) {
   if (query.include === undefined) {
     // This ensures `crate.versions` are always fetched from another request.
-    query.include = 'keywords,categories,downloads';
+    query.include = 'keywords,categories,downloads,default_version';
   }
 
   return query;

tests/adapters/crate-test.js

@@ -23,20 +23,28 @@ module('Adapter | crate', function (hooks) {
     assert.strictEqual(bar?.name, 'bar');
   });
 
-  test('findRecord requests do not include versions by default', async function (assert) {
-    let _foo = this.server.create('crate', { name: 'foo' });
-    let version = this.server.create('version', { crate: _foo });
+  test('findRecord requests only include `default_version` by default', async function (assert) {
+    let crate = this.server.create('crate', { name: 'foo' });
+    let versions = [
+      this.server.create('version', { crate, num: '0.0.1' }),
+      this.server.create('version', { crate, num: '0.0.2' }),
+      this.server.create('version', { crate, num: '0.0.3' }),
+    ];
+    let default_version = versions.find(v => v.num === '0.0.3');
 
     let store = this.owner.lookup('service:store');
 
     let foo = await store.findRecord('crate', 'foo');
     assert.strictEqual(foo?.name, 'foo');
 
-    // versions should not be loaded yet
+    // Only `defaul_version` should be loaded
     let versionsRef = foo.hasMany('versions');
-    assert.deepEqual(versionsRef.ids(), []);
+    assert.deepEqual(versionsRef.ids(), [default_version.id]);
 
     await versionsRef.load();
-    assert.deepEqual(versionsRef.ids(), [version.id]);
+    assert.deepEqual(
+      versionsRef.ids(),
+      versions.map(v => v.id),
+    );
   });
 });