Feature(articles): getting article reading time

-showing time it takes to read an article [Delivers #164489943]
andela · Apr 15, 2019 · a74c804 · a74c804
1 parent 5abbf4b
commit a74c804
Show file tree

Hide file tree

Showing 7 changed files with 27 additions and 4 deletions.
diff --git a/controllers/article.js b/controllers/article.js
@@ -1,4 +1,5 @@
 import models from '../models/index';
+import readingTime from '../helpers/readingTime';
 
 const Article = models.article;
 /**
@@ -82,8 +83,9 @@ class ArticleController {
         if (!article) {
           return res.status(404).json({ error: 'Sorry the requested resource could not be found.' });
         }
+        const time = readingTime(article.title + article.body);
         // @return article
-        return res.status(200).json({ status: 200, article });
+        return res.status(200).json({ status: 200, article, readingTime: `${time} min` });
       })
       .catch(error => res.status(500).json({ error: `Something wrong please try again later. ${error}` }));
   }

diff --git a/helpers/readingTime.js b/helpers/readingTime.js
@@ -0,0 +1,16 @@
+
+const readingTime = (body) => {
+  let time;
+  const numberOfWords = body.split(' ').length;
+  const minutes = (numberOfWords / 265).toFixed(2);
+  const seconds = minutes - Math.floor(minutes);
+  if (seconds >= 0.30) {
+    time = Math.floor(minutes) + 1;
+  } else {
+    time = Math.floor(minutes);
+  }
+
+  return time;
+};
+
+export default readingTime;
diff --git a/migrations/20190402140158-article.js b/migrations/20190402140158-article.js
@@ -23,6 +23,7 @@ const articleMigration = {
       author: {
         type: Sequelize.INTEGER,
         allowNull: false,
+        onDelete: 'CASCADE',
         references: {
           model: 'users',
           key: 'id'

diff --git a/migrations/20190408131611-create-comments.js b/migrations/20190408131611-create-comments.js
@@ -12,13 +12,15 @@ module.exports = {
     articleId: {
       type: Sequelize.INTEGER,
       allowNull: false,
+      onDelete: 'CASCADE',
       references: {
         model: 'articles', key: 'article_id'
       }
     },
     author: {
       type: Sequelize.INTEGER,
       allowNull: false,
+      onDelete: 'CASCADE',
       references: {
         model: 'users', key: 'id'
       }

diff --git a/migrations/20190409161017-create-rate.js b/migrations/20190409161017-create-rate.js
@@ -9,6 +9,7 @@ const rateMigration = {
     userId: {
       type: Sequelize.INTEGER,
       allowNull: false,
+      onDelete: 'CASCADE',
       references: {
         model: 'users',
         key: 'id',
@@ -18,6 +19,7 @@ const rateMigration = {
     articleId: {
       type: Sequelize.INTEGER,
       allowNull: false,
+      onDelete: 'CASCADE',
       references: {
         model: 'articles',
         key: 'article_id',

diff --git a/models/article.js b/models/article.js
@@ -20,8 +20,8 @@ const articleModel = (Sequelize, DataTypes) => {
       foreignKey: 'articleId',
       allowNull: false
     });
-    Article.belongsTo(models.user, { as: 'authorfkey', foreignKey: 'author' });
-    Article.hasMany(models.rate, { foreignKey: 'articleId' });
+    Article.belongsTo(models.user, { as: 'authorfkey', foreignKey: 'author', onDelete: 'CASCADE' });
+    Article.hasMany(models.rate, { foreignKey: 'articleId', onDelete: 'CASCADE' });
   };
   return Article;
 };

diff --git a/testingdata/user.json b/testingdata/user.json
@@ -40,7 +40,7 @@
     "email": "blaise@gmail.com"
   },
   "googleValidToken": {
-    "access_token": "ya29.GlvpBnXYdrCRtsIchnu6i01ejs_52UwfyI6wztxXe-Fyu15M4kPOBS3V-sXYGIX89tLRKluMgGIZG_3AhO8rfHbSzs0JcHDhcuukpVdJTbopyWn7dnJ0ao2ZS1Vr"
+    "access_token": "ya29.GlzsBkXVGjsECcatNR04ApJ72Jf96hzq7N5dJ0vOI3Nr1ZWFIRZFyBOETI3HyVs7uuS5Xn16VCrH39PTw8_E0DTLbfpvN58XWiGOL8DSoamHLMohkdY_WgwzGJ_Hpg"
   },
   "googleInvalidToken": {
     "access_token": "ya29.GlvhBpzY2hl2ShgOMrpkni8obGgwyX0mr85Oendf2kmblu3BrRNTmYK2DVQiPciVOBFkLvR57YE90qDyffgJOqgzV68zutO3-Y9QDKooAPuxPvwsbsWM36wwVPHT"