Move setup usage of factory to beforeEach blocks

Author: talyssonoc

test/api/users/listUsers.spec.js

@@ -4,14 +4,16 @@ const { expect } = require('chai');
 
 describe('API :: GET /api/users', () => {
   context('when there are users', () => {
+    beforeEach(() => {
+      return factory.createMany('user', 2, [
+        { name: 'First' },
+        { name: 'Second' }
+      ]);
+    });
+
     it('return success with array of users', () => {
-      return factory
-        .createMany('user', 2, [
-          { name: 'First' },
-          { name: 'Second' }
-        ])
-        .then(() => request().get('/api/users')
-          .expect(200))
+      return request().get('/api/users')
+        .expect(200)
         .then(({ body }) => {
           expect(body).to.have.lengthOf(2);
 

test/infra/user/SequelizeUsersRepository.spec.js

@@ -6,15 +6,17 @@ const { User: UserModel } = require('src/infra/database/models');
 
 describe('Infra :: User :: SequelizeUsersRepository', () => {
   describe('#getAll', () => {
+    beforeEach(() => {
+      return factory.createMany('user', 2, [
+        { name: 'User 1' },
+        { name: 'User 2' }
+      ]);
+    });
+
     it('returns all users from the database', () => {
       const repository = new SequelizeUsersRepository({ UserModel });
 
-      return factory
-        .createMany('user', 2, [
-          { name: 'User 1' },
-          { name: 'User 2' }
-        ])
-        .then(() => repository.getAll())
+      return repository.getAll()
         .then((users) => {
           expect(users).to.have.lengthOf(2);