Update Question model

Author: sivayogasubramanian

questions/src/controllers/questions.ts

@@ -1,6 +1,6 @@
 import express from "express";
 import { QuestionDao } from "../models/questions";
-import { ApiResponse, EMPTY_OBJECT, StatusMessageType } from "../types";
+import { ApiResponse, StatusMessageType } from "../types";
 import { handleCustomError, handleServerError } from "../utils";
 
 // GET /questions/:id
@@ -41,15 +41,27 @@ export const createQuestion = async (
   res: express.Response
 ) => {
   try {
-    const { title, description } = req.body;
-    if (!title || !description) {
+    const { title, description, tags, difficulty } = req.body;
+    if (!title || !description || !difficulty) {
       handleCustomError(res, {
         type: StatusMessageType.ERROR,
-        message: "Title and description must be provided",
+        message: "Title, description and difficulty must be provided",
       });
     }
 
-    const question = await QuestionDao.createQuestion(title, description);
+    if (!tags || tags.length === 0) {
+      handleCustomError(res, {
+        type: StatusMessageType.ERROR,
+        message: "At least one tag must be provided",
+      });
+    }
+
+    const question = await QuestionDao.createQuestion(
+      title,
+      description,
+      tags,
+      difficulty
+    );
     const response: ApiResponse = {
       payload: question,
       statusMessage: {
@@ -70,15 +82,29 @@ export const updateQuestion = async (
 ) => {
   try {
     const id = req.params.id;
-    const { title, description } = req.body;
-    if (!id || !title || !description) {
+    const { title, description, tags, difficulty } = req.body;
+    if (!id || !title || !description || !difficulty) {
+      handleCustomError(res, {
+        type: StatusMessageType.ERROR,
+        message:
+          "Question ID, title, description and difficulty must be provided",
+      });
+    }
+
+    if (!tags || tags.length === 0) {
       handleCustomError(res, {
         type: StatusMessageType.ERROR,
-        message: "Question ID, title, and description must be provided",
+        message: "At least one tag must be provided",
       });
     }
 
-    const question = await QuestionDao.updateQuestion(id, title, description);
+    const question = await QuestionDao.updateQuestion(
+      id,
+      title,
+      description,
+      tags,
+      difficulty
+    );
     if (!question) {
       handleCustomError(res, {
         type: StatusMessageType.ERROR,

questions/src/models/questions.ts

@@ -1,28 +1,66 @@
 import mongoose from "mongoose";
 
 const QuestionSchema = new mongoose.Schema({
-  title: { type: String, required: true },
-  description: { type: String, required: true },
+  title: {
+    type: String,
+    required: true,
+    validate: {
+      validator: (title: string) => title.length > 0,
+      message: "Title is required",
+    },
+  },
+  difficulty: {
+    type: String,
+    enum: ["Easy", "Medium", "Hard"],
+    required: true,
+    ValidityState: {
+      values: ["Easy", "Medium", "Hard"],
+      message: "{VALUE} is not supported",
+    },
+  },
+  tags: {
+    type: Array,
+    required: true,
+    validate: {
+      validator: (tags: [String]) => tags.length > 0,
+      message: "At least one tag is required",
+    },
+  },
+  description: {
+    type: String,
+    required: true,
+    validate: {
+      validator: (description: string) => description.length > 0,
+      message: "Description is required",
+    },
+  },
 });
 
-const QuestionModel = mongoose.model("Question", QuestionSchema);
+export const QuestionModel = mongoose.model("Question", QuestionSchema);
 
 const getQuestionById = async (id: string) =>
   QuestionModel.findById(id).then((question) => question?.toObject());
 
-const createQuestion = async (title: string, description: string) => {
-  const question = new QuestionModel({ title, description });
+const createQuestion = async (
+  title: string,
+  description: string,
+  tags: [String],
+  difficulty: string
+) => {
+  const question = new QuestionModel({ title, description, tags, difficulty });
   return question.save().then((question) => question.toObject());
 };
 
 const updateQuestion = async (
   id: string,
   title: string,
-  description: string
+  description: string,
+  tags: [String],
+  difficulty: string
 ) => {
   const updatedQuestion = await QuestionModel.findByIdAndUpdate(
     id,
-    { title, description },
+    { title, description, tags, difficulty },
     { new: true }
   );
   return updatedQuestion?.toObject();

questions/src/utils/index.ts

@@ -25,7 +25,7 @@ export const handleServerError = (error: Error, res: express.Response) => {
     payload: EMPTY_OBJECT,
     statusMessage: {
       type: StatusMessageType.ERROR,
-      message: "Something went wrong! Please try again later.",
+      message: error.message,
     },
   };