routes testes and all bugs are fixed

Author: codingEdg

src/controllers/user.controller.js

@@ -5,9 +5,14 @@ import { User } from "../models/user.model.js";
 import { uploadOnCloudinary } from "../utils/fileUploadsCloudinary.js";
 import { ApiResponse } from "../utils/ApiResponse.js";
 import jwt from "jsonwebtoken";
+import mongoose from "mongoose";
 import { toBeDeletedAvatar } from "../utils/toBeDeletedAvatar.js";
 import { deleteCoverImage } from "../utils/deleteCoverImage.js";
 
+const testFunc = asyncHandler(async (req, res) => {
+  return res.status(200).json(new ApiResponse(201, {}, "success full tested"));
+});
+
 const options = {
   httpOnly: true,
   secure: true,
@@ -173,6 +178,7 @@ const loginUser = asyncHandler(async (req, res) => {
 const loggedOutUser = asyncHandler(async (req, res) => {
   // steps to logout user
   // 1. get user details from postman (as detals are mentioned in user model file)
+
   await User.findByIdAndUpdate(
     req.user._id,
     {
@@ -191,7 +197,7 @@ const loggedOutUser = asyncHandler(async (req, res) => {
 });
 
 // another controller for user end point for frontend side to refresh the access token to get login again
-const refreshAccessToken = asyncHandler(async (req, res, next) => {
+const refreshAccessToken = asyncHandler(async (req, res) => {
   // 1. get user details from postman (as detals are mentioned in user model file)
   // usually we get details from req.body or req.url so
   const incomingRefreshToken =
@@ -233,9 +239,10 @@ const refreshAccessToken = asyncHandler(async (req, res, next) => {
   }
 });
 
-const changeCurrentUserPassword = asyncHandler(async (req, res, next) => {
+const changeCurrentUserPassword = asyncHandler(async (req, res) => {
   // take all required field from user (req.body)
   const { oldPassword, newPassword } = req.body;
+  // console.log({ oldPassword });
 
   // bcoz user is loggedin, means midddleware had run then (req.user = user) so find the user from user._id
   const user = await User.findById(req.user._id);
@@ -248,7 +255,7 @@ const changeCurrentUserPassword = asyncHandler(async (req, res, next) => {
   // change password
   user.password = newPassword;
   await user.save({ validateBeforeSave: false });
-
+  console.log("password changed");
   return res
     .status(200)
     .json(new ApiResponse(200, {}, "Password Changed SuccessFully !!!"));
@@ -269,10 +276,10 @@ const getCurrentUser = asyncHandler(async (req, res, next) => {
 const updateAccountDetails = asyncHandler(async (req, res) => {
   // take all required field from user (req.body)
   const { fullName, userName, email } = req.body;
-  if (!(userName || fullName || email))
+  if (!(userName && fullName && email))
     throw new ApiError(400, "All fields are required "); // 99 90 83 74 27
 
-  const user = User.findByIdAndUpdate(
+  const user = await User.findByIdAndUpdate(
     req.user._id,
     {
       $set: {
@@ -289,13 +296,14 @@ const updateAccountDetails = asyncHandler(async (req, res) => {
   return res
     .status(200)
     .json(
-      new ApiResponse(200, { user }, "Account Details Updated SuccessFully !!!")
+      new ApiResponse(200, user, "Account Details Updated SuccessFully !!!")
     );
 });
 
 const updateUserAvatar = asyncHandler(async (req, res) => {
+  // console.log(req.file.path)
   // take all required field from user (req.body)
-  const avatarLocalpath = req.file?.avatar?.path;
+  const avatarLocalpath = req.file?.path;
   if (!avatarLocalpath) throw new ApiError(404, "Avatar file is required");
 
   const avatar = await uploadOnCloudinary(avatarLocalpath);
@@ -312,7 +320,7 @@ const updateUserAvatar = asyncHandler(async (req, res) => {
     { new: true }
   ).select("-password");
 
-  toBeDeletedAvatar(avatarLocalpath);
+  // toBeDeletedAvatar(avatarLocalpath);
 
   return res
     .status(200)
@@ -322,7 +330,7 @@ const updateUserAvatar = asyncHandler(async (req, res) => {
 const updateCoverImage = asyncHandler(async (req, res) => {
   {
     // take all required field from user (req.body)
-    const coverImageLocalpath = req.file?.coverImage?.path;
+    const coverImageLocalpath = req.file?.path;
     if (!coverImageLocalpath)
       throw new ApiError(404, "cover Image file is missing");
 
@@ -339,7 +347,7 @@ const updateCoverImage = asyncHandler(async (req, res) => {
       },
       { new: true }
     ).select("-password");
-    deleteCoverImage(coverImageLocalpath);
+    // deleteCoverImage(coverImageLocalpath);
     return res
       .status(200)
       .json(new ApiResponse(200, user, "cover Image Updated SuccessFully !!!"));
@@ -487,6 +495,7 @@ const getUserWatchHistory = asyncHandler(async (req, res) => {
 });
 
 export {
+  testFunc,
   registerUser,
   loginUser,
   loggedOutUser,

src/middlewares/auth.middleware.js

@@ -11,12 +11,13 @@ export const verifyJWT = asyncHandler(async (req, res, next) => {
       (await req.header("Authorization")?.replace("Bearer ", ""));
 
     if (!token) throw new ApiError(401, "Unauthorized Access");
+    // console.log({ token });
 
     // verify token
     const decodedToken = jwt.verify(token, process.env.ACCESS_TOKEN_SECRET);
 
     // get user from token
-    const user = User.findById(decodedToken?._id).select(
+    const user = await User.findById(decodedToken?._id).select(
       "-password -refreshToken"
     );
     if (!user) throw new ApiError(401, "Invalid Access token");
@@ -27,6 +28,10 @@ export const verifyJWT = asyncHandler(async (req, res, next) => {
     // user is verified now proceed further
     next();
   } catch (err) {
-    throw new ApiError(401, err?.message || "invalid access token");
+    console.log(err);
+    // if (err.name === "JsonWebTokenError") {
+    //   throw new ApiError(401, "Unauthorized Access");
+    // }
+    // throw new ApiError(401, err?.message || "invalid access token found !!!");
   }
 });

src/routes/user.routes.js

@@ -11,6 +11,7 @@ import {
   getUserChannelProfile,
   getUserWatchHistory,
   updateAccountDetails,
+  testFunc,
 } from "../controllers/user.controller.js";
 import { upload } from "../middlewares/multer.middleware.js";
 import { verifyJWT } from "../middlewares/auth.middleware.js";
@@ -33,12 +34,13 @@ router.route("/register").post(
   registerUser
 );
 
+router.route("/token").post(testFunc);
 router.route("/login").post(loginUser);
 
 // secure routes
 router.route("/logout").post(verifyJWT, loggedOutUser);
 
-router.route("/refreshtoken").post(refreshAccessToken);
+router.route("/refresh-token").post(refreshAccessToken);
 
 router.route("/change-password").post(verifyJWT, changeCurrentUserPassword);
 
@@ -50,7 +52,7 @@ router
   .route("/update-avatar")
   .patch(verifyJWT, upload.single("avatar"), updateUserAvatar);
 router
-  .route("/update-cover-image")
+  .route("/cover-image")
   .patch(verifyJWT, upload.single("coverImage"), updateCoverImage);
 
 router.route("/c/:username").get(verifyJWT, getUserChannelProfile);

src/utils/ApiError.js

@@ -15,7 +15,6 @@ class ApiError extends Error {
     if (stack) {
       this.stack = stack;
     } else {
-      console.log("errror captured");
       Error.captureStackTrace(this, this.constructor);
     }
   }

src/utils/deleteCoverImage.js

@@ -1,3 +1,4 @@
+import fs from "fs";
 const deleteCoverImage = (path) => {
   fs.unlink(path, (err) => err && console.log(err));
 };

src/utils/toBeDeletedAvatar.js

@@ -1,7 +1,14 @@
+import fs from "fs";
+
 const toBeDeletedAvatar = (avatarLocalPath) => {
-  fs.unlink(avatarLocalPath, (err) => {
-    if (err) console.log(err);
-  });
+  try {
+    fs.unlink(avatarLocalPath, (err) => {
+      if (err) console.log(err);
+    });
+  } catch (err) {
+    console.log("error while deleting local path");
+    console.log(err);
+  }
 };
 
 export { toBeDeletedAvatar };