error-in-test

Author: samrth07

src/app.ts

@@ -30,7 +30,7 @@ app.use(json())
 app.use(urlencoded({ extended: true }))
 
 // 3) Handle file uploads (in-memory)
-const upload = multer({ storage: multer.memoryStorage() })
+ const upload = multer({ storage: multer.memoryStorage() })
 
 // 4) Mount your routes, injecting `upload` middleware where needed
 //    For endpoints that accept file uploads, you can do e.g.:

src/controllers/project.controller.ts

@@ -2,6 +2,8 @@ import * as projectService from "../services/project.service";
 import { Request ,  Response } from "express";
 import { ApiError } from "../utils/apiError";
 import { error } from "console";
+import { uploadImage } from "../utils/imageUtils";
+import { supabase } from "../app"; 
 
 export const getProjects = async ( req : Request , res : Response ) => {
 
@@ -31,15 +33,27 @@ export const getProjectById = async ( req : Request , res : Response ) => {
 export const createProject = async ( req : Request , res : Response ) => {
 
         try {
-                    const projectContent = req.body;
-                    const AdminId = req.body.AdminId;
-                    if( projectContent.name.length === 0 || !projectContent.githubUrl || !AdminId || !projectContent.imageUrl) throw new ApiError( " Field is missing !!!" , 400);
+                const file = req.file;
+                if(!file) throw new ApiError('Image file is not find' , 400);
 
-                const project = await projectService.createProject(projectContent , AdminId);
-                res.status(200).json(project);
+                const imageUrl = await uploadImage( supabase , file , 'projects');
+
+                if(!imageUrl) throw new ApiError("Image url is missing" , 400);
+
+                    const projectContent = {
+                        name: req.body.projectData.project,
+                        imageUrl: imageUrl,
+                        githubUrl: req.body.projectData.githubUrl,
+                        deployUrl: req.body.projectData.deployUrl,
+                        AdminId: req.body.projectData.AdminId,
+                    };
+
+                const project = await projectService.createProject( projectContent );
+                res.status(200).json(project); 
 
         } catch (error) {
-                throw new ApiError( error as string , 500);
+                console.log(error);
+                throw new ApiError( error as string , 404);
         }
 };
 

src/routes/index.ts

@@ -15,28 +15,3 @@ export default function routes(upload: Multer, supabase: SupabaseClient) {
   return router
 }
 
-/*
-write your routes like this, skip the paramaeters in the exported function if your routes don't need supabase and multer
-
-import { Router } from 'express'
-import { Multer } from 'multer'
-import * as memberCtrl from '../controllers/member.controller'
-import { SupabaseClient } from '@supabase/supabase-js'
-
-export default function membersRouter(
-  upload: Multer,
-  supabase: SupabaseClient
-) {
-  const router = Router()
-
-  // Photo upload endpoint:
-  router.post(
-    '/:memberId/photo',
-    upload.single('photo'),
-    (req, res, next) => memberCtrl.uploadPhoto(req, res, next, supabase)
-  )
-
-  return router
-}
-
-*/

src/routes/projects.ts

@@ -1,5 +1,5 @@
-import { Router } from 'express'
-import { Multer } from 'multer'
+import { NextFunction, Request, Response, Router } from 'express'
+import  { Multer }  from 'multer'
 import { SupabaseClient } from '@supabase/supabase-js'
 import {
     addMembers,
@@ -13,6 +13,18 @@ import {
 } from '../controllers/project.controller'
 
 
+function parseCreateProjectData(req : Request, res : Response  , next : NextFunction) {
+
+            if(req.body.projectData){
+                    try{
+                        const parse = JSON.parse(req.body.projectData);
+                        req.body = parse;
+                    }catch(e){
+                        return res.status(400).json({ message: 'Invalid JSON in projectData field' });
+                    }
+            }
+            next();
+}
 
 export default function projectsRouter(
     upload: Multer,
@@ -60,7 +72,7 @@ export default function projectsRouter(
      * @apiError { error 400}   some fields are missing 
      * @apiError { error 500}   error related to the db interaction
      */
-    router.post('/create', createProject )
+    router.post('/create', upload.single('image') , parseCreateProjectData , createProject )
 
     //  Update Project
         /**
@@ -150,12 +162,6 @@ export default function projectsRouter(
 
     router.delete( '/:projectId/members/:memberId', removeMembers)
 
-    // Photo upload endpoint:
-    //   router.post(
-    //     '/:projectId/photo',
-    //     upload.single('photo'),
-    //     (req, res, next) => memberCtrl.uploadPhoto(req, res, next, supabase)
-    //   )
 
 
     return router

src/services/project.service.ts

@@ -16,15 +16,15 @@ export const getProjectById = async ( projectId : number ) => {
         })
 };
 
-export const createProject = async ( projectContent : projectContent  , AdminId : string) => {
+export const createProject = async ( projectContent : projectContent ) => {
 
     return await prisma.project.create({
         data : {
             name : projectContent.name,
             imageUrl : projectContent.imageUrl,
             githubUrl : projectContent.githubUrl,
             deployUrl : projectContent.deployUrl,
-            createdById :  AdminId,
+            createdById :  projectContent.AdminId,
         },
     })
    };

src/types/project.d.ts

@@ -8,7 +8,8 @@ declare global {
         name: string,
         imageUrl: string ,
         githubUrl: string,
-        deployUrl: string?
+        deployUrl: string?,
+        AdminId : string
     }
 
     type addMembersData = {

tests/Project.test.ts

@@ -1,12 +1,13 @@
-import { describe } from 'node:test';
-import { createProject  , getProjectById , updateProjects , deleteProjects , getMembersByProjectId , addMembers , removeMembers} from '../src/controllers/project.controller';
+import { createProject  , getProjectById , updateProjects , deleteProjects
+   , getMembersByProjectId , addMembers , removeMembers} from '../src/controllers/project.controller';
 import * as projectService from '../src/services/project.service';
 import { ApiError } from '../src/utils/apiError';
+import { Response , Request} from 'express';
 
 // modify the respose Object
 
    const mockRes = () => {
-    const res: any = {};
+    const res = {} as Response;
     res.status = jest.fn().mockReturnValue(res);
     res.json = jest.fn().mockReturnValue(res);
     return res;
@@ -20,16 +21,18 @@ describe('createProjectHandler', () => {
 
 
   it('should return 200 and created project with valid input', async () => {
-    const req: any = {
+    const req = {
       body: {
         name: 'EventHub',
-        imageUrl: 'https://example.com/image.png',
         githubUrl: 'https://github.com/example/eventhub',
         deployUrl: 'https://eventhub.example.com',
         AdminId,
       },
+      file : {
+          name : "image"
+      }
 
-    };
+    } as unknown as Request;
 
     const res = mockRes();
 

tsconfig.json

@@ -110,7 +110,7 @@
     // "skipDefaultLibCheck": true,                      /* Skip type checking .d.ts files that are included with TypeScript. */
     "skipLibCheck": true                                 /* Skip type checking all .d.ts files. */
   },
-  "include": ["src/**/*"], // Include your source files
+  "include": ["src/**/*", "tests/imageUtils.test.ts"], // Include your source files
   "exclude": ["node_modules", "tests/**/*"] // Exclude test files from compilation
 }