implement getting all logs for one person. Each log currently has an …

…account instance attached, as per the TODO

models/index.js

@@ -25,7 +25,7 @@ const Log = sequelize.import('./log.model');
 const Plugin = sequelize.import('./plugin.model');
 
 Account.belongsTo(Plugin);
-Account.hasMany(Log);
+Log.belongsTo(Account);
 
 export { Account, Log, Plugin };
 

server/controller/log.controller.js

@@ -1,7 +1,7 @@
 import httpErrors from 'httperrors';
 import { Op } from 'sequelize';
 
-import { Log } from '../../models';
+import { Account, Log } from '../../models';
 
 export async function create({ accountUuid, nativeLocation }) {
   // validate data
@@ -22,14 +22,23 @@ export async function create({ accountUuid, nativeLocation }) {
   }
 }
 
-export async function readAll({ accountUuid }) {
+export async function readAll({ accountUuid, personUuid }) {
   // create filter
   const condition = {};
+  const include = [];
   if (accountUuid) condition.accountUuid = accountUuid;
+  if (personUuid) {
+    // TODO this adds the account to the fetched data,
+    // even though we want the account only for filtering
+    include.push({
+      model: Account,
+      where: { personUuid },
+    });
+  }
 
   // query database
   try {
-    const log = await Log.findAll({ where: condition });
+    const log = await Log.findAll({ where: condition, include });
     return log;
   } catch (err) {
     throw new httpErrors[500](err.message || 'An error occurred...');

test/controller/log.spec.js

@@ -65,6 +65,21 @@ describe('Log Controller', () => {
       ]);
     }
 
+    // read all by person
+    {
+      const logs = await Log.readAll({ personUuid });
+      // toMatchObject because sequelize model instances are not plain objects
+      expect(logs).toMatchObject([
+        {
+          id: expect.any(Number),
+          accountUuid,
+          createdAt: expect.any(Date),
+          updatedAt: expect.any(Date),
+          nativeLocation: 'foo',
+        },
+      ]);
+    }
+
     // update
     {
       await Log.update(id, {