[Feature #65412893] highlight text in article

src/config/dbConfig.js

@@ -9,23 +9,26 @@ const config = {
     password: process.env.DB_PASSWORD_DEV,
     database: process.env.DB_NAME_DEV,
     host: process.env.DB_HOST_DEV,
-    dialect: 'postgres'
+    dialect: 'postgres',
+    seederStorage: 'sequelize'
   },
   test: {
     use_env_variable: 'DATABASE_URL_TEST',
     username: process.env.DB_USER_TEST,
     password: process.env.DB_PASSWORD_TEST,
     database: process.env.DB_NAME_TEST,
     host: process.env.DB_HOST_TEST,
-    dialect: 'postgres'
+    dialect: 'postgres',
+    seederStorage: 'sequelize'
   },
   production: {
     use_env_variable: 'DATABASE_URL',
     username: process.env.DB_USER,
     password: process.env.DB_PASSWORD,
     database: process.env.DB_NAME,
     host: process.env.DB_HOST,
-    dialect: 'postgres'
+    dialect: 'postgres',
+    seederStorage: 'sequelize'
   }
 };
 

src/controllers/HighlightController.js

@@ -20,7 +20,7 @@ export default class Highlights {
 
     const contentLength = highlightedText.split('').length;
     const indexesLength = stopIndex - startIndex;
-    if (contentLength !== indexesLength + 1) {
+    if (contentLength !== indexesLength) {
       return res.status(status.BAD_REQUEST).json({
         message: 'Sorry the length of your highlightedText does not match with start and end index'
       });
@@ -79,8 +79,8 @@ export default class Highlights {
           message: `the highlight with id ${id} does not exist`
         }))
       || res.status(status.OK).json({
-        message: 'You have successfully remove your highlight',
-        highlight
+        message: 'You have successfully removed your highlight',
+        highlightId: id
       })
     );
   }

src/helpers/queryHelper.js

@@ -3,11 +3,12 @@ const dbFindSingle = async (model, whereCondition = {}) => model.findOne({
   logging: false
 });
 
-const dbFindAll = async (model, whereCondition, offset = 0, limit = 20) => model.findAll({
+const dbFindAll = async (model, whereCondition, offset = 0, limit = 20, include) => model.findAll({
   offset,
   limit,
   where: whereCondition,
-  logging: false
+  logging: false,
+  include
 });
 
 const dbCreate = async (model, condition) => model.create(condition, { logging: false });

src/helpers/validation/createHighlight.js

@@ -4,14 +4,12 @@ export default (input) => {
   const schema = Joi.object().keys({
     highlightedText: Joi.string()
       .min(5)
-      .max(255)
       .required(),
     startIndex: Joi.number().required(),
     stopIndex: Joi.number().required(),
     comment: Joi.string()
-      .min(5)
-      .max(255)
-      .required()
+      .allow(null, '')
+      .optional()
   });
 
   return Joi.validate(input, schema, { abortEarly: false });

src/middlewares/checkArticle.js

@@ -16,7 +16,7 @@ const checkArticle = async (req, res, next) => {
   });
   if (!findArticle) {
     return res.status(status.NOT_FOUND).send({
-      message: 'That article does not exist'
+      message: 'This article does not exist'
     });
   }
   next();

src/models/highlight.js

@@ -14,13 +14,13 @@ module.exports = (sequelize, DataTypes) => {
       },
       userId: {
         type: DataTypes.INTEGER,
-        allowNull: true,
+        allowNull: false,
         references: {
           model: 'Users',
           key: 'id'
         },
         onUpdate: 'CASCADE',
-        onDelete: 'SET NULL'
+        onDelete: 'CASCADE'
       },
       highlightedText: {
         type: DataTypes.STRING,
@@ -52,10 +52,7 @@ module.exports = (sequelize, DataTypes) => {
     {}
   );
   Highlight.associate = (models) => {
-    Highlight.belongsTo(models.User, {
-      foreignKey: 'userId',
-      onDelete: 'CASCADE'
-    });
+    Highlight.belongsTo(models.User, { foreignKey: 'userId', as: 'commentAuthor' });
   };
   return Highlight;
 };

src/queries/highlights/getAll.js

@@ -1,6 +1,12 @@
 import db from '../../models';
 import { dbFindAll } from '../../helpers/queryHelper';
 
-const getAll = async (condition = {}) => dbFindAll(db.Highlight, condition);
+const getAll = async (condition = {}) => dbFindAll(db.Highlight, condition, null, null, [
+  {
+    model: db.User,
+    as: 'commentAuthor',
+    attributes: ['firstName', 'lastName', 'username', 'email', 'image']
+  }
+]);
 
 export default getAll;

src/seeders/20190605075225-permissions.js

@@ -2,10 +2,10 @@ import * as Factory from '../helpers/factory';
 
 const permissionsNormal = Factory.permissionsNormal.build();
 const permissionsAdmin = Factory.permissionsAdmin.build();
-permissionsNormal.createdAt = '2019-05-12T22:00:00';
-permissionsNormal.updatedAt = '2019-05-12T22:00:00';
-permissionsAdmin.createdAt = '2019-05-12T22:00:00';
-permissionsAdmin.updatedAt = '2019-05-12T22:00:00';
+permissionsNormal.createdAt = new Date();
+permissionsNormal.updatedAt = new Date();
+permissionsAdmin.createdAt = new Date();
+permissionsAdmin.updatedAt = new Date();
 
 export default {
   up: queryInterface => queryInterface.bulkInsert('Permissions', [permissionsNormal, permissionsAdmin], {}),

src/seeders/20190715123639-admin-user.js

@@ -8,9 +8,9 @@ const userAdmin = {
   email: 'admin@admin.com',
   password: password.hash('admin'),
   role: 'admin',
-  permissions: Factory.permissionsAdmin.build(),
-  createdAt: '2019-05-12T22:00:00',
-  updatedAt: '2019-05-12T22:00:00'
+  permissions: Factory.permissionsAdmin.build().permissions,
+  createdAt: new Date(),
+  updatedAt: new Date()
 };
 
 export default {

src/tests/routes/highlight.test.js

@@ -33,16 +33,6 @@ describe('Highlight on Article', () => {
         cascade: true,
         logging: false
       });
-      await db.Article.destroy({
-        truncate: true,
-        cascade: true,
-        logging: false
-      });
-      await db.Highlight.destroy({
-        truncate: true,
-        cascade: true,
-        logging: false
-      });
 
       createdUser = (await db.User.create(newUser, { logging: false })).dataValues;
       newArticle.userId = createdUser.id;
@@ -52,6 +42,7 @@ describe('Highlight on Article', () => {
         { expiresIn: '1d' }
       );
       createdArticle = (await db.Article.create(newArticle, { logging: false })).dataValues;
+      newHighlight.userId = createdArticle.userId;
       newHighlight.articleSlug = createdArticle.slug;
       createdHighlight = await db.Highlight.create(newHighlight, { logging: false });
       newArticleSlug = createdHighlight.articleSlug;
@@ -69,7 +60,7 @@ describe('Highlight on Article', () => {
       .send({
         highlightedText: 'on sera ensemble bientotssssss',
         startIndex: 0,
-        stopIndex: 29,
+        stopIndex: 30,
         comment: 'welcomme to the party'
       })
       .end((err, res) => {
@@ -99,7 +90,7 @@ describe('Highlight on Article', () => {
       .send({
         highlightedText: 'on sera ensemble bientotssssss',
         startIndex: 0,
-        stopIndex: 30,
+        stopIndex: 39,
         comment: 'welcomme to the party'
       })
       .end((err, res) => {