Merge pull request #30 from BaljinderHothi/testing

Testing

Author: NirathH

.DS_Store

@@ -0,0 +1,93 @@
+name: CI Test Pipeline
+
+on:
+  pull_request:
+    branches: [ main ]
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    continue-on-error: false
+
+    env:
+      PYTHONPATH: ${{ github.workspace }}
+      NEXT_PUBLIC_MOCK_GEMINI: true
+      NEXT_PUBLIC_SUPABASE_URL: ${{ secrets.SUPABASE_URL }}
+      NEXT_PUBLIC_SUPABASE_KEY: ${{ secrets.SUPABASE_KEY }}
+
+    steps:
+      - name: 🧾 Checkout code
+        uses: actions/checkout@v3
+
+      - name: ⚙️ Setup Node.js
+        uses: actions/setup-node@v3
+        with:
+          node-version: 18
+
+      - name: ⚙️ Setup Python
+        uses: actions/setup-python@v4
+        with:
+          python-version: '3.10'
+
+      - name: ♻️ Cache Python dependencies
+        uses: actions/cache@v3
+        with:
+          path: ~/.cache/pip
+          key: ${{ runner.os }}-pip-${{ hashFiles('**/requirements.txt') }}
+          restore-keys: ${{ runner.os }}-pip-
+
+      - name: ♻️ Cache Node modules
+        uses: actions/cache@v3
+        with:
+          path: ./fitcheck/node_modules
+          key: ${{ runner.os }}-node-${{ hashFiles('fitcheck/package-lock.json') }}
+          restore-keys: ${{ runner.os }}-node-
+
+      - name: Install Python dependencies
+        run: |
+          python -m pip install --upgrade pip
+          pip install -r requirements.txt
+          pip install pytest pytest-cov playwright
+          playwright install
+
+      - name: Install frontend dependencies
+        working-directory: ./fitcheck
+        run: npm install
+
+      - name: Build Next.js app
+        working-directory: ./fitcheck
+        env:
+          NEXT_PUBLIC_SUPABASE_URL: ${{ secrets.SUPABASE_URL }}
+          NEXT_PUBLIC_SUPABASE_KEY: ${{ secrets.SUPABASE_KEY }}
+          NEXT_PUBLIC_MOCK_GEMINI: true
+        run: npm run build
+
+      - name: Start Next.js server in background
+        working-directory: ./fitcheck
+        env:
+          NEXT_PUBLIC_SUPABASE_URL: ${{ secrets.SUPABASE_URL }}
+          NEXT_PUBLIC_SUPABASE_KEY: ${{ secrets.SUPABASE_KEY }}
+          NEXT_PUBLIC_MOCK_GEMINI: true
+        run: |
+          nohup npm start > ../nextjs.log 2>&1 &
+
+      - name: Wait for server to be ready
+        run: |
+          for i in {1..10}; do
+            curl --fail http://localhost:3000/login && break || sleep 3
+          done || (echo "Server failed to start or /login not reachable" && cat fitcheck/../nextjs.log && exit 1)
+
+      - name: Run Python tests with coverage
+        run: pytest --cov=./ --cov-report=html
+
+      - name: Run Playwright E2E test (Login + Dashboard)
+        run: python tests/test_login_e2e.py
+
+      - name: Run Playwright E2E test (Extension UI)
+        run: xvfb-run -a python tests/test_popup_ui.py
+
+      - name: Upload coverage report
+        uses: actions/upload-artifact@v4
+        with:
+          name: coverage-report
+          path: htmlcov

.github/workflows/cipipeline.yml

@@ -43,3 +43,5 @@ node_modules/
 babel.config.js
 .env.local
 
+__pycache__/
+*.py[cod]

.gitignore

@@ -2,8 +2,6 @@
 import supabase from '../../../utils/supabase/config';
 import recommendationService from '../../../utils/supabase/recommendationService';
 
