feat: add integration tests for API routes and setup test environment for Vitest

Author: davidsnowden0072

edurate/backend/index.test.js

@@ -1,22 +1,16 @@
 /**
  * index.test.js
  * 
- * Tests for backend API endpoints.
- * Validates route handling, error responses, and input validation.
+ * Integration tests for backend API endpoints.
+ * Tests actual route handling, database interactions, and input validation.
  */
 
-import { describe, it, expect, vi } from 'vitest';
+import { describe, it, expect, beforeAll } from 'vitest';
 import request from 'supertest';
 import express from 'express';
 import cors from 'cors';
-
-// Mock the supabase config to avoid requiring env vars
-vi.mock('./config/supabase.js', () => ({
-  supabase: {}
-}));
-
-// Import routes AFTER mocking
-const courseRoutes = await import('./routes/courses.js').then(m => m.default);
+import 'dotenv/config';
+import courseRoutes from './routes/courses.js';
 
 // Create test app instance
 const app = express();
@@ -52,6 +46,62 @@ describe('Backend API Tests', () => {
   });
 });
 
+describe('API Routes Integration Tests', () => {
+  
+  describe('GET /api/courses', () => {
+    it('should return list of courses with ratings', async () => {
+      const response = await request(app)
+        .get('/api/courses')
+        .expect(200);
+      
+      expect(response.body).toHaveProperty('success', true);
+      expect(response.body).toHaveProperty('data');
+      expect(Array.isArray(response.body.data)).toBe(true);
+      
+      // Check structure of first course if any exist
+      if (response.body.data.length > 0) {
+        const course = response.body.data[0];
+        expect(course).toHaveProperty('id');
+        expect(course).toHaveProperty('Course_name');
+        expect(course).toHaveProperty('rating');
+        expect(course).toHaveProperty('numReviews');
+      }
+    });
+  });
+
+  describe('GET /api/courses/:id', () => {
+    it('should return a single course with rating', async () => {
+      const response = await request(app)
+        .get('/api/courses/1')
+        .expect(200);
+      
+      expect(response.body).toHaveProperty('success', true);
+      expect(response.body.data).toHaveProperty('id', 1);
+      expect(response.body.data).toHaveProperty('rating');
+      expect(response.body.data).toHaveProperty('numReviews');
+    });
+
+    it('should return 404 for non-existent course', async () => {
+      const response = await request(app)
+        .get('/api/courses/99999')
+        .expect(404);
+      
+      expect(response.body).toHaveProperty('success', false);
+    });
+  });
+
+  describe('GET /api/courses/:id/reviews', () => {
+    it('should return reviews for a course', async () => {
+      const response = await request(app)
+        .get('/api/courses/1/reviews')
+        .expect(200);
+      
+      expect(response.body).toHaveProperty('success', true);
+      expect(Array.isArray(response.body.data)).toBe(true);
+    });
+  });
+});
+
 describe('Review Validation Tests', () => {
   it('should reject review with missing rating', async () => {
     const response = await request(app)
@@ -100,4 +150,62 @@ describe('Review Validation Tests', () => {
     expect(response.body.success).toBe(false);
     expect(response.body.error).toContain('required');
   });
+
+  it('should create a new review with valid data', async () => {
+    const newReview = {
+      rating: 5,
+      comment: 'Automated test review - excellent course!',
+      student_name: 'Test Student'
+    };
+
+    const response = await request(app)
+      .post('/api/courses/1/reviews')
+      .send(newReview)
+      .expect(201);
+    
+    expect(response.body).toHaveProperty('success', true);
+    expect(response.body.data).toHaveProperty('id');
+    expect(response.body.data.rating).toBe(5);
+    expect(response.body.data.comment).toBe('Automated test review - excellent course!');
+  });
+});
+
+describe('Vote System Tests', () => {
+  let testReviewId;
+
+  // Get a review ID for testing votes
+  beforeAll(async () => {
+    const response = await request(app).get('/api/courses/1/reviews');
+    if (response.body.data && response.body.data.length > 0) {
+      testReviewId = response.body.data[0].id;
+    }
+  });
+
+  it('should increment upvote count', async () => {
+    if (!testReviewId) {
+      console.log('Skipping upvote test - no reviews available');
+      return;
+    }
+
+    const response = await request(app)
+      .post(`/api/courses/1/reviews/${testReviewId}/upvote`)
+      .expect(200);
+    
+    expect(response.body).toHaveProperty('success', true);
+    expect(response.body.data).toHaveProperty('upvotes');
+  });
+
+  it('should increment downvote count', async () => {
+    if (!testReviewId) {
+      console.log('Skipping downvote test - no reviews available');
+      return;
+    }
+
+    const response = await request(app)
+      .post(`/api/courses/1/reviews/${testReviewId}/downvote`)
+      .expect(200);
+    
+    expect(response.body).toHaveProperty('success', true);
+    expect(response.body.data).toHaveProperty('downvotes');
+  });
 });

edurate/backend/package-lock.json

@@ -15,7 +15,7 @@
   },
   "devDependencies": {
     "nodemon": "^3.0.2",
-    "supertest": "^7.1.4",
+    "supertest": "^7.2.2",
     "vitest": "^4.0.15"
   }
-}
+}

edurate/backend/package.json

@@ -0,0 +1,15 @@
+/**
+ * setupTests.js
+ * 
+ * Test environment setup for Vitest.
+ * Runs before all test files.
+ */
+
+import { expect, afterEach } from 'vitest';
+import { cleanup } from '@testing-library/react';
+import '@testing-library/jest-dom';
+
+// Cleanup after each test
+afterEach(() => {
+  cleanup();
+});