From a300eced76965d22bad37a087d297752d2306f5e Mon Sep 17 00:00:00 2001 From: Saufi Date: Mon, 11 May 2026 12:26:07 +0800 Subject: [PATCH 1/2] role listing --- .claude/settings.local.json | 9 ++++ app/Http/Controllers/RoleController.php | 20 +++++++++ app/Models/Role.php | 15 +++++++ database/factories/RoleFactory.php | 25 +++++++++++ .../2026_05_11_042341_create_roles_table.php | 29 +++++++++++++ database/seeders/RoleSeeder.php | 22 ++++++++++ resources/views/layouts/navigation.blade.php | 6 +++ resources/views/roles/_table.blade.php | 34 +++++++++++++++ resources/views/roles/index.blade.php | 42 +++++++++++++++++++ routes/web.php | 2 + tests/Feature/RoleIndexTest.php | 41 ++++++++++++++++++ 11 files changed, 245 insertions(+) create mode 100644 .claude/settings.local.json create mode 100644 app/Http/Controllers/RoleController.php create mode 100644 app/Models/Role.php create mode 100644 database/factories/RoleFactory.php create mode 100644 database/migrations/2026_05_11_042341_create_roles_table.php create mode 100644 database/seeders/RoleSeeder.php create mode 100644 resources/views/roles/_table.blade.php create mode 100644 resources/views/roles/index.blade.php create mode 100644 tests/Feature/RoleIndexTest.php diff --git a/.claude/settings.local.json b/.claude/settings.local.json new file mode 100644 index 0000000..b94054c --- /dev/null +++ b/.claude/settings.local.json @@ -0,0 +1,9 @@ +{ + "permissions": { + "allow": [ + "mcp__laravel-boost__database-schema", + "Bash(php artisan *)", + "Bash(vendor/bin/pint --dirty --format agent)" + ] + } +} diff --git a/app/Http/Controllers/RoleController.php b/app/Http/Controllers/RoleController.php new file mode 100644 index 0000000..f2dbc01 --- /dev/null +++ b/app/Http/Controllers/RoleController.php @@ -0,0 +1,20 @@ +paginate(10); + + if ($request->ajax()) { + return view('roles._table', compact('roles')); + } + + return view('roles.index', compact('roles')); + } +} diff --git a/app/Models/Role.php b/app/Models/Role.php new file mode 100644 index 0000000..a68d904 --- /dev/null +++ b/app/Models/Role.php @@ -0,0 +1,15 @@ + */ + use HasFactory; + + protected $fillable = ['name', 'description']; +} diff --git a/database/factories/RoleFactory.php b/database/factories/RoleFactory.php new file mode 100644 index 0000000..ffbe866 --- /dev/null +++ b/database/factories/RoleFactory.php @@ -0,0 +1,25 @@ + + */ +class RoleFactory extends Factory +{ + /** + * Define the model's default state. + * + * @return array + */ + public function definition(): array + { + return [ + 'name' => fake()->unique()->word(), + 'description' => fake()->sentence(), + ]; + } +} diff --git a/database/migrations/2026_05_11_042341_create_roles_table.php b/database/migrations/2026_05_11_042341_create_roles_table.php new file mode 100644 index 0000000..5ea3089 --- /dev/null +++ b/database/migrations/2026_05_11_042341_create_roles_table.php @@ -0,0 +1,29 @@ +id(); + $table->string('name')->unique(); + $table->string('description')->nullable(); + $table->timestamps(); + }); + } + + /** + * Reverse the migrations. + */ + public function down(): void + { + Schema::dropIfExists('roles'); + } +}; diff --git a/database/seeders/RoleSeeder.php b/database/seeders/RoleSeeder.php new file mode 100644 index 0000000..1933f21 --- /dev/null +++ b/database/seeders/RoleSeeder.php @@ -0,0 +1,22 @@ + 'Admin', 'description' => 'Full access to all resources'], + ['name' => 'Editor', 'description' => 'Can create and edit content'], + ['name' => 'Viewer', 'description' => 'Read-only access'], + ]; + + foreach ($roles as $role) { + Role::firstOrCreate(['name' => $role['name']], $role); + } + } +} diff --git a/resources/views/layouts/navigation.blade.php b/resources/views/layouts/navigation.blade.php index c64bf64..a2b2431 100644 --- a/resources/views/layouts/navigation.blade.php +++ b/resources/views/layouts/navigation.blade.php @@ -15,6 +15,9 @@ {{ __('Dashboard') }} + + {{ __('Roles') }} + @@ -70,6 +73,9 @@ {{ __('Dashboard') }} + + {{ __('Roles') }} + diff --git a/resources/views/roles/_table.blade.php b/resources/views/roles/_table.blade.php new file mode 100644 index 0000000..b6b38b2 --- /dev/null +++ b/resources/views/roles/_table.blade.php @@ -0,0 +1,34 @@ + + + + + + + + + + + @foreach ($roles as $role) + + + + + + + @endforeach + + @if ($roles->isEmpty()) + + + + @endif + +
#{{ __('Name') }}{{ __('Description') }}{{ __('Created') }}
{{ $roles->firstItem() + $loop->index }}{{ $role->name }}{{ $role->description ?? '—' }}{{ $role->created_at->format('d M Y') }}
+ {{ __('No roles found.') }} +
+ +@if ($roles->hasPages()) +
+ {{ $roles->links() }} +
+@endif diff --git a/resources/views/roles/index.blade.php b/resources/views/roles/index.blade.php new file mode 100644 index 0000000..114f6de --- /dev/null +++ b/resources/views/roles/index.blade.php @@ -0,0 +1,42 @@ + + +

