hamed/clinicpro
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

feat(patients): phase D — call-center tab (patient call log)

Browse Source 
Add a record-scoped PatientCall entity (subject, summary, outcome
success/missed, called_at, personnel) with its repository and three
owner-gated endpoints on PatientController:

  GET    /patient/{uuid}/calls   — call log, newest first, optional ?outcome
  POST   /patient/{uuid}/call    — log a call (subject required)
  DELETE /patient/call/{uuid}    — delete an entry

Wire the previously-placeholder "کال سنتر" tab as a CallCenterTab: a register
form (date/time/subject/summary + success/missed toggle, personnel taken from
the logged-in user) beside a filterable call history (all / success / missed).
With this every patient-detail tab is now backed by a real endpoint, so the
generic Placeholder is no longer reachable. PatientCallTest covers create/list/
delete, the outcome filter, the invalid-outcome fallback, and ownership scoping.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
hamed
2026-07-13 16:01:16 +03:30
co-authored by Claude Opus 4.8
parent 61981a45d0
commit 665a210ef8
8 changed files with 474 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
tests/Patient/PatientCallTest.php
+92
View File
@@ -0,0 +1,92 @@
<?php
namespace App\Tests\Patient;
use App\Doctor\Entity\Doctor;
use App\Patient\Entity\PatientRecord;
use App\Tests\ApiTestCase;
/**
* Patient call log (کال سنتر): create, list, outcome filter, delete + ownership scoping.
*/
class PatientCallTest extends ApiTestCase
{
/** @return array{0: \App\Auth\Entity\User, 1: PatientRecord} */
private function recordFor(): array
{
$owner = $this->createUser(['ROLE_DOCTOR']);
$doctor = new Doctor($owner, 'دکتر');
$this->em->persist($doctor);
$this->em->flush();
$patient = $this->createUser(['ROLE_USER']);
$record = new PatientRecord('doctor', $doctor->getId(), $patient, 'doctor', $doctor->getId());
$this->em->persist($record);
$this->em->flush();
return [$owner, $record];
}
public function testCreateListDelete(): void
{
[$owner, $record] = $this->recordFor();
$created = $this->authJson('POST', '/api/v1/patient/' . $record->getUuid() . '/call', $owner, [
'subject' => 'پیگیری نوبت', 'summary' => 'جابه‌جایی تاریخ', 'outcome' => 'success', 'personnel' => 'مریم امینی',
]);
self::assertSame(201, $this->responseCode());
self::assertSame('پیگیری نوبت', $created['data']['subject']);
self::assertSame('success', $created['data']['outcome']);
self::assertSame('مریم امینی', $created['data']['personnel']);
$cUuid = $created['data']['uuid'];
$list = $this->authJson('GET', '/api/v1/patient/' . $record->getUuid() . '/calls', $owner);
self::assertCount(1, $list['data']);
$this->authJson('DELETE', '/api/v1/patient/call/' . $cUuid, $owner);
self::assertSame(200, $this->responseCode());
$after = $this->authJson('GET', '/api/v1/patient/' . $record->getUuid() . '/calls', $owner);
self::assertCount(0, $after['data']);
}
public function testRequiresSubject(): void
{
[$owner, $record] = $this->recordFor();
$this->authJson('POST', '/api/v1/patient/' . $record->getUuid() . '/call', $owner, ['subject' => '']);
self::assertSame(422, $this->responseCode());
}
public function testOutcomeFilter(): void
{
[$owner, $record] = $this->recordFor();
$this->authJson('POST', '/api/v1/patient/' . $record->getUuid() . '/call', $owner, ['subject' => 'a', 'outcome' => 'success']);
$this->authJson('POST', '/api/v1/patient/' . $record->getUuid() . '/call', $owner, ['subject' => 'b', 'outcome' => 'missed']);
$missed = $this->authJson('GET', '/api/v1/patient/' . $record->getUuid() . '/calls?outcome=missed', $owner);
self::assertCount(1, $missed['data']);
self::assertSame('missed', $missed['data'][0]['outcome']);
}
public function testInvalidOutcomeFallsBackToSuccess(): void
{
[$owner, $record] = $this->recordFor();
$created = $this->authJson('POST', '/api/v1/patient/' . $record->getUuid() . '/call', $owner, [
'subject' => 'x', 'outcome' => 'garbage',
]);
self::assertSame('success', $created['data']['outcome']);
}
public function testOwnershipScoped(): void
{
[$owner, $record] = $this->recordFor();
$created = $this->authJson('POST', '/api/v1/patient/' . $record->getUuid() . '/call', $owner, ['subject' => 'x']);
$cUuid = $created['data']['uuid'];
[$other] = $this->recordFor();
$this->authJson('DELETE', '/api/v1/patient/call/' . $cUuid, $other);
self::assertSame(404, $this->responseCode());
$this->authJson('GET', '/api/v1/patient/' . $record->getUuid() . '/calls', $other);
self::assertSame(404, $this->responseCode());
}
}