-
-
 export default async function handler(req, res) {
   if (req.method !== 'GET') {
     return res.status(405).json({ error: 'Only GET supported' });
@@ -14,53 +12,59 @@ export default async function handler(req, res) {
     return res.status(401).json({ error: 'Missing or invalid Authorization header' });
   }
 
-  const token = authHeader.split('Bearer ')[1];
-  const { data: userData, error: userError } = await supabase.auth.getUser(token);
+  try {
+    const token = authHeader.split('Bearer ')[1];
+    const { data: userData, error: userError } = await supabase.auth.getUser(token);
 
-  if (userError || !userData?.user) {
-    return res.status(401).json({ error: 'Invalid or expired token' });
-  }
+    if (userError || !userData?.user) {
+      console.error('Auth error:', userError?.message || 'No user data');
+      return res.status(401).json({ error: 'Invalid or expired token' });
+    }
 
-  const email = userData.user.email;
+    const email = userData.user.email;
 
-  // Get user ID from 'users' table
-  const { data: userRow, error: userLookupError } = await supabase
-    .from('users')
-    .select('userid')
-    .eq('email', email)
-    .single();
+    const { data: userRow, error: userLookupError } = await supabase
+      .from('users')
+      .select('userid')
+      .eq('email', email)
+      .single();
 
-  if (userLookupError || !userRow) {
-    return res.status(404).json({ error: 'User not found' });
-  }
+    if (userLookupError || !userRow) {
+      console.error('User lookup failed:', userLookupError?.message || 'User not found');
+      return res.status(404).json({ error: 'User not found' });
+    }
 
-  // Get sentiment reviews
-  const { data: sentimentData, error: sentimentError } = await supabase
-    .from('sentiment')
-    .select('*')
-    .eq('userid', userRow.userid)
-    .order('created_at', { ascending: false });
+    const { data: sentimentData, error: sentimentError } = await supabase
+      .from('sentiment')
+      .select('*')
+      .eq('userid', userRow.userid)
+      .order('created_at', { ascending: false });
 
-  if (sentimentError) {
-    return res.status(500).json({ error: 'Failed to fetch sentiment data' });
-  }
+    if (sentimentError) {
+      console.error('Sentiment fetch error:', sentimentError.message);
+      return res.status(500).json({ error: 'Failed to fetch sentiment data' });
+    }
+
+    const reviews = sentimentData.map(item => ({
+      productName: item.product_name,
+      rating: item.star_rating,
+      likedFeatures: item.sentiment_label === 'positive' ? 'style, comfort' : '',
+      dislikedFeatures: item.sentiment_label === 'negative' ? 'fit, quality' : '',
+      comment: item.review_text
+    }));
 
-  // Transform reviews into Gemini-compatible format
-  const reviews = sentimentData.map(item => ({
-    productName: item.product_name,
-    rating: item.star_rating,
-    likedFeatures: item.sentiment_label === 'positive' ? 'style, comfort' : '',
-    dislikedFeatures: item.sentiment_label === 'negative' ? 'fit, quality' : '',
-    comment: item.review_text
-  }));
+    const userPayload = {
+      reviews: reviews.slice(0, 3),
+      stylePreferences: {},
+      purchaseHistory: [],
+      budget: { weeklyAmount: 100, spentAmount: 0 }
+    };
 
-  const userPayload = {
-    reviews: reviews.slice(0, 3),  // ✅ limit to 3 to avoid token limit
-    stylePreferences: {},
-    purchaseHistory: [],
-    budget: { weeklyAmount: 100, spentAmount: 0 }
-  };
+    const recommendations = await recommendationService.generateRecommendations(userPayload);
+    return res.status(200).json({ recommendations });
 
-  const recommendations = await recommendationService.generateRecommendations(userPayload);
-  return res.status(200).json({ recommendations });
+  } catch (err) {
+    console.error('Unexpected server error in getReviews:', err);
+    return res.status(500).json({ error: 'Internal Server Error' });
+  }
 }

fitcheck/pages/api/user_rec/getRec.js

@@ -0,0 +1,12 @@
+fastapi
+requests
+pytest
+pytest-cov
+pytest-asyncio
+pytest-tornasync
+pytest-trio
+pytest-twisted
+playwright
+python-dotenv
+twisted
+git+https://github.com/supabase-community/supabase-py.git