+ {{ __('Roles') }} +

+
+ +
+
+
+
+
+ {{ __('Loading...') }} +
+ +
+ @include('roles._table') +
+
+
+
+
+
diff --git a/routes/web.php b/routes/web.php index 74bb7ca..9c022e8 100644 --- a/routes/web.php +++ b/routes/web.php @@ -1,6 +1,7 @@ middleware(['auth', 'verified'])->name('dashboard'); Route::middleware('auth')->group(function () { + Route::get('/roles', [RoleController::class, 'index'])->name('roles.index'); Route::get('/profile', [ProfileController::class, 'edit'])->name('profile.edit'); Route::patch('/profile', [ProfileController::class, 'update'])->name('profile.update'); Route::delete('/profile', [ProfileController::class, 'destroy'])->name('profile.destroy'); diff --git a/tests/Feature/RoleIndexTest.php b/tests/Feature/RoleIndexTest.php new file mode 100644 index 0000000..78a2f2f --- /dev/null +++ b/tests/Feature/RoleIndexTest.php @@ -0,0 +1,41 @@ +get('/roles')->assertRedirect('/login'); +}); + +test('authenticated users can view the roles page', function () { + $this->actingAs(User::factory()->create()) + ->get('/roles') + ->assertOk() + ->assertViewIs('roles.index'); +}); + +test('roles page passes paginated roles to view', function () { + $this->actingAs(User::factory()->create()) + ->get('/roles') + ->assertOk() + ->assertViewHas('roles'); +}); + +test('ajax request returns roles table partial', function () { + $this->actingAs(User::factory()->create()) + ->withHeader('X-Requested-With', 'XMLHttpRequest') + ->get('/roles') + ->assertOk() + ->assertViewIs('roles._table'); +}); + +test('ajax pagination returns correct page', function () { + Role::factory()->count(15)->create(); + + $this->actingAs(User::factory()->create()) + ->withHeader('X-Requested-With', 'XMLHttpRequest') + ->get('/roles?page=2') + ->assertOk() + ->assertViewIs('roles._table') + ->assertViewHas('roles', fn ($roles) => $roles->currentPage() === 2); +}); From eaf0ba7a4d8283e440b0bcac421e3983515fc1d3 Mon Sep 17 00:00:00 2001 From: Saufi Date: Mon, 11 May 2026 12:29:11 +0800 Subject: [PATCH 2/2] add new role functionality --- .claude/settings.local.json | 3 +- app/Http/Controllers/RoleController.php | 16 +++++++ app/Http/Requests/StoreRoleRequest.php | 28 +++++++++++++ resources/views/roles/create.blade.php | 56 +++++++++++++++++++++++++ resources/views/roles/index.blade.php | 23 ++++++++-- routes/web.php | 2 + tests/Feature/StoreRoleTest.php | 52 +++++++++++++++++++++++ 7 files changed, 176 insertions(+), 4 deletions(-) create mode 100644 app/Http/Requests/StoreRoleRequest.php create mode 100644 resources/views/roles/create.blade.php create mode 100644 tests/Feature/StoreRoleTest.php diff --git a/.claude/settings.local.json b/.claude/settings.local.json index b94054c..59f1246 100644 --- a/.claude/settings.local.json +++ b/.claude/settings.local.json @@ -3,7 +3,8 @@ "allow": [ "mcp__laravel-boost__database-schema", "Bash(php artisan *)", - "Bash(vendor/bin/pint --dirty --format agent)" + "Bash(vendor/bin/pint --dirty --format agent)", + "mcp__laravel-boost__search-docs" ] } } diff --git a/app/Http/Controllers/RoleController.php b/app/Http/Controllers/RoleController.php index f2dbc01..cb60165 100644 --- a/app/Http/Controllers/RoleController.php +++ b/app/Http/Controllers/RoleController.php @@ -2,8 +2,11 @@ namespace App\Http\Controllers; +use App\Http\Requests\StoreRoleRequest; use App\Models\Role; +use Illuminate\Http\RedirectResponse; use Illuminate\Http\Request; +use Illuminate\View\View; class RoleController extends Controller { @@ -17,4 +20,17 @@ class RoleController extends Controller return view('roles.index', compact('roles')); } + + public function create(): View + { + return view('roles.create'); + } + + public function store(StoreRoleRequest $request): RedirectResponse + { + Role::create($request->validated()); + + return redirect()->route('roles.index') + ->with('status', 'role-created'); + } } diff --git a/app/Http/Requests/StoreRoleRequest.php b/app/Http/Requests/StoreRoleRequest.php new file mode 100644 index 0000000..ef2c983 --- /dev/null +++ b/app/Http/Requests/StoreRoleRequest.php @@ -0,0 +1,28 @@ +|string> + */ + public function rules(): array + { + return [ + 'name' => ['required', 'string', 'max:255', 'unique:roles,name'], + 'description' => ['nullable', 'string', 'max:500'], + ]; + } +} diff --git a/resources/views/roles/create.blade.php b/resources/views/roles/create.blade.php new file mode 100644 index 0000000..3a24f01 --- /dev/null +++ b/resources/views/roles/create.blade.php @@ -0,0 +1,56 @@ + + +

