fix bug role delete
WMS API/ERP-API/pipeline/head This commit looks good
Details
WMS API/ERP-API/pipeline/head This commit looks good
Details
This commit is contained in:
parent
4ce31cd7ae
commit
6e5f30df4d
|
|
@ -6,7 +6,7 @@ use App\Http\Controllers\Controller;
|
|||
use Illuminate\Http\Request;
|
||||
use App\Http\Requests\Auth\Role\DeleteRequest;
|
||||
use App\Http\Resources\Auth\Role\ListResource as RowResource;
|
||||
use App\Repositories\Auth\UserRepository;
|
||||
use App\Repositories\Auth\RoleRepository;
|
||||
use App\Models\Role;
|
||||
|
||||
class DeleteController extends Controller
|
||||
|
|
@ -14,9 +14,9 @@ class DeleteController extends Controller
|
|||
/**
|
||||
* Handle the incoming request.
|
||||
*/
|
||||
public function __invoke(DeleteRequest $request, Role $user, UserRepository $repository)
|
||||
public function __invoke(DeleteRequest $request, Role $role, RoleRepository $repository)
|
||||
{
|
||||
$data = $repository->delete($user);
|
||||
$data = $repository->delete($role);
|
||||
return response()->json([
|
||||
"success" => true
|
||||
]);
|
||||
|
|
|
|||
|
|
@ -23,7 +23,7 @@ class DeleteTest extends TestCase
|
|||
*/
|
||||
public function test_success(): void
|
||||
{
|
||||
$permission = Permission::where("code","auth.user:delete")->first();
|
||||
$permission = Permission::where("code","auth.role:delete")->first();
|
||||
$role = Role::factory()->create();
|
||||
$role->permissions()->attach($permission->id);
|
||||
$user = User::factory()->create();
|
||||
|
|
@ -31,9 +31,9 @@ class DeleteTest extends TestCase
|
|||
|
||||
Sanctum::actingAs($user);
|
||||
|
||||
$data = User::factory()->create();
|
||||
$data = Role::factory()->create();
|
||||
|
||||
$response = $this->post('/auth/user/'.$data->id.'/delete');
|
||||
$response = $this->post('/auth/role/'.$data->id.'/delete');
|
||||
|
||||
$response->assertStatus(200);
|
||||
}
|
||||
|
|
|
|||
Loading…
Reference in New Issue