Sprint3/frontend backend api integration 3.2

backend/main.py

@@ -21,6 +21,11 @@
     allow_headers=["*"],
 )
 
+def check_empty_text(v: str) -> str:
+    if not v or not v.strip():
+        raise ValueError("text must not be empty")
+    return v
+
 
 # ============================================
 # REQUEST/RESPONSE SCHEMAS
@@ -37,11 +42,28 @@ class GenerateRequest(BaseModel):
     @field_validator("text")
     @classmethod
     def text_must_not_be_empty(cls, v: str) -> str:
-        if not v or not v.strip():
-            raise ValueError("text must not be empty")
+        return check_empty_text(v)
+
+
+class StudyPackRequest(GenerateRequest):
+    """
+    Request body for POST /generate-study-pack
+    - text: The user's study notes to process
+    """
+    @field_validator("text")
+    @classmethod
+    def text_length_constraint(cls, v: str) -> str:
+        v = check_empty_text(v)
+        stripped = v.strip()
+        # validate length
+        if len(stripped) < 10:
+            raise ValueError(f"text must not be less than 10 characters")
+        if len(stripped) > 10000:
+            raise ValueError("text must not be more than 10000 characters")
         return v
 
 
+
 class QuizQuestion(BaseModel):
     """A single quiz question with multiple choice options"""
     question: str
@@ -59,32 +81,6 @@ class GenerateResponse(BaseModel):
     quiz: list[QuizQuestion]
 
 
-# ============================================
-# STUDY PACK REQUESTS
-# ============================================
-
-class StudyPackRequest(BaseModel):
-    """
-    Request body for POST /generate-study-pack
-    - text: The user's study notes to process
-    """
-    text: str
-
-    @field_validator('text')
-    @classmethod
-    def validate_text(cls, v: str) -> str:
-        # do not include whitespace
-        stripped = v.strip()
-        # validate emptiness
-        if not v or not stripped:
-            raise ValueError("text must not be empty")
-        # validate length
-        if len(stripped) < 10:
-            raise ValueError(f"text must not be less than 10 characters")
-        if len(stripped) > 10000:
-            raise ValueError("text must not be more than 10000 characters")
-        return v
-
 
 # ============================================
 # STUDY PACK HELPER FUNCTIONS
@@ -280,7 +276,7 @@ async def generate_study_pack(request: StudyPackRequest):
     if response is None:
         raise HTTPException(
             status_code=500,
-            detail="Gemini API is unavailable. Please ensure GEMINI_API_KEY is set in .env file."
+            detail="Gemini unavailable. Please try again."
         )
 
     try:

backend/tests/test_studypack_unit.py

@@ -346,7 +346,7 @@ def test_handles_gemini_api_unavailable(self, mock_gemini, client):
         )
 
         assert response.status_code == 500
-        assert "GEMINI_API_KEY" in response.json()["detail"]
+        assert "Gemini unavailable" in response.json()["detail"]
 
     @patch.object(GeminiService, 'call_gemini', new_callable=AsyncMock)
     def test_handles_invalid_json_from_gemini(self, mock_gemini, client):
@@ -453,7 +453,7 @@ def test_various_input_formats(self, mock_gemini, client):
             )
             assert response.status_code == 200, f"Failed for input: {notes}"
 
-
+    
 
 if __name__ == "__main__":
     pytest.main([__file__, "-v"])

frontend/README.md

@@ -34,3 +34,18 @@ You can check out [the Next.js GitHub repository](https://github.com/vercel/next
 The easiest way to deploy your Next.js app is to use the [Vercel Platform](https://vercel.com/new?utm_medium=default-template&filter=next.js&utm_source=create-next-app&utm_campaign=create-next-app-readme) from the creators of Next.js.
 
 Check out our [Next.js deployment documentation](https://nextjs.org/docs/app/building-your-application/deploying) for more details.
+
+## Testing
+
+Install dependencies:
+
+```bash
+npm install --save-dev @testing-library/react @testing-library/jest-dom @testing-library/user-event
+npm install --save-dev ts-jest @types/jest
+```
+
+Run test:
+
+```bash
+npm test
+```