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

@@ -49,7 +49,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

@@ -29,18 +29,26 @@ module('Adapter | crate', function (hooks) {
 
   test('findRecord requests do not include versions by default', async function (assert) {
     let _foo = this.db.crate.create({ name: 'foo' });
-    let version = this.db.version.create({ crate: _foo });
+    this.db.version.create({ crate: _foo, num: '0.0.1' });
+    this.db.version.create({ crate: _foo, num: '0.0.2' });
+    this.db.version.create({ crate: _foo, num: '0.0.3' });
+
+    let versions = this.db.version.getAll().reverse();
+    let default_version = versions.find(it => it.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(it => `${it.id}`),
+    );
   });
 });