Merge pull request #18 from getmarkus/cm-branch-16

refactor: API endpoints to includeing in tests

Author: getmarkus

app/core/usecases/analyze_issue.py

@@ -25,6 +25,7 @@ def analyze(self) -> Issue:
         issue = self.repo.get_by_id(self.issue_number)
         logger.info(f"issue: {issue}")
         if issue.issue_number == 0:
+            logger.info(f"not found issue: {self.issue_number}")
             raise NotFoundException(
                 message="Issue not found",
                 detail=f"Issue with number {self.issue_number} does not exist",

app/tests/conftest.py

@@ -18,7 +18,7 @@ def session_fixture():
 
 
 @pytest.fixture(name="client")
-def client_fixture(session: Session):
+def client_fixture():
 
     with TestClient(app) as client:
         yield client

app/tests/test_issues.py

@@ -35,9 +35,9 @@ def test_analyze_issue_client(client: TestClient, session: Session):
     repository.add(test_issue)
     repository.commit()
 
-    response = client.post("/issues/1/analyze")
-    # assert response.status_code == 200
-    assert response.json() == {"version": 1, "issue_number": 1}
+    response = client.post("/v1/issues/1/analyze")
+    assert response.status_code == 200
+    assert response.json() == {"issue_state": "OPEN", "version": 0, "issue_number": 1}
 
 
 def test_analyze_issue_not_found(client: TestClient, session: Session):
@@ -53,15 +53,14 @@ def test_analyze_issue_not_found(client: TestClient, session: Session):
         use_case.analyze()
     assert exc_info.value.message == "Issue not found"
 
-    response = client.post("/issues/1/analyze")
+    response = client.post("/v1/issues/1/analyze")
     assert response.status_code == 404
-    assert response.json() == {"version": 1, "issue_number": 1}
 
 
 def test_analyze_issue_invalid_number(client: TestClient, session: Session):
     """Test analyzing an issue with an invalid issue number."""
 
-    response = client.post("/issues/abc/analyze")
+    response = client.post("/v1/issues/abc/analyze")
     assert response.status_code == 422
     # Validate error response structure
     error_detail = response.json()["detail"]
@@ -72,6 +71,6 @@ def test_analyze_issue_invalid_number(client: TestClient, session: Session):
 
 def test_analyze_issue_unauthorized(client: TestClient):
     # Test case 3: Unauthorized access
-    response = client.post("/issues/456/analyze")
+    response = client.post("/v1/issues/456/analyze")
     assert response.status_code == 401
-    assert response.js_on() == {"detail": "Unauthorized"}
+    assert response.json() == {"detail": "Unauthorized"}