Merge branch 'main' into fetch-testexecution-status-from-c3

backend/test_observer/controllers/artefacts/artefacts.py

@@ -35,7 +35,7 @@
     get_reports_ids,
     get_statuses_ids,
 )
-from .models import ArtefactBuildDTO, ArtefactDTO
+from .models import ArtefactBuildDTO, ArtefactDTO, ArtefactPatch
 
 router = APIRouter()
 
@@ -65,6 +65,19 @@ def get_artefact(artefact_id: int, db: Session = Depends(get_db)):
     return artefact
 
 
+@router.patch("/{artefact_id}")
+def patch_artefact(
+    artefact_id: int, request: ArtefactPatch, db: Session = Depends(get_db)
+):
+    artefact = db.get(Artefact, artefact_id)
+
+    if artefact is None:
+        raise HTTPException(status_code=404, detail="Artefact not found")
+
+    artefact.status = request.status
+    db.commit()
+
+
 @router.get("/{artefact_id}/builds", response_model=list[ArtefactBuildDTO])
 def get_artefact_builds(
     artefact_id: int,

backend/test_observer/controllers/artefacts/models.py

@@ -22,7 +22,7 @@
 from pydantic import AliasPath, BaseModel, ConfigDict, Field
 
 from test_observer.external_apis.c3.models import TestResult
-
+from test_observer.data_access.models_enums import ArtefactStatus
 
 class TestExecutionStatus(str, Enum):
     IN_PROGRESS = "IN_PROGRESS"
@@ -70,3 +70,7 @@ class ArtefactBuildDTO(BaseModel):
     architecture: str
     revision: int | None
     test_executions: list[TestExecutionDTO]
+
+
+class ArtefactPatch(BaseModel):
+    status: ArtefactStatus

backend/tests/controllers/artefacts/test_artefacts.py

@@ -32,6 +32,7 @@
     SubmissionStatus,
 )
 from test_observer.main import app
+from test_observer.data_access.models_enums import ArtefactStatus
 from tests.helpers import create_artefact
 
 
@@ -211,3 +212,17 @@ def test_correct_test_execution_status(
             ],
         }
     ]
+
+
+def test_artefact_signoff(db_session: Session, test_client: TestClient):
+    artefact = create_artefact(db_session, "candidate")
+
+    response = test_client.patch(
+        f"/v1/artefacts/{artefact.id}",
+        json={"status": ArtefactStatus.APPROVED},
+    )
+
+    db_session.refresh(artefact)
+
+    assert response.status_code == 200
+    assert artefact.status == ArtefactStatus.APPROVED