mirage/keywords: Calculate crates_cnt property

Author: Turbo87

mirage/factories/keyword.js

@@ -6,6 +6,4 @@ export default Factory.extend({
   id() {
     return this.keyword;
   },
-
-  crates_cnt: 0,
 });

mirage/fixtures/keywords.js

@@ -1,84 +1,70 @@
 export default [
   {
-    crates_cnt: 38,
     created_at: '2014-11-23T06:47:40Z',
     id: 'network',
     keyword: 'network',
   },
   {
-    crates_cnt: 1,
     created_at: '2014-11-23T06:47:40Z',
     id: 'rust',
     keyword: 'rust',
   },
   {
-    crates_cnt: 1,
     created_at: '2014-11-23T06:47:40Z',
     id: 'plugin',
     keyword: 'plugin',
   },
   {
-    crates_cnt: 1,
     created_at: '2014-11-23T06:47:40Z',
     id: 'code-generation',
     keyword: 'code-generation',
   },
   {
-    crates_cnt: 1,
     created_at: '2014-11-23T06:47:40Z',
     id: 'python',
     keyword: 'python',
   },
   {
-    crates_cnt: 1,
     created_at: '2014-11-23T06:47:40Z',
     id: 'ruby',
     keyword: 'ruby',
   },
   {
-    crates_cnt: 1,
     created_at: '2014-11-23T06:47:40Z',
     id: 'shell',
     keyword: 'shell',
   },
   {
-    crates_cnt: 1,
     created_at: '2014-11-23T06:47:40Z',
     id: 'string',
     keyword: 'string',
   },
   {
-    crates_cnt: 1,
     created_at: '2014-11-23T06:47:40Z',
     id: 'case',
     keyword: 'case',
   },
   {
-    crates_cnt: 1,
     created_at: '2014-11-23T06:47:40Z',
     id: 'camel',
     keyword: 'camel',
   },
   {
-    crates_cnt: 1,
     created_at: '2014-11-23T06:47:40Z',
     id: 'snake',
     keyword: 'snake',
   },
   {
-    crates_cnt: 1,
     created_at: '2014-11-23T06:47:40Z',
     id: 'inflection',
     keyword: 'inflection',
   },
   {
-    crates_cnt: 1,
     created_at: '2014-11-23T06:47:40Z',
     id: 'elastic',
     keyword: 'elastic',
   },
   {
-    crates_cnt: 1,
     created_at: '2014-11-23T06:47:40Z',
     id: 'elasticsearch',
     keyword: 'elasticsearch',

mirage/serializers/keyword.js

@@ -0,0 +1,24 @@
+import BaseSerializer from './application';
+
+export default BaseSerializer.extend({
+  getHashForResource() {
+    let [hash, addToIncludes] = BaseSerializer.prototype.getHashForResource.apply(this, arguments);
+
+    if (Array.isArray(hash)) {
+      for (let resource of hash) {
+        this._adjust(resource);
+      }
+    } else {
+      this._adjust(hash);
+    }
+
+    return [hash, addToIncludes];
+  },
+
+  _adjust(hash) {
+    let allCrates = this.schema.crates.all();
+    let associatedCrates = allCrates.filter(it => it.keywordIds.includes(hash.id));
+
+    hash.crates_cnt = associatedCrates.length;
+  },
+});

tests/mirage/crates-test.js

@@ -314,7 +314,7 @@ module('Mirage | Keywords', function(hooks) {
       assert.deepEqual(responsePayload.crate.keywords, ['no-std']);
       assert.deepEqual(responsePayload.keywords, [
         {
-          crates_cnt: 0,
+          crates_cnt: 1,
           id: 'no-std',
           keyword: 'no-std',
         },

tests/mirage/keywords-test.js

@@ -107,5 +107,24 @@ module('Mirage | Keywords', function(hooks) {
         },
       });
     });
+
+    test('calculates `crates_cnt` correctly', async function(assert) {
+      this.server.create('keyword', { keyword: 'cli' });
+      this.server.createList('crate', 7, { keywordIds: ['cli'] });
+      this.server.create('keyword', { keyword: 'not-cli' });
+      this.server.createList('crate', 3, { keywordIds: ['not-cli'] });
+
+      let response = await fetch('/api/v1/keywords/cli');
+      assert.equal(response.status, 200);
+
+      let responsePayload = await response.json();
+      assert.deepEqual(responsePayload, {
+        keyword: {
+          id: 'cli',
+          crates_cnt: 7,
+          keyword: 'cli',
+        },
+      });
+    });
   });
 });