💚 ?

server/api/auth.test.js

@@ -6,7 +6,7 @@ import { flushdb, seed } from '../test/support';
 const server = new TestServer(app.callback());
 
 beforeEach(flushdb);
-afterAll(() => server.close());
+afterAll(server.close);
 
 describe.skip('#auth.signup', async () => {
   it('should signup a new user', async () => {

server/api/documents.js

@@ -13,12 +13,13 @@ router.post('documents.list', auth(), pagination(), async ctx => {
   if (direction !== 'ASC') direction = 'DESC';
 
   const user = ctx.state.user;
-  const documents = await Document.findAll({
+  const userId = user.id;
+  const starredScope = { method: ['withStarred', userId] };
+  const documents = await Document.scope('defaultScope', starredScope).findAll({
     where: { teamId: user.teamId },
     order: [[sort, direction]],
     offset: ctx.state.pagination.offset,
     limit: ctx.state.pagination.limit,
-    include: [{ model: Star, as: 'starred', where: { userId: user.id } }],
   });
 
   const data = await Promise.all(

server/api/documents.test.js

@@ -6,7 +6,7 @@ import { flushdb, seed } from '../test/support';
 const server = new TestServer(app.callback());
 
 beforeEach(flushdb);
-afterAll(() => server.close());
+afterAll(server.close);
 
 describe('#documents.list', async () => {
   it('should return documents', async () => {

server/api/user.test.js

@@ -3,12 +3,12 @@ import TestServer from 'fetch-test-server';
 import app from '..';
 import { User } from '../models';
 
-import { flushdb, seed, sequelize } from '../test/support';
+import { flushdb, seed } from '../test/support';
 
 const server = new TestServer(app.callback());
 
 beforeEach(flushdb);
-afterAll(() => server.close());
+afterAll(server.close);
 
 describe('#user.info', async () => {
   it('should return known user', async () => {

server/models/Document.js

@@ -130,6 +130,11 @@ Document.associate = models => {
     },
     { override: true }
   );
+  Document.addScope('withStarred', userId => ({
+    include: [
+      { model: models.Star, as: 'starred', where: { userId }, required: false },
+    ],
+  }));
 };
 
 Document.findById = async id => {

server/models/User.test.js

@@ -4,8 +4,6 @@ beforeEach(flushdb);
 
 it('should set JWT secret and password digest', async () => {
   const { user } = await seed();
-  await user.save();
-
   expect(user.passwordDigest).toBeTruthy();
   expect(user.getJwtToken()).toBeTruthy();