Merge pull request #15512 from marcusmoore/testing/fmcs

Added tests for delete methods in api
This commit is contained in:
snipe
2024-09-18 13:44:44 +01:00
committed by GitHub
28 changed files with 1239 additions and 55 deletions
@@ -188,10 +188,6 @@ class AssetMaintenancesController extends Controller
// Check if the asset maintenance exists
$assetMaintenance = AssetMaintenance::findOrFail($assetMaintenanceId);
if (! Company::isCurrentUserHasAccess($assetMaintenance->asset)) {
return response()->json(Helper::formatStandardApiResponse('error', null, 'You cannot delete a maintenance for that asset'));
}
$assetMaintenance->delete();
return response()->json(Helper::formatStandardApiResponse('success', $assetMaintenance, trans('admin/asset_maintenances/message.delete.success')));
@@ -220,7 +220,6 @@ class LicensesController extends Controller
*/
public function destroy($id) : JsonResponse
{
//
$license = License::findOrFail($id);
$this->authorize('delete', $license);