fix(events): use relative route paths to match Fastify prefix registration (#229)

All five route handlers in eventRoutes defined absolute /api/events* paths
while app.ts also registered the plugin with prefix: '/api/events'. Fastify
concatenates registration prefix and route path, producing double-prefixed
endpoints (/api/events/api/events, /api/events/api/events/:slug, etc.) that
are unreachable in production.

Strip the /api/events prefix from every route definition so paths are
relative (/, /:slug, /:slug/join, /:slug/leave, /:slug/attendees),
consistent with every other route plugin in the codebase.

Update the test buildApp() to register with { prefix: '/api/events' },
matching production. Inject URLs in existing tests already use the full
/api/events/* paths and require no changes.

Fixes #224.

Author: Ridanshi

apps/backend/src/__tests__/event.test.ts

@@ -78,7 +78,9 @@ async function buildApp(): Promise<FastifyInstance> {
     return mockJwtVerify();
   });
 
-  await app.register(eventRoutes);
+  // Register with the same prefix used in production (app.ts) so that
+  // tests exercise routes at their real paths — /api/events, /api/events/:slug, etc.
+  await app.register(eventRoutes, { prefix: '/api/events' });
   await app.ready();
   return app;
 }

apps/backend/src/routes/event.ts

@@ -36,13 +36,13 @@ type PaginatedAttendeesResponse = {
 }
 
 export async function eventRoutes(app:FastifyInstance) {
-    app.post('/api/events' , async(request: FastifyRequest<{
+    app.post('/' , async(request: FastifyRequest<{
         Body: {
             name: string,
-            description?: string, 
-            startDate: string, 
+            description?: string,
+            startDate: string,
             location: string,
-            endDate: string, 
+            endDate: string,
             isPublic?: boolean
     }}>, reply: FastifyReply) => {
         let decoded; 
@@ -98,7 +98,7 @@ export async function eventRoutes(app:FastifyInstance) {
     })
 
     //Returns event details and attendees count
-    app.get('/api/events/:slug', async(request: FastifyRequest<{Params: {slug: string}}>, reply: FastifyReply) => {
+    app.get('/:slug', async(request: FastifyRequest<{Params: {slug: string}}>, reply: FastifyReply) => {
         const paramsSlug = request.params.slug; 
         const details = await app.prisma.event.findUnique({
             where: {
@@ -132,7 +132,7 @@ export async function eventRoutes(app:FastifyInstance) {
         return response; 
     })
 
-    app.post('/api/events/:slug/join' , async(request: FastifyRequest<{Params: {slug: string}}>, reply: FastifyReply) => {
+    app.post('/:slug/join' , async(request: FastifyRequest<{Params: {slug: string}}>, reply: FastifyReply) => {
         let decoded; 
         try {
             decoded = await request.jwtVerify() as any; 
@@ -172,7 +172,7 @@ export async function eventRoutes(app:FastifyInstance) {
 
     })
 
-    app.delete('/api/events/:slug/leave',async(request: FastifyRequest<{Params: {slug: string}}>, reply: FastifyReply) => {
+    app.delete('/:slug/leave',async(request: FastifyRequest<{Params: {slug: string}}>, reply: FastifyReply) => {
         let decoded; 
         try {
             decoded = await request.jwtVerify() as any
@@ -211,7 +211,7 @@ export async function eventRoutes(app:FastifyInstance) {
         }
     })
 
-    app.get('/api/events/:slug/attendees', async(request: FastifyRequest<{Params: {slug: string}, Querystring: {page?:string; limit?: string}}>, reply: FastifyReply) => {
+    app.get('/:slug/attendees', async(request: FastifyRequest<{Params: {slug: string}, Querystring: {page?:string; limit?: string}}>, reply: FastifyReply) => {
         const paramsSlug = request.params.slug; 
         const page = Math.max(1, Number(request.query.page) || 1); 
         const limit = Math.min(50, Number(request.query.limit) || 10);