added cookie-parser, endpoint functionality tested

Author: liamroh

package-lock.json

@@ -163,8 +163,8 @@
     "socket.io": "^4.6.1",
     "socket.io-client": "^4.6.1",
     "source-map-support": "^0.5.21",
-    "typescript-coverage-report": "^0.7.0",
     "ts-node": "^10.9.1",
+    "typescript-coverage-report": "^0.7.0",
     "uniqid": "^5.4.0",
     "use-debounce": "^8.0.3",
     "uuid": "^8.3.2"

package.json

@@ -33,8 +33,8 @@ const marketplaceController: MarketplaceController = {
   publishProject: (req, res, next) => {
     const { _id, project, comments, userId, username, name } = req.body;
     const createdAt = Date.now();
-   
     if (userId === req.cookies.ssid) {
+      console.log('inside');
       Projects.findOneAndUpdate(
         // looks in projects collection for project by Mongo id
         { _id },
@@ -58,13 +58,15 @@ const marketplaceController: MarketplaceController = {
         }
       );
     }
-    // we should not expect a user to be able to access another user's id, but included error handling for unexpected errors
-    return next({
-      log: 'Error in marketplaceController.publishProject', 
-      message: {
-        err: 'Error in marketplaceController.publishProject, check server logs for details'
-      }
-    })
+    else {
+      // we should not expect a user to be able to access another user's id, but included error handling for unexpected errors
+      return next({
+        log: 'Error in marketplaceController.publishProject', 
+        message: {
+          err: 'Error in marketplaceController.publishProject, check server logs for details'
+        }
+      })
+    }
   },
 
   /**
@@ -91,13 +93,16 @@ const marketplaceController: MarketplaceController = {
         return next();
       });
     }
-    // we should not expect a user to be able to access another user's id, but included error handling for unexpected errors
-    return next({
-      log: `Error in marketplaceController.unpublishProject`,
-      message: {
-        err: 'Error in marketplaceController.unpublishProject, userId of project does not match cookies.ssid'
-      }
-    })
+    else {
+      // we should not expect a user to be able to access another user's id, but included error handling for unexpected errors
+      return next({
+        log: `Error in marketplaceController.unpublishProject`,
+        message: {
+          err: 'Error in marketplaceController.unpublishProject, userId of project does not match cookies.ssid'
+        }
+      })
+
+    }
   }
 };
 export default marketplaceController;

server/controllers/marketplaceController.ts

@@ -10,7 +10,7 @@ const { json, urlencoded } = bodyParser;
 const { makeExecutableSchema } = require('@graphql-tools/schema');
 
 import express from 'express';
-// import cookieParser from 'cookie-parser';
+import cookieParser from 'cookie-parser';
 
 import config from '../config.js';
 const { API_BASE_URL, DEV_PORT } = config;
@@ -41,7 +41,7 @@ const isTest = process.env.NODE_ENV === 'test';
 
 app.use(express.json({ limit: '100mb' }));
 app.use(express.urlencoded({ limit: '100mb', extended: true }));
-
+app.use(cookieParser());//added cookie parser
 // Routes
 // const stylesRouter = require('./routers/stylesRouter');
 import stylesRouter from './routers/stylesRouter';