mirage/crates: Replace _owner_teams|users with crate-ownership model

Author: Turbo87

mirage/route-handlers/crates.js

@@ -18,12 +18,12 @@ export function register(server) {
 
     if (request.queryParams.user_id) {
       let userId = parseInt(request.queryParams.user_id, 10);
-      crates = crates.filter(crate => (crate._owner_users || []).indexOf(userId) !== -1);
+      crates = crates.filter(crate => schema.crateOwnerships.findBy({ crateId: crate.id, userId }));
     }
 
     if (request.queryParams.team_id) {
       let teamId = parseInt(request.queryParams.team_id, 10);
-      crates = crates.filter(crate => (crate._owner_teams || []).indexOf(teamId) !== -1);
+      crates = crates.filter(crate => schema.crateOwnerships.findBy({ crateId: crate.id, teamId }));
     }
 
     if (request.queryParams.sort === 'alpha') {

tests/acceptance/dashboard-test.js

@@ -33,7 +33,8 @@ module('Acceptance | Dashboard', function (hooks) {
     }
 
     {
-      let crate = this.server.create('crate', { name: 'nanomsg', _owner_users: [42] });
+      let crate = this.server.create('crate', { name: 'nanomsg' });
+      this.server.create('crate-ownership', { crate, user });
       this.server.create('version', { crate, num: '0.1.0' });
     }
 

tests/mirage/crates-test.js

@@ -134,31 +134,19 @@ module('Mirage | Crates', function (hooks) {
     });
 
     test('supports a `user_id` parameter', async function (assert) {
-      this.server.create('crate', { name: 'foo' });
-      this.server.create('version', { crateId: 'foo' });
-      this.server.create('crate', { name: 'bar', _owner_users: [42] });
-      this.server.create('version', { crateId: 'bar' });
-      this.server.create('crate', { name: 'baz', _owner_users: [13] });
-      this.server.create('version', { crateId: 'baz' });
-
-      let response = await fetch('/api/v1/crates?user_id=42');
-      assert.equal(response.status, 200);
-
-      let responsePayload = await response.json();
-      assert.equal(responsePayload.crates.length, 1);
-      assert.equal(responsePayload.crates[0].id, 'bar');
-      assert.equal(responsePayload.meta.total, 1);
-    });
+      let user1 = this.server.create('user');
+      let user2 = this.server.create('user');
 
-    test('supports a `team_id` parameter', async function (assert) {
       this.server.create('crate', { name: 'foo' });
       this.server.create('version', { crateId: 'foo' });
-      this.server.create('crate', { name: 'bar', _owner_teams: [42] });
+      let bar = this.server.create('crate', { name: 'bar' });
+      this.server.create('crate-ownership', { crate: bar, user: user1 });
       this.server.create('version', { crateId: 'bar' });
-      this.server.create('crate', { name: 'baz', _owner_teams: [13] });
+      let baz = this.server.create('crate', { name: 'baz' });
+      this.server.create('crate-ownership', { crate: baz, user: user2 });
       this.server.create('version', { crateId: 'baz' });
 
-      let response = await fetch('/api/v1/crates?team_id=42');
+      let response = await fetch(`/api/v1/crates?user_id=${user1.id}`);
       assert.equal(response.status, 200);
 
       let responsePayload = await response.json();
@@ -168,14 +156,19 @@ module('Mirage | Crates', function (hooks) {
     });
 
     test('supports a `team_id` parameter', async function (assert) {
+      let team1 = this.server.create('team');
+      let team2 = this.server.create('team');
+
       this.server.create('crate', { name: 'foo' });
       this.server.create('version', { crateId: 'foo' });
-      this.server.create('crate', { name: 'bar', _owner_teams: [42] });
+      let bar = this.server.create('crate', { name: 'bar' });
+      this.server.create('crate-ownership', { crate: bar, team: team1 });
       this.server.create('version', { crateId: 'bar' });
-      this.server.create('crate', { name: 'baz', _owner_teams: [13] });
+      let baz = this.server.create('crate', { name: 'baz' });
+      this.server.create('crate-ownership', { crate: baz, team: team2 });
       this.server.create('version', { crateId: 'baz' });
 
-      let response = await fetch('/api/v1/crates?team_id=42');
+      let response = await fetch(`/api/v1/crates?team_id=${team1.id}`);
       assert.equal(response.status, 200);
 
       let responsePayload = await response.json();