+ {{ __('New Role') }} +

+
+ +
+
+
+
+
+
+

+ {{ __('Role Details') }} +

+ +

+ {{ __('Create a new role to assign to users in the system.') }} +

+
+ +
+ @csrf + +
+ + + +
+ +
+ + + +
+ +
+ {{ __('Create Role') }} + + + {{ __('Cancel') }} + +
+
+
+
+
+
+
+
diff --git a/resources/views/roles/index.blade.php b/resources/views/roles/index.blade.php index 114f6de..dae2c3c 100644 --- a/resources/views/roles/index.blade.php +++ b/resources/views/roles/index.blade.php @@ -1,12 +1,29 @@ -

- {{ __('Roles') }} -

+
+

+ {{ __('Roles') }} +

+ + {{ __('New Role') }} + +
+ @if (session('status') === 'role-created') +
+ {{ __('Role created successfully.') }} +
+ @endif +
group(function () { Route::get('/roles', [RoleController::class, 'index'])->name('roles.index'); + Route::get('/roles/create', [RoleController::class, 'create'])->name('roles.create'); + Route::post('/roles', [RoleController::class, 'store'])->name('roles.store'); Route::get('/profile', [ProfileController::class, 'edit'])->name('profile.edit'); Route::patch('/profile', [ProfileController::class, 'update'])->name('profile.update'); Route::delete('/profile', [ProfileController::class, 'destroy'])->name('profile.destroy'); diff --git a/tests/Feature/StoreRoleTest.php b/tests/Feature/StoreRoleTest.php new file mode 100644 index 0000000..c6b0994 --- /dev/null +++ b/tests/Feature/StoreRoleTest.php @@ -0,0 +1,52 @@ +get('/roles/create')->assertRedirect('/login'); +}); + +test('authenticated users can access the create role page', function () { + $this->actingAs(User::factory()->create()) + ->get('/roles/create') + ->assertOk() + ->assertViewIs('roles.create'); +}); + +test('authenticated users can create a role', function () { + $this->actingAs(User::factory()->create()) + ->post('/roles', ['name' => 'Manager', 'description' => 'Manages things']) + ->assertRedirect(route('roles.index')) + ->assertSessionHas('status', 'role-created'); + + $this->assertDatabaseHas('roles', ['name' => 'Manager', 'description' => 'Manages things']); +}); + +test('role name is required', function () { + $this->actingAs(User::factory()->create()) + ->post('/roles', ['name' => '', 'description' => 'Some description']) + ->assertSessionHasErrors('name'); +}); + +test('role name must be unique', function () { + Role::factory()->create(['name' => 'Admin']); + + $this->actingAs(User::factory()->create()) + ->post('/roles', ['name' => 'Admin']) + ->assertSessionHasErrors('name'); +}); + +test('description is optional', function () { + $this->actingAs(User::factory()->create()) + ->post('/roles', ['name' => 'Viewer']) + ->assertRedirect(route('roles.index')); + + $this->assertDatabaseHas('roles', ['name' => 'Viewer', 'description' => null]); +}); + +test('guests cannot create a role', function () { + $this->post('/roles', ['name' => 'Admin'])->assertRedirect('/login'); + + $this->assertDatabaseMissing('roles', ['name' => 'Admin']); +});