feat(set-article-price): set article price

- update create article methods to set price
- update update article controllers
- update model and migration files
- add validation methods for price input
- add test for associated edge cases

[Delivers #159568537]

config/config.js

@@ -1,4 +1,5 @@
 import { } from 'dotenv/config';
+
 module.exports = {
   development: {
     username: process.env.DB_USER,

controllers/ArticleController.js

@@ -18,13 +18,13 @@ class ArticleController {
      */
   static createArticle(req, res) {
     const {
-      title, description, body, tagList, imageUrl,
+      title, description, body, tagList, imageUrl, isPaidFor, price
     } = req.body.article;
 
     const { userId } = req;
 
     const articleObject = {
-      title, description, body, tagList, imageUrl, userId
+      title, description, body, tagList, imageUrl, isPaidFor, price, userId
     };
     /**
      * check if image was provided in the request
@@ -119,13 +119,15 @@ class ArticleController {
   * requests that fail
   */
   static editArticle(req, res) {
-    const { title, description, body } = req.body.article;
+    const { title, description, body, isPaidFor, price } = req.body.article;
     const { count } = req;
     const { slug } = req.params;
     return Article.update({
       title,
       description,
       body,
+      isPaidFor,
+      price,
       updatedCount: Utilities.increaseCount(count)
     }, {
       where: {

helpers/createArticleHelper.js

@@ -11,10 +11,11 @@ import { Article, User } from '../models';
  */
 
 const createArticleHelper = (res, articleObject, imageUrl = null) => {
+  // articleObject.price = articleObject.price.toFixed(2);
+  articleObject.price = (articleObject.price) ? articleObject.price.toFixed(2) : undefined;
   const {
-    title, description, body, tagList, userId
+    title, description, body, tagList, userId, isPaidFor, price,
   } = articleObject;
-
   return Article
     .create({
       slug: generateUniqueSlug(title),
@@ -24,6 +25,8 @@ const createArticleHelper = (res, articleObject, imageUrl = null) => {
       userId,
       tagList,
       imageUrl,
+      isPaidFor,
+      price
     })
     .then(article => Article.findById(article.id, {
       include: [{

middlewares/validatePrice.js

@@ -0,0 +1,32 @@
+const validatePrice = (req, res, next) => {
+  if (req.body.article.price && typeof req.body.article.price !== 'number') {
+
+    return res.status(400).json({
+      errors: {
+        body: [
+          'Article price must be in figure'
+        ]
+      }
+    });
+  }
+  if (req.body.article.isPaidFor && !req.body.article.price) {
+    return res.status(400).json({
+      errors: {
+        body: [
+          'Article is to be paid for, but price is not set'
+        ]
+      }
+    });
+  }
+  if (req.body.article.price <= 0.28 || req.body.article.price >= 5.53) {
+    return res.status(400).json({
+      errors: {
+        body: [
+          'Price can only be between $0.28 to $5.53'
+        ]
+      }
+    });
+  }
+  next();
+};
+export default validatePrice;

migrations/20180730142739-create-article.js

@@ -51,6 +51,12 @@ module.exports = {
         as: 'userId',
       },
     },
+    isPaidFor: {
+      type: Sequelize.BOOLEAN
+    },
+    price: {
+      type: Sequelize.DECIMAL
+    }
   }),
   down: (queryInterface/* , Sequelize */) => {
     queryInterface.dropTable('Articles');

models/Article.js

@@ -21,7 +21,13 @@ module.exports = (sequelize, DataTypes) => {
     tagList: DataTypes.ARRAY(DataTypes.STRING),
     favorited: DataTypes.BOOLEAN,
     favoritesCount: DataTypes.INTEGER,
-    imageUrl: DataTypes.STRING
+    imageUrl: DataTypes.STRING,
+    isPaidFor: {
+      type: DataTypes.BOOLEAN
+    },
+    price: {
+      type: DataTypes.DECIMAL
+    }
   }, {});
 
   Article.associate = (models) => {