mahdahar/clqms-be
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

test: align account patch status expectation with update semantics

Browse Source 
PATCH requests in this API now follow update semantics and return 200 instead of 201. Update the feature test assertion so it validates the standardized behavior and avoids false failures.
This commit is contained in:
mahdahar 2026-04-08 04:18:16 +07:00
parent 61ec0cbb8a
commit 02a6a1f883
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
tests/feature/MasterDataPatchTest.php
View File

@ -162,7 +162,7 @@ class MasterDataPatchTest extends CIUnitTestCase
->withBodyFormat('json') ->withBodyFormat('json')
->call('patch', "api/organization/account/{$id}", ['AccountName' => 'Updated account']); ->call('patch', "api/organization/account/{$id}", ['AccountName' => 'Updated account']);
$patch->assertStatus(201); $patch->assertStatus(200);
$show = $this->withHeaders($this->authHeaders())->call('get', "api/organization/account/{$id}"); $show = $this->withHeaders($this->authHeaders())->call('get', "api/organization/account/{$id}");
$show->assertStatus(200); $show->assertStatus(200);
$showData = json_decode($show->getJSON(), true)['data']; $showData = json_decode($show->getJSON(), true)['data'];