Set incident status to fixed when incident update status is fixed (#349)

Author: 406-mot-acceptable

src/Actions/Update/CreateUpdate.php

@@ -22,6 +22,7 @@ public function handle(Incident|Schedule $resource, CreateIncidentUpdateRequestD
         $resource->updates()->save($update);
 
         if ($resource instanceof Incident && $data->status === IncidentStatusEnum::fixed) {
+            $resource->update(['status' => IncidentStatusEnum::fixed]);
             $this->updateComponentsToOperational($resource);
         }
 

tests/Unit/Actions/Update/CreateUpdateTest.php

@@ -41,6 +41,38 @@
         ->latestStatus->toEqual(IncidentStatusEnum::identified);
 });
 
+it('transitions parent incident status to fixed when incident update status is fixed', function () {
+    $incident = Incident::factory()->create([
+        'status' => IncidentStatusEnum::investigating,
+    ]);
+
+    $data = CreateIncidentUpdateRequestData::from([
+        'message' => 'This issue has been fixed.',
+        'status' => IncidentStatusEnum::fixed,
+    ]);
+
+    app(CreateUpdate::class)->handle($incident, $data);
+
+    expect($incident->fresh())
+        ->status->toEqual(IncidentStatusEnum::fixed);
+});
+
+it('does not change parent incident status when incident update status is not fixed', function () {
+    $incident = Incident::factory()->create([
+        'status' => IncidentStatusEnum::investigating,
+    ]);
+
+    $data = CreateIncidentUpdateRequestData::from([
+        'message' => 'Still looking into this.',
+        'status' => IncidentStatusEnum::identified,
+    ]);
+
+    app(CreateUpdate::class)->handle($incident, $data);
+
+    expect($incident->fresh())
+        ->status->toEqual(IncidentStatusEnum::investigating);
+});
+
 it('sets linked component status to operational when incident update status is fixed', function () {
     $incident = Incident::factory()->create([
         'status' => IncidentStatusEnum::investigating,