Complete reports system integration - connect reportRoutes to main app

Author: safranowith

README.md

@@ -108,9 +108,8 @@ npm run test:verbose
 - `GET /api/files` - Retrieve file list
 
 ### Reports
-- `GET /api/reports` - Retrieve all reports
-- `POST /api/reports` - Create new report
-- `GET /api/reports/:id` - Retrieve specific report
+- `GET /reports/generate` - Generate a report from current data
+- `GET /reports/summary` - Get summary statistics of data
 
 ## Technologies
 
@@ -163,15 +162,13 @@ const response = await fetch('/api/files/upload', {
 
 ### Generate Report
 ```javascript
-const response = await fetch('/api/reports', {
-  method: 'POST',
-  headers: { 'Content-Type': 'application/json' },
-  body: JSON.stringify({
-    title: 'Monthly Revenue Report',
-    period: '2025-01',
-    type: 'revenue'
-  })
-});
+// Get summary statistics
+const summaryResponse = await fetch('/reports/summary');
+const summary = await summaryResponse.json();
+
+// Generate full report
+const reportResponse = await fetch('/reports/generate');
+const report = await reportResponse.json();
 ```
 
 ## Environment Variables

src/app.ts

@@ -3,6 +3,7 @@ import mongoose from 'mongoose';
 import { connectDatabase } from './config/database';
 import { setFileRoutes } from './routes/fileRoutes';
 import { setDataRoutes } from './routes/dataRoutes';
+import { setReportRoutes } from './routes/reportRoutes';
 
 const app = express();
 const PORT = process.env.PORT || 3000;
@@ -17,6 +18,7 @@ connectDatabase();
 // Set up routes
 setFileRoutes(app);
 setDataRoutes(app);
+setReportRoutes(app);
 
 // Start the server
 app.listen(PORT, () => {

src/routes/reportRoutes.ts

@@ -0,0 +1,10 @@
+import { Router } from 'express';
+import { ReportController } from '../controllers/reportController';
+
+const router = Router();
+const reportController = new ReportController();
+
+export function setReportRoutes(app: Router) {
+    app.get('/reports/generate', reportController.generateReport.bind(reportController));
+    app.get('/reports/summary', reportController.getSummary.bind(reportController));
+}

tests/app.test.ts

@@ -1,5 +1,5 @@
 import request from 'supertest';
-import express from 'express';
+import express, { Request, Response } from 'express';
 
 // Mock dependencies
 jest.mock('../src/config/database');
@@ -24,10 +24,10 @@ describe('App', () => {
         // Mock implementations
         mockConnectDatabase.mockImplementation(() => {});
         mockSetFileRoutes.mockImplementation((app) => {
-            app.get('/test-file', (req, res) => res.json({ message: 'file route' }));
+            app.get('/test-file', (req: Request, res: Response) => res.json({ message: 'file route' }));
         });
         mockSetDataRoutes.mockImplementation((app) => {
-            app.get('/test-data', (req, res) => res.json({ message: 'data route' }));
+            app.get('/test-data', (req: Request, res: Response) => res.json({ message: 'data route' }));
         });
 
         // Set up routes
@@ -56,7 +56,7 @@ describe('App', () => {
     });
 
     it('should handle JSON middleware', async () => {
-        app.post('/test-json', (req, res) => {
+        app.post('/test-json', (req: Request, res: Response) => {
             res.json({ received: req.body });
         });
 
@@ -69,7 +69,7 @@ describe('App', () => {
     });
 
     it('should handle URL encoded middleware', async () => {
-        app.post('/test-form', (req, res) => {
+        app.post('/test-form', (req: Request, res: Response) => {
             res.json({ received: req.body });